review-pr-in-worktree

review-pr-in-worktree

人間専用。 人間から名前を指定して依頼された場合にのみ開始してください。別のスキルからここにたどり着いた場合は、停止し、どの手順も実行する前に明示的な確認を得てください。

PR を専用の worktree にチェックアウトし、そこでレビューします。コミットされた内容、テストが合格するか、実装するとしている計画に沿っているか、その計画からどこが逸脱しているかを確認します。報告してください。修正はしないでください。

このスキルがどの PR を対象にするかを決め、それ以外は共通です。 PR のチェックアウト、ラウンドごとのレビュー、チェックアウトの後片付けは、/review-pr-in-lane も利用する worktree review で行います。レビューそのもの、つまり何を読み、ゲートをどう実行し、コードをどの基準で評価し、最終的に yes または no で終わる固定レポートをどう作成するかは、/review-a-pr-and-report で行います。このフローでは、ラウンドごとにこれを呼び出します。

プロセス

1. リポジトリと PR を特定する — 手元にある情報を使い、検索せずに質問する

すでに目の前にある情報から、次の順番で両方を特定します。

1. 引数があれば、その内容（PR 番号、PR の URL、owner/name）。
2. 現在の作業ディレクトリの git リモートとブランチ — gh はクローン内のリポジトリを推測し、gh pr status は現在のブランチに対応する PR を示します。
3. 曖昧さのない既存のコンテキスト — このセッションで PR を開いた、または人間が先に PR を指定し、それ以降に別の PR を示す情報がない場合。

どれを使っても特定できない場合、または複数の情報が食い違う場合は、直ちに質問してください。最初に行うのは、質問を 1 つすることです。最新の PR、唯一のオープン PR、あるいはタイトルから人間の話題だと思われる PR を選んではいけません。誤った PR のレビューはセッション全体を無駄にするだけでなく、断定的な報告として受け取られてしまいます。

完了条件： 検索せずに、リポジトリ 1 つと PR 番号 1 つを特定できていること。

2. worktree でレビューする

worktree-review.md の最初のセクションから、対象のリポジトリと PR を指定して手順に従います。PR を調査し、デタッチ状態でチェックアウトし、各ラウンドをレビューして判定ブロックを出力します。ラウンド間はあなたの指示を待ち、後片付けを行い、PR がどうなったかを確認します。

完了条件： そのフローの最後のセクションで、PR の状態が報告されていること。

フロー内での位置づけ

/create-pr-for-branch が PR を作成し、/review-pr-in-worktree が作成者の必要に応じて何度でもレビューし、/squash-merge-and-clean-up がマージします。それぞれは、人間が個別に開始する独立した手順です。それぞれが作成者、レビュアー、メンテナーという別の担当者に対応するためです。通常、マージはまったく別のセッションで行われます。そのため、このフローでは自分の報告が最終判断だと決めつけず、トラッカーに PR のその後を確認して終了します。

このスキルでは、レビューそのものを /review-a-pr-and-report として切り分けています。チェックアウト元がどこであっても同じ判断を行うためです。チェックアウトを安全に行い、最後に片付けるのが worktree review です。これは /review-pr-in-lane と共通で、異なるのは PR の見つけ方だけです。/review-pr-in-lane は lane が PR を作成するまで待ちます。