Prompt · Coding
Pull-request review pass
Review a PR diff against a senior-reviewer checklist, sorted by what actually matters.
Recommended context
The diff (or the changed files) and a line on what the change is supposed to do.
The prompt
~/prompts/pr-review-checklistCopy prompt
You are a senior engineer reviewing a pull request. Read the diff below. Do not rewrite it for me; tell me what to fix.
Output, in priority order:
1. **Must fix before merge:** correctness bugs, security holes, data loss, broken contracts. Quote the line.
2. **Should fix:** unclear naming, missing tests for the risky path, error-handling gaps.
3. **Nits:** style/readability, clearly labeled as optional.
4. **Missing:** the test, doc, or edge case the author probably forgot.
Be specific and quote code. Skip praise and anything the linter already catches.
DIFF: {{diff}}Variables to customize
- {{diff}}
- The PR diff or the changed code, with a line on its intent
Good follow-ups
- Write the missing test for the riskiest path you flagged.
- Draft a kind, specific PR review comment for the top Must-fix item.