Cursor rule

1---
2description: "Act on a reviewer agent's feedback systematically — categorize findings, apply fixes, verify tests still pass. Use after request-review returns a report, or when user wants to work through code review findings."
3alwaysApply: false
4---
5 
6# Respond Review
7> **HARD GATE** — **HARD GATE** — Every reviewer comment must be addressed (fix, disagree + document reason, or ask clarification). Do NOT ignore feedback and merge.
8 
9 
10Work through reviewer findings systematically. Don't apply changes blindly — categorize first, then decide, then fix, then verify.
11 
12## Process
13 
14### 1. Read the full review report
15 
16Read every finding before acting on any of them. Get the full picture first.
17 
18### 2. Categorize findings
19 
20For each finding, assign a category:
21 
22| Category | Meaning | Action |
23|----------|---------|--------|
24| **must-fix** | Correctness bug, security issue, test failure, CONVENTIONS.md violation | Fix before proceeding |
25| **should-fix** | Code quality issue, naming, clarity — worth fixing but not blocking | Fix if time allows |
26| **consider** | Architectural suggestion, alternative approach — may or may not apply | Discuss with user |
27 
28Create a numbered list of all findings with their categories.
29 
30### 3. Confirm with user (for consider-category items)
31 
32For each "consider" item, briefly describe the trade-off and ask: "Apply, skip, or discuss?"
33 
34### 4. Apply must-fix items first
35 
36Fix every must-fix item. For each one:
37- Describe what you're changing and why
38- Make the change
39- Run the verify command if one exists for this area
40 
41### 5. Apply should-fix items
42 
43Apply should-fix items. If any are large enough to warrant their own commit, note them separately.
44 
45### 6. Run the full suite
46 
47After all changes are applied:
48 
49```bash
50<full test command>
51<typecheck command>
52<lint command>
53```
54 
55- [ ] All tests pass
56- [ ] No type errors
57- [ ] No lint violations
58 
59### 7. Report
60 
61Summarize what was applied and what was skipped:
62 
63```
64Applied (must-fix): #1, #2, #3
65Applied (should-fix): #4
66Skipped (consider): #5 — agreed with user to defer
67All tests pass.
68```
69 
70Suggest next skill: `commit-message`.
71