coverage-and-failure-path-gapslisted

## 既定 CI レビューでは発火しない / Not triggered on the default CI review path このスキルは repo 全体の既存テスト木（変更されていないファイルを含む）を要するため、既定 runner の供給コンテキスト（`RUNNER_SUPPLIED_CONTEXTS = ['diff', 'prDescription', 'fullFile']`、`scripts/validate-skills.mjs`）では発火しない。`RIVER_AVAILABLE_CONTEXTS` を拡張して `tests`（repo-wide test tree）を供給する構成でのみ有効になる（`GRANDFATHERED_UNSUPPLIED_CONTEXT` に登録済み、#1606）。 ## Pattern declaration Primary pattern: Reviewer Secondary patterns: Inversion Why: カバレッジギャップ検出はチェックリスト型評価が主だが、テスト対象コードが差分に含まれない場合は実行を止める必要がある。 ## Rule / ルール - 主要フローと失敗フローの両方にテストがあることを確認する。 - 例外系・タイムアウト・リトライなどのエラーハンドリングをテストする。 - 変更によって追加/変更された分岐・境界値・フォールバックをカバーする。 ## Heuristics / 判定の手がかり - 新しい条件分岐・ガードが追加されたのに対応するテストがない。 - 例外処理やエラーリターンに対するアサーションが見当たらない。 - 大きな refactor でテストの網羅対象が変わっているのに、テスト差分が少ない��� - クリティカルパス（認証/課金/データ保存など）にテストが不足。 - `coverage_report` でステートメント/ブランチ/ラインの低下が確認できるのに差分テストが増えていない。 ## Good / Bad Examples - Good: 成功・失敗・境界を分けた `describe` / `it` を追加し、エラーメッセージも検証。 - Bad: `happy path` のみのテストで、例外時や空入力時の検証がない。 - Good: `coverage_report` を確認し、差分ファイルのステートメント/ブランチカバレッジを改善。 ## Actions / 改善案 - 新規/変更分岐ごとに正常系・異常系テストを追加する（例外メッセージも含めて検証）。 - タイムアウト/リトライ/フォールバックをモックし、意図した失敗動作を確認する。 - クリティカルパスのカバレッジを `coverage_report` ベースで確認し、不足を埋めるテストを提案する。 - 差分ファイルの各分岐・ガードに対して「期待される成功/失敗シナリオ + 入力例」を列挙し、テストケースとして提示する。 ## Non-goals / 扱わないこと - 既存テストの全面的な書き換えやリファクタリング。 - 実行環境や外部サービスの障害注入（カオス試験）の設計。 - プロダクション監視の網羅性評価。 ## Pre-execution Gate / 実行前ゲート このスキルは以下の条件がすべて満たされない限り`NO_REVIEW`を返す。 - [ ] 差分にソースコード（`src/**/*`, `lib/**/*`）またはテストフ