Skip to content

feat: merge-train/avm#23332

Open
AztecBot wants to merge 3 commits into
nextfrom
merge-train/avm
Open

feat: merge-train/avm#23332
AztecBot wants to merge 3 commits into
nextfrom
merge-train/avm

Conversation

@AztecBot
Copy link
Copy Markdown
Collaborator

@AztecBot AztecBot commented May 16, 2026

BEGIN_COMMIT_OVERRIDE
feat!: add immutables_hash to contract instance (#23091)
feat!: update address derivation (#23151)
feat(avm)!: add immutables_hash member to get contract instance opcode (#23152)
END_COMMIT_OVERRIDE

Part 1 of the immutables_hash changes as part of
[AZIP-9](AztecProtocol/governance#20)

This pr just adds the `immutables_hash` field to the various
`ContractInstance` structs across the codebase.

It also updates the `publish_contract_instance` function and bumps
various constants for`ContractInstance` versions and
`PXE_DATA_SCHEMA_VERSION`
Copy link
Copy Markdown
Contributor

@IlyasRidhuan IlyasRidhuan left a comment

Choose a reason for hiding this comment

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

Holding this train up as we want to atomically merge azip changes

@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/6b01ddd4b18cb51f�6b01ddd4b18cb51f8;;�): yarn-project/scripts/run_test.sh bb-prover/src/avm_proving_tests/avm_check_circuit_amm.test.ts (9s) (code: 0)

Part 2 of the immutables_hash changes as part of
[AZIP-9](AztecProtocol/governance#20)

Updates address derivation to include the `immutables_hash`.
Specifically, `immutables_hash` is now part of the
`salted_initialization_hash` computation

This also includes a constant update from `DOM_SEP__CONTRACT_ADDRESS_V1`
-> `DOM_SEP__CONTRACT_ADDRESS_V2`
#23152)

Part 3 of the immutables_hash changes as part of
[AZIP-9](AztecProtocol/governance#20)

This updates the `GETCONTRACTINSTANCE` opcode so that the
`immutables_hash` can be retrieved from the contract instance via the
opcode.
@IlyasRidhuan IlyasRidhuan requested a review from sirasistant as a code owner May 16, 2026 09:42
Copy link
Copy Markdown
Collaborator

@ludamad ludamad left a comment

Choose a reason for hiding this comment

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

🤖 Auto-approved

@AztecBot AztecBot enabled auto-merge May 16, 2026 14:26
@AztecBot
Copy link
Copy Markdown
Collaborator Author

🤖 Auto-merge enabled after 4 hours of inactivity. This PR will be merged automatically once all checks pass.

@IlyasRidhuan IlyasRidhuan disabled auto-merge May 16, 2026 17:39
@AztecBot AztecBot enabled auto-merge May 16, 2026 18:16
@AztecBot
Copy link
Copy Markdown
Collaborator Author

🤖 Auto-merge enabled after 4 hours of inactivity. This PR will be merged automatically once all checks pass.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants