$npx -y skills add tobihagemann/turbo --skill review-codeReview code for bugs, security vulnerabilities, API misuse, consistency issues, simplicity problems, or test coverage gaps by running internal reviews and a peer review in parallel and returning combined findings. Single-concern with a type argument, or full review with no argume
| 1 | # Review Code |
| 2 | |
| 3 | Review code against type-specific criteria. Runs internal reviews and `/peer-review` in parallel by default. Returns combined structured findings. |
| 4 | |
| 5 | **Types:** `correctness`, `security`, `api-usage`, `consistency`, `simplicity`, `coverage` |
| 6 | |
| 7 | With a type argument, runs a single-concern internal review plus the peer review. With no type argument, runs all six internal reviews plus the peer review. |
| 8 | |
| 9 | ## Step 1: Determine the Scope |
| 10 | |
| 11 | Determine what to review: |
| 12 | |
| 13 | - If a specific **diff command** was provided (e.g., `git diff --cached`, `git diff main...HEAD`), use that. |
| 14 | - If a **file list or directory** was provided, review those files directly (read the full files, not a diff). |
| 15 | - If **neither** was provided, default to diffing against the repository's default branch (detect via `gh repo view --json defaultBranchRef --jq '.defaultBranchRef.name'`). If there are no changes against the default branch, stop and state that there is nothing to review. |
| 16 | |
| 17 | ## Step 2: Run Reviews in Parallel |
| 18 | |
| 19 | Each active type maps to a criteria reference file: |
| 20 | |
| 21 | - **Correctness** — [references/correctness-review.md](references/correctness-review.md) |
| 22 | - **Security** — [references/security-review.md](references/security-review.md) |
| 23 | - **API usage** — [references/api-usage-review.md](references/api-usage-review.md) |
| 24 | - **Consistency** — [references/consistency-review.md](references/consistency-review.md) |
| 25 | - **Simplicity** — [references/simplicity-review.md](references/simplicity-review.md) |
| 26 | - **Coverage** — [references/coverage-review.md](references/coverage-review.md) |
| 27 | |
| 28 | Full review activates all six types; a single-concern argument activates one. Skip peer review when instructed (e.g., "without peer review", "no peer", "internal only"). |
| 29 | |
| 30 | Use the Agent tool to launch all agents below in a single assistant message so they run concurrently. Run them in the foreground so all their results return in this turn. Each Agent call uses `model: "opus"`. For full review that is seven Agent tool calls (six internal + one peer); for single-concern it is two (one internal + one peer). Every agent's prompt must direct it to treat the shared working tree and its git index as read-only and to assess findings by reading and reasoning. For a check that genuinely requires mutating code (such as testing whether a finding holds), the agent works in an isolated `git worktree` it discards afterward. |
| 31 | |
| 32 | - **Internal Agent (one per active type):** Launch a separate Agent tool call for each active type. The subagent's prompt must include the scope, the path to the type's reference file (`~/.claude/skills/review-code/references/<type>-review.md`), the output format below, and this directive: read that reference file directly, apply its determination criteria as the bar for a real finding, then report every finding that clears that bar tagged with its priority. Coverage is the goal at this stage, so surface everything that qualifies and let the priority tags convey severity. The subagent must also return the Overall Verdict block for its type, using the verdict label from the reference file it read. |
| 33 | - **Peer review Agent (unless skipping):** Launch an Agent tool call whose prompt instructs the subagent to invoke `/peer-review` via the Skill tool with a request describing: (a) the scope to review; (b) all active types covered in one single-pass review run that evaluates every dimension, each judged independently against its criteria file, rather than a per-dimension parallel fan-out; (c) for each dimension, the criteria live in `~/.claude/skills/review-code/references/<type>-review.md` — the reviewer should read that file directly, use its priority scale and verdict label, and include any extra metadata fields it specifies; (d) the output format below, including the `**Failure scenario:**` line. The prompt must also state explicitly that the subagent's final assistant message must contain the verbatim findings text `/peer-review` produced. |
| 34 | |
| 35 | Aggregate the findings and per-type verdicts the subagents return, with attribution (reviewer: "internal" or "peer"; type; file path). Present them in the output format below. |
| 36 | |
| 37 | Then use the TaskList tool and proceed to any remaining task. |
| 38 | |
| 39 | ## Output Format |
| 40 | |
| 41 | Format each finding as: |
| 42 | |
| 43 | ``` |
| 44 | ### [P<N |