code-review

/code-review

見慣れないプレースホルダーがある場合や、接続されているツールを確認する必要がある場合は、CONNECTORS.md を参照してください。

セキュリティ、パフォーマンス、正確性、保守性の観点から、コードの変更を体系的にレビューします。

使用方法

/code-review <PR URL or file path>

提供されたコードの変更をレビューします：@$1

レビュー対象のファイルや URL が指定されていない場合は、何をレビューするか質問します。

動作の仕組み

┌─────────────────────────────────────────────────────────────────┐
│                      CODE REVIEW                                   │
├─────────────────────────────────────────────────────────────────┤
│  STANDALONE (always works)                                       │
│  ✓ Paste a diff, PR URL, or point to files                      │
│  ✓ Security audit (OWASP top 10, injection, auth)               │
│  ✓ Performance review (N+1, memory leaks, complexity)           │
│  ✓ Correctness (edge cases, error handling, race conditions)    │
│  ✓ Style (naming, structure, readability)                        │
│  ✓ Actionable suggestions with code examples                    │
├─────────────────────────────────────────────────────────────────┤
│  SUPERCHARGED (when you connect your tools)                      │
│  + Source control: Pull PR diff automatically                    │
│  + Project tracker: Link findings to tickets                     │
│  + Knowledge base: Check against team coding standards           │
└─────────────────────────────────────────────────────────────────┘

レビューの観点

セキュリティ

• SQL インジェクション、XSS、CSRF
• 認証と認可の不備
• コード内のシークレットや認証情報
• 安全でないデシリアライゼーション
• パストラバーサル
• SSRF

パフォーマンス

• N+1 クエリ
• 不要なメモリ割り当て
• アルゴリズムの計算量（ホットパスでの O(n²)）
• データベースインデックスの不足
• 上限のないクエリやループ
• リソースリーク

正確性

• エッジケース（空の入力、null、オーバーフロー）
• 競合状態や並行処理の問題
• エラー処理と伝播
• オフバイワンエラー
• 型安全性

保守性

• 分かりやすい命名
• 単一責任
• 重複
• テストカバレッジ
• 自明でないロジックのドキュメント

出力

## Code Review: [PR title or file]

### Summary
[1-2 sentence overview of the changes and overall quality]

### Critical Issues
| # | File | Line | Issue | Severity |
|---|------|------|-------|----------|
| 1 | [file] | [line] | [description] | 🔴 Critical |

### Suggestions
| # | File | Line | Suggestion | Category |
|---|------|------|------------|----------|
| 1 | [file] | [line] | [description] | Performance |

### What Looks Good
- [Positive observations]

### Verdict
[Approve / Request Changes / Needs Discussion]

コネクターが利用可能な場合

~~ソース管理が接続されている場合：

• URL から PR の差分を自動的に取得します
• CI の状態とテスト結果を確認します

~~プロジェクトトラッカーが接続されている場合：

• 指摘事項を関連チケットに紐付けます
• PR が明記された要件に対応しているか確認します

~~ナレッジベースが接続されている場合：

• チームのコーディング規約やスタイルガイドに照らして変更を確認します

ヒント

1. 背景情報を提供する —「これはホットパスです」や「これは PII を扱います」と伝えると、重点を絞ってレビューできます。
2. 懸念事項を指定する —「セキュリティに重点を置いてください」と指定すると、レビューの範囲を絞れます。
3. テストを含める — テストのカバレッジと品質も確認します。