القائمة
Find missing tests for critical paths, edge cases, and failure handling in changed code.
يُحدَّد Prompt الذي يراجع المصدر أولًا بشكل افتراضي. يمكنك التبديل إلى أمر مباشر أو تنزيل نسخة محلية.
اقرأ SKILL.md وأي ملفات مرافقة يعرضها SkillsMP قبل أن تقرر التثبيت.
التثبيت باستخدام Codex أو Claude انسخ هذا Prompt والصقه في Codex أو Claude أو مساعد آخر ليراجع صفحة Skill ويثبّتها لك.
يتجاوز الأمر المباشر Prompt المخصّص للمراجعة. افحص المصدر قبل تشغيله.
npx skills add https://github.com/s977043/river-review --skill coverage-and-failure-path-gapsيبقى الأمر في سطر واحد. مرّر أفقيًا لمراجعته كاملًا قبل النسخ.
تفضّل نسخة محلية؟ نزّل الملفات المتاحة حاليًا لدى SkillsMP.
6つの専門レビュアーロールを並列実行し、consensusLevel（複数ロールの合意度）と Tech Lead レポート（top3指摘・blindSpots・consensusSummary）で結果を統合する マルチエージェントレビュー entry skill。 Parallel multi-role review with consensus scoring (consensusLevel) and Tech Lead report. Use when a major release needs exhaustive multi-angle review, or when a single-perspective review is not enough and you want confidence that no reviewer angle was missed（重要リリース前の網羅レビュー・多視点の確証が 欲しいとき）。
検査器（linter / ガード / CI ゲート / バリデータ / フィルタ）そのものの検出力が差分で静かに落ちていないか、また新設した検査が実際に検出できることが実証されているかを diff-time で検出する。Check 1 unverified detection reduction（検査器の検出ロジック（正規表現・除外パターン・解決集合・allowlist・baseline）を変更して検出件数が減ったのに、減った 1 件ずつが誤検出であった根拠が示されないまま「誤検出を潰した」「N → M 件に減少」と改善として主張している）、Check 2 unproven guard detection power（新しい検査ロジック・CI ゲート・バリデーションを追加・変更したのに、検出すべき入力を注入して検出されること／検出すべきでない入力で黙ることを示す変異注入の証跡が無く、「追加した」「CI が緑」を検出力の証拠として扱っている）、の 2 Check を対象とする report-only。テスト自身のアサーションが実質何も検証していない構造は test-assertion-effectiveness、レビュー基準・品質ゲートの明示的な弱体化（ルールの削除・閾値の引き下げ・suppression entry 追加・lint ルールの無効化）は review-criteria-integrity、リファクタ完了主張一般の検証は refactor-claim-audit、workflow の permissions と action pin は gha-workflow-security、設定ファイルの構文・型の妥当性は config-json へ委譲する
pbi-input, plan, todo, test-cases 間の整合性をチェックし、実装着手前の仕様漏れを検知する
استنادا إلى تصنيف SOC المهني
عرض SKILL.md
| id | coverage-gap |
| name | Coverage and Failure Path Gaps |
| description | Find missing tests for critical paths, edge cases, and failure handling in changed code. |
| version | 0.1.0 |
| category | downstream |
| phase | downstream |
| applyTo | ["src/**/*","lib/**/*","**/*.test.*","**/*.spec.*"] |
| tags | ["tests","coverage","reliability","downstream"] |
| severity | major |
| inputContext | ["diff","tests"] |
| outputKind | ["tests","findings","actions","summary"] |
| modelHint | balanced |
| dependencies | ["test_runner","coverage_report"] |
このスキルは repo 全体の既存テスト木（変更されていないファイルを含む）を要するため、既定 runner の供給コンテキスト（RUNNER_SUPPLIED_CONTEXTS = ['diff', 'prDescription', 'fullFile']、scripts/validate-skills.mjs）では発火しない。RIVER_AVAILABLE_CONTEXTS を拡張して tests（repo-wide test tree）を供給する構成でのみ有効になる（GRANDFATHERED_UNSUPPLIED_CONTEXT に登録済み、#1606）。
Primary pattern: Reviewer Secondary patterns: Inversion Why: カバレッジギャップ検出はチェックリスト型評価が主だが、テスト対象コードが差分に含まれない場合は実行を止める必要がある。
coverage_report でステートメント/ブランチ/ラインの低下が確認できるのに差分テストが増えていない。describe / it を追加し、エラーメッセージも検証。happy path のみのテストで、例外時や空入力時の検証がない。coverage_report を確認し、差分ファイルのステートメント/ブランチカバレッジを改善。coverage_report ベースで確認し、不足を埋めるテストを提案する。このスキルは以下の条件がすべて満たされない限りNO_REVIEWを返す。
src/**/*, lib/**/*）またはテストファイル（*.test.*, *.spec.*）の変更が含まれているゲート不成立時の出力: NO_REVIEW: coverage-gap — カバレッジギャップ検出の対象となるコード変更が検出されない