docs: /developをCopilotレビュー対応まで自律実行にする

[kyu08]

概要

/developを起動すると、計画の合意後はブランチ作成からPR作成、Copilotレビュー対応までを確認なしで進めるようにする。これまでは「ブランチ名は任せる」「ローカルレビュー後にPRを作り、Copilotレビューに/review-respondで対応する」と毎回指示しないと全自動で進まなかった。

変更内容

• /developの確認必須フェーズをPhase 1（計画）だけにした。ブランチ名の決定、コミット、PR作成は承認を待たない
• ブランチ作成をgit switch -c <branch> origin/mainに変えた
  • mainが別のworktreeでチェックアウトされているとgit checkout mainが失敗するため
• Phase 7でPR作成後にgh pr readyする
  • このリポジトリではCopilotがPR作成時にreviewerに自動追加されるが、レビューを始めるのはready for reviewになってから（#225のtimelineで確認）
• Phase 8を追加した。Copilotのレビューをバックグラウンドのポーリング（30秒間隔、上限20分）で待ち、/review-respond <PR番号> --autoで対応する
• /prと/review-respondに--autoモードを追加した。引数なしの単体実行は従来どおり確認を挟む
  • /review-respond --autoでも、仕様・設計方針の変更を求める指摘や計画と矛盾する指摘ではユーザーに確認する

テスト計画

N/A（スキル定義とドキュメントのみの変更）

備考

Copilotの再レビューは1回分しか待たない。push後の再レビューへの対応は自動では回らない。

---

Generated with Claude Code

[Copilot]

🟡 Changes recommended

front matterの構文エラーと、pushされていないブランチで自律PR作成が停止する問題があります。

3 open findings

• PR作成前に同名リモートブランチをpushしていない · New
• YAMLのflow sequenceが不正でfront matterを解析できない · New
• Phase 5・6の変更をコミット・pushせず作業ツリーがdirtyになる · New

What changed in this PR

計画合意後の /develop を、PR作成とCopilotレビュー対応まで自律実行する変更です。

Changes:

• /develop の確認フェーズを計画時のみに変更
• /pr と /review-respond に --auto を追加
• Copilotレビューの待機・対応フローを追加

File	Description
CONTRIBUTING.md	推奨開発フローを更新
.claude/​skills/​review-respond/​SKILL.md	自律レビュー対応を追加
.claude/​skills/​pr/​SKILL.md	自律PR作成を追加
.claude/​skills/​develop/​SKILL.md	Phase 2以降の自律実行を定義
.claude/​HARNESS.md	ハーネス上の役割を更新

🧠 Review effort: Balanced

---

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

🔵 Needs a closer look

diff 基準の不整合、レビュー対象の過剰範囲、およびポーリングのエラー処理に修正が必要です。

0 open findings

3 resolved since last review

• YAMLのflow sequenceが不正でfront matterを解析できない
• PR作成前に同名リモートブランチをpushしていない
• Phase 5・6の変更をコミット・pushせず作業ツリーがdirtyになる

Previously missed (3)

In code that hasn't changed since last review

Use origin/main as the self-review comparison base

.claude/​skills/​develop/​SKILL.md:69

origin/main から分岐しても、別 worktree で checkout 中のローカル main は更新されません。一方、Phase 6 が呼ぶ /self-review は .claude/skills/self-review/SKILL.md:20 で git diff main...HEAD を使うため、ローカル main が遅れている環境では upstream の既存コミットまで今回の変更としてレビューし、自律修正対象にし得ます。/self-review の比較基準も origin/main（またはブランチ作成時に記録した base）へ揃えてください。

Exit immediately when reviewer API queries fail

.claude/​skills/​develop/​SKILL.md:150

gh api が認証・通信・API エラーで失敗すると n が空のまま数値比較だけが失敗し、ループは継続して20分後に「timeout」と報告します。実際の障害を reviewer 未追加やレビュー遅延と誤認するため、API 取得失敗時は直ちにエラーを報告して終了してください。

Restrict automatic responses to Copilot reviewer comments

.claude/​skills/​develop/​SKILL.md:156

この呼び出し先は全 reviewer の未対応行コメントを取得し、投稿者で絞り込みません（.claude/skills/review-respond/SKILL.md:49）。待機中に人間のコメントが付くと、それまで --auto で確認なしに修正・返信され、Phase 8 の「Copilot レビュー対応」および CONTRIBUTING.md:119 の人間レビュー手順と食い違います。reviewer を指定できる引数を追加し、ここでは Copilot bot のコメントだけを対象にしてください。

🧠 Review effort: Balanced

[Copilot]

🔵 Needs a closer look

レビュー対象の絞り込みとポーリング失敗時の処理に修正が必要です。

0 open findings

Previously missed (2)

In code that hasn't changed since last review

API取得エラーを未到着として扱い再試行している

.claude/​skills/​develop/​SKILL.md:154

gh api が認証・権限・通信エラーで失敗すると n が空になり、数値比較エラーを出しながらループを継続して、API 障害を「レビュー未到着」として最大20分待ってしまいます。取得失敗はその場で別エラーとして終了・報告し、成功した空結果だけを再試行してください。

レビュー対象をCopilotのコメントに限定できていない

.claude/​skills/​develop/​SKILL.md:157

この呼び出しは Copilot の指摘だけを自動処理する意図ですが、review-respond の Phase 1 は .claude/skills/review-respond/SKILL.md:44-49 で PR の全行コメントを reviewer の絞り込みなしに取得します。待機中に人間がコメントすると、その指摘まで --auto で承認なしに修正・返信され、CONTRIBUTING.md:119 の対話的な人間レビュー対応と食い違います。bot login を引数として渡し、取得・件数判定・処理対象をその reviewer に限定してください。

🧠 Review effort: Balanced