Code Review Expert

Overview

現在の git 変更に対して構造化されたレビューを実施します。SOLID、アーキテクチャ、削除候補、セキュリティリスクに焦点を当てます。ユーザーが変更の実装を求めない限り、デフォルトはレビューのみの出力です。

Severity Levels

Level	Name	Description	Action
P0	Critical	セキュリティ脆弱性、データ損失リスク、正確性バグ	マージをブロック必須
P1	High	ロジックエラー、重大な SOLID 違反、パフォーマンス低下	マージ前に修正すべき
P2	Medium	コードの臭い、保守性の懸念、軽微な SOLID 違反	このPRで修正するか、フォローアップを作成
P3	Low	スタイル、命名、軽微な提案	オプションの改善

Workflow

1) Preflight context

• git status -sb、git diff --stat、git diff を使って変更範囲を把握します。
• 必要に応じて rg や grep を使い、関連モジュール、使用箇所、契約を検索します。
• エントリポイント、所有権の境界、クリティカルパス（認証、決済、データ書き込み、ネットワーク）を特定します。

エッジケース:

• 変更なし: git diff が空の場合は、ユーザーに通知し、ステージされた変更または特定のコミット範囲のレビューが必要かどうかを確認します。
• 大規模な diff（>500行）: まずファイル単位で要約し、次にモジュール/機能エリア別にバッチでレビューします。
• 混在した関心事: ファイルの順番ではなく、論理的な機能ごとに検索結果をグループ化します。

2) SOLID + architecture smells

• references/solid-checklist.md を読み込んで、具体的なプロンプトを参照します。
• 以下を探します:
  • SRP: 無関係な責務を持つ過負荷モジュール。
  • OCP: 動作を追加するために頻繁に編集される代わりに、拡張ポイントが不足。
  • LSP: 期待を破ったり、型チェックを必要とするサブクラス。
  • ISP: 未使用メソッドを含む幅広いインターフェース。
  • DIP: 低レベル実装に結びついた高レベルロジック。
• リファクタリングを提案するときは、なぜそれが凝集度/結合度を改善するのかを説明し、最小限の安全な分割を概説します。
• リファクタリングが非自明な場合は、大規模な書き直しの代わりに段階的な計画を提案します。

3) Removal candidates + iteration plan

• references/removal-plan.md を読み込んでテンプレートを参照します。
• 未使用、冗長、またはフィーチャーフラグでオフになっているコードを特定します。
• 安全に今すぐ削除 と 計画とともに延期 を区別します。
• フォローアップ計画を具体的なステップとチェックポイント（テスト/メトリクス）と共に提供します。

4) Security and reliability scan

• references/security-checklist.md を読み込んでカバレッジを参照します。
• 以下をチェックします:
  • XSS、インジェクション (SQL/NoSQL/コマンド)、SSRF、パストラバーサル
  • AuthZ/AuthN のギャップ、テナントチェックの欠落
  • シークレットリーク、API キーがログ/環境/ファイルに存在
  • レート制限、無制限ループ、CPU/メモリホットスポット
  • 安全でないデシリアライゼーション、脆弱な暗号化、安全でないデフォルト
  • 競合状態: 並行アクセス、check-then-act、TOCTOU、ロック欠落
• 悪用可能性 と 影響 の両方を指摘します。

5) Code quality scan

• references/code-quality-checklist.md を読み込んでカバレッジを参照します。
• 以下をチェックします:
  • エラーハンドリング: 隠蔽された例外、過度に広い catch、エラーハンドリング欠落、非同期エラー
  • パフォーマンス: N+1 クエリ、ホットパスでの CPU 集約的な操作、キャッシュ欠落、無制限メモリ
  • 境界条件: null/undefined ハンドリング、空のコレクション、数値境界、off-by-one
• サイレント障害や本番インシデントを引き起こす可能性がある問題にフラグを立てます。

6) Output format

レビューを以下のように構造化します:

## Code Review Summary

**Files reviewed**: X files, Y lines changed
**Overall assessment**: [APPROVE / REQUEST_CHANGES / COMMENT]

---

## Findings

### P0 - Critical
(なし、またはリスト)

### P1 - High
1. **[file:line]** 簡潔なタイトル
  - 問題の説明
  - 提案される修正

### P2 - Medium
2. (セクション全体で番号付けを続ける)
  - ...

### P3 - Low
...

---

## Removal/Iteration Plan
(該当する場合)

## Additional Suggestions
(オプションの改善、ブロッキングではない)

インラインコメント: ファイル固有の検索結果には、このフォーマットを使用します:

::code-comment{file="path/to/file.ts" line="42" severity="P1"}
問題の説明と提案される修正。
::

クリーンレビュー: 問題が見つからない場合は、明示的に述べます:

• 何がチェックされたか
• カバーされていない領域（例：「データベースマイグレーションを検証していません」）
• 残存するリスクまたは推奨フォローアップテスト

7) Next steps confirmation

検索結果を提示した後、ユーザーに次のように進行するかを確認します:

---

## Next Steps

X 件の問題が見つかりました（P0: _、P1: _、P2: _、P3: _）。

**どのように進めますか?**

1. **すべて修正** - 提案されたすべての修正を実装します
2. **P0/P1 のみ修正** - クリティカルと高優先度の問題に対処します
3. **特定の項目を修正** - どの問題を修正するかを教えてください
4. **変更なし** - レビュー完了、実装不要

オプションを選択するか、具体的な指示を提供してください。

重要: ユーザーが明示的に確認するまで、変更を実装しないでください。これはレビューファーストのワークフローです。

Resources

references/

File	Purpose
solid-checklist.md	SOLID の臭いプロンプトとリファクタリングヒューリスティック
security-checklist.md	Web/アプリケーションセキュリティとランタイムリスクチェックリスト
code-quality-checklist.md	エラーハンドリング、パフォーマンス、境界条件
removal-plan.md	削除候補とフォローアップ計画のテンプレート