chore: refresh pinned Chonk IVC inputs (fix nightly debug build)#23981
Draft
AztecBot wants to merge 2 commits into
Draft
chore: refresh pinned Chonk IVC inputs (fix nightly debug build)#23981AztecBot wants to merge 2 commits into
AztecBot wants to merge 2 commits into
Conversation
Generated by ci-refresh-chonk. Only the pinned Chonk input hash is committed here; the immediate follow-up CI run is skipped intentionally. --ci-skip
Collaborator
Author
|
Pinned Chonk inputs refreshed to |
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.
Nightly Debug Build failure — root cause and fix
The Nightly Debug Build on
next(7e94c2c) failed inbbapi_tests ChonkPinnedIvcInputsTest.AllPinnedFlows:Root cause
The pinned Chonk IVC inputs (
barretenberg/cpp/chonk-pinned-flows, pinned bybarretenberg/cpp/scripts/chonk-inputs.hash=209dde8e69a27c9f) are stale. The pin was last bumped on 2026-05-21 (2ab01bb56fa). Since then, 10 commits changednoir-projects/noir-protocol-circuits, several of them breaking protocol-circuit changes (e.g.feat!: add message signing and fallback keys#23510,feat!: shift remaining protocol-contract addresses to 1-3#23218,feat!: demote auth registry to non-protocol contract#23106). These changed the kernel/app circuits and their VKs, so the recorded databus calldata in the pinned proofs no longer lines up with what the current circuits produce.The run log confirms a wholesale VK drift — every selector/sigma reports
Commitment mismatchandDoes the precomputed vk match with the one generated during accumulation? false— followed by the fatalkernel_return_data_matchassertion.Why only the nightly debug build catches it
The consistency check in
barretenberg/cpp/src/barretenberg/chonk/chonk.cppisBB_ASSERT_DEBUG, so it is compiled out in release. Two guardrails that should catch drift do not:chonk_inputs.sh checkrunsbb checkwith--disable_assertsin debug and only fails on a hardVK mismatch detectedstring → did not flag the drift, so it accumulated silently across ~500 commits.The nightly debug build runs the gtest with assertions enabled, which is the only place this drift surfaces.
Fix
This PR carries the
ci-refresh-chonklabel, the documented mechanism (seebarretenberg/cpp/CLAUDE.md). On this PR's CI, the chonk-input-update post-action runsbarretenberg/cpp/scripts/ci_update_chonk_inputs.sh, which:barretenberg/cpp/scripts/chonk-inputs.hash,ChonkPinnedIvcInputsTest.AllPinnedFlows(native + bb.js),The committed source change is a comment on the test documenting that it is the canary for pinned-input drift, that it only fails in debug, and how to refresh — so the next person who hits this in nightly finds the fix immediately.
Verification
The refresh + the test re-run is performed by this PR's CI (it requires the build/capture infra and S3 artifact upload, so it cannot be reproduced in the investigation container). Do not merge until CI has pushed the regenerated
chonk-inputs.hashand the greenChonkPinnedIvcInputsTest.AllPinnedFlows.Created by claudebox · group:
slackbot