PRの審査

概要

徹底したPR審査専用の評価を実施し、/preparepr準備状況について構造化された推奨事項を返します。

入力

• PRの番号またはURLを要求してください。
• 指定されていない場合は、必ず質問してください。会話から自動検出しないでください。
• 曖昧な場合は、確認してください。

セキュリティ

• mainまたはorigin/mainへのプッシュを絶対に行わないでください。審査中も、その他の場合も、行いません。
• 審査中にgit pushを実行しないでください。審査を読み取り専用として扱ってください。
• ゲートウェイを停止または終了しないでください。ゲートウェイ停止コマンドを実行しないでください。ポート18792のプロセスを終了しないでください。

実行ルール

• ワークフローを実行してください。TODOチェックリストを出力した後で停止しないでください。
• 委譲する場合、委譲先がコマンドを実行して出力をキャプチャすることを要求してください。計画ではなく。

既知の失敗パターン

• 「fatal: not a git repository」が表示される場合、間違ったディレクトリにいます。~/openclawを使用してください。
• チェックリストを出力した後で停止しないでください。それは完了ではありません。

出力の文体

• カジュアルで直接的に書いてください。
• ダッシュの使用は避けてください。コンマまたは文を分けてください。

完了基準

• ワークツリーでコマンドを実行してPRを直接検査してください。
• セクションA〜Jの構造化された審査を作成してください。
• 完全な審査をワークツリー内の.local/review.mdに保存してください。

最初に: TODOチェックリストを作成

すべての審査ステップのチェックリストを作成して出力し、その後続行してコマンドを実行してください。

セットアップ: ワークツリーを使用

すべての審査作業に隔離されたワークツリーを使用してください。

cd ~/Development/openclaw
# 確認: リポジトリにいることを確認
git rev-parse --show-toplevel

WORKTREE_DIR=".worktrees/pr-<PR>"
git fetch origin main

# 既存ワークツリーがあれば再利用、なければ新規作成
if [ -d "$WORKTREE_DIR" ]; then
  cd "$WORKTREE_DIR"
  git checkout temp/pr-<PR> 2>/dev/null || git checkout -b temp/pr-<PR>
  git fetch origin main
  git reset --hard origin/main
else
  git worktree add "$WORKTREE_DIR" -b temp/pr-<PR> origin/main
  cd "$WORKTREE_DIR"
fi

# /reviewpr から /preparepr から /mergepr にかけて保持されるローカルスクラッチスペースを作成
mkdir -p .local

ワークツリーディレクトリ内ですべてのコマンドを実行してください。 origin/mainから開始して、PRコードを見る前に既存実装をチェックできるようにしてください。

ステップ

1. PRメタデータと背景を特定

gh pr view <PR> --json number,title,state,isDraft,author,baseRefName,headRefName,headRepository,url,body,labels,assignees,reviewRequests,files,additions,deletions --jq '{number,title,url,state,isDraft,author:.author.login,base:.baseRefName,head:.headRefName,headRepo:.headRepository.nameWithOwner,additions,deletions,files:.files|length,body}'

2. PRブランチを見る前に、これが既にmainに存在するかチェック

• PR タイトルと説明からコア機能または修正を特定してください。
• PRタイトル、変更ファイルパス、およびdiffからの関数またはコンポーネント名からのキーワードを使用して既存実装を検索してください。

# PR タイトルと変更ファイルからのキーワードを使用
rg -n "<keyword_from_pr_title>" -S src packages apps ui || true
rg -n "<function_or_component_name>" -S src packages apps ui || true

git log --oneline --all --grep="<keyword_from_pr_title>" | head -20

既に存在する場合は、BLOCKERまたは少なくともIMPORTANTとして指摘してください。

3. PRをアサイン

他のメンバーに審査中であることを知らせるために自分自身をアサインしてください。PRがスパムのように見える場合、またはクローズを推奨する予定のドラフトの場合はスキップしてください。

gh_user=$(gh api user --jq .login)
gh pr edit <PR> --add-assignee "$gh_user"

