feat(aztec-nr): wire constrained tagging into message delivery#23867
Closed
vezenovm wants to merge 9 commits into
Closed
feat(aztec-nr): wire constrained tagging into message delivery#23867vezenovm wants to merge 9 commits into
vezenovm wants to merge 9 commits into
Conversation
This was referenced Jun 4, 2026
vezenovm
force-pushed
the
mv/f-669-constrained-delivery-helpers
branch
from
69abd69 to
e2a43cd
Compare
vezenovm
requested review from
IlyasRidhuan,
LeilaWang,
MirandaWood,
jeanmon and
sirasistant
as code owners
vezenovm
force-pushed
the
mv/f-670-constrained-delivery
branch
from
0afb5db to
11e7eff
Compare
vezenovm
removed request for
IlyasRidhuan,
LeilaWang,
MirandaWood,
jeanmon and
sirasistant
vezenovm
added a commit
that referenced
this pull request
Jun 10, 2026
) Fixes [F-697](https://linear.app/aztec-labs/issue/F-697/aztec-nr-extend-the-onchaindelivery-builder-for-secret-origin) Fixes [F-649](https://linear.app/aztec-labs/issue/F-649/fixaztec-nr-skip-note-log-linkage-for-onchain-constrained-delivery-to) ## Summary - Extends `MessageDelivery` with separate typed builders for unconstrained and constrained on-chain delivery. - Adds internal tag-secret derivation config: - offchain: none - onchain unconstrained: address-pair by default, with optional non-interactive handshake selection - onchain constrained: non-interactive handshake - Introduces `OnchainDeliveryMode` as the typed on-chain mode used by tagging-index helpers and the HandshakeRegistry ABI instead of raw `u8` values. - Keeps invalid delivery/tag-derivation combinations rejected at compile time where possible. - Stops linking constrained-tagged note logs to note-hash squashing, so removing a squashed note does not break the recipient tag index chain. - Re-pins the HandshakeRegistry standard contract and updates the generated standard-contract metadata. Constrained tagging is still not fully wired up: the delivery config is validated, but tag emission remains mocked through the existing wallet-derived unconstrained path behind TODO(#14565). Stack: #23875, then this PR, then #23866, then #23867. --------- Co-authored-by: AztecBot <tech@aztec-labs.com> Co-authored-by: Nicolas Chamo <nicolas@chamo.com.ar>
vezenovm
force-pushed
the
mv/f-669-constrained-delivery-helpers
branch
from
6337393 to
c552943
Compare
vezenovm
force-pushed
the
mv/f-670-constrained-delivery
branch
from
cf306d5 to
03a026c
Compare
…nstrained-delivery Propagates the rename of calculate_secret_and_index to resolve_secret_and_index (and the test contract's calculate_and_return to resolve_and_return) into the emit helper, TXE tests, and e2e test.
…nstrained-delivery
The registry source changes from #23938 shift the standard registry's artifact hash, class id, and address. Regenerates the standard-contract data and address pins and repacks the registry entry in pinned-standard-contracts.tar.gz from the merged build.
…nstrained-delivery
…nstrained-delivery
Contributor
Author
|
Closing in favor of #23866 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes F-670
do_private_message_deliveryresolves the handshake secret, emits the per-send chain nullifier, and derives the constrained log tag.compute_constrained_log_tagandemit_nullifier_and_compute_constrained_tag.Stacked on #23865 (F-697) and #23866 (F-669).