Skip to content

refactor(pxe)!: derive oracle interfaces from registry and introduce aztec_misc_ prefix#23840

Merged
nchamo merged 7 commits into
merge-train/fairiesfrom
nchamo/oracle-hash-calculation
Jun 4, 2026
Merged

refactor(pxe)!: derive oracle interfaces from registry and introduce aztec_misc_ prefix#23840
nchamo merged 7 commits into
merge-train/fairiesfrom
nchamo/oracle-hash-calculation

Conversation

@nchamo

@nchamo nchamo commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Why we are doing this

Oracle handler interfaces (IUtilityExecutionOracle, IPrivateExecutionOracle) were hand-written and had to be kept in sync with ORACLE_REGISTRY entries manually. Three cross-cutting oracles (log, getRandomField, assertCompatibleOracleVersion) were incorrectly scoped under aztec_utl_ despite being available in all execution contexts.

Our fix

  • Derive IUtilityExecutionOracle and IPrivateExecutionOracle from ORACLE_REGISTRY using TypeScript mapped types (HandlersForPrefix). Adding a new oracle entry to the registry now automatically updates the handler interface.
  • Introduce aztec_misc_ prefix for the 3 cross-cutting oracles and derive IMiscOracle from it.
  • Move NoteData from interfaces.ts to noir-structs/note_data.ts.

Note

I haven't tackled this same refactor for TXE yet, since we need to do more clean up before we get to it

@nchamo nchamo self-assigned this Jun 3, 2026
/**
* Information about a note needed during execution.
*/
export interface NoteData {

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.

Moved this from interfaces.ts to its own file

@nchamo nchamo changed the title refactor(pxe): derive oracle handler interfaces from registry refactor(pxe)!: derive oracle interfaces from registry and introduce aztec_misc_ prefix Jun 4, 2026
@AztecBot

AztecBot commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator

Flakey Tests

🤖 says: This CI run detected 2 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (8;;http://ci.aztec-labs.com/c10787b8355377fc�c10787b8355377fc8;;�):  yarn-project/end-to-end/scripts/run_test.sh simple src/e2e_p2p/preferred_gossip_network.test.ts (168s) (code: 0) group:e2e-p2p-epoch-flakes
\033FLAKED\033 (8;;http://ci.aztec-labs.com/4d3e6f2cebea25dd�4d3e6f2cebea25dd8;;�):  yarn-project/end-to-end/scripts/run_test.sh simple src/e2e_epochs/epochs_invalidate_block.parallel.test.ts "proposer invalidates multiple checkpoints" (557s) (code: 0) group:e2e-p2p-epoch-flakes

@nchamo nchamo merged commit 585829d into merge-train/fairies Jun 4, 2026
12 checks passed
@nchamo nchamo deleted the nchamo/oracle-hash-calculation branch June 4, 2026 13:05
spypsy pushed a commit that referenced this pull request Jun 8, 2026
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.

3 participants