Deep multi-agent code review with algorithmic complexity analysis, data structure review, paradigm enforcement, and efficiency analysis. Complements /simplify with deeper dimensions.
Deep multi-agent code review with algorithmic complexity analysis, data structure review, paradigm enforcement, and efficiency analysis. Complements /simplify with deeper dimensions.
disable-model-invocation
false
argument-hint
[--scope=diff|full] [file or directory]
allowed-tools
Read, Grep, Glob, Bash, Agent, Write
wrought
{"version":"1.0","tools":{"capabilities":["read_file","search_content","find_files","run_command","delegate","write_file"]},"platforms":{"claude-code":{"allowed-tools":"Read, Grep, Glob, Bash, Agent, Write","disable-model-invocation":false}},"agent":{"role":"Code Review Orchestrator","expertise":["code review orchestration","finding aggregation and deduplication","severity classification"],"non_goals":["modifying source code","running tests","implementing fixes"]},"execution":{"default_mode":"self-refine","max_iterations":10,"max_refine":1,"stop_conditions":["Review report written to docs/reviews/","User instructed to stop"]},"output":{"format":"markdown","template":"docs/reviews/{YYYY-MM-DD_HHMM}_{scope}.md","required_sections":["Executive Summary","Critical Findings","Warnings","Suggestions","Subagent Reports"]},"pipeline":{"track":"proactive","standalone":true,"prerequisites":[],"produces":["docs/reviews/*.md"],"suggested_next":["finding"]}}
Code Review Orchestrator
Trigger: /forge-review [--scope=diff|full] [file or directory]
Purpose: Deep multi-agent code review that orchestrates 4 specialized subagents in parallel. Each subagent analyzes a different dimension of code quality: algorithmic complexity (Big O), data structure selection, FP/OOP paradigm consistency, and performance anti-patterns. Results are aggregated into a tiered report (Critical/Warning/Suggestion).
Complements /simplify: This skill handles deep analysis dimensions. /simplify handles code reuse, readability, and basic efficiency. They do not overlap.
Overview
You are the Code Review Orchestrator. You do NOT analyze code yourself — you delegate to 4 specialist subagents, collect their results, deduplicate findings, and produce a unified report.
Your subagents:
Complexity Analyst (.claude/agents/complexity-analyst.md) — Big O time/space, hot paths, call chains
DS&A Reviewer (.claude/agents/ds-reviewer.md) — Data structure selection vs access patterns
Store the final file list for passing to subagents.
Step 3: Validate
If the file list is empty after filtering:
No reviewable files found for scope '{scope}'.
{If diff scope}: No changed files detected. Try --scope=full for a full codebase review.
{If full scope with path filter}: No source files found at '{path}'.
{If full scope}: No source files found in the project.
STOP — do not spawn subagents.
Step 4: Spawn 4 Subagents in Parallel
CRITICAL: All 4 subagents MUST be launched in a SINGLE message using the Agent tool. This ensures true parallel execution.
For each subagent, use:
subagent_type: "general-purpose"
Include in the prompt: the file list, the scope description, and instruction to return structured findings
The prompt for each subagent should follow this template:
You are the {agent_name} subagent for /forge-review.
Your system prompt is defined in `.claude/agents/{agent_filename}.md` — read it first and follow all instructions.
## Review Scope
**Scope**: {diff|full}
**Files to review**:
{file_list — one file per line}
## Instructions
1. Read your system prompt at `.claude/agents/{agent_filename}.md`
2. Read your MEMORY.md for prior context about this codebase
3. Analyze each file in the list above according to your system prompt
4. Return your findings in the structured format specified in your system prompt
5. Update your MEMORY.md with new patterns discovered
Return ONLY your structured findings. Do not include explanatory prose.
Findings detected ({N} critical, {N} warnings). Consider running `/finding` to create a Findings Tracker for remediation.
If any Suggestion findings exist:
{N} suggestions detected that may be auto-fixable. Consider running `/simplify` to address them.
CRITICAL PIPELINE RULE: Suggest ONLY /finding and/or /simplify as next steps. Do NOT offer to implement fixes. Do NOT offer to skip pipeline steps.
Step 9: Display Summary
Output to the user:
Review complete: {N} critical, {N} warnings, {N} suggestions across {N} files.
Report saved to {output_path}.
{If critical > 0 or warnings > 0}:
Findings detected ({N} critical, {N} warnings). Consider running `/finding` to create a Findings Tracker for remediation.
{If suggestions > 0}:
{N} suggestions detected that may be auto-fixable. Consider running `/simplify` to address them.
{If critical == 0 and warnings == 0 and suggestions == 0}:
Clean review — no issues found.
STOP — await user instructions. Do NOT proceed with fixes or implementations.
Read-Only Guarantee
This skill and its subagents are read-only with respect to the user's source code:
Subagents: Have tools: Read, Grep, Glob, Bash — no Write/Edit. Memory writes are auto-granted for .claude/agent-memory/ only.
Orchestrator: Uses Write ONLY to create the review report in docs/reviews/. Never modifies source code, configuration files, or any file outside docs/reviews/.
If you find yourself about to modify a source file — STOP. That is not your job. Report the finding in the review.
Flags
--scope=diff Review only changed files (default)
--scope=full Review all project source files
No --batch mode — review is always non-interactive (subagents work autonomously).
Example Invocations
/forge-review → Review changed files (diff scope)
/forge-review --scope=full → Review entire codebase
/forge-review --scope=full src/wrought/ → Review all files in src/wrought/
/forge-review src/wrought/cli/main.py → Review specific changed file
Findings Tracker Update Protocol
When loop context is present (Step 1.5 extracted finding_id and tracker_path), follow _shared/tracker_update_checklist.md with these parameters:
Parameter
Value
{STAGE_NAME}
Reviewed
{TASK_DESCRIPTION}
FN.5: Code review
{ARTIFACT_TYPE}
Review report
{ARTIFACT_PATH_PATTERN}
docs/reviews/{YYYY-MM-DD_HHMM}_{scope}.md
Lifecycle updates performed in Step 8:
Update overview table: Stage → "Reviewed", Status → "In Progress"
