# Code Review

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

- Skill: `infinith4/code-review` (Agent Skill)
- Install (CLI): `npx skillmds@latest add infinith4/code-review`
- Raw SKILL.md: https://api.skillmd.com/api/skills/infinith4/code-review/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Coding & Dev Tools
- Author: infinith4 (https://skillmd.com/u/infinith4)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/infinith4/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)
- 依存性注入が可能か
- モック可能な設計か
- テストカバレッジは十分か

## 重要度分類

| 重要度 | 説明 | 対応 |
|--------|------|------|
| **Blocking** | 修正必須。セキュリティ脆弱性、重大バグ | マージ前に必ず修正 |
| **Major** | 重要な問題。パフォーマンス、設計問題 | 修正を強く推奨 |
| **Minor** | 軽微な問題。スタイル、命名 | 可能なら修正 |
| **Advisory** | 参考情報。改善提案 | 次回以降で検討 |

## レビュー実行

### 差分確認
```bash
# 変更ファイル一覧
git diff --name-only HEAD~1

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

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

### 静的解析ツール

#### TypeScript
```bash
npm run lint
npm run typecheck
```

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

#### C#
```bash
dotnet build /warnaserror
```

#### Java
```bash
./gradlew check
```

## Codexとの連携

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

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

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

## 出力形式

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

```markdown
## コードレビュー結果

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

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

### Major Issues
1. ...

### Minor Issues
1. ...

### Advisory
- [改善提案]

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

## レビュー後のアクション

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