docs: fix aztec-nr v4.2.0 documentation issues from audit#22166
Open
critesjosh wants to merge 2 commits intonextfrom
Open
docs: fix aztec-nr v4.2.0 documentation issues from audit#22166critesjosh wants to merge 2 commits intonextfrom
critesjosh wants to merge 2 commits intonextfrom
Conversation
added 2 commits
March 30, 2026 16:45
Fix 52 issues across all aztec-nr doc pages identified during a manual review against nargo 1.0.0-beta.18 and local Aztec network v4.2.0-aztecnr-rc.2. Key fixes: - Update deprecated import paths (dep::aztec:: -> aztec::, protocol_types -> protocol) - Add missing scope parameter to capsule API examples - Correct partial note commitment structure (storage_slot moves to completion phase) - Replace non-existent methods (simulate_utility -> execute_utility, msg_sender().unwrap() -> msg_sender()) - Fix NoteMessagePendingDelivery -> NoteMessage struct name - Fix out-of-bounds array access in sort_efficient example - Correct dependency URLs from aztec-packages to aztec-nr repo - Fix ONCHAIN_CONSTRAINED link target that incorrectly pointed to ONCHAIN_UNCONSTRAINED - Remove stale TODO comments, placeholder images, and delegate_call references - Update pedersen -> poseidon2, return_values BoundedVec -> return_hash Field
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
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.
Summary
nargo 1.0.0-beta.18and local Aztec networkv4.2.0-aztecnr-rc.2docs-developers/(source) anddeveloper_versioned_docs/version-v4.2.0-aztecnr-rc.2/(versioned copy)scope), non-existent methods (simulate_utility,msg_sender().unwrap()), incorrect struct names, out-of-bounds array access in code examples, wrong dependency URLs, stale content removalCritical fixes (would cause compilation failures if copied from docs)
scopeparameter to all capsule API examplesCapsuleArray::at()call signature (2 args → 3)sort_efficientexampledebug_logimport path (dep::aztec::oracle::debug_log→aztec::oracle::logging)simulate_utility()withexecute_utility()NoteMessagePendingDelivery→NoteMessagestorage_slotin completion phase, not partial phase)Other fixes
dep::import prefixes to current unprefixed styleaztec-packagesrepo toaztec-nrrepoONCHAIN_CONSTRAINEDlink that pointed toONCHAIN_UNCONSTRAINEDreturn_values: BoundedVec→return_hash: FieldTWO_POW_32→TWO_POW_16)unconstrainedkeyword to utility function examples#[derive]attributes onNFTNotein AIP-721Test plan
yarn buildin docs/)v4.2.0-aztecnr-rc.2tag