Skip to content

feat!: add immutables_hash to contract instance#23091

Merged
IlyasRidhuan merged 3 commits into
merge-train/avmfrom
ir/04-30-feat_add_immutables_hash_to_contract_instance
May 16, 2026
Merged

feat!: add immutables_hash to contract instance#23091
IlyasRidhuan merged 3 commits into
merge-train/avmfrom
ir/04-30-feat_add_immutables_hash_to_contract_instance

Conversation

@IlyasRidhuan

@IlyasRidhuan IlyasRidhuan commented May 8, 2026

Copy link
Copy Markdown
Contributor

Part 1 of the immutables_hash changes as part of AZIP-9

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 forContractInstance versions and PXE_DATA_SCHEMA_VERSION

IlyasRidhuan commented May 8, 2026

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@IlyasRidhuan IlyasRidhuan force-pushed the ir/04-30-feat_add_immutables_hash_to_contract_instance branch from a6725b2 to ab7fc16 Compare May 11, 2026 15:33
@IlyasRidhuan IlyasRidhuan force-pushed the ir/04-30-feat_add_immutables_hash_to_contract_instance branch from ab7fc16 to 30d8c61 Compare May 11, 2026 16:30
* caller.current_class_id,
* caller.init_hash // situational - only if caller needs it
* caller.init_hash, // situational - only if caller needs it
* caller.immutables_hash // situational - only if caller needs it

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.

Enabled here but only properly used in the PR that adds immutable_hash retrievability as an opcode

@IlyasRidhuan IlyasRidhuan marked this pull request as ready for review May 12, 2026 12:54
@IlyasRidhuan IlyasRidhuan removed the request for review from LeilaWang May 12, 2026 12:54
@IlyasRidhuan IlyasRidhuan force-pushed the ir/04-30-feat_add_immutables_hash_to_contract_instance branch 4 times, most recently from 8a3f592 to 276df60 Compare May 14, 2026 21:07

@jeanmon jeanmon 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!

@mverzilli mverzilli 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.

The parts I have context about look good: Aztec.js, PXE, stdlib and TXE.

I defer to others for the rest of the PR!

@IlyasRidhuan IlyasRidhuan force-pushed the ir/04-30-feat_add_immutables_hash_to_contract_instance branch from 276df60 to 4ed2bc8 Compare May 15, 2026 20:56
@IlyasRidhuan IlyasRidhuan merged commit a1e7193 into merge-train/avm May 16, 2026
15 checks passed
@IlyasRidhuan IlyasRidhuan deleted the ir/04-30-feat_add_immutables_hash_to_contract_instance branch May 16, 2026 08:55
@AztecBot AztecBot mentioned this pull request May 16, 2026
danielntmd pushed a commit to danielntmd/aztec-packages that referenced this pull request Jun 4, 2026
BEGIN_COMMIT_OVERRIDE
feat!: add immutables_hash to contract instance (AztecProtocol#23091)
feat!: update address derivation (AztecProtocol#23151)
feat(avm)!: add immutables_hash member to get contract instance opcode
(AztecProtocol#23152)
feat!: azip 8 public key hashes (AztecProtocol#23159)
feat(avm)!: Remove `is_infinite` flag from AVM ECC & update noir
submodule with serialization changes (AztecProtocol#23342)
fix(avm): addressing claude review of interaction builders code (AztecProtocol#23431)
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.

3 participants