Skip to content

feat(resolver): record the block of each contenthash write - #283

Draft
charlesHetterich wants to merge 1 commit into
masterfrom
charles/contenthash-updated-at-block
Draft

feat(resolver): record the block of each contenthash write#283
charlesHetterich wants to merge 1 commit into
masterfrom
charles/contenthash-updated-at-block

Conversation

@charlesHetterich

@charlesHetterich charlesHetterich commented Sep 9, 2026

Copy link
Copy Markdown

Closes #193

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

CI Summary

Check Result
4naly3er Analysis Found 38 issues: 5 medium, 8 low, 14 gas, 11 informational - View Report
Slither Analysis Found 181 issues: 4 high, 40 medium, 80 low, 57 informational - View Report
Contract Tests (Unit + Fuzz) All tests passed (633 total) - View Report
Contract Tests (Invariant) All tests passed (59 total) - View Report
Gas Report 9 contracts analyzed - View Report
Coverage 74.1% line coverage - View Report
Format & Lint Passed - Code formatted correctly
File Validation Passed - All tracked files valid
Deploy Contracts Reproduces the committed manifest; resume verified
PR Title PR Title Valid
Labels Unknown
Secret Scan Passed - No secrets detected

Per-section details omitted: combined body exceeded the 65000-char comment limit. Follow the View Report links above for each section's full output.

@github-actions github-actions Bot added smartcontracts Any issue relating to smart contracts scope: resolver type: test labels Sep 9, 2026
Store the block number of every successful setContenthash per node and
expose it through contenthashUpdatedAtBlock(node), so clients can read how
recently a name's content changed without scanning ContentHashUpdated logs.
The mapping takes one reserved slot from __gap; existing slots are untouched.
Entries written before this change read zero until their next write.

Closes #193
@charlesHetterich
charlesHetterich force-pushed the charles/contenthash-updated-at-block branch from fe6b88f to e48f2ef Compare September 9, 2026 06:41
@re-gius

re-gius commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Just a reminder to not merge this until we cut the final release for R1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: resolver smartcontracts Any issue relating to smart contracts type: test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feat]: Expose when a node contenthash last changed

2 participants