review-findings

1# Review Findings
2 
3Addresses and fixes QA findings using the Forja (Dev) agent.
4 
5## When to Use This Skill
6 
7- After receiving a code review from Centinela (QA)
8- When a review report has findings that need to be addressed
9- Fixing critical and warning issues before re-verification
10 
11## What This Skill Does
12 
131. Runs the SIGN IN checklist
142. Reads the review report and understands each finding's root cause
153. Plans fix order: Critical first, then Warnings
164. Implements fixes with updated tests for each finding
175. Scans for dead code after all fixes
186. Runs Implementation Complete and Pre-Delivery checklists (TIME OUT)
197. Verifies every Critical finding addressed, every Warning addressed or deferred with justification
208. Prepares a fix report for QA re-verification
21 
22## How to Use
23 
24### Basic Usage
25 
26```
27/review-findings
28```
29 
30### With Specific Review
31 
32```
33/review-findings docs/reviews/user-auth-review.md
34```
35 
36## Example
37 
38**User**: `/review-findings docs/reviews/webhook-system-review.md`
39 
40**Output**:
41- All Critical findings fixed with tests
42- All Warning findings fixed or explicitly deferred with justification
43- Fix report documenting what was changed and why
44- Ready for QA re-verification
45 
46## Tips
47 
48- If no review is specified, the most recent review in `docs/reviews/` is used
49- The agent understands root causes before writing any fix
50- Conflicting fixes are identified and planned around during the pre-fix phase