my-pr

name	my-pr
description	Unified pull request workflow: prepare a safe branch, run read-only review, fix Required findings, create/update a GitHub draft PR, and verify CI plus automated review comments. Subcommands: `create` (simplify + PR, skip code review), `review` (read-only findings only), `fix` (Required fixes + commit, no push), `simplify` (simplification only), `verify` (existing PR checks/reviews only). Use when creating or reviewing PRs, self-reviewing changes, simplifying PR changes, or requesting "PR作成", "レビュー", "簡素化". Do NOT use for responding to review comments or a lightweight review of the current working diff without the PR workflow (use the built-in /code-review instead).
argument-hint	[create|review|fix|simplify|verify]

PR — Pull Request Workflow

PR 作成を安全に進めるため、ブランチ準備、簡素化、レビュー、修正、PR作成、PR後検証を明確に分ける。

リポジトリ状態(呼び出し時点のスナップショット)

git status --short --branch

このスナップショットはモード判定と Safety gate の初期判断にのみ使う。以後の各ステップでは、規定のコマンド・スクリプトで取得する最新状態を正とする。

0. モード判定

$ARGUMENTS の先頭を確認し、references/commands.md を読んで実行パスを決める。

引数	動作
空（デフォルト）	full PR workflow
create	simplify + PR, skip local code review
review	read-only quality review and finding integration
fix	Required fixes + verification + commit, no push
simplify	simplification-only apply, no PR/push
verify	existing PR checks/reviews only

不明な引数の場合は停止してユーザーに確認する。

bundled scripts は chezmoi により $HOME/.claude/skills/my-pr/ へ配置される。最初のスクリプト実行前に配置を検証する。

test -f "$HOME/.claude/skills/my-pr/SKILL.md"
test -x "$HOME/.claude/skills/my-pr/scripts/prepare-review-artifacts.sh"

以後、bundled scripts は各 shell call から $HOME/.claude/skills/my-pr/scripts/... で直接参照する。前の shell call で設定した変数が残ると仮定しない。

---

1. Safety gate

verify 以外では、references/branching.md を読み、変更対象の分離、protected branch / worktree 処理、worktree 移動後の元ブランチ cleanup、upstream safety を確認する。

ここで対象変更と作業ブランチを確定し、protected branch に対象差分が残っていないことを確認するまで、simplify、review、fix、PR作成へ進まない。

---

2. Base and PR state

verify 以外では base branch と既存 PR を確認する。base branch の取得に失敗した場合は推測で main にしない。

base 取得では、ローカル main / master などの protected branch ref を直接進めない。git fetch origin "$BASE_BRANCH:$BASE_BRANCH" は禁止する。別 worktree の checked-out branch ref だけが進み、worktree/index に古い内容が逆差分として残ることがある。

BASE_BRANCH=$(gh repo view --json defaultBranchRef -q .defaultBranchRef.name)
BASE_REF="origin/$BASE_BRANCH"
git fetch origin "+refs/heads/${BASE_BRANCH}:refs/remotes/origin/${BASE_BRANCH}"
git rev-parse --verify "$BASE_REF^{commit}" >/dev/null
git diff "$BASE_REF"...HEAD --stat
git log "$BASE_REF"..HEAD --oneline
bash "$HOME/.claude/skills/my-pr/scripts/prepare-review-artifacts.sh" "$BASE_REF"

PR scope を確認する。未コミット差分だけでなく、base branch との差分量を必ず報告する。

最後に出力された絶対パスを、この run の artifact state file として保持する。以後の例にある /absolute/path/to/artifact.env は必ずその実パスへ置き換える。latest-env.sh の推測や、前の shell call の環境変数へ依存しない。

既存 PR の有無を確認する。

bash "$HOME/.claude/skills/my-pr/scripts/prepare-pr-context.sh" "/absolute/path/to/artifact.env"
source "/absolute/path/to/artifact.env"
cat "$MY_PR_SCOPE_SUMMARY"
cat "$MY_PR_CONTEXT"

MY_PR_SCOPE_GATE が ok 以外の場合は停止する。large はユーザーがそのブランチ全体を PR 対象として明示済みの場合だけ続行できる。untracked は対象ファイルを明示して stage / git add -N するか、対象外と確認してから artifact を作り直す。

MY_PR_CONTEXT_STATE=found の場合は、PR 本文、top-level comments、reviews、inline review comments を review 入力として扱う。no_existing_pr の場合は、PR 本文と過去のやり取りが存在しないことを明示し、推測で補わない。

---

3. Quality workflow

3-1. create / simplify

コードレビューをスキップするモードでは、references/simplify/overview.md を読み、integrated simplify を apply mode で実行する。特に指定がない場合は Codex CLI の simplify profile（model_reasoning_effort="medium"）で行う。

• create: local correctness review なしで simplify apply → 修正があれば commit → PR 作成/更新へ進む。
• simplify: 簡素化専用。simplify apply → 修正・検証・必要な commit まで行い、PR作成・push せず終了する。

3-2. デフォルト / review / fix: read-only quality review

