Skip to content

fix(txe): correct TXE_ORACLE_INTERFACE_HASH to match current oracle interface#23460

Merged
nchamo merged 1 commit into
merge-train/fairiesfrom
cb/f1c09746ff72
May 21, 2026
Merged

fix(txe): correct TXE_ORACLE_INTERFACE_HASH to match current oracle interface#23460
nchamo merged 1 commit into
merge-train/fairiesfrom
cb/f1c09746ff72

Conversation

@AztecBot

Copy link
Copy Markdown
Collaborator

Problem

The merge-train/fairies build failed in yarn-projectcompile_all (ci.aztec-labs.com/3b9623ba54db88b4), on the new TXE oracle version check added by #23324:

Error: The TXE oracle interface has changed. Update TXE_ORACLE_INTERFACE_HASH to
c5e3567b08790c8181254c0831de6b5af49e221e14d36dded3fc5d8787fba482 ...

The check is correct; the pinned literal is wrong. #23324 committed TXE_ORACLE_INTERFACE_HASH = '6ab5922a…', which corresponds to the TXE oracle interface before #23289 ("feat: add basic gas settings to testenv") added a gasSettings: GasSettings parameter to two ITxeExecutionOracle methods. #23289 has been on next for a week, so interfaces.ts already carries gasSettings and the live interface hashes to c5e3567b…. The literal was computed against a stale base.

Fix

Re-pin TXE_ORACLE_INTERFACE_HASH to the value the check computes against the current interface (c5e3567b…), which is exactly what the failing build emitted.

No version bump: #23289's gasSettings already shipped under TXE oracle version 1.0.0, and both the TS (txe_oracle_version.ts) and Noir (txe_oracles.nr) sides are at MAJOR=1/MINOR=0 with matching signatures. This pins the baseline hash to the current interface rather than recording a new interface change on top of an already-correct baseline.

The train re-run validates the value end-to-end.


Created by claudebox · group: slackbot

@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label May 21, 2026
@nchamo nchamo self-assigned this May 21, 2026
@nchamo nchamo marked this pull request as ready for review May 21, 2026 09:52
@AztecBot

Copy link
Copy Markdown
Collaborator Author

Flakey Tests

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

\033FLAKED\033 (8;;http://ci.aztec-labs.com/8d196e279b787c2f�8d196e279b787c2f8;;�):  yarn-project/end-to-end/scripts/run_test.sh simple src/e2e_epochs/epochs_mbps.parallel.test.ts "builds multiple blocks per slot with L2 to L1 messages" (352s) (code: 0) group:e2e-p2p-epoch-flakes

@nchamo nchamo merged commit 7f674bb into merge-train/fairies May 21, 2026
37 checks passed
@nchamo nchamo deleted the cb/f1c09746ff72 branch May 21, 2026 10:25
github-merge-queue Bot pushed a commit that referenced this pull request May 25, 2026
BEGIN_COMMIT_OVERRIDE
feat(txe): add TXE oracle version check to bootstrap (#23324)
fix(txe): correct TXE_ORACLE_INTERFACE_HASH to match current oracle
interface (#23460)
feat(aztec-nr): add Serialize/Deserialize for EphemeralArray (#23417)
refactor: move validation and error handling out of transport layer
(#23422)
feat(pxe)!: add source and block-range filtering to get_logs_by_tag
(#23326)
fix(txe): update TXE oracle interface hash for new AVM oracle methods
(#23492)
chore(ci): capture sandbox diagnostics on acceptance test failure
(#23495)
feat(aztec-nr)!: rename push_nullifier to push_nullifier_unsafe (#23488)
feat(aztec-nr)!: add explicit custom_sync_state hook to AztecConfig
(#23446)
fix(ci): skip aztec-cli notify job when acceptance test is skipped
(#23534)
fix: released contract artifact aztec version (forward port of #23470)
(#23500)
fix: merge train conflicts (#23548)
END_COMMIT_OVERRIDE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants