docs(brief): scope Success criterion to what C4-verify shows at Check - #76
Merged
Conversation
…#74) The Success criterion is the load-bearing field Check tests "did this work" against. A criterion scoped to a T3 whole-suite pass or a fork-CI green can't be shown at Check time — those are advisory/baseline-red and only clear post-merge — so the bundle either can't satisfy its own criterion or the human rubber-stamps it. Anchor the field guidance to the per-fix C4-verify (red→green on the patch in isolation), which is demonstrable at the moment Check runs. Template-side so it lands for all instances rather than diverging instance-side. Closes #74. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: Eduard Ralph <15236434+eduralph@users.noreply.github.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
1 participant
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.
Summary
Closes #74. Tightens the
Success criterionfield guidance intemplates/brief.md.tplso a planner doesn't scope the criterion to evidence that only appears after merge.The criterion is the load-bearing field Check tests "did this work" against. Scoped to a T3 whole-suite pass or a fork-CI green, it can't be shown at Check time (those gates are advisory/baseline-red and only clear post-merge) — so the bundle either can't satisfy its own criterion or the human rubber-stamps it. Anchoring it to the per-fix C4-verify (red→green on the patch in isolation) keeps it demonstrable at the moment Check runs.
Template-side so it lands for all instances rather than diverging instance-side (surfaced in gramps-testbed-v2).
Change
One field's guidance in
templates/brief.md.tpl— no code. Offline suite green (137 tests).🤖 Generated with Claude Code