feat(bb): chonk inputs revisit, smaller default bb.js CRS#23400
Merged
Conversation
ludamad
approved these changes
May 19, 2026
ludamad
changed the title
ludamad
changed the title
4 tasks
Ports the Chonk inputs refresh workflow proposal from https://gist.github.com/ludamad/7fbd491753b17f4f9c94f0c7875de906 Source proposal commit (pre-port): 5fe441d9c2fab3e8410be30dcf94d3e29168e493 Base: AztecProtocol/aztec-packages merge-train/barretenberg @ 4da6ab0
ludamad
added a commit
that referenced
this pull request
May 19, 2026
Follow-up from reviewing #23400. --------- Co-authored-by: AztecBot <tech@aztecprotocol.com>
spalladino
added a commit
that referenced
this pull request
May 27, 2026
This causes Codex sandbox to fail and the apply_patch command to fail. Fix is to remove the symlinks for all the .codex folders, and instead create actual folders with symlinks in their contents. A pre-commit hook checks that all contents are symlinked. > The issue is the tracked symlink: > > yarn-project/.codex -> .claude > > The sandbox is trying to enforce /home/santiago/Projects/aztec-4/yarn-project/.codex as a read-only > path, but yarn-project is also a writable root. Since .codex is a symlink inside that writable root, > bubblewrap refuses to set up the sandbox: > > Fatal error: cannot enforce sandbox read-only path .../.codex > because it crosses writable symlink .../.codex > > So apply_patch is not uniquely broken. I reproduced the same sandbox setup failure with simple > sandboxed commands like pwd and ls. Commands that are already approved or explicitly escalated can > still run because they bypass that sandbox path setup. This issue had been introduced in #23400.
danielntmd
pushed a commit
to danielntmd/aztec-packages
that referenced
this pull request
Jun 4, 2026
BEGIN_COMMIT_OVERRIDE feat(ci): refresh Chonk inputs (AztecProtocol#23400) fix(ci): harden Chonk refresh post-action (AztecProtocol#23419) END_COMMIT_OVERRIDE
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
2 participants
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.
Centralizes pinned Chonk input management and wires PR CI to refresh the input hash via
ci-refresh-chonkor a--ci-refresh-chonkhead-commit marker.The refresh path skips the main CI body, regenerates/uploads the pinned inputs, proves one small refreshed flow in C++ and bb.js, then pushes only
barretenberg/cpp/scripts/chonk-inputs.hashback to the PR.Also makes Chonk tests/benchmarks consume the pinned inputs directly, lowers the default bb.js CRS size to
2 ** 18, and makesAGENTS.mda symlink toCLAUDE.md.