review
Review changes for architecture violations, clean code, and security
SKILL.md.claude/skills/review/SKILL.md
GitHubCode Review
Context
!git status --short
!git diff --stat
!git diff --cached --stat
Scope
staged→git diff --cachedbranch→git diff main...HEAD- a path → read the file directly
- empty → whichever of staged / unstaged / branch diff has content
Instructions
Delegate to the clean-code-reviewer agent with the scope above. It holds the review criteria and reads .claude/rules/* itself — do not restate the criteria here.
For a diff touching module boundaries (a new module, a moved class, a cross-module import, a changed Facade signature), also delegate to hexagonal-architect and merge both reports.
Before reporting
Verify each finding against the actual file — a diff hunk lacks the surrounding context that decides whether something is really wrong. Drop anything you cannot confirm.
Output
Merge into one list, most severe first:
**[SEVERITY]** file:line — Category
What's wrong. What to do instead.
CRITICAL (security, data loss, broken behavior) · MAJOR (architecture violation, real smell) · MINOR · SUGGESTION
End with: verdict (approve / request changes), counts by severity, and the top items to fix first.