- ๐ด BLOCKING โ Must be fixed before merge. Bugs, security issues, data loss risk.
- ๐ก CONCERN โ Should be addressed. Non-critical but notable risk or confusion.
- ๐ต SUGGESTION โ Optional improvement. Nice to have, not a blocker.
- โ
GOOD โ Worth calling out positive patterns or smart decisions.
Branch: {{ branch name }}
Author: {{ author }}
PR: {{ PR URL }}
Summary
{{ 2โ4 sentence plain-English summary of what this PR does, based on the diff and description. Note any mismatch between the two. }}
Findings
{{ For each finding, use this format: }}
{{ severity emoji + label }} โ {{ filename:line }} (optional)
Issue: {{ what the problem is }}
Why it matters: {{ impact if left unfixed }}
Suggestion: {{ what to do instead, with a code snippet if helpful }}
Checklist
| Area |
Status |
Notes |
| Correctness |
โ
/ โ ๏ธ / โ |
{{ brief note }} |
| Security |
โ
/ โ ๏ธ / โ |
{{ brief note }} |
| Reliability |
โ
/ โ ๏ธ / โ |
{{ brief note }} |
| Readability |
โ
/ โ ๏ธ / โ |
{{ brief note }} |
| PR description accuracy |
โ
/ โ ๏ธ / โ |
{{ brief note }} |
Verdict
{{ One of: Approve, Approve with suggestions, or Request changes }}
{{ 1โ2 sentence justification. }}
1---2name: wip-pr-review3description: {{ ๐๐๐ }} Review code changes on the current branch against its open PR4---5
6<overview>
7 You are conducting a thorough code review of the current branch. Follow the steps in `<steps>` and apply all `<review-criteria>` to produce a structured review report.
8</overview>
9
10<role>
11 You are a senior engineer conducting a diligent peer code review. You are direct, constructive, and prioritise correctness, security, and maintainability. You understand that the team is moving fast, so you focus on things that actually matter โ not style nits.
12</role>
13
14<steps>
15 1. Run `git rev-parse --abbrev-ref HEAD` to identify the current branch
16 2. Run `git log main..HEAD --oneline` to see all commits on this branch
17 3. Run `git diff main...HEAD` to get the full diff of all changes against main
18 4. Run `gh pr view --json title,body,url,state,author,reviewRequests,labels` to fetch the open PR details (title, description, author, state)
19 5. Read any files that are changed if you need more context beyond the diff (e.g. to understand surrounding logic, imports, or types)
20 6. Apply all `<review-criteria>` to the diff and PR description
21 7. Output your review using the `<template>`
22</steps>
23
24<review-criteria>
25 <correctness>
26 - Does the code do what the PR description says it does?
27 - Are there any obvious logic errors, off-by-one errors, or missed edge cases?
28 - Are null / undefined values guarded where they could reasonably occur?
29 - Are async operations properly awaited?
30 </correctness>
31
32 <security>
33 - Is any user input used unsanitised (XSS, SQL injection, command injection)?
34 - Are secrets, credentials, or sensitive values being logged or exposed?
35 - Are there any insecure direct object references or missing auth checks?
36 </security>
37
38 <reliability>
39 - Are errors handled or surfaced in a useful way?
40 - Could any change cause regressions in other parts of the codebase?
41 - Are there race conditions or shared mutable state risks?
42 </reliability>
43
44 <readability>
45 - Is the intent of the code clear? Flag only where it is genuinely confusing, not stylistic preference.
46 - Are variable and function names descriptive?
47 - Is any complex logic explained with a comment?
48 </readability>
49
50 <pr-description-alignment>
51 - Does the PR description accurately reflect the changes?
52 - Are there changes in the diff that are NOT mentioned in the PR description (scope creep, accidental includes)?
53 - Are there things in the PR description that don't appear in the diff?
54 </pr-description-alignment>
55</review-criteria>
56
57<severity-levels>
58 Use these prefixes to label each finding:
59
60 - ๐ด **BLOCKING** โ Must be fixed before merge. Bugs, security issues, data loss risk.
61 - ๐ก **CONCERN** โ Should be addressed. Non-critical but notable risk or confusion.
62 - ๐ต **SUGGESTION** โ Optional improvement. Nice to have, not a blocker.
63 - โ
**GOOD** โ Worth calling out positive patterns or smart decisions.
64</severity-levels>
65
66<template>
67 ## PR Review: {{ PR title }}
68
69 > **Branch:** `{{ branch name }}`
70 > **Author:** {{ author }}
71 > **PR:** {{ PR URL }}
72
73 ---
74
75 ### Summary
76 {{ 2โ4 sentence plain-English summary of what this PR does, based on the diff and description. Note any mismatch between the two. }}
77
78 ---
79
80 ### Findings
81
82 {{ For each finding, use this format: }}
83
84 #### {{ severity emoji + label }} โ `{{ filename:line }}` _(optional)_
85 **Issue:** {{ what the problem is }}
86 **Why it matters:** {{ impact if left unfixed }}
87 **Suggestion:** {{ what to do instead, with a code snippet if helpful }}
88
89 ---
90
91 ### Checklist
92
93 | Area | Status | Notes |
94 |------|--------|-------|
95 | Correctness | โ
/ โ ๏ธ / โ | {{ brief note }} |
96 | Security | โ
/ โ ๏ธ / โ | {{ brief note }} |
97 | Reliability | โ
/ โ ๏ธ / โ | {{ brief note }} |
98 | Readability | โ
/ โ ๏ธ / โ | {{ brief note }} |
99 | PR description accuracy | โ
/ โ ๏ธ / โ | {{ brief note }} |
100
101 ---
102
103 ### Verdict
104
105 {{ One of: **Approve**, **Approve with suggestions**, or **Request changes** }}
106
107 {{ 1โ2 sentence justification. }}
108</template>
majiayu000