| name | code-review |
| description | Use this skill after completing multiple, complex software development tasks before informing the user that work is complete. |
Code Review After Completing Complex Software Development Tasks
Run a structured review over the changes in scope. Two modes:
- Reviewing work you just completed, or told to fix - review, then apply and verify the fixes.
- Asked only to review - stop at clear, actionable feedback and let the user decide what to act on. Don't start editing.
Workflow
- Spawn parallel sub-agents to critically review the changes, splitting the work by review axis (below) or by area of the codebase.
- Compile their findings into a concise numbered list, each tagged critical/medium/low.
- Verify every finding against the actual code before reporting or acting on it. Sub-agents report false positives, and acting on a phantom issue makes the code worse.
Applying Fixes
Fix mode only. If you were asked only to review, stop at step 3 and report your findings.
- Fix the confirmed issues.
- Re-run the project's lint/test/build pipeline.
- Read each fix to confirm it resolves its finding and didn't break the original task or introduce a new problem. A green pipeline proves mechanical correctness only.
- Stop after this single verification pass; don't recurse into a fresh full review.
If a finding is especially complex or keeps recurring, use the systematic-debugging skill to get to the root cause.
Review Axes
Direct sub-agents to evaluate the changes across these dimensions. The questions below are illustrative, not a fixed checklist. Apply the ones that fit and raise the concerns that actually matter for this codebase's language, domain, and conventions:
- Correctness - Does it do what the task required? Are edge cases (null, empty, boundary) and error paths handled, not just the happy path? Do the tests actually exercise the new behaviour?