Skip to content

feat: plumb immutables hash#23512

Merged
Thunkar merged 7 commits into
merge-train/fairiesfrom
gj/plumb_immutables_hash
May 25, 2026
Merged

feat: plumb immutables hash#23512
Thunkar merged 7 commits into
merge-train/fairiesfrom
gj/plumb_immutables_hash

Conversation

@Thunkar
Copy link
Copy Markdown
Contributor

@Thunkar Thunkar commented May 22, 2026

While it was added to aztec.js and nr, the higher level helpers hardcoded Fr.ZERO

Thunkar added 3 commits May 22, 2026 15:01
Adds an optional `AccountManagerCreateOptions` argument to
`AccountManager.create` so callers can commit an `immutablesHash`
(and override `deployer` / `publicKeys`) without bypassing the
standard account-setup pipeline.

Motivation: enables third-party wallets to set up AZIP-9
initializerless accounts that bind their signing key via
`ContractInstance.immutables_hash` (see #23151) through the normal
`AccountManager.create` path. Today they have to construct the
instance themselves and reach for the package-private positional
constructor.

The change is backwards compatible: callers that don't pass `opts`
keep producing instances with `immutablesHash = Fr.ZERO`, matching
today's behavior.
@Thunkar Thunkar requested review from mverzilli and nventuro May 22, 2026 15:06
@Thunkar Thunkar self-assigned this May 22, 2026
```

If you need to call a public function *with* a sender, use `call_public` instead.
If you need to call a public function _with_ a sender, use `call_public` instead.
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.

Just noting you're updating unrelated parts of the migr notes

@Thunkar Thunkar enabled auto-merge (squash) May 25, 2026 11:31
@Thunkar Thunkar merged commit a98c763 into merge-train/fairies May 25, 2026
14 checks passed
@Thunkar Thunkar deleted the gj/plumb_immutables_hash branch May 25, 2026 11:48
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