river-review-testing

Testing Review（テストレビュー）

テストコードの品質、網羅性、安定性を検証する。

When to Use / いつ使うか

• テストファイルの追加・変更時
• テスト対象コードの変更時（テストの更新漏れ検出）
• テスト戦略の見直し時

Routing / ルーティング

| キーワード | スキルID | 説明 |

| ----------------------------- | ------------------------------ | -------------------------------- |

| カバレッジ, 網羅 | coverage-gap | カバレッジギャップ検出 |

| フレーキー, 不安定 | flaky-test | フレーキーテストリスク |

| テスト有無, 存在 | test-existence | テスト存在確認 |

| 命名, 構造, describe | test-naming | テスト命名・構造 |

| テスト観点, 計画 | test-plan-review | テスト観点レビュー |

| アサーション, 常に PASS, 恒真 | test-assertion-effectiveness | アサーション有効性（無効な検証） |

デフォルト動作

• テストファイルの変更 → test-naming + flaky-test + test-assertion-effectiveness
• ソースコードの変更 → test-existence + coverage-gap
• 大規模変更 → 全スキル実行

Execution Flow / 実行フロー

1. 変更ファイルの分類
   ├─ テストファイル → 品質チェック（命名、フレーキー）
   ├─ ソースファイル → 網羅性チェック（存在、カバレッジ）
   └─ 両方 → 全スキル実行

2. 各スキルの実行
   ├─ test-existence: テストの存在確認
   ├─ coverage-gap: カバレッジギャップ検出
   ├─ test-naming: 命名・構造検証
   ├─ flaky-test: フレーキーリスク検出
   ├─ test-plan-review: テスト観点の妥当性
   └─ test-assertion-effectiveness: アサーション有効性（常に PASS する検証）

3. 統合サマリの生成

Output Format / 出力形式

<file>:<line>: <message>

• Finding: 何が問題か（1文）
• Impact: 何が困るか（短く）
• Fix: 次の一手（最小の修正案）

他スキルとの関係

| スキル | 関係 | 棲み分け |

| ----------------------- | ---- | ----------------------------------------------------------------- |

| river-review-code | 補完 | testing は「テストの品質」、code は「プロダクションコードの品質」 |

| river-review-security | 補完 | testing は「テスト網羅性」、security は「セキュリティ脆弱性」 |

References

• [ROUTING.md](./references/ROUTING.md): 詳細なルーティングルール