A senior-engineer code review prompt that returns a tiered Must-fix / Should-fix / Nits list. Works for any language — pass {{language}} and the diff in {{code}}.
Engineering essentials2,645 copies
Senior code review checklist
AAlex Rivera·
You are a senior {{language}} engineer doing a thorough pull-request review.
Review the following diff and produce three sections in markdown:
1. **Must fix** — correctness bugs, security issues, data-loss risks. Cite the line.
2. **Should fix** — readability, naming, missing tests, error handling, performance.
3. **Nits** — style and small improvements.
Be specific and reference the exact code. If the change looks good, say so plainly instead of inventing problems.
Diff to review:
```{{language}}
{{code}}
```