Skip to content

fix(pxe): backport anchor header threading from #22679 to v4-next#22705

Merged
nchamo merged 1 commit into
backport-to-v4-next-stagingfrom
claudebox/backport-22679-pxe-to-v4-next
Apr 24, 2026
Merged

fix(pxe): backport anchor header threading from #22679 to v4-next#22705
nchamo merged 1 commit into
backport-to-v4-next-stagingfrom
claudebox/backport-22679-pxe-to-v4-next

Conversation

@AztecBot

Copy link
Copy Markdown
Collaborator

Backports the PXE-only portion of #22679 to v4-next.

The full fix in #22679 spans barretenberg world-state, the aztec-node and PXE; only the PXE refactor is portable to v4-next without dragging the C++ / world-state changes.

What this changes

In yarn-project/pxe/src/pxe.ts, capture the anchor header once per proveTx / simulateTx / profileTx (right after blockStateSynchronizer.sync()) and thread it through to both #executePrivate and #prove, instead of re-reading anchorBlockStore independently inside each call.

This is a defensive refactor — the anchor cannot drift today because both callees live inside the same job-queue slot — but it makes the invariant explicit and type-checked going forward.

What this does not include from #22679

  • C++ WorldStateRevision::LATEST sentinel and is_historical() migration (barretenberg/cpp/src/barretenberg/world_state/*, avm_fuzzer/.../dbs.cpp)
  • Matching TS sentinel in yarn-project/stdlib/src/world-state/world_state_revision.ts and yarn-project/world-state/src/native/native_world_state.ts
  • AztecNodeService.getWorldState initial-header path change in yarn-project/aztec-node/src/aztec-node/server.ts (+ test)

These are intentionally omitted per the request to backport only the PXE changes.

Notes

Original PR: #22679

ClaudeBox log: https://claudebox.work/s/6ab5b666456f5be7?run=1

@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels Apr 21, 2026
@nchamo nchamo marked this pull request as ready for review April 24, 2026 14:53
@AztecBot AztecBot force-pushed the claudebox/backport-22679-pxe-to-v4-next branch from e38e1ba to 5f0eb00 Compare April 24, 2026 15:06
@nchamo nchamo self-assigned this Apr 24, 2026
@nchamo nchamo merged commit 8825db7 into backport-to-v4-next-staging Apr 24, 2026
9 checks passed
@nchamo nchamo deleted the claudebox/backport-22679-pxe-to-v4-next branch April 24, 2026 15:58
AztecBot added a commit that referenced this pull request Apr 27, 2026
BEGIN_COMMIT_OVERRIDE
feat(aztec)!: add counter template for aztec init (#22751)
cherry-pick: fix(pxe): restrict setSenderForTags override to current
call (F-564) (#22672) (with conflicts)
fix: backport restrict setSenderForTags override to current call
(#22672) (#22767)
fix(pxe): backport anchor header threading from #22679 to v4-next
(#22705)
END_COMMIT_OVERRIDE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants