shiiman Review PR
PR をレビューし、結果をローカル表示または GitHub に投稿します。
注意: 自分の PR に付いたレビューコメントに対応する場合は
pr-review-checkを使用してください。
Help
$ARGUMENTS に --help が含まれる場合、以下を表示して終了:
/shiiman-github:pr-review - PR レビュー
概要:
PR をレビューし、結果をローカル表示または GitHub に投稿する。
既定はローカル表示のみ。投稿・承認は発話から判断し、実行前に必ず確認する。
使用方法:
/shiiman-github:pr-review [PR番号]
オプション:
--help このヘルプを表示
出力先・レビュー結果の伝え方（フラグの代わり）:
指定なし → ローカル表示のみ（GitHub には投稿しない）
「GitHub に投稿して」 → レビューコメントを GitHub に投稿
「approve」「承認」「LGTM」 → Approve として投稿
「要修正」「変更要求」 → Request Changes として投稿
例:
/shiiman-github:pr-review # 現在のブランチの PR をレビュー（ローカル表示）
/shiiman-github:pr-review 123 # PR #123 をレビュー（ローカル表示）
「PR #123 をレビューして投稿して」 # レビューして GitHub に投稿
「PR #123 を approve して」 # PR #123 を承認
起動時の自動判断（フラグの代わり）
引数・発話から出力先とレビュー結果を判定する。明示がなければローカル表示のみ（誤投稿防止の安全側）とする。
| 項目 | 既定 | 判断ルール |
|---|---|---|
| 出力先 | ローカル表示 | 「投稿して」「GitHub に」「approve」「要修正」等あれば投稿 |
| レビュー結果 | コメント | 「approve」「承認」「LGTM」→ Approve／「要修正」→ Request Changes |
GitHub への投稿・承認・変更要求は取り消しにくい外部操作のため、実行前に必ず AskUserQuestion で確認する（後述のステップ 6）。判断に迷う場合はローカル表示にとどめる。
ワークフロー
1. 対象 PR の特定
- PR 番号が指定されている場合: その PR をレビュー
- PR 番号が指定されていない場合: 現在のブランチの PR を自動検出
# PR 番号未指定時の自動検出
gh pr view --json number,title,url
2. レビュータイプの確認
ユーザーにレビュータイプを確認（複数選択可）:
| タイプ | 説明 | チェック観点 |
|---|---|---|
general |
全体レビュー | コード品質、設計、テスト |
security |
セキュリティ | OWASP Top 10、入力検証、認証・認可 |
performance |
パフォーマンス | N+1、メモリリーク、アルゴリズム効率 |
デフォルト: general
3. PR 内容の取得
# PR 情報取得
gh pr view {pr番号} --json title,body,files,additions,deletions
# 変更差分取得
gh pr diff {pr番号}
4. レビュー実行
全体レビュー（general）:
- コードスタイル・命名規則
- 設計パターン・責務分離
- エラーハンドリング
- テストカバレッジ
- ドキュメント
セキュリティレビュー（security）:
- インジェクション（SQL, XSS, Command）
- 認証・認可の実装
- 機密情報の扱い
- 入力検証
- セッション管理
パフォーマンスレビュー（performance）:
- N+1 クエリ
- メモリリーク
- 不要なループ・計算
- キャッシュ戦略
- 非同期処理
5. 重要度の分類
| レベル | 説明 |
|---|---|
| Critical | セキュリティリスク、アーキテクチャ違反 |
| High | パフォーマンス問題、重要な規約違反 |
| Medium | コード品質向上 |
| Low | スタイル提案 |
6. 結果の出力
ローカル表示のみ（既定）: ローカルに結果を表示し、ここで終了する
## コードレビュー結果
### サマリー
{overall_assessment}
### 良い点
- {good_point_1}
### 改善提案
- [ ] {suggestion_1}
### 重要度: 高
{high_priority_issues}
### 重要度: 中
{medium_priority_issues}
### 重要度: 低
{low_priority_issues}
投稿と判定した場合: GitHub に投稿する前に、AskUserQuestion で必ず確認する（外部・不可逆操作）。
question: "レビュー結果を GitHub に投稿しますか？"
options:
- コメントとして投稿: gh pr review --comment で投稿
- Approve として投稿: gh pr review --approve で承認
- Request Changes として投稿: gh pr review --request-changes で変更要求
- 投稿しない: ローカル表示のみで終了
確認で投稿が選ばれた場合のみ、対応するコマンドを実行する。
# コメントとして投稿
gh pr review {pr番号} --comment --body "{レビュー内容}"
# Approve として投稿
gh pr review {pr番号} --approve --body "{レビュー内容}"
# Request Changes として投稿
gh pr review {pr番号} --request-changes --body "{レビュー内容}"
7. 結果報告
PR #{pr番号} のレビューを完了しました。
レビュータイプ: {types}
出力先: {ローカル表示|GitHub 投稿}
重要な指摘: {high_priority_count}件
改善提案: {suggestion_count}件
簡易承認フロー（「approve」「LGTM」のみ伝えられた場合）
レビュー依頼なしで「approve」「承認」「LGTM」だけ伝えられた場合、詳細レビューをスキップして承認のみ行う。承認は GitHub への外部・不可逆操作のため、実行前に必ず AskUserQuestion で確認する。
1. PR 状態の確認
gh pr view {pr番号} --json title,state,reviews,mergeable,statusCheckRollup
以下を確認:
- 未対応の必須修正がないか
- CI が通っているか
- マージ可能な状態か
2. 承認実行（確認後）
AskUserQuestion で承認の実行可否を確認し、承認が選ばれた場合のみ実行する。
gh pr review {pr番号} --approve --body "{承認コメント}"
3. 結果報告
PR #{pr番号} を承認しました。
ステータス: ✅ Approved
マージ可能: {mergeable}
マージする場合:
gh pr merge {pr番号}
承認コメントテンプレート
シンプル:
LGTM! 🎉
詳細:
## Approved
{approval_comment}
### 確認済み項目
- [x] コード品質
- [x] テスト
- [x] ドキュメント
LGTM! 🎉
重要な注意事項
- ✅ 投稿の意図が明示されない場合はローカル表示のみ（誤投稿防止の安全側）
- ✅ GitHub への投稿・承認・変更要求は実行前に必ず AskUserQuestion で確認する
- ✅ 「approve」「LGTM」のみの場合は簡易承認フローへ（確認後に承認）
- ✅ 具体的な改善提案を含める
- ✅ 良い点も指摘する
- ✅ 重要度を明確にする
- ❌ 曖昧なコメントを避ける
- ❌ 個人攻撃的なコメントは禁止