메뉴
Check whether changed code paths have corresponding tests and suggest minimal coverage.
기본적으로 소스를 먼저 확인하는 Prompt가 선택됩니다. 직접 명령으로 전환하거나 로컬 사본을 다운로드할 수도 있습니다.
설치 여부를 결정하기 전에 SKILL.md와 SkillsMP에 표시된 보조 파일을 읽어 보세요.
Codex 또는 Claude로 설치 이 Prompt를 복사해 Codex, Claude 또는 다른 어시스턴트에 붙여 넣으면 Skill 페이지를 검토하고 설치를 진행할 수 있습니다.
직접 명령은 검토 Prompt를 거치지 않습니다. 실행하기 전에 소스를 확인하세요.
npx skills add https://github.com/s977043/river-review --skill test-presence-for-changed-code명령은 한 줄로 유지됩니다. 복사하기 전에 가로로 스크롤해 전체 내용을 확인하세요.
로컬 사본을 원하시나요? 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 | test-existence |
| name | Test Presence for Changed Code |
| description | Check whether changed code paths have corresponding tests and suggest minimal coverage. |
| version | 0.1.0 |
| category | downstream |
| phase | downstream |
| applyTo | ["src/**/*","lib/**/*","**/*.test.*","**/*.spec.*"] |
| tags | ["tests","coverage","downstream"] |
| severity | major |
| inputContext | ["diff","tests"] |
| outputKind | ["tests","findings","actions"] |
| 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: テスト存在チェックはチェックリスト型評価が主だが、実行パスへ影響するコード変更が差分にない場合は実行を止める必要がある。
このスキルは以下の条件がすべて満たされない限りNO_REVIEWを返す。
src/**/*, lib/**/*）またはテストファイル（*.test.*, *.spec.*）の変更が含まれているゲート不成立時の出力: NO_REVIEW: test-existence — テスト存在チェックの対象となるコード変更が検出されない
*.test.* / *.spec.*）が含まれている場合。*.test.* / *.spec.* が無い、または差分がゼロ。coverage_report を参照し、変更ファイルのステートメント/ブランチカバレッジが上がる具体的なテストケースを提案する。Finding: / Evidence: / Impact: / Fix: / Severity: / Confidence: を含む短いメッセージにする。Fix は 1〜3 件のテスト観点に絞る（例: 新分岐、例外、境界）。