4. PR説明を注意深く読む

ステップ1の本文を使用します。目標、スコープ、および欠落しているコンテキストを要約してください。

5. diffを徹底的に読む

最小限:

gh pr diff <PR>

完全なコードコンテキストがローカルに必要な場合、PRヘッドをローカル参照にフェッチしてdiffを表示します。マージコミットを作成しないでください。

git fetch origin pull/<PR>/head:pr-<PR>
# ワーキングツリーを変更せずに変更を表示

git diff --stat origin/main..pr-<PR>
git diff origin/main..pr-<PR>

PR版のファイルを直接ブラウズしたい場合、ワークツリーでpr-<PR>を一時的にチェックアウトしてください。コミットまたはプッシュを行わないでください。その後、temp/pr-<PR>に戻してorigin/mainにリセットしてください。

# 必要な場合のみ使用
# git checkout pr-<PR>
# ...ファイルを検査...

git checkout temp/pr-<PR>
git reset --hard origin/main

6. 変更の必要性と価値を検証

率直にしてください。低品質のAI生成コンテンツを指摘してください。

7. 実装品質を評価

正確性、設計、パフォーマンス、および使いやすさを確認してください。

8. セキュリティレビューを実施

OpenClawサブエージェントはgit、gh、シェルを含むフルディスクアクセス権で実行されることを想定してください。認証、入力検証、シークレット、依存関係、ツール安全性、およびプライバシーをチェックしてください。

9. テストと検証をレビュー

既存の内容、不足している内容、および最小限のリグレッションテストについて特定してください。

10. ドキュメントをチェック

PRが関連ドキュメント(README、docs、インラインAPIドキュメント、またはコンフィグ例など)を含むコードに触れているかどうかをチェックしてください。

• 変更された領域のドキュメントが存在し、PRがドキュメントを更新しない場合、IMPORTANTとしてフラグを立ててください。
• PRが新機能またはコンフィグオプションを追加して、ドキュメントがない場合、IMPORTANTとしてフラグを立ててください。
• 変更がユーザーに見える影響がない内部的なものの場合、スキップしてください。

11. チェンジログをチェック

CHANGELOG.mdが存在するか、およびPRがエントリを保証するかどうかをチェックしてください。

• プロジェクトにチェンジログがあり、PRがユーザーに見える場合、欠落したエントリをIMPORTANTとしてフラグを立ててください。
• /preparepr用に変更を残してください。ここではフラグを立てるだけです。

12. 重要な質問に答える

/prepareprが問題を修正できるか、それとも貢献者がPRを更新する必要があるかを決定してください。

13. ワークツリーに検出結果を保存

セクションA〜Jの完全な構造化レビューを.local/review.mdに書き込みます。 ファイルを作成または上書きして、ファイルが存在して空でないことを確認してください。

ls -la .local/review.md
wc -l .local/review.md

14. 構造化レビューを出力

.local/review.mdに保存した内容に一致するレビューを作成してください。

A) 要約推奨事項

• 次のいずれか: /preparepr準備完了 | 作業が必要 | 検討が必要 | 不要(クローズ)
• 1〜3文。

B) 何が変更されたか

C) 良い点

D) セキュリティ検出事項

E) 懸念事項または質問(実行可能な)

• 番号付きリスト。
• 各項目をBLOCKER、IMPORTANT、またはNITとしてマークしてください。
• 各項目について、ファイルまたは領域を指摘し、具体的な修正を提案してください。

F) テスト

G) ドキュメント状態

• 関連ドキュメントが最新、欠落、または該当なしかを述べてください。

H) チェンジログ

• CHANGELOG.mdにエントリが必要かどうか、およびどのカテゴリかを述べてください。

I) フォローアップ(オプション)

J) 推奨PRコメント(オプション)

ガードレール

• ワークツリーのみ。
• 審査後にワークツリーを削除しないでください。
• 審査のみ。マージしたり、プッシュしたりしないでください。