Skip to content

feat: add raw PrioUpdateRegistry V2 - #3

Open
dvush wants to merge 6 commits into
mainfrom
dvush/pur-2-v2
Open

feat: add raw PrioUpdateRegistry V2#3
dvush wants to merge 6 commits into
mainfrom
dvush/pur-2-v2

Conversation

@dvush

@dvush dvush commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

Remove registry-level freshness and packed headers. Store full-width raw slots, preserve updater writes, and add permanent per-lane decoders using STATICCALL.

Move freshness validation into ExamplePropAmm, add V2 tests, and split the V1 and V2 documentation.

Remove registry-level freshness and packed headers. Store full-width raw slots, preserve updater writes, and add permanent per-lane decoders using STATICCALL.

Move freshness validation into ExamplePropAmm, add V2 tests, and split the V1 and V2 documentation.
dvush and others added 5 commits August 19, 2026 15:39
Minimises the trust surface. The two immutable allowlisted addresses and
the state-changing CALL that reached them are removed, so the decoder
STATICCALL is now the only external interaction on any write path: a
transaction addressed to the registry can write nothing but the
registry's own storage.

- Remove the constructor, both trusted-target immutables, and
  isTrustedCallTarget.
- Drop the calls argument, allowlist check, call-hash and call loop from
  updateStateWithDecoder.
- Drop trustedCallsHash and callResults from IPrioUpdateDecoder; both
  existed only to carry trusted-call output.
- Keep the callback lock: it still stops a decoder reading lanes
  mid-update.
- Replace the trusted-target callback test with a decoder-driven probe so
  the reentrancy coverage survives the removal.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0145oJMRwscWeEFEDSU8BAtF
The removed parameters left signatures that now fit on one line, which
forge fmt --check flagged. Whitespace only.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0145oJMRwscWeEFEDSU8BAtF
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants