Repository navigation
Ratchet SG-0 CI discipline wiring - #2599
Merged
Merged
Conversation
briansrls
marked this pull request as ready for review
May 10, 2026 15:33
briansrls
commented
May 10, 2026
briansrls
left a comment
Contributor
Author
There was a problem hiding this comment.
Review metadata
- Provider / model:
codex/unknown - Commit:
6befbfe0· Trigger:schedule - Thinking:
175s wall
Non-blocking — Strengths
scripts/check-pr-sg0-net-shrink-discipline.shThe added CI-wiring assertion is implementation-only, fail-closes on missing SG-0 discipline steps, and matches the P5 SG-0 PR-body gate intent.
✅ No blocking concerns; the reconstructed PR-head script self-test passes.
Contributor
Author
|
Manager read: #2599 is not a duplicate of the merged gate-closure receipt #2588. #2588 remains the canonical gate #75 closure artifact ( I verified locally on the PR branch:
All passed. Treat this as optional/beneficial hardening, not as blocking gate #75 closure. — sent from silent-ram-834 |
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.
Summary
Strengthens R3 gate #75 (
pr_anticipation_discipline_ci_active) by makingscripts/check-pr-sg0-net-shrink-discipline.sh --self-testverify the workflow wiring that makes the gate active in CI.The self-test now fails if
.github/workflows/ci.ymlstops running:Per-PR debt-paydown receipt (required for all PRs)
pr_anticipation_discipline_ci_activeindocs/r3-program-plan.md§1 gate table /docs/r3-structure.mdPR-authoring-discipline gate; retirement mechanism is a same-script CI-wiring receipt inscripts/check-pr-sg0-net-shrink-discipline.sh --self-test, so local and CI self-test coverage now fail closed if the workflow stops enforcing the PR-authoring contract.Test plan
bash -n scripts/check-pr-sg0-net-shrink-discipline.shbash scripts/check-pr-sg0-net-shrink-discipline.sh --self-testgit diff --check