/code-review

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

セキュリティ、パフォーマンス、正確性、保守性という構造的な観点からコード変更をレビューします。

使用方法

/code-review <PR URL またはファイルパス>

提供されたコード変更をレビューする: @$1

特定のファイルまたは URL が提供されない場合は、何をレビューするかをお尋ねします。

仕組み

┌─────────────────────────────────────────────────────────────────┐
│                      CODE REVIEW                                   │
├─────────────────────────────────────────────────────────────────┤
│  スタンドアロン (常に機能)                                        │
│  ✓ diff、PR URL、またはファイルを貼り付け可能                    │
│  ✓ セキュリティ監査 (OWASP Top 10、インジェクション、認証)       │
│  ✓ パフォーマンスレビュー (N+1、メモリリーク、複雑性)            │
│  ✓ 正確性 (エッジケース、エラーハンドリング、競合状態)           │
│  ✓ スタイル (命名、構造、可読性)                                 │
│  ✓ コード例を含むアクション可能な提案                            │
├─────────────────────────────────────────────────────────────────┤
│  スーパーチャージ版 (ツール連携時)                               │
│  + ソース管理: PR diff を自動取得                                 │
│  + プロジェクトトラッカー: 検出結果をチケットにリンク            │
│  + ナレッジベース: チーム コーディング標準に照らし合わせて確認   │
└─────────────────────────────────────────────────────────────────┘

レビュー対象

セキュリティ

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

パフォーマンス

• N+1 クエリ
• 不要なメモリ割り当て
• アルゴリズム複雑性 (ホットパスでの O(n²))
• データベースインデックスの欠落
• 制限なしクエリまたはループ
• リソースリーク

正確性

• エッジケース (空入力、null、オーバーフロー)
• 競合状態と並行処理の問題
• エラーハンドリングと伝播
• 1 つずれたエラー
• 型安全性

保守性

• 命名の明確性
• 単一責任の原則
• 重複
• テストカバレッジ
• 自明でないロジックのドキュメント

出力

## Code Review: [PR タイトルまたはファイル名]

### Summary
[変更内容と全体的な品質の 1～2 文での概要]

### Critical Issues
| # | File | Line | Issue | Severity |
|---|------|------|-------|----------|
| 1 | [ファイル] | [行番号] | [説明] | 🔴 Critical |

### Suggestions
| # | File | Line | Suggestion | Category |
|---|------|------|------------|----------|
| 1 | [ファイル] | [行番号] | [説明] | Performance |

### What Looks Good
- [ポジティブな指摘]

### Verdict
[承認 / 変更要求 / 要議論]

コネクタ利用時

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

• URL から PR diff を自動取得
• CI ステータスとテスト結果を確認

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

• 検出結果を関連するチケットにリンク
• PR が記載要件に対応していることを確認

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

• チーム コーディング標準とスタイルガイドに照らし合わせて変更を確認

ヒント

1. コンテキストを提供する — 「これはホットパスです」または「これは PII を処理します」と書くと、焦点を絞れます。
2. 懸念を指定する — 「セキュリティに焦点を当てて」とすればレビュー内容を絞り込めます。
3. テストを含める — テストカバレッジと品質もチェックします。