| name | receiving-code-review |
| description | Use when code review feedback has arrived, before implementing the suggestions |
Receiving Code Review
- Read all the feedback before reacting to any of it.
- Restate each item as a technical requirement in your own words. If any item is unclear, stop and ask about it before implementing anything — items are often related, and partial understanding produces the wrong fix.
- Check each item against the codebase. Does it hold here? Does it break something that currently works? Is there a reason the code is the way it is? Does the reviewer have the full context?
- Where a suggestion adds a feature nothing uses, grep for callers first and say what you found. An endpoint nothing calls should be removed, not implemented properly.
- Where a suggestion is wrong, push back with technical reasoning and point at the tests or code that show it. Where it conflicts with a decision your partner already made, raise it with them rather than choosing for them.
- Where you cannot verify a claim, say what you would need to verify it instead of proceeding on assumption.
- Implement in order: things that break or are insecure, then simple fixes, then complex ones. Test each fix on its own and check for regressions.
- Answer correct feedback by stating the fix and where it landed. Skip the agreement and the thanks — the change is the answer.
- If you pushed back and turned out to be wrong, say what you checked and what it showed, then fix it. No apology, no defence of the pushback.
On GitHub, reply inside the comment thread (gh api repos/{owner}/{repo}/pulls/{pr}/comments/{id}/replies), not as a new top-level PR comment.
Terminal state: invoke verification-before-completion.