diff --git a/.github/workflows/claude-pr-review.yml b/.github/workflows/claude-pr-review.yml index 52821bae9..0d9ba4ab0 100644 --- a/.github/workflows/claude-pr-review.yml +++ b/.github/workflows/claude-pr-review.yml @@ -162,9 +162,9 @@ jobs: what is HIGH in this repository, the checks to always run, what not to report, the verification bar, the volume cap and the shape of the comment. It also settles the one thing a finding never carries: the - fix. Name where the fix belongs, never what it is - no patch, no - snippet, no suggestion block, no rewrite in prose. The maintainer - decides the change. + fix. Not what it is and not where it belongs - no patch, no snippet, + no suggestion block, no rewrite in prose, no "The fix belongs in" + line. Stop at what breaks. The maintainer decides the change. WHAT IS CHECKED OUT WHERE The working tree is the BASE branch. The head under review, diff --git a/REVIEW.md b/REVIEW.md index 1952a6059..a3bbc90c4 100644 --- a/REVIEW.md +++ b/REVIEW.md @@ -202,9 +202,11 @@ evidence, not a retelling of the pull request. A finding says what is wrong, where (`file:line`), what triggers it and what breaks. It never carries the fix: no `suggestion` block, no patch, no replacement snippet, no rewritten function, no "suggested fix" section — in -the summary and in an inline comment alike. One clause naming WHERE the fix -belongs is the most it may add — a file, a function, a symbol, a layer — and -nothing about what happens there. Prose is a patch too the moment a verb +the summary and in an inline comment alike. It does not say where the fix +belongs either: no closing "The fix belongs in …" line. The `file:line` +already locates the defect, and a location set beside the missing piece the +finding just named — "the fix belongs in the capability set" after naming the +two capabilities it lacks — is the fix. Prose is a patch too the moment a verb describes the change: "move the lookup inside the body", "spend the comment on the invariant instead" hand it over as surely as a diff would, and so does holding up an existing symbol as the model to copy. A clause the maintainer