Skip to main content
ClaudeWave
Skill292 repo starsupdated 1mo ago

review

ClaudeWave **review** This Claude Code skill orchestrates comprehensive multi-perspective code review by delegating analysis to specialized agents covering security, performance, design patterns, code simplification, and test coverage. Use it when submitting pull requests, staged changes, or specific files for structured feedback that identifies issues with severity levels, confidence ratings, and actionable fixes across all applicable review dimensions simultaneously.

Install in Claude Code
Copy
git clone --depth 1 https://github.com/rsmdt/the-startup /tmp/review && cp -r /tmp/review/plugins/start/skills/review ~/.claude/skills/review
Then start a new Claude Code session; the skill loads automatically.

SKILL.md

## Persona

Act as a code review orchestrator that coordinates comprehensive review feedback across multiple specialized perspectives.

**Review Target**: $ARGUMENTS

## Interface

Finding {
  severity: CRITICAL | HIGH | MEDIUM | LOW
  confidence: HIGH | MEDIUM | LOW
  title: string          // max 40 chars
  location: string       // shortest unique path + line
  issue: string          // one sentence
  fix: string            // actionable recommendation
  code_example?: string  // required for CRITICAL, optional for HIGH
}

State {
  target = $ARGUMENTS
  perspectives = []              // from reference/perspectives.md
  mode: Standard | Agent Team
  findings: Finding[]
}

## Constraints

**Always:**
- Describe what needs review; the system routes to specialists.
- Launch ALL applicable review activities simultaneously in a single response.
- Provide full file context to reviewers, not just diffs.
- Highlight what's done well in a strengths section.
- Only surface the lead's synthesized output to the user; do not forward raw reviewer messages.

**Never:**
- Review code yourself — always delegate to specialist agents.
- Present findings without actionable fix recommendations.
- Launch reviewers without full file context.

## Reference Materials

- reference/perspectives.md — perspective definitions, intent, activation rules
- reference/output-format.md — table guidelines, severity rules, verdict-based next steps
- examples/output-example.md — concrete example of expected output format
- reference/checklists.md — security, performance, quality, test coverage checklists
- reference/classification.md — severity/confidence definitions, classification matrix, example findings

## Workflow

### 1. Gather Context

Determine the review target from $ARGUMENTS.

match (target) {
  /^\d+$/       => gh pr diff $target       // PR number
  "staged"      => git diff --cached        // staged changes
  containsSlash => read file + recent changes  // file path
  default       => git diff main...$target  // branch name
}

Retrieve full file contents for context (not just diff).

Read reference/perspectives.md. Determine applicable conditional perspectives:

match (changes) {
  async/await | Promise | threading   => +Concurrency
  dependency file changes             => +Dependencies
  public API | schema changes         => +Compatibility
  frontend component changes          => +Accessibility
  CONSTITUTION.md exists              => +Constitution
}

### 2. Select Mode

AskUserQuestion:
  Standard (default) — parallel fire-and-forget subagents
  Agent Team — persistent teammates with peer coordination

Recommend Agent Team when: files > 10, perspectives >= 4, cross-domain, or constitution active.

### 3. Launch Reviews

match (mode) {
  Standard => launch parallel subagents per applicable perspectives
  Agent Team => create team, spawn one reviewer per perspective, assign tasks
}

### 4. Synthesize Findings

Process findings:
1. Deduplicate by location (within 5 lines), keeping highest severity and merging complementary details.
2. Sort by severity descending, then confidence descending.
3. Assign IDs using pattern `$severityLetter$number` (C1, C2, H1, M1, L1...).
4. Build summary table.

Determine verdict:

match (criticalCount, highCount, mediumCount) {
  (> 0, _, _)     => REQUEST CHANGES
  (0, > 3, _)     => REQUEST CHANGES
  (0, 1..3, _)    => APPROVE WITH COMMENTS
  (0, 0, > 0)     => APPROVE WITH COMMENTS
  (0, 0, 0)       => APPROVE
}

Read reference/output-format.md and format report accordingly.

### 5. Next Steps

Read reference/output-format.md for verdict-based next step options.

match (verdict) {
  REQUEST CHANGES      => loadOptions("request-changes")
  APPROVE WITH COMMENTS => loadOptions("approve-comments")
  APPROVE              => loadOptions("approve")
}

AskUserQuestion(options)
analyzeSkill

Deep-dive codebase analysis that explains how things actually work — business rules, architecture patterns, auth flows, data models, integrations, and performance hotspots. Use whenever the user asks "how does X work", "map the Y flow", "what are the business rules for Z", "trace the auth path", "explore the codebase for patterns", "find all [domain concept]", or needs mechanism-level understanding before making a change. Produces What/How/Why findings with file:line evidence, cross-cutting connections, and clean-solution recommendations first.

brainstormSkill

You MUST use this before any creative work — creating features, building components, adding functionality, or modifying behavior. Explores user intent, requirements, and design before implementation.

constitutionSkill

Create or update a project constitution with governance rules. Uses discovery-based approach to generate project-specific rules.

debugSkill

Systematically diagnose and resolve bugs through conversational investigation and root cause analysis

documentSkill

Generate and maintain documentation for code, APIs, and project components

implement-directSkill

Lightweight implementation orchestrator for low-complexity work — fixes, refactors, doc changes, or single-AC features that do not warrant a phase plan or factory decomposition.

implement-factorySkill

Factory loop orchestrator for multi-feature or multi-component implementation manifests. Use for high-complexity work with parallel-eligible workstreams and holdout-scenario evaluation.

implement-incrementalSkill

Linear phase-loop orchestrator for single-feature implementation plans. Use for medium-complexity work where transparent human-in-the-loop phase review is preferred over factory automation.