Harness Review (v3)
Harness v3 の統合レビュースキル。
以下の旧スキルを統合:
harness-review — コード・プラン・スコープ多角的レビューcodex-review — Codex CLI によるセカンドオピニオンverify — ビルド検証・エラー復旧・レビュー修正適用troubleshoot — エラー・障害の診断と修復
Quick Reference
| ユーザー入力 | サブコマンド | 動作 |
|---|
| "レビューして" / "review" | code（自動） | コードレビュー（直近の変更） |
"harness-plan 実行後" | plan（自動） | 計画レビュー |
| "スコープ確認" | scope（自動） | スコープ分析 |
harness-review code | code | コードレビュー強制 |
harness-review plan | plan | 計画レビュー強制 |
harness-review scope | scope | スコープ分析強制 |
レビュータイプ自動判定
| 直前のアクティビティ | レビュータイプ | 観点 |
|---|
harness-work 後 | Code Review | Security, Performance, Quality, Accessibility |
harness-plan 後 | Plan Review | Clarity, Feasibility, Dependencies, Acceptance |
| タスク追加後 | Scope Review | Scope-creep, Priority, Feasibility, Impact |
Code Review フロー
Step 1: 変更差分を収集
git diff HEAD~1 --stat
git diff HEAD~1 -- <changed_files>
Step 2: 4観点でレビュー
| 観点 | チェック内容 |
|---|
| Security | SQLインジェクション, XSS, 機密情報露出, 入力バリデーション |
| Performance | N+1クエリ, 不要な再レンダリング, メモリリーク |
| Quality | 命名, 単一責任, テストカバレッジ, エラーハンドリング |
| Accessibility | ARIA属性, キーボードナビ, カラーコントラスト |
Step 3: レビュー結果出力
## レビュー結果
### APPROVE / REQUEST_CHANGES
**重大な問題**: なし / {{詳細}}
| 観点 | 評価 | 詳細 |
|------|------|------|
| Security | OK / NG | {{詳細}} |
| Performance | OK / NG | {{詳細}} |
| Quality | OK / NG | {{詳細}} |
| Accessibility | OK / NG | {{詳細}} |
### 推奨改善点（必須ではない）
- {{改善提案}}
Step 4: コミット判定
- APPROVE: 自動コミット実行（
--no-commit でなければ）
- REQUEST_CHANGES: 問題箇所と修正方針を提示。
harness-work で修正後に再レビュー
Plan Review フロー
- Plans.md を読み込む
- 以下の 5 観点 でレビュー:
- Clarity: タスク説明が明確か
- Feasibility: 技術的に実現可能か
- Dependencies: タスク間の依存関係が正しいか（Depends カラムと実際の依存が一致しているか）
- Acceptance: 完了条件（DoD カラム）が定義され、検証可能か
- Value: このタスクはユーザー課題を解くか？
- 「誰の、どんな問題」が明示されているか
- 代替手段（作らない選択肢）は検討されたか
- Elephant（全員気づいているが放置されている問題）はないか
- DoD / Depends カラムの品質チェック:
- DoD が空欄のタスク → 警告（「完了条件が未定義です」）
- DoD が検証不能（「いい感じ」「ちゃんと動く」等） → 警告 + 具体化提案
- Depends に存在しないタスク番号 → エラー
- 循環依存 → エラー
- 改善提案を提示
Scope Review フロー
- 追加されたタスク/機能をリスト化
- 以下の観点で分析:
- Scope-creep: 当初スコープからの逸脱
- Priority: 優先度は適切か
- Feasibility: 現在のリソースで実現可能か
- Impact: 既存機能への影響
- リスクと推奨アクションを提示
異常検知
| 状況 | アクション |
|---|
| セキュリティ脆弱性 | 即座に REQUEST_CHANGES |
| テスト改ざん疑い | 警告 + 修正要求 |
| force push 試み | 拒否 + 代替案提示 |
Codex Environment
Codex CLI 環境（CODEX_CLI=1）では一部ツールが利用不可のため、以下のフォールバックを使用する。
| 通常環境 | Codex フォールバック |
|---|
TaskList でタスク一覧取得 | Plans.md を Read して WIP/TODO タスクを確認 |
TaskUpdate でステータス更新 | Plans.md のマーカーを Edit で直接更新（例: cc:WIP → cc:完了） |
| レビュー結果を Task に書き込み | レビュー結果を stdout に出力 |
検出方法
if [ "${CODEX_CLI:-}" = "1" ]; then
# Codex 環境: Plans.md ベースのフォールバック
fi
Codex 環境でのレビュー出力
Task ツール非対応のため、レビュー結果は標準出力にマークダウン形式で出力する。
Lead エージェントまたはユーザーが結果を読み取り、次のアクションを判断する。
関連スキル
harness-work — レビュー後に修正を実装harness-plan — 計画を作成・修正harness-release — レビュー通過後にリリース