Skip to content

refactor(pxe)!: remove legacy oracle compat, rename V2 oracles#23425

Merged
nchamo merged 1 commit into
merge-train/fairiesfrom
nchamo/v5-is-here
May 20, 2026
Merged

refactor(pxe)!: remove legacy oracle compat, rename V2 oracles#23425
nchamo merged 1 commit into
merge-train/fairiesfrom
nchamo/v5-is-here

Conversation

@nchamo

@nchamo nchamo commented May 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Removes the LEGACY_ORACLE_VERSION = 12 shim and 18 legacy oracle name aliases in legacy_oracle_mappings.ts — the v5 protocol contract redeployment means all contracts now run on the current oracle version.
  • Drops 4 unused V1 capsule-backed oracles: getPendingTaggedLogs, validateAndStoreEnqueuedNotesAndEvents, getLogsByTag, getMessageContextsByTxHash.
  • Renames the remaining *V2 oracle methods and handlers (e.g. aztec_utl_getPendingTaggedLogs_v2aztec_utl_getPendingTaggedLogs) across the Noir, TS PXE, and TXE layers.
  • Bumps ORACLE_VERSION_MAJOR 23 → 24 and recomputes ORACLE_INTERFACE_HASH.

Fixes F-416

@nchamo nchamo requested a review from nventuro as a code owner May 20, 2026 08:59
@nchamo nchamo self-assigned this May 20, 2026
describe('utilityResolveMessageContexts', () => {
const requestSlot = Fr.random();
const responseSlot = Fr.random();
describe('getMessageContextsByTxHash', () => {

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just repurposed the v1 tests for v2, since we didn't have any

@nchamo nchamo requested review from benesjan and mverzilli May 20, 2026 09:24

@benesjan benesjan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

BTW there is a whole project with some more issues related to this

@nchamo nchamo merged commit d071195 into merge-train/fairies May 20, 2026
14 checks passed
@nchamo nchamo deleted the nchamo/v5-is-here branch May 20, 2026 09:55
danielntmd pushed a commit to danielntmd/aztec-packages that referenced this pull request Jun 4, 2026
BEGIN_COMMIT_OVERRIDE
refactor(pxe)!: remove legacy oracle compat, rename V2 oracles (AztecProtocol#23425)
refactor(pxe)!: drop V1 oracle remnants, simplify nullifier witness
(AztecProtocol#23428)
END_COMMIT_OVERRIDE
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