Skip to content

docs: add map and state variable docs (backport #22824)#22880

Merged
nventuro merged 1 commit intobackport-to-v4-next-stagingfrom
claudebox/backport-22824-v4-next
May 4, 2026
Merged

docs: add map and state variable docs (backport #22824)#22880
nventuro merged 1 commit intobackport-to-v4-next-stagingfrom
claudebox/backport-22824-v4-next

Conversation

@AztecBot
Copy link
Copy Markdown
Collaborator

Summary

Backport of #22824 to v4-next.

The PR is purely a docs update to aztec-nr/aztec/src/state_vars/:

  • expanded Map documentation
  • new module-level docs in state_vars/mod.nr (intro, public/private state variable comparison tables, storage slots, Context parameter)
  • minor doc tweaks in public_mutable.nr and state_variable.nr

Conflict resolution

Three test files conflicted: propagated_note_hashes_tests.nr, propagated_nullifiers_tests.nr, propagated_private_logs_tests.nr.

The conflicts were incidental adjacent-line drift (a use super::rebuild_bounded_vec; import and a couple of reformatted BoundedVec.pop() rebuild call sites) that belong to a separate BoundedVec.pop() change which has not yet been backported to v4-next. I kept the v4-next (HEAD) state in those three files so this PR contains only the docs changes that PR #22824 was actually about.

Full analysis: https://gist.github.com/AztecBot/36b70c2ad5fdfe6a9e4101ec56230cc1

ClaudeBox log: https://claudebox.work/s/126c3faff0c2af32?run=4

These contain a basic explainer on some state variable concepts, add
some missing bits there (like a table with different kinds of statevars
and their usecases), and greatly expands on (completes) `Map`
documentation.
@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels Apr 30, 2026
@nventuro nventuro marked this pull request as ready for review May 4, 2026 18:11
@nventuro nventuro self-requested a review as a code owner May 4, 2026 18:11
@nventuro nventuro merged commit 562d5e3 into backport-to-v4-next-staging May 4, 2026
33 of 38 checks passed
@nventuro nventuro deleted the claudebox/backport-22824-v4-next branch May 4, 2026 18:11
AztecBot added a commit that referenced this pull request May 5, 2026
BEGIN_COMMIT_OVERRIDE
docs: add map and state variable docs  (#22824)
fix: e2e compat should not fail for contracts added after legacy stables
(#22900)
chore: fix kv-store browser tests hangs (#22721)
feat: kv-store sqlite backend with page level encryption (#22759)
fix: install node 22 for aztec-cli acceptance test (#22917)
feat: backport kv-store sqlite encryption (#22759) to v4-next (#22927)
fix(docs): correct llms.txt links for versioned developer docs (#22819)
feat(docs): improve discoverability of Aztec.nr API reference docs
(#22861)
feat(docs): backport improve discoverability of Aztec.nr API reference
docs (#22861) to v4-next (#22931)
feat(aztec-nr): add call_self stubs for utility functions (#22885)
docs: add map and state variable docs (backport #22824) (#22880)
refactor: `getPackageVersion` fn cleanup (#22941)
fix(ci): skip acceptance test for canary -commit. tags (#22951)
fix: closing db, correct stub side effects (#22939)
END_COMMIT_OVERRIDE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants