review-findings

# Review Findings Addresses and fixes QA findings using the Forja (Dev) agent. ## When to Use This Skill - After receiving a code review from Centinela (QA) - When a review report has findings that need to be addressed - Fixing critical and warning issues before re-verification ## What This Skill Does 1. Runs the SIGN IN checklist 2. Reads the review report and understands each finding's root cause 3. Plans fix order: Critical first, then Warnings 4. Implements fixes with updated tests for each finding 5. Scans for dead code after all fixes 6. Runs Implementation Complete and Pre-Delivery checklists (TIME OUT) 7. Verifies every Critical finding addressed, every Warning addressed or deferred with justification 8. Prepares a fix report for QA re-verification ## How to Use ### Basic Usage ``` /review-findings ``` ### With Specific Review ``` /review-findings docs/reviews/user-auth-review.md ``` ## Example **User**: `/review-findings docs/reviews/webhook-system-review.md` **Output**: - All Critical findings fixed with tests - All Warning findings fixed or explicitly deferred with justification - Fix report documenting what was changed and why - Ready for QA re-verification ## Tips - If no review is specified, the most recent review in `docs/reviews/` is used - The agent understands root causes before writing any fix - Conflicting fixes are identified and planned around during the pre-fix phase