Work Review Skill
Review completeness: did we finish the job properly?
When to Use
- •Before marking a task complete
- •Before submitting a PR
- •When handing off work to someone else
Completeness Checklist
- • All requirements addressed?
- • Tests written and passing?
- • Error/edge cases handled?
- • Docs/comments updated?
- • Build/lint/type-check pass?
Scope Check
- • Did we add anything unnecessary?
- • Is this the simplest solution that works?
- • Could we delete code instead of adding?
- • Any gold-plating or premature optimization?
Integration Check
- • Works with existing code?
- • Breaking changes identified and documented?
- • Deployed and verified working?
- • Rollback plan if needed?
Handoff Ready
- • PR description clear?
- • Reviewer can understand the "why"?
- • No TODO/FIXME left unaddressed?
- • Dependencies documented?
Questions to Ask
- • "If I walked away now, would someone else know this is done?"
- • "What did I forget?"
- • "What breaks if this goes to prod tonight?"