analyze-pr

Analyze PR

目的

PR を読んで、次の handoff 設計に必要な事実をまとめる。

• PR 概要
• related issue / acceptance criteria / intent context
• changed files
• 既存テストと layer applicability

この skill は、内部的に PR 取得・変更分析・テスト対応付け・リスク評価を順に実行し、結果を統合する。

デフォルトの考え方

• PR -> test understanding -> manual test design -> QA issue handoff が主フロー。
• ユーザーには内部 step 名ではなく、「PR と既存テストを理解する skill」として見せる。
• local DB は resumable cache として使い、user-facing の主語にしない。
• 先に CLI / GitHub / diff / issue から取れる事実を集め、それでも不足する文脈だけを AskUserQuestion で確認する。
• 対象 repository の root で実行するのが基本。別 cwd から実行する場合は --repository-root <path> を必ず付ける。

実行手順

1. 対象 repository の root で CLI を実行する。plugin ディレクトリなど別 cwd なら --repository-root <path> を付ける。
2. bun run dev analyze-pr --pr <number> を実行する。
   • 内部で PR 取得 → 変更分析 → テスト対応付け → リスク評価を順に実行し、結果を統合する。
   • config.json と local DB がなければ自動初期化する。
   • 返却 JSON に internal ID は含まれない。
3. 返却 JSON から intent context / changed files / test coverage / risk highlights / layer applicability を読み取る。layer applicability は { assessment, phase: "pre-allocation", note } の形式で、allocation 前の暫定評価であることが明示されている。
4. 事実だけでは intent や acceptance criteria が足りない場合は、AskUserQuestion で不足文脈を確認する。
5. 整理結果を要約する。
6. AskUserQuestion で design-handoff に進んでよいか確認する。

CLI:

# 対象 repository の root で実行する場合
bun run dev analyze-pr --pr <number>

# plugin ディレクトリなど別 cwd から実行する場合
bun run dev analyze-pr --pr <number> --repository-root /path/to/target-repo

会話テンプレート

開始時

• いまから PR / related issue / diff / existing tests を確認し、handoff 設計の前提を整理することを短く伝える。
• 最初は質問せず、CLI と GitHub から取れる事実を先に集める。

例:

• 「PR と related issue、既存テストを確認し、handoff 設計の前提を整理します。まずは diff と issue から取れる事実を集めます。」

質問してよい条件

• PR 本文や related issue を読んでも acceptance criteria が曖昧
• intent context が複数解釈に割れる
• QA scope に影響する前提が diff から読めない

この場合だけ AskUserQuestion を使う。質問は、その skill の判断に必要な不足文脈だけに絞る。

要約時

• 少なくとも次を短く整理する。
  • PR の狙い
  • related issue / acceptance criteria の要点
  • changed files の大枠
  • existing tests と layer applicability の要点
  • 残っている不確実性

次へ進む確認

• 要約のあと、AskUserQuestion で design-handoff に進んでよいかを明示的に確認する。

例:

• 「analysis は以上です。この前提で design-handoff に進めてよいですか。」

完了条件

• linked issue や PR 本文から intent context が読めている。
• changed files と categories が整理されている。
• 既存テスト候補、coverage confidence、layer applicability が確認できている。
• 後続の design-handoff が内部 ID や step 順序を意識せずに進められる形に要約されている。
• 次の skill に進む前に、AskUserQuestion でユーザー確認が取れている。

次の Step

• design-handoff