---

name: code-review description: コード差分を詳細にレビュー argument-hint: [base-branch] [--issue NUMBER] [--uncommitted] disable-model-invocation: false

/code-review

コード差分を経験豊富なシニアエンジニアとして、本番運用を考慮しながら詳細にレビューし、Approve可能か判断

使用方法

/code-review [base-branch] [--issue ISSUE_NUMBER] [--uncommitted]

引数:

• base-branch: 比較対象のブランチ（省略時: PRがあればPRのベースブランチ、なければデフォルトブランチを自動判定）
• --issue ISSUE_NUMBER: 指定したIssue要件を満たしているか確認
• --uncommitted: 未コミットの差分をレビュー（git diffを使用）

例:

/code-review                              # PRがあればPRのベースブランチ、なければデフォルトブランチとの差分をレビュー
/code-review main                         # mainブランチとの差分をレビュー
/code-review origin/feature-auth          # feature-authブランチとの差分をレビュー
/code-review main --issue 123             # Issue #123の要件も確認
/code-review --uncommitted                # 未コミットの差分をレビュー
/code-review --issue 456 --uncommitted    # 未コミット差分をIssue #456の観点で確認

実行内容

1. 引数の解析

• 第1引数（--で始まらない）があればベースブランチ名として使用
• 第1引数がない、または--で始まる場合はベースブランチを自動判定:
  1. gh pr view --json baseRefName --jq '.baseRefName' でPRのベースブランチを取得し、origin/ を付加
  2. PRがなければ git symbolic-ref refs/remotes/origin/HEAD でデフォルトブランチを取得
• --issue ISSUE_NUMBER: Issue番号を抽出
• --uncommitted: 未コミット差分フラグを設定

解析例:

/code-review feature/auth --issue 123
→ base-branch: "feature/auth", issue: 123, uncommitted: false

/code-review --issue 456 --uncommitted
→ base-branch: (自動判定: PRがあれば "origin/<PRのベースブランチ>"), issue: 456, uncommitted: true

/code-review origin/main
→ base-branch: "origin/main", issue: null, uncommitted: false

2. PRコンテキスト確認（自動）

• セクション1でPRからベースブランチを取得できた場合、同時にPR情報も取得:
  1. PRの説明・目的を確認
  2. 関連Issueがあれば要件を確認（--issue未指定時）
  3. レビュー後、説明と実際の変更の整合性を評価
• PRが存在しない場合はスキップ

3. 差分取得と確認

• --uncommittedが指定されている場合:
  • git diff で既存ファイルの未コミット差分を取得
  • git ls-files --others --exclude-standard でトラックされていない新規ファイルを確認
  • トラックされていないファイルがあれば、その内容も読み込んでレビュー対象に含める
  • トラックされていないファイルがある場合は、レビュー結果で明示的に言及する
• 指定されていない場合:
  • 必ず解析済みのベースブランチを使用 (git diff <base-branch>...HEAD)
  • 差分が大きい場合は git diff <base-branch>...HEAD --name-only で変更ファイル一覧を先に確認
  • 各ファイルを git diff <base-branch>...HEAD -- <ファイル名> で個別に確認
  • 全体を把握してからレビュー評価を実施

重要原則:

• 全差分確認: 必ず全ての変更内容を確認してから評価する
• 段階的確認: ファイル一覧 → 個別差分 → 全体評価の順で進める
• 部分的判断禁止: 一部の差分や最初の部分だけを見て結論を出さない
• 見落とし防止: 特に新規追加ファイルや定数・インターフェース変更は必ず確認する

4. Issue情報取得（--issue指定時）

• gh issue view <ISSUE_NUMBER> でIssue内容を取得
• 要件・仕様を確認

5. レビュー実行

以下の観点から詳細にレビュー:

基本観点

• コードの品質（可読性、保守性）
• バグや潜在的な問題の有無
• セキュリティリスク（OWASP Top 10等）
• パフォーマンスへの影響
• テストコードの妥当性
  • 無駄なテストがないか（過剰なモック、重複テスト等）
  • 必要なテストの抜け漏れがないか（正常系・異常系・エッジケース）
  • テストの可読性（テスト名、構造、意図の明確さ）
• エラーハンドリングの適切性
• 命名規則やコーディングスタイルの一貫性

本番運用観点

• 本番環境でのログ・監視は十分か
• ロールバック可能な実装か
• データベースマイグレーションの安全性
• エッジケースや境界値の考慮
• 障害時の影響範囲と復旧方法
• スケーラビリティへの影響

Issue指定時の追加観点

• Issue要件を満たしているか
• 仕様から逸脱していないか
• 関連する要件がすべて実装されているか

6. レビュー結果出力

以下の形式で出力:

## レビュー結果

### 総合評価
- Approve可能 / 修正が必要 / 要議論

### 良い点
- ...

### 指摘事項
#### 必須修正
- ...

#### 推奨修正
- ...

#### 質問・確認事項
- ...

### Issue要件の充足状況（--issue指定時のみ）
- ...

注意事項

• ベースブランチが存在しない場合はエラーを通知
• Issue番号が不正な場合はエラーを通知
• ベースブランチの自動判定優先順位: PRのベースブランチ → デフォルトブランチ → origin/main（フォールバック）