/harness-review

name: harness-review
description: "HAR: Multi-angle code, plan, scope review. Security/quality check. Trigger: review, code review, plan review, scope analysis. Do NOT load for: implementation, new features, bugfix, setup, release."
description-en: "HAR: Multi-angle code, plan, scope review. Security/quality check. Trigger: review, code review, plan review, scope analysis. Do NOT load for: implementation, new features, bugfix, setup, release."
description-ja: "HAR:コード・プラン・スコープを多角的にレビュー。セキュリティ・品質チェック。レビューして、レビュー、コードレビュー、プランレビュー、スコープ分析で起動。実装・新機能・バグ修正・セットアップ・リリースには使わない。"
kind: workflow
purpose: "Review code, plans, scope, and evidence before acceptance"
trigger: "review, レビューして, code review, plan review, scope analysis"
shape: evaluate
role: evaluator
pair: harness-work
owner: harness-core
since: "2026-05-05"
allowed-tools: ["Read", "Grep", "Glob", "Bash", "Task", "Monitor", "AskUserQuestion"]
argument-hint: "[code|plan|scope|--quick|--codex-closeout|--dual|--team-debate|--security|--ui-rubric|--blind-judge]"
context: fork
effort: high
user-invocable: true

Harness Review

Harness の統合レビュースキル。 この `SKILL.md` は薄い dispatcher であり、詳細な品質基準は `references/` を読む。

if $ARGUMENTS == "": → 「今までの作業のレビュー」と解釈し、Review target detection を実行する → review target が 1 つに確定できる場合だけ自動開始する → review target が不明または複数候補の場合は AskUserQuestion で選択肢を出し、認識を揃えてから開始する

<!-- 上記 3 行は AUTO-START CONTRACT。skill-editing.md の「最冒頭 3 行以内」ルールに従い fence / HTML コメントで押し下げない -->

Output Contract (P35: 「止まったように見える」UX 対策)

`<local-command-stdout>` で host へ結果を中継する場合だけ、output の **最後の 1 行**に次の literal を含める:

`↑この結果は Claude が要約します。Enter キーで次へ進むか、新規 prompt で別の指示を出してください。`

これは `<local-command-stdout>` 経由で text response として表示されると user が「止まった」と感じる UX 問題への明示的な instruction (patterns.md P35)。 host の最終回答は verdict、判断理由、確認した証拠、未検証点を返す。要約の予告だけで終了しない。

Dispatcher Contract

この skill の責務は review 判定だけ。 commit / push / release は既定では行わない。

• review default read-only boundary: 既定は read-only。`APPROVE` でも自動 commit しない
• Do not push just to review: review 目的だけで push しない
• commit が必要な場合は、ユーザー明示依頼、`harness-work`、または `harness-release` の Work Commit Gate に委譲する
• `--commit-on-approve` のような明示 opt-in が設計されるまで、この skill 単体の default side effect は禁止

Quick Reference

| Command | Mode | Purpose | |---|---|---| | `/harness-review` | `code` | 今までの作業を自動検出して review | | `/harness-review --quick` | `quick` | 小さな dirty change を軽く closeout | | `/harness-review --codex-closeout` | `codex-closeout` | Codex 助言 + focused tests で closeout | | `/harness-review --dual` | `dual` | Claude + Codex second opinion | | `--pre-review cursor` | `code+pre-review` | fresh-context composer advisory pre-review before brain verdict（read-only） | | `/harness-review --cursor` | `code+cursor-second-opinion` | core review gates + cursor second-opinion（brain 一次レビュー必須） | | `HARNESS_IMPL_BACKEND=cursor harness-review` | `code+cursor-second-opinion` | default ON 時も core review gates に cursor second-opinion を自動加算。primary verdict は brain 固定 | | `/harness-review --team-debate` | `team-debate` | TeamAgent Debate を強制 | | `/harness-review --security` | `security` | security 専用 review | | `/harness-review plan` | `plan` | `Plans.md` の計画 review | | `/harness-review scope` | `scope` | scope creep / 漏れ review |

Mode Decision

引数から実行 mode を決定し、必要な `references/` を選択ロードする。

| 入力 | mode | 読む reference | |---|---|---| | 引数なし / `code` | `code` | `references/code-review.md`, `references/governance.md` | | `--quick` | `quick` | `references/codex-closeout.md`, `references/code-review.md` | | `--codex-closeout` | `codex-closeout` | `references/codex-closeout.md` | | `--dual` | `dual` | `references/dual-review.md`, `references/team-debate.md` | | `--team-debate` | `team-debate` | `references/team-debate.md`, `references/governance.md` | | `--security` | `security` | `references/security-profile.md`, `references/governance.md` | | `--ui-rubric` | `ui-rubric` | `references/ui-rubric.md` | | `plan` | `plan` | `references/plan-review.md`, `references/governance.md` | | `scope` | `scope` | `references/scope-review.md`, `references/governance.md` | | `--cursor` or resolver result `cursor` for no-arg / `code` review only | `code+cursor-second-opinion` | `references/code-review.md`, `references/governance.md`, `references/cursor-review.md`, `references/dual-review.md` | | `full` | `full` | `references/code-review.md`, `references/team-debate.md`, `references/dual-review.md` |

`quick` と `codex-closeout` は軽量 path。 小さな dirty change、single commit、PR branch の closeout を速く見る。 品質 gate を捨てるものではない。

Cursor Default ON

mode 判定時、明示 mode words (`plan`, `scope`, `full`) と明示フラグを先に確定する。no-arg / `code` review の場合だけ helper root を解決して resolver を 1 回だけ実行し、resolver 不在時は `claude` とみなす。

HARNESS_PLUGIN_ROOT="${HARNESS_PLUGIN_ROOT:-${CLAUDE_PLUGIN_ROOT:-}}"; if [ -z "$HARNESS_PLUGIN_ROOT" ] && [ -n "${CLAUDE_SKILL_DIR:-}" ]; then probe="$(cd "${CLAUDE_SKILL_DIR}" && pwd)"; while [ "$probe" != "/" ] && [ ! -d "$probe/scripts" ]; do probe="$(cd "$probe/.." && pwd)"; done; [ -d "$probe/scripts" ] && HARNESS_PLUGIN_ROOT="$probe"; fi
if [ -x "${HARNESS_PLUGIN_ROOT:-}/scripts/resolve-impl-backend.sh" ]; then resolved_backend="$(bash "${HARNESS_PLUGIN_ROOT}/scripts/resolve-impl-backend.sh" --role reviewer)"; else resolved_backend="claude"; fi

no-arg / `code` review で結果が `cursor` の場合は `--cursor` と同じ `cursor-second-opinion` を追加するが、core review gates (`references/code-review.md`, `references/governance.md`) は必ず先に読み、Cursor reference は additive にだけ扱う。primary verdict は brain 側で維持し、cursor は `dual_review.cursor_verdict` の advisory に限る。`plan` / `scope` など明示 mode word は resolver result より優先し、cursor default によって plan/scope references も code/governance references も置き換えない。結果が `claude` / `codex` の場合は従来どおりで、review の primary 判定面は変えない。

Pre-Review Cursor (`--pre-review cursor`)

`/harness-review --pre-review cursor` は `bash scripts/pre-review-cursor.sh [--base ref]` で read-only fresh-context composer pre-review（`model-routing.sh --host cursor --t