references/review.md を読み、そこに定義された Reviewer A/B/C と統合ルールに従う。

最初に simplify apply を実行しない。先に apply すると、Claude/Codex が古い diff をレビューするため。

prepare-review-artifacts.sh が作成した repo-local artifact を使う。/tmp の diff や「現在のファイル状態」レビューへ暗黙に切り替えない。

以下 3 つを同時に起動し、起動した reviewer が完了するまで統合しない。

• Reviewer A: integrated simplify review (stdin-embedded Codex medium effort, byte-chunked when needed, capped findings)
• Reviewer B: Claude correctness review (Claude Code Agent when available; Claude CLI otherwise)
• Reviewer C: stdin-embedded Codex correctness review via scripts/run-codex-review.sh

Reviewer A/C の失敗、quota、permission、diff access 不可、timeout は review incomplete として停止する。Reviewer B の実行・入力エラーも同様に停止する。Reviewer B が読み取りを完了したが所定の本文構造を返さない場合だけ、同じ session で形式修正を1回要求する。再度不正なら Reviewer B 全体を skip し、A/C の結果を COMPLETE_WITH_SKIPS として統合する。

3-3. 統合

references/review.md の Integration rules と Integration output に従う。3つの結果を重複排除し、各指摘を Required / Recommended / Not needed のどれか1つに分類する。

background 実行した reviewer が残っている間は最終回答しない。やむを得ず待機に入る場合は、再開に必要な artifact path、reviewer output path、次の手順を保存し、CCV の background monitor が利用可能なら監視登録する。

review は read-only なのでここで終了する。ファイル編集、検証、commit、push、PR作成をしない。

3-4. Required fix

デフォルト / fix では、🔴 Required だけ修正する。🟡 Recommended は修正しない。

変更の性質	対応
タイポ、未使用 import、dead code、重複削除	修正する
振る舞いが変わらない簡素化	修正する
バグ・セキュリティ修正	root cause を確認して修正する
振る舞い変更、API/schema/CLI/config 変更	修正せず停止して報告する
好み、style、clever one-liner 化	修正しない

修正後は関連検証を実行し、修正を commit する。fix は push、PR作成、verify を行わずここで終了する。

---

4. PR create / update

review / fix / simplify の場合はスキップする。

references/pr-body.md を読み、PR title/body を作る。push 前には references/branching.md の Push destination safety を実行する。

既存 PR がなければ draft PR を作成し、既存 PR があれば本文を更新する。

---

5. Verify

デフォルト、create、verify で実行する。review / fix / simplify ではスキップする。

references/verify.md を読み、既存 PR の checks polling、automated review 確認、必要な修正を行う。ready 化は行わない。

---

注意事項

• PR 本文は日本語で書く。
• draft PR を作成する。ready 化は行わない。
• --assignee @me を付ける。
• commit message は Conventional Commits 形式の英語。
• create でも integrated simplify apply は必ず実行する。
• review は read-only。指摘の収集と統合だけを行う。
• read-only reviewer は repo 内外を問わずファイルを書かない。.plans、/tmp、.tmp/my-pr への追加メモも禁止する。main orchestrator が作る .tmp/my-pr/ artifact だけは例外。
• fix は Required だけ修正・検証・commit し、push しない。
• デフォルトでは read-only review → Required fix → PR作成/更新 → verify の順に実行する。
• Simplify は Codex CLI の model_reasoning_effort="medium" を使う。model は上書きしない。
• Simplify は full diff を優先する。references/review.md の line/byte chunking 条件を満たす場合だけ file-boundary chunking し、各 run/chunk の Required と Recommended をそれぞれ最大 5 件に制限する。
• chunk の単一ファイル上限を超える差分はそのファイルだけレビューから除外し、ファイル名と byte 数を最終結果に明示する。レビュー済みとして扱わない。
• Reviewer A/C は scripts/run-codex-review.sh を使う。runner は context-file 引数から artifact root を決定するため、別 process へ MY_PR_ARTIFACT_DIR を引き継ぐ必要はない。artifact path を Codex の shell tool に読ませず、stdin 埋め込み、末尾 nonce、SHA-256 receipt で完全性を検証する。Codex は対象 repo ではなく隔離済み artifact-local repo から起動する。
• Claude correctness review は host-aware に実行する。Claude Code Agent が使えるセッションでは Agent を使い、それ以外では Claude CLI の --json-schema で完全な review Markdown を受け取る。統合前に scripts/validate-reviewer-b-output.sh で必須見出しを検証する。
• Codex、Claude reviewer、diff artifact 取得のいずれかに失敗したら停止する。Reviewer B の実行成功後に本文構造検証だけが失敗する場合は、1回の形式修正後に Reviewer B を skip する。暗黙に他 reviewer や local review へ切り替えない。
• fallback、default substitution、broad catch を追加しない。
• 好みの問題や style 指摘は修正対象にしない。
• テストが壊れる修正はしない。
• worktree 使用時は、PR 作成後も自動削除しない。
• .tmp/my-pr/ は local artifact 置き場。stage / commit しない。