Hard Limits (MANDATORY)
| Limit | Value | Why |
|---|---|---|
| Max tasks per review | 5 | Forces prioritization |
| Max P2/P3 addressed | 2 | Focus on real bugs |
| Max criteria per task | 3 | Essential tests only |
15 issues? CHOOSE THE 5 THAT MATTER.
Finding Categories
Classify each finding BEFORE planning:
| Category | Action |
|---|---|
| DELETE | Remove the code |
| SIMPLIFY | Inline, flatten, reduce |
| FIX | Minimal targeted fix |
| SKIP | Don't create task |
Waste Pattern Detection:
- •"Add validation for..." → Delete the code path
- •"Add error handling for..." → Simplify so errors can't happen
- •"Add tests for edge cases..." → Remove the edge cases
- •"Refactor for readability..." → Delete the abstraction
3-Phase Process
Phase 1: Parse and Filter
For EACH finding:
- •"Would deleting the code be better than fixing it?"
- •"Is this pointing at a waste pattern?"
- •"What real bug slips through if we ignore this?"
Classify as DELETE / SIMPLIFY / FIX / SKIP.
Phase 2: Group and Size
| Size | Scope |
|---|---|
| 30 min | Single file, <20 lines |
| 1 hr | Single file, 20-50 lines |
| 1-2 hr | 2-3 files, <100 lines |
Split if >2hr. Combine if <30min.
Phase 3: Task Specification
markdown
### Task: {Title}
**Type:** DELETE | SIMPLIFY | FIX
**Files:** file:line, file:line
**What:** One sentence
**Why:** Bug prevented OR waste removed
**Criteria (max 3):**
- [ ] Criterion 1
- [ ] Criterion 2
- [ ] Tests pass
Output Format
After generating task specifications, output in spec YAML format for ft epic decompose:
markdown
# Review Fix Plan: {epic}
## Summary
- Total findings: N
- DELETE: X | SIMPLIFY: Y | FIX: Z | SKIP: W
## Tasks (max 5)
### 1. {Title} [DELETE|SIMPLIFY|FIX]
**Files:** file:line
**What:** One sentence
**Why:** Bug prevented OR waste removed
**Criteria:**
- [ ] ...
Find the Stupid
| Stupid | Why |
|---|---|
| Task for every finding | 15 tasks = nothing gets done |
| Patching bad code | Delete > Fix |
| 8 acceptance criteria | Gold plating |
| P2 tasks before P0 done | Priority inversion |