Skip to content

Commit 1556c41

Browse files
igerberclaude
andcommitted
Address PR #415 R4 review (1 P3 directional wording)
R4 P3 — The new claim-vs-shipped audit (in the Single-Pass Completeness Mandate at lines 100-125) said "per the Deferred Work Acceptance section above", but that section is at lines 95-110 — BELOW the Mandate, not above. Trivial directional fix: above -> below in both pr_review.md and the matching substitution old-half in openai_review.py. R4 verdict was already ✅ (P3 doesn't block under the new rules); this commit is cosmetic cleanup so the remaining nit is gone. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent ff43bd2 commit 1556c41

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

.claude/scripts/openai_review.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -996,7 +996,7 @@ def estimate_cost(
996996
claim of correctness) or **P1** (missing assumption check).
997997
- **Tests**: a behavioral regression test exists for the claimed behavior.
998998
Missing test for shipped behavior is **P2** per the deferral rule
999-
(per the Deferred Work Acceptance section above) — TODO.md tracking does
999+
(per the Deferred Work Acceptance section below) — TODO.md tracking does
10001000
NOT downgrade this.
10011001
- **Public docstrings**: affected method/class docstrings mention the new
10021002
behavior (parameters, return-shape additions, side effects). Missing is

.github/codex/prompts/pr_review.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ Before finalizing, confirm you have run each of these audits on the diff:
103103
claim of correctness) or **P1** (missing assumption check).
104104
- **Tests**: a behavioral regression test exists for the claimed behavior.
105105
Missing test for shipped behavior is **P2** per the deferral rule
106-
(per the Deferred Work Acceptance section above) — TODO.md tracking does
106+
(per the Deferred Work Acceptance section below) — TODO.md tracking does
107107
NOT downgrade this.
108108
- **Public docstrings**: affected method/class docstrings mention the new
109109
behavior (parameters, return-shape additions, side effects). Missing is

0 commit comments

Comments
 (0)