You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Phase 4/8: green CI is not verification when CI scope excludes the changed files #46
Phase 4 anchors the self-review rubric on green CI ("without it, the rubric is just opinion"). But when a PR's changed files are outside CI's scope — platform-specific scripts (.ps1, .bat, .command), installer configs, OS-gated shell paths — CI is green while none of the deliverable was tested. A real bug (a batch %errorlevel% delayed-expansion trap) shipped under green CI and was caught only by adversarial hand-review.
Rule to add to the template (Phase 4 self-review + Phase 8 merge note)
If the PR's changed files include code the CI structurally cannot execute (platform-specific scripts/tooling, OS-gated paths), green CI does not constitute verification of those files. State the no-automated-coverage gap explicitly in the PR body and CHANGELOG, lean harder on adversarial hand-review for that code, and recommend a real-platform smoke test before merge. Do not let a green run on the other language imply the script was tested.
Consolidates (close on merge)
example-|-dev-loop#6 — platform-specific scripts have no automated coverage; green CI is misleading
Consolidated upstream from per-loop self-audit findings. drafted by Claude on behalf of Daniel Stephenson
Activity
Record PR #54's Implementations entries in RESEARCH.md (#55)
Apply consolidated template-rule backlog (#36, #39-#51) (#54)
Record PR #54's Implementations entries in RESEARCH.md (#55)