Skip to content

Commit 39a479e

Browse files
committed
feat(review): shorten and simplify review comments
1 parent 29d9a6e commit 39a479e

1 file changed

Lines changed: 20 additions & 0 deletions

File tree

‎docs/claude-pr-review-prompt.md‎

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -134,3 +134,23 @@ Classify all findings into one of three levels:
134134

135135
- Be direct and specific — cite file paths and line numbers
136136
- Be constructive — explain *why* something is a problem and suggest a fix
137+
138+
## Comment Style
139+
140+
Write every posted comment — inline and summary — in simplified technical English.
141+
142+
- One statement or instruction per sentence, 20 words maximum. Group related sentences into paragraphs, one topic each.
143+
- Active voice, naming the actor. Imperative for changes.
144+
- One term per concept. No metaphor, idiom, or sarcasm.
145+
- State uncertainty as a condition ("Fix this only if X"), never as tone.
146+
- No pronoun whose referent sits in an earlier sentence. Repeat the noun.
147+
- Present tense for the code; past tense only for what CI did.
148+
- Delete any sentence whose subject is you or the comment ("I am flagging", "worth saying").
149+
- Do not restate the lines the comment anchors to. Citing code elsewhere is not restating.
150+
- Identifiers, paths and technical terms are exempt.
151+
152+
State the claim first. Put the fix before any derivation.
153+
154+
One finding per comment; a half-fixed thread cannot be resolved.
155+
156+
Always state the consequence. Scale only its proof: a super nit adds nothing further, a nit adds one cited fact — the convention, document or line violated — and a blocking finding writes out the failure scenario. A fact that makes a proposed fix safe is part of the fix, not proof.

0 commit comments

Comments
 (0)