Skip to content

fix: change init and single claim nullif to incl owner address, add testing utilities - #24892

Merged
nventuro merged 3 commits into
merge-train/fairies-v5from
nico/triage-196-privatemutable-initialization-nullifier-shared-between-two
Jul 21, 2026
Merged

fix: change init and single claim nullif to incl owner address, add testing utilities#24892
nventuro merged 3 commits into
merge-train/fairies-v5from
nico/triage-196-privatemutable-initialization-nullifier-shared-between-two

Conversation

@nventuro

Copy link
Copy Markdown
Contributor

Reimplementation of #24837 - same nullifier fix, this time extending TestEnvironment so that we can test the fix works as intended.

@nventuro
nventuro requested a review from nchamo July 21, 2026 18:46
Comment thread noir-projects/aztec-nr/aztec/src/state_vars/private_mutable/test.nr
@nchamo

nchamo commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Actually, @nventuro , should we have some migration notes?

@nventuro
nventuro requested a review from LeilaWang as a code owner July 21, 2026 19:13
@nventuro
nventuro enabled auto-merge (squash) July 21, 2026 19:13
…able-initialization-nullifier-shared-between-two
@nventuro
nventuro merged commit f66808c into merge-train/fairies-v5 Jul 21, 2026
12 checks passed
@nventuro
nventuro deleted the nico/triage-196-privatemutable-initialization-nullifier-shared-between-two branch July 21, 2026 19:42
nchamo pushed a commit that referenced this pull request Jul 23, 2026
…esting utilities (#24892)

Reimplementation of
#24837 - same
nullifier fix, this time extending `TestEnvironment` so that we can test
the fix works as intended.

(cherry picked from commit f66808c)
rangozd pushed a commit to rangozd/aztec-packages that referenced this pull request Aug 5, 2026
…ztecProtocol#24931)

Forward-ports the **noir / aztec-nr / contracts** slice of the v5-next →
next backlog (work merged to `v5-next` after the ~2026-07-08 cut that
reshaped `next`).

## Applied (clean cherry-picks, chronological)
- fix: prevent reception of messages too far into the future (AztecProtocol#24645)
- fix(aztec-nr): reject infinity ephemeral key in message encryption
(AztecProtocol#24665)
- fix(aztec-nr): tolerate malformed partial-note completion logs
(AztecProtocol#24668)
- docs(aztec-nr): document partial note completion trust model (AztecProtocol#24816)
- refactor(aztec-nr): shared no-op sync handler for stateless contracts
(AztecProtocol#24844)
- fix: dont panic on note msgs on contracts with no notes (AztecProtocol#24852)
- docs(noir-contracts): document standard-contract re-pin consequences
(AztecProtocol#24890)

## ⚠️ Needs owner conflict-resolution (conflict against reshaped `next`;
not included here)
Cherry-pick onto this branch and resolve:
- [ ] `git cherry-pick -x 9f1167e` — feat!: make inbox secrets be
multiple fields (AztecProtocol#24599)
- [ ] `git cherry-pick -x 4490597` — fix(aztec-nr): prevent recipient
forging a colliding handshake (AztecProtocol#24403)
- [ ] `git cherry-pick -x 8b1903c` — feat!: forbid external note
validation checks (AztecProtocol#24644)
- [ ] `git cherry-pick -x 10e339a` — fix(aztec-nr)!: compute note
property selectors from the packed layout (AztecProtocol#24689)
- [ ] `git cherry-pick -x 15c7a1e` — chore: clarify scope of packable
impl detection (AztecProtocol#24820)
- [ ] `git cherry-pick -x f66808c` — fix: change init and single
claim nullif to incl owner address, add testing utilities (AztecProtocol#24892)

Part of the manual v5-next → next backlog sweep. Draft until conflicts
are resolved and CI is green.
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.

2 participants