review-pr

PR レビュー（投稿しないドラフト作成）

GitHub PR をレビューし、個別インラインコメントのドラフトを作成して提示します。

最重要ルール

この Skill は GitHub へ一切投稿しない。

• gh pr review、gh api --method POST .../reviews、gh api --method POST .../comments、gh pr comment を実行してはならない。
• 読み取り専用の gh 呼び出し（gh pr view / gh pr diff / gh api の GET）のみ使用する。
• 投稿は利用者がドラフトを確認したうえで /post-review を実行したときにのみ行われる。
• 利用者から「そのまま投稿して」と言われた場合も、この Skill 内では投稿せず /post-review に引き継ぐ。

引数

$ARGUMENTS から以下を解析:

1. 対象 PR（いずれか）
   • URL: https://github.com/{owner}/{repo}/pull/{number}
   • 番号のみ: 6958（カレントリポジトリの PR とみなす）
   • 省略: gh pr view --json number,url でカレントブランチに紐づく PR を特定。特定できなければ利用者に確認して終了
2. 追加観点（自由記述、省略可）
   • 例: 「セキュリティを重点的に」「個人情報保護法・GDPR との兼ね合いもチェック」
   • 例: 「関連 issue の再現手順を確認して」
   • 例: 「~/git-repos/stripe-payment-plugin への影響調査もお願いします」
   • 追加観点は通常のレビュー観点に上乗せする。指定されたものだけを見るのではない

手順

1. コンテキストの収集

# PR メタ情報（headRefOid = HEAD コミット SHA。ドラフトの commit_id に使う）
gh pr view {number} --repo {owner}/{repo} \
  --json number,title,body,author,baseRefName,headRefName,headRefOid,state,isDraft,mergeable,additions,deletions,changedFiles,labels,url

# 差分（ローカル git diff ではなくこれを使う）
gh pr diff {number} --repo {owner}/{repo}

# 既存レビュー・既存インラインコメント（重複指摘の回避）
gh api repos/{owner}/{repo}/pulls/{number}/reviews --jq '.[] | {user: .user.login, state, body: (.body[0:200])}'
gh api repos/{owner}/{repo}/pulls/{number}/comments --paginate \
  --jq '.[] | {user: .user.login, path, line, body: (.body[0:200])}'

# CI 状況
gh pr checks {number} --repo {owner}/{repo}

関連 issue の確認: PR 本文の Closes #N / Refs #N / Fixes #N や、利用者が提示した issue URL があれば gh issue view N --repo {owner}/{repo} --comments で本文とコメントを読む。issue に再現手順が書かれている場合は、それが PR の変更で解消されるかを差分から追う。

既出指摘の除外: CodeRabbit / Gemini Code Assist / 他レビュアーが既に指摘済みの内容は、ドラフトに重複して載せない。既に別 issue 化されているものも同様。

2. 差分の読解

差分だけで判断せず、変更されたファイルの前後関係を Read で確認する。以下は毎回見る観点:

• 正確性: 分岐漏れ、null/未定義、境界値、例外経路、トランザクション境界
• 後方互換性: 公開 API・エンティティ・テンプレート・イベント・設定キーの削除や意味変更
• セキュリティ: 入力検証、権限チェック、CSRF、XSS、SQL/コマンドインジェクション、パストラバーサル、機密情報のログ出力
• 影響範囲: 変更されたメソッド・定数・設定の呼び出し元を rg で洗い出す（1 箇所ずつではなく全件まとめて）
• テスト: 変更に対応するテストの有無、テストが実際に退行を検出できるか
• PR の記述と実装の一致: 本文の説明どおりの変更になっているか