fix(ci): allow ci-refresh-chonk PRs to run while draft so pinned-input refresh can complete#23756
Closed
AztecBot wants to merge 1 commit into
Closed
fix(ci): allow ci-refresh-chonk PRs to run while draft so pinned-input refresh can complete#23756AztecBot wants to merge 1 commit into
AztecBot wants to merge 1 commit into
Conversation
…t refresh can complete
Collaborator
Author
|
Automatically closing this stale claudebox draft PR (no updates for 5+ days). Re-open if still needed. |
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.
Problem
The Nightly Debug Build (run 26739379231) fails on the test
bbapi_tests ChonkPinnedIvcInputsTest.AllPinnedFlows. The build/compile succeed; the failing flow (CI log) shows:Immediate cause: stale pinned Chonk inputs
The pinned Chonk flows are an S3 tarball keyed by
barretenberg/cpp/scripts/chonk-inputs.hash, last refreshed 2026-05-21 (2ab01bb56fa). Since then, breaking protocol-circuit / contract changes landed onnext— protocol-contract address shift to 1–3 (#23218),multi_call_entrypoint/public_checks/ auth-registry demotions (#23197, #23217, #23106), and several aztec-nr changes — which alter the kernel public-input layout and VKs. The+4public-inputs shift (669→673) and the wholesale precomputed-VK mismatch are consistent with legitimate circuit evolution, so the fix is to refresh the pin (perbarretenberg/cpp/CLAUDE.mdand thechonk-inputsskill).Why it isn't self-healing (the actual fix here)
The nightly already auto-dispatches a ClaudeBox session on failure, which opens a
ci-refresh-chonkPR (e.g. #23751, #23744). The refresh push-back (ci3_success.sh→ci_update_chonk_inputs.sh) runs inside thecijob. But that job is gated:create_propens these refresh PRs as drafts, so thecijob is skipped (confirmed on #23751: the CI3 run was skipped/cancelled in ~9s, head never advanced,chonk-inputs.hashunchanged). The refresh never runs → the nightly stays red → another duplicate draft refresh PR is opened. A loop.Fix
Add
ci-refresh-chonkto the draft escape-hatch on thecijob (same pattern as the existingci-draftlabel). Aci-refresh-chonkPR runsCI_MODE=skipfor the main body anyway (ci3_labels_to_env.sh) and only performs the lightweight regenerate + small-flow smoke-test + hash push-back, so running it on drafts is cheap and is exactly the intent of the label.Follow-up to actually green the nightly
This PR fixes the mechanism but does not itself regenerate the pin. To clear the current failure, an existing refresh PR (#23751, already labeled
ci-refresh-chonk, basenext) needs to run its refresh and be merged. Once this gate change is onnext, re-triggering #23751 (mark ready / re-push) runs the refresh, which pushes the newchonk-inputs.hash; merging it turns the nightly green. The two open duplicates (#23751, #23744) should be deduplicated to one.Created by claudebox · group:
slackbot