reviewing-code

コードレビュー

ユーザーがコードレビューやコードへのフィードバックを求めている場合、またはコードの品質を確認するよう依頼している場合に、このスキルを使用します。

手順

1. 変更内容を理解する — ファイルまたは差分を読み、コードの目的を理解します。変更の範囲（新機能、バグ修正、リファクタリング）を特定します。

2. 正確性を確認する

   • エッジケース（空の入力、null、ゼロ、負の数）に対応していますか？
   • エラー状態（try/catch、エラーバウンダリ、フォールバック UI）に対応していますか？
   • 非同期コードは競合状態、キャンセル、タイムアウトに対応していますか？
   • ループや配列アクセスにオフバイワンエラーはありませんか？

3. 保守性を確認する

   • 関数は単一の責務に集中していますか？
   • 変数名や関数名は内容を適切に表していますか？
   • 抽出すべき不要な重複はありませんか？
   • マジックナンバーは名前付き定数に置き換えられていますか？
   • コードの複雑さは妥当ですか（深くネストされた条件分岐、長い関数など）？

4. パフォーマンスを確認する

   • データベースアクセスに N+1 クエリのパターンはありませんか？
   • レンダリングループ内でコストの高い計算や API 呼び出しが行われていませんか？
   • 大きなリストに仮想化やページネーションがありませんか？
   • よく使われるデータベースクエリに必要なインデックスがありませんか？
   • メモ化は適切に使用されていますか（過剰に適用されていませんか）？

5. 型安全性を確認する（TypeScript プロジェクトの場合）

   • 絞り込むべき any 型はありませんか？
   • 公開 API の関数の戻り値の型は明示されていますか？
   • ユニオン型は網羅的に処理されていますか？

6. テストを確認する

   • 新規または変更されたコードのテストはありますか？
   • テストは正常系とエラーケースの両方をカバーしていますか？
   • テストは分離されていますか（変更可能な状態を共有していませんか）？

7. フィードバックを提供する — 指摘事項を重要度ごとに整理します。

   • 必須修正：バグ、セキュリティ上の問題、データ損失のリスク
   • 修正推奨：パフォーマンスの問題、保守性に関する懸念
   • 軽微な指摘：スタイルの好み、軽微な提案

   各指摘には、ファイル、行、問題点、修正案を含めます。

注意事項

• 建設的な姿勢を保ちます。問題である理由を説明し、問題だと指摘するだけにとどめないでください。
• 改善が必要な点だけでなく、適切にできている点も認めます。
• リンターやフォーマッターで処理できるスタイル上の問題に、過度にこだわらないでください。