piv-review-changes

コードレビュー

最近変更されたファイルの技術的なコードレビューを行います。

基本原則

レビューの理念：

• シンプルさこそ究極の洗練です。すべての行に存在意義が必要です
• コードは書くよりもはるかに頻繁に読まれます。読みやすさを優先してください
• 最善のコードとは、多くの場合、書かないコードです
• 意図の明確さと表現の簡潔さから、洗練さが生まれます

レビュー対象

まず、コードベースの標準とパターンを理解するために、コードベースのコンテキストを把握します。

最初に次の項目を確認します：

• CLAUDE.md
• README.md
• コアモジュールの主要ファイル
• docs ディレクトリに記載された標準（および .claude/references/ 内のドキュメント）

十分に理解できたら、次のコマンドを実行します：

git status
git diff HEAD
git diff --stat HEAD

次に、新しいファイルの一覧を確認します：

git ls-files --others --exclude-standard

新しいファイルはすべて全体を読みます。変更されたファイルも、全体のコンテキストを理解するために差分だけでなく全体を読みます。

変更または追加されたファイルごとに、次の点を分析します：

1. ロジックエラー

   • オフバイワンエラー
   • 条件式の誤り
   • エラーハンドリングの欠落
   • 競合状態

2. セキュリティ上の問題

   • SQL インジェクションの脆弱性
   • XSS の脆弱性
   • 安全でないデータ処理
   • 秘密情報や API キーの露出

3. パフォーマンスの問題

   • N+1 クエリ
   • 非効率なアルゴリズム
   • メモリーリーク
   • 不要な計算

4. コード品質

   • DRY 原則への違反
   • 過度に複雑な関数
   • 不適切な命名
   • 型ヒント／アノテーションの欠落

5. コードベースの標準や既存パターンへの準拠

   • docs ディレクトリに記載された標準への準拠
   • リント、型付け、フォーマットの標準
   • ロギングの標準
   • テストの標準

問題が実在することを確認する

• 見つかった問題に対する個別のテストを実行します
• 型エラーが実際に問題となることを確認します
• コンテキストを踏まえてセキュリティ上の懸念を検証します

出力形式

.claude/code-reviews/[appropriate-name].md に新しいファイルを保存します

統計：

• 変更されたファイル数：0
• 追加されたファイル数：0
• 削除されたファイル数：0
• 追加された行数：0
• 削除された行数：0

見つかった問題ごとに：

severity: critical|high|medium|low
file: path/to/file.py
line: 42
issue: [one-line description]
detail: [explanation of why this is a problem]
suggestion: [how to fix it]

問題が見つからなかった場合：「コードレビューに合格しました。技術的な問題は検出されませんでした。」

重要

• 具体的に記載します（曖昧な指摘ではなく行番号を示してください）
• スタイルではなく、実際のバグに焦点を当てます
• 不満を述べるだけでなく、修正案を提案します
• セキュリティ上の問題には CRITICAL のフラグを付けます