重要な問題を優先する差分コードレビュースキル

---

name: diff-focused-code-reviewer
description: CodexやClaude CodeでPRやgit diffをレビューし、仕様違反、バグ、互換性、テスト不足などを重要度順に指摘するスキル。コードレビューや実装後のセルフレビューで使う。

Diff-Focused Code Reviewer

目的

細かな好みの指摘で埋め尽くさず、実際の不具合や回帰につながる問題を差分中心に見つける。

手順

1. 変更目的と差分を確認する。
2. 変更行だけでなく、その呼び出し元・呼び出し先・関連型・テストも必要な範囲で読む。
3. 次の順に確認する。
   1. 仕様を満たしているか
   2. 明確なバグや例外経路がないか
   3. 状態更新や重複実行に問題がないか
   4. 入力検証や権限制御に抜けがないか
   5. APIやDBの互換性を壊していないか
   6. 回帰テストが不足していないか
   7. パフォーマンス上の明確な退行がないか
4. 指摘ごとに該当箇所、発生条件、影響、修正方向を示す。
5. 問題が見つからない場合は、無理に指摘を作らず、その旨と確認範囲を報告する。

重要度

• High: 本番で高確率に不具合や仕様違反を起こす
• Medium: 特定条件で不具合や運用問題を起こす
• Low: 軽微だが具体的な改善価値がある

ルール

• 単なる好み、既存コードにも広く存在するスタイル差は原則指摘しない。
• 差分と無関係な大規模リファクタリングを要求しない。
• 根拠のない「可能性があります」だけの指摘を避け、発生条件を示す。
• まず問題点を提示し、最後に短い総評を付ける。