Code Review Skill
Purpose: Perform automated code review against project standards.
Trigger
When: PR review requested OR user asks for review
Context Needed: Changed files, diff content, PR description
MCP Tools: get_errors, grep_search, read_file
Checks
TypeScript
- • No
anytypes - • Explicit return types on public methods
- • No unused imports
- • Proper error handling
Architecture
- • Controllers are thin
- • Business logic in services
- • DTOs for all inputs
Angular (if frontend)
- • Standalone components
- • OnPush change detection
- • Signals for state
- • @if/@for control flow
Testing
- • Tests for new code
- • No skipped tests
- • Coverage maintained
Documentation
- • Updated if needed
- • Correct template used
Severity
| Level | Action |
|---|---|
| 🔴 Error | Must fix |
| 🟡 Warning | Should fix |
| 🔵 Info | Consider |