| name | review-branch |
| description | 現在のブランチの実装をmainとの差分でレビューする。PR作成前のセルフレビュー用途。コード品質・セキュリティ・パフォーマンス・テストの観点でフィードバックを返す。修正は行わずレビュー結果のみ報告 (READ-ONLY)。 |
| when_to_use | PR作成前にローカルブランチの実装をセルフレビューしたいとき。Triggers: 'ブランチをレビュー', 'PR前にレビュー', 'セルフレビュー', 'review my branch', 'review before PR', 'self review', 'pre-PR review' |
| argument-hint | [focus-area] (例: security, performance, tests) |
| allowed-tools | ["Bash","Read","Glob","Grep"] |
ブランチ実装レビューコマンド
現在のブランチで実装された変更をPR作成前にコードレビューする。mainブランチとの差分を分析し、コード品質・セキュリティ・パフォーマンスの観点でフィードバックを返す。
/ndf:review との使い分け
| 観点 | review-branch | review |
|---|
| 対象 | ローカルブランチの差分(PR前) | GitHub上の既存PR |
| 判定 | フィードバックを返す | Approve / Request Changes を判定 |
| 用途 | PR作成前のセルフレビュー | PR作成後のレビュー |
使用方法
/ndf:review-branch # 全般レビュー
/ndf:review-branch security # セキュリティに焦点
/ndf:review-branch performance # パフォーマンスに焦点
/ndf:review-branch tests # テスト網羅性に焦点
/ndf:review-branch "ビジネスロジック" # 任意のフォーカス
レビュー手順
1. 変更の把握
git diff main --name-only
git diff main --stat
git log main..HEAD --oneline
2. 変更内容の分析
各変更ファイルに対して以下を確認:
- 追加・変更されたロジック: 意図が明確か、正しく実装されているか
- テストカバレッジ: 適切なテストが追加されているか
- コーディング規約: プロジェクトの規約に準拠しているか
3. 品質チェック観点
コード品質
- 命名規則の一貫性
- 関数/メソッドの責務(単一責任原則)
- DRY原則(重複コードの排除)
- 可読性・保守性
- 過剰な抽象化がないか(YAGNI)
セキュリティ
- SQLインジェクション対策
- XSS対策
- CSRF対策
- 入力値バリデーション
- 認証・認可の適切性
- 機密情報(トークン、キー、PII)の取り扱い
パフォーマンス
- N+1 クエリの有無
- 不要なデータベースアクセス
- メモリ使用量
- インデックスの活用
エラーハンドリング
- 例外が適切に捕捉されているか
- ログ出力の妥当性(詳細は
/ndf:logging-guidelines)
- リトライ/タイムアウトの設計
4. レビュー結果の報告
## レビュー結果
### 概要
- 変更ファイル数: X
- 追加行数: +XXX
- 削除行数: -XXX
### Good(良い点)
- ...
### Suggestions(改善提案)
- `path/to/file.ext:123` — 提案内容
### Issues(要修正)
- `path/to/file.ext:456` — 問題点と修正方針
使用例
/ndf:review-branch
/ndf:review-branch security
/ndf:review-branch performance
/ndf:review-branch tests
注意事項
- 大量の変更がある場合、重要な変更から優先的にレビューする
- 自動品質チェック(linter, formatter, type checker)は事前実行済みを前提とする
- レビュー結果は提案であり、最終判断は開発者が行う
- コード修正は行わない(分析とフィードバックのみ。修正は
/ndf:fix で別途実行)
関連
/ndf:review — PR単位レビュー (Approve/Request Changes判定)
/ndf:review-pr-comments — 既存PRコメントの分類
/ndf:fix — PRレビューコメントの修正対応
/ndf:logging-guidelines — ログ設計