River review testing

Skill s977043/river-review/skills/agent-skills/river-review-testing

テスト観点のレビューエージェント。テスト網羅性、命名規則、フレーキーテスト、カバレッジギャップの個別スキルへルーティングする。From its SKILL.md

Install

npx -y skills add s977043/river-review --skill river-review-testing

Assembled from the repository path, not quoted from the project. Check it against their README if it does not work.

One thing to look at

2 stars2 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.

What its file declares

Copied from the file, not written here

The file declares its own license as MIT. That is the author’s claim about this one file, and it is not the same thing as the license GitHub reports for the repository, which is listed with the other numbers below.

SKILL.md

3.5 KB, as published. Nobody here has run it

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: 詳細なルーティングルール