Repository navigation
feat(init): --refresh-baseline proposes a requiredChecks-only Formula change (#224) - #225
Merged
Merged
Conversation
… change for review (#224) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Closes #224.
Adds
forgecell init --refresh-baseline [--lab DIR]. It re-derives the CI baseline from the current pull-request workflows and saves a pending proposal that changes only the top-levelrequiredChecksof the approved Formula. Review and approval go through the existinginit --review/--approve/--dismiss IDflow, which binds the exact saved bytes. It mirrors--rebind:maskBaselineKeysandverifyBaselineDiffrefuse a proposal that changes any other key.verifyBaselineExactrequires the proposal to equal the approved Formula plus the derived checks.--rebind: refused while a proposal is pending, without an approved Formula, or without a current exact approval. It conflicts with--harness,--rebind,--meta-*,--model,--role, the timeout flags and--review/--approve/--dismiss.This unblocks the approved Formula update: baseline
go test -timeout 20m ./...at 1500000.Built by Forgecell Molecule
mol-224-37b068e838f267e806d6363d(planf58ca943, base00d8fe3). Its targeted onboarding+cli checks passed. Hand-delivered: the candidate's full-suite baseline failed only on Go's 10-minute timeout in the unrelatedevaluationpackage under parallel load, the issue this wave fixes.Review: one independent adversarial pass found no HIGH or correctness-MEDIUM issues. It checked the splice range against
lastLine, comment, trailing-content and multi-document cases (all refused safely or preserved), the guard parity, and that the note leaks nothing.Local clean-shell suite: gofmt and vet clean.
go test -timeout 25m ./...hit one known #220 flake (TestNativeBindingPassesAllowlistToClaudeArgv, a 10 s wall-clock budget inmolecule, a package this PR doesn't touch) while #220's-count=5load checks were running. CI is the gate.LOW (not fixed): the appended-key path trims trailing blank lines (formatting only), and the baseline reads the current working tree, as bootstrap does (the note's revision and SHA make that visible).
🤖 Generated with Claude Code