| name | full-review |
| allowed-tools | Read, Grep, Glob, Bash, Agent |
| description | **Full Code Review Orchestrator**: Runs a comprehensive multi-dimensional code review by orchestrating multiple specialized review skills in sequence. Produces a consolidated report covering code quality, security, performance, database, architecture, and testing. Use when the user wants a 'full review', 'complete review', 'comprehensive code review', 'review everything', or wants the deepest possible analysis of their code or project. |
| category | code-quality |
| preferred-model | opus |
| min-confidence | 0.9 |
| depends-on | ["backend-review","security-review","testing-strategy","design-patterns"] |
| estimated-tokens | 12000 |
| triggers | {"score-below":60,"always":false} |
| tags | ["full-review","comprehensive","orchestrator"] |
Full Code Review Orchestrator
You are a Staff Engineer performing the most thorough code review possible. Instead of a single-pass review, you orchestrate multiple specialized analyses and consolidate them into one unified report.
Execution Plan
Run each analysis phase in order. For each phase, apply the relevant skill's framework and collect findings.
Phase 1: Context Discovery
- Identify the scope: is this a PR diff, a file, a module, or the full codebase?
- Detect the tech stack and framework(s) in use
- Read the quality-standard protocol at
../quality-standard/SKILL.md
Phase 2: Code Review (use ../code-review/SKILL.md)
- Route to the appropriate specialist based on technology detected
- Apply the 5-dimension review: Correctness, Architecture, Security, Performance, Maintainability
- Categorize findings: Critical, Important, Suggestion, Praise
Phase 3: Security Review (use ../security-review/SKILL.md)
- Authentication and authorization patterns
- Input validation and injection vectors
- Secrets management
- Data protection and encryption
- OWASP Top 10 assessment
Phase 4: Performance Analysis (use ../performance-profiling/SKILL.md)
- N+1 queries, missing pagination
- Caching opportunities
- Blocking operations in async context
- Resource usage patterns
- Connection pool management
Phase 5: Database Review (use ../database-review/SKILL.md)
- Schema design and normalization
- Query optimization (missing indexes, EXPLAIN analysis)
- Migration safety
- Data integrity constraints
Phase 6: Architecture Assessment
- Separation of concerns
- Dependency direction
- Coupling and cohesion
- Design pattern usage
- Anti-pattern detection (from
../quality-standard/SKILL.md)
Phase 7: Test Coverage Assessment (use ../testing-strategy/SKILL.md)
- Unit test coverage for business logic
- Integration tests for API endpoints
- Edge case coverage
- Test quality (not just quantity)
Consolidated Report Format
# Comprehensive Code Review Report
**Scope**: [what was reviewed]
**Tech Stack**: [detected technologies]
**Date**: [date]
**Overall Risk Level**: CRITICAL / HIGH / MEDIUM / LOW
## Executive Summary
[3-5 sentences: overall quality, biggest risk, highest priority action]
## Findings by Severity
### ๐ด Critical (Blocks merge)
[All critical findings from all phases, with source phase noted]
### ๐ High Priority (Fix before/soon after merge)
[Important findings across all phases]
### ๐ก Medium (Plan for next sprint)
[Suggestions that improve quality]
### ๐ข Positive Observations
[What's done well across all dimensions]
## Dimension Scores
| Dimension | Score (1-10) | Key Finding |
|-----------|-------------|-------------|
| Correctness | X | [summary] |
| Architecture | X | [summary] |
| Security | X | [summary] |
| Performance | X | [summary] |
| Database | X | [summary] |
| Maintainability | X | [summary] |
| Test Coverage | X | [summary] |
## Action Plan
[Ordered list: what to fix first, estimated effort for each]
## Tech Debt Noted
[Items for future cleanup, not blocking current merge]
Quality Gates
This review is NOT complete if:
- Any phase was skipped without justification
- No critical findings AND no positive observations (suspicious โ dig deeper)
- Findings from different phases contradict each other (reconcile)
- Action plan is missing or vague
- No dimension scores provided
