コードレビュー — AIによる品質ゲート

コードを書いたら、すぐにレビューする。この習慣が、AIコーディングの品質を決定的に分ける。

なぜ即時レビューが重要か

AIが書いたコードは「もっともらしい」が、以下のリスクを含む可能性がある。

• セキュリティ脆弱性: SQLインジェクション、XSS、CSRF
• ロジックエラー: エッジケースの見落とし
• パフォーマンス問題: N+1クエリ、不要な再レンダリング
• 保守性の低下: 過度に複雑な実装、命名の不統一

コードを書いてから時間が経つほど、問題の修正コストは上がる。書いた直後にレビューすることで、問題を最小コストで修正できる。

code-reviewer エージェント

> 今の変更をcode-reviewerエージェントでレビューして

code-reviewerは以下の観点でコードを分析する。

レビュー観点

1. 可読性: 命名、構造、コメントの適切さ
2. 保守性: 関数の長さ、ファイルの凝集度、結合度
3. パフォーマンス: 不要な計算、メモリリーク
4. エラーハンドリング: 例外処理、エッジケース対応
5. コーディング規約: プロジェクトの規約との整合性

重要度レベル

security-reviewer エージェント

> security-reviewerエージェントでセキュリティチェックを実行して

セキュリティに特化したレビューを行う。

チェック項目

• OWASP Top 10: インジェクション、認証不備、XSS等
• 秘密情報の漏洩: ハードコードされたAPIキー、パスワード
• 入力バリデーション: ユーザー入力の未検証
• 暗号化: 安全でない暗号アルゴリズムの使用
• 依存関係: 既知の脆弱性を持つパッケージ

医療データの特別な考慮

医療プロジェクトでは追加のセキュリティ要件がある。

• 患者データの暗号化
• アクセスログの記録
• セッション管理の適切性
• データの最小化原則

レビューワークフロー

推奨フロー

コード作成 → code-reviewer → 修正 → security-reviewer → 修正 → コミット

並列レビュー

時間を節約するため、2つのレビュアーを並列で実行することもできる。

> 以下を並列で実行して:
> 1. code-reviewerでコード品質をチェック
> 2. security-reviewerでセキュリティをチェック

PRレビュー

GitHub上のPRをClaude Codeでレビューすることもできる。

> gh pr view 42 の変更内容をレビューして

PRの差分を分析し、コード品質とセキュリティの両面からフィードバックを生成する。