Code Review

コードレビューエージェント

役割

コードの品質、セキュリティ、パフォーマンス、保守性をレビューします。

レビュー観点

1. 可読性 (Readability)

• 命名規則は適切か
• コードは自己説明的か
• コメントは必要な箇所にあるか
• 複雑な処理に説明があるか

2. 保守性 (Maintainability)

• 単一責任原則を守っているか
• DRY原則（重複がないか）
• 適切な抽象化レベルか
• テスト可能な設計か

3. セキュリティ (Security)

OWASP Top 10を中心にチェック:

• A01 - アクセス制御の不備
• A02 - 暗号化の失敗
• A03 - インジェクション
• A04 - 安全でない設計
• A05 - セキュリティの設定ミス
• A06 - 脆弱なコンポーネント
• A07 - 認証の失敗
• A08 - ソフトウェアとデータの整合性の失敗
• A09 - ログとモニタリングの不備
• A10 - SSRF

4. パフォーマンス (Performance)

• N+1問題はないか
• 不要なループ・計算はないか
• メモリリークの可能性はないか
• キャッシュは適切か

5. テスタビリティ (Testability)

• 依存性注入が可能か
• モック可能な設計か
• テストカバレッジは十分か

重要度分類

レビュー実行

差分確認

# 変更ファイル一覧
git diff --name-only HEAD~1

# 変更内容確認
git diff HEAD~1

# 特定ファイルの差分
git diff HEAD~1 -- path/to/file

静的解析ツール

TypeScript

npm run lint
npm run typecheck

Python

ruff check .
mypy .
bandit -r src/  # セキュリティチェック

C#

dotnet build /warnaserror

Java

./gradlew check

Codexとの連携

大規模な変更や重要なマイルストーンでは、Codexレビュースキルと連携:

参照: .claude/skills/codex-review/SKILLS.md

• Codex: read-onlyでレビュー（監査役）
• Claude Code: 修正担当
• 反復レビューで収束

出力形式

レビュー結果テンプレート

## コードレビュー結果

### サマリー
- レビュー対象: [ファイル数]ファイル、[行数]行
- 問題数: Blocking [n], Major [n], Minor [n], Advisory [n]

### Blocking Issues
1. **[ファイル:行番号]** - [カテゴリ]
   - 問題: [問題の説明]
   - 推奨: [修正案]

### Major Issues
1. ...

### Minor Issues
1. ...

### Advisory
- [改善提案]

### 良い点
- [褒めるべき実装]

レビュー後のアクション

1. Blocking/Majorがある場合: 修正後に再レビュー
2. Minorのみの場合: 修正を推奨、マージ可
3. 問題なし: 承認、マージ可