ojowwalker77/claude-matrix/review

Matrix Code Review

Perform comprehensive, context-aware code review using 5-agent architecture with 95% false positive reduction.

Tip: This skill runs in a forked context for an unbiased perspective - similar to how a human reviewer would approach the code for the first time.

Architecture

ORCHESTRATOR (parses target, routes, aggregates)
     │
     ├── DETECTION AGENT → security, runtime, breaking, logic flaws, hygiene (nuke)
     ├── IMPACT AGENT → blast radius, transitive graph, test coverage
     ├── TRIAGE AGENT → tier assignment, confidence calibration, noise filter
     ├── REMEDIATION AGENT → context-aware fixes, regression checks
     └── VERIFICATION AGENT → build, test, lint validation

Usage

Parse user arguments from the skill invocation (text after the trigger phrase).

Expected format: <target> [mode]

• target: File path, PR number, or "staged" for staged changes
• mode (optional): default | lazy (default: from config or default)

Modes

Default Mode (Comprehensive)

Full 5-agent review pipeline with maximum index utilization:

• Detection: Security vulns, runtime issues, breaking changes, hygiene (nuke scan)
• Impact: Transitive blast radius (2-3 levels), service boundaries, test coverage
• Triage: Tier classification, >80% signal ratio target
• Remediation: Context-aware fixes matching codebase patterns
• Verification: Run build, test, lint commands automatically

Lazy Mode (Quick)

Detection agent only:

• Direct code inspection
• Critical issues only (Tier 1)
• ~2-3 comments, no blast radius

Review Pipeline

Follow the 5-agent orchestration detailed in references/review-phases.md:

1. Orchestrator - Parse target, dispatch agents, aggregate results
2. Detection Agent - Find security, runtime, breaking, logic issues
3. Impact Agent - Calculate transitive blast radius, test coverage
4. Triage Agent - Classify tiers, calibrate confidence, filter noise
5. Remediation Agent - Generate fixes, check regression risk
6. Verification Agent - Run build, test, lint; report results

Early exit: If Detection finds nothing critical, skip Impact/Triage depth.

Examples

/matrix:review src/utils/auth.ts          # Default mode (comprehensive)
/matrix:review staged                     # Review staged changes
/matrix:review staged lazy                # Quick review of staged changes
/matrix:review 123                        # Review PR #123
/matrix:review 123 lazy                   # Quick review of PR #123

Additional Resources

Reference Files

• `references/review-phases.md` - Orchestrator + 5-agent pipeline
• `references/agents/detection.md` - Detection patterns and output format
• `references/agents/impact.md` - Blast radius algorithm
• `references/agents/triage.md` - Tier classification and signal ratio
• `references/agents/remediation.md` - Fix generation and regression checks
• `references/agents/verification.md` - Build/test/lint command detection and execution