Xs Code Reviewer

プルリクエストコードレビュー

GitHub CLIを使用してプルリクエストを分析し、バグ・リスク・回帰・不足テストを優先してレビューする。

重要: デフォルトはチャット上でのレビュー報告。GitHubへのコメント投稿、review投稿、approve/request changes は外部状態変更なので、ユーザーの明示OKを取ってから行う。

使用方法

PR#123をレビューして
owner/repo の PR#123 をレビューして
このPRを複数AIで見て

レビュー完了の定義

通常レビュー:

• PR情報と差分を確認した
• 必要に応じて xs-multi-agent で外部AIにも見せた
• findings を重要度順に統合した
• チャットでレビュー結果を報告した

GitHub投稿まで依頼された場合:

• 投稿対象（repo / PR / コメント種別）を明示してユーザーのOKを取った
• GitHubに投稿した
• 投稿URLまたはPR URLを報告した

レビュー手順

Step 1: PR情報の確認

# PRテンプレートがあれば確認
test -f .github/pull_request_template.md && cat .github/pull_request_template.md

# PR詳細
gh pr view <PR番号> --repo <owner>/<repo> --json title,body,headRefOid,baseRefName,headRefName,files,additions,deletions,url

# 差分
gh pr diff <PR番号> --repo <owner>/<repo>

# 変更ファイル一覧
gh pr diff <PR番号> --repo <owner>/<repo> --name-only

--repo <owner>/<repo> を明示する。remote が複数ある環境では、gh が別リポジトリを選ぶ事故が起きやすい。

Step 2: ファイル別詳細分析

各変更ファイルについて必要に応じて確認する。

gh pr diff <PR番号> --repo <owner>/<repo> -- <ファイルパス>

ローカルにブランチを取得する場合は、既存未コミット変更を確認してから行う。

Step 3: レビュー観点

優先順位:

• セキュリティ、データ損失、権限、公開範囲の問題
• バグ、回帰、エラーハンドリング不足
• テスト不足、検証不足
• 仕様・ドキュメントとの不整合
• 保守性、可読性、設計上の懸念
• 好みの問題や軽微なスタイル

コードレビューでは、良い点より先に findings を出す。問題がない場合は「ブロッカーなし」と明示し、残るリスクや未検証項目を書く。

Step 4: マルチエージェント検証

複数AIで見る依頼、重要PR、大きめの差分では xs-multi-agent を併用する。

CONFIG_PATH="$(bash skills/xs-multi-agent/scripts/check_agents.sh)"
cat "$CONFIG_PATH"

依頼プロンプトはファイルに保存してから、利用可能な外部エージェントへ渡す。

cat > /tmp/pr-review-prompt.txt <<'EOF'
以下のPR差分をレビューしてください。
バグ、セキュリティ、回帰、不足テストを優先し、好みの指摘は後回しにしてください。

<PR情報と差分>
EOF

bash skills/xs-multi-agent/scripts/run_agent.sh codex /tmp/pr-review-prompt.txt "$PWD"

外部AIが使えない場合は、自分だけでレビューしてよい。その場合は「外部AIは検出したが使用できなかった」と報告する。

Step 5: 結果統合

複数AIの意見はそのまま並べず、重複をまとめて重要度順に統合する。

重要度:

• blocker: セキュリティ、データ損失、明確なバグ、重大な回帰
• major: 修正した方がよい設計・仕様・テスト不足
• minor: 軽微な改善
• nit: 好みや表記揺れ。必要な時だけ

誤検出の可能性がある指摘は、必ず差分やファイルを再確認してから出す。

出力フォーマット

レビュー結果

- blocker: 0件
- major: 1件
- minor: 2件

Findings

1. major: <タイトル>
   - file: <path>:<line>
   - 内容:
   - 理由:
   - 修正案:

2. minor: <タイトル>
   - file:
   - 内容:

確認したこと
- `gh pr view ...`
- `gh pr diff ...`
- Codex: 使用 / 未使用（理由）

残るリスク
- ...

問題が見つからない場合:

ブロッカーなし。

確認したこと
- ...

残るリスク
- <未実行テストや見ていない範囲>

GitHubに投稿する場合

GitHub投稿は外部状態変更。以下を明示してOKを取る。

これから public/private repo <owner>/<repo> の PR #<番号> に以下を投稿します。
- review comment: <件数>
- summary review: 1件

OK？

OK後に投稿する。

サマリーのみ投稿

gh pr review <PR番号> --repo <owner>/<repo> --comment --body-file /tmp/review-summary.md

インラインコメント投稿

GitHubのインラインコメントは diff position や line/side 指定が必要。失敗しやすいので、投稿前に対象行と diff を確認する。

COMMIT_ID=$(gh pr view <PR番号> --repo <owner>/<repo> --json headRefOid --jq .headRefOid)

gh api repos/<owner>/<repo>/pulls/<PR番号>/comments \
  -F body="<コメント本文>" \
  -F commit_id="$COMMIT_ID" \
  -F path="<ファイルパス>" \
  -F line=<行番号> \
  -F side=RIGHT

投稿できない場合は、サマリーコメントにファイルパス・行番号付きでまとめる。

チェックリスト

• [ ] gh pr view でPR情報を取得した
• [ ] gh pr diff で差分を確認した
• [ ] 重要な変更ファイルを直接確認した
• [ ] 必要に応じて xs-multi-agent を使った
• [ ] findings を重要度順にした
• [ ] GitHub投稿が必要な場合、投稿前にユーザーのOKを取った
• [ ] 投稿した場合、投稿先URLを報告した

注意事項

• GitHub CLI（gh）がインストール・認証済みであることを確認
• PRテンプレートが存在しない場合は、標準的なレビュー観点を適用
• 大量の変更がある場合は、重要度の高い問題から優先する
• 建設的で具体的なフィードバックにする
• 初学者でも分かるように、専門用語には補足を入れる
• 外部AIの出力をそのまま信用しない。差分で再確認してから採用する