Purpose
Define the common review workflow shared by code-review, design-review, and unit-test skills across all platforms.
Severity definitions
| Severity | Meaning |
|---|---|
| must-fix | Security vulnerability, broken functionality, blocks deployment or App Store submission |
| should-fix | Standards violation, maintainability concern, tech debt |
| nice-to-have | Style preference, minor improvement, optional optimization |
Approval gate
If issues are found and --no-fix is not set:
"Found X issues (Y must-fix). Approve fixes? (yes/no)"
Do not modify code before approval.
Fix and confirm (if approved)
- •Apply fixes
- •Re-run automated gates (lint, format, typecheck)
- •Run unit tests to confirm no regressions
- •Report final status
Fix constraints
- •Don't add dependencies unless required
- •Don't weaken assertions to make tests pass
- •Prefer targeted fixes over broad refactors
- •Don't disable or skip tests to make them pass