review-code について
review-codeは、他のワークフロー内で実行するコードレビュー用スキルです。編集やリファクタリングは行わず、必須チェックリストに沿って根拠と重要度を添えた指摘を返します。
主な機能
- 01指定範囲の変更をレビュー
- 02ロジックとエラー処理を確認
- 03重複コードと未使用コードを検出
- 04命名・依存関係・参照を検証
- 05重要度別に指摘を報告
活用シーン
- 01デフォルトブランチとの差分をレビュー
- 02指定パスやglobの変更を確認
- 03コードレビューの指摘一覧を作成
MCP 対応
入力には、このスキルとMCPの関係について記載されていません。
インストール
コマンドをコピーしてターミナルで実行します。
npx skills add https:/インストール後、Claude Code で/review-codeを入力して読み込まれたか確認します。
詳細
コードレビュー（内部）
review-change／product-audit が会話内で作成します。どのエージェントも、振り分けられたステップとしてこのファイルに従ってください。指摘事項のみ。編集やリファクタリングはしません。
対象範囲
呼び出し元が指定する差分またはパス／glob。デフォルトでは、デフォルトブランチに対する現在の変更です。返す表の先頭に対象範囲を記載してください。
チェックリスト（すべての項目を評価してください。省略不可。該当なしの場合はその旨を記載してください）
✓ 変更されたパスにロジックエラーがない（各変更関数について、境界値を含む入力から出力まで追跡する） ✓ すべてのエラーパスが処理されている（例外の握りつぶし、空の catch、戻りコードの無視がない） ✓ ロジックの重複がない（変更ブロックが既存のヘルパーを再実装していない。該当する場合は既存のヘルパーを示す） ✓ デッドコードが追加されていない（未使用の関数、パラメーター、インポート、到達不能な分岐） ✓ 差分に未処理の TODO／FIXME／HACK マーカーが残っていない ✓ 命名規則とファイル規約がプロジェクトのドキュメントに準拠している（先にドキュメントを読み、違反している規約を示す） ✓ SPEC が要求する範囲を超える新たな抽象化がない（実装が1つだけのインターフェース／基底クラスは指摘事項） ✓ SPEC で正当化されていない新しい依存関係がない ✓ 簡素化：動作を損なわずに行数を減らせる変更ブロックがない（変更前／変更後を示す） ✓ 参照の追跡：変更されたすべてのシンボル／APIについて参照検索が行われている（更新されていない呼び出し元は指摘事項） ✓ SPEC の開発シナリオで挙げられたエッジケースが、テストだけでなくコードでも処理されている
重要度の基準
ユーザーの結果が変わる場合、またはプロジェクトが明示的に定めた規則に違反している場合に限り、根拠の横に違反している規則を示して行を報告してください。指摘事項ではないもの：コメントや句読点の誤り、書式だけのずれ、根拠となる規則のないスタイル上の好み、SPEC に挙げられていないシナリオに対する仮定上の堅牢性。適切な変更に対しては、Decision: PASS を含む空の表を返すのが期待される結果です。表を埋めるためだけに指摘事項を追加してはいけません。
次の形式のみを返す
REVIEW CODE — scope: <scope>
| # | Finding | Sev | Evidence | Suggested fix |
|---|---------|-----|----------|---------------|
| 1 | <what> | critical|major|minor | <file:line> | <smallest action> |
Checklist: <n> evaluated, <n> pass, <n> findings, <n> n/a (<which + why>)
Summary: <1-2 sentences>
Decision: PASS | FAIL
重大または主要な指摘事項が未解決の場合は FAIL。それ以外は PASS。軽微な指摘事項はブロック条件にならず、呼び出し元のトリアージステップに回されます。
完了条件
- すべてのチェックリスト項目について、根拠（ファイル：行番号またはコマンド出力）を示すか、理由を添えて明示的に該当なしとする。
- 上記の固定形式ブロックのみを返し、それ以外は何も返さず、コードを変更していないこと。