pr-ready

name	pr-ready
description	PR を出す前のチェックリスト。動くことを確認し、差分がレビュー可能で、文脈と検証の証拠が付いているかを見る。PR を開く前に必ず走らせる。

PR 準備チェック

レビューしていないコードを他人に押し付けない。 自分で検証していない PR を 出すのは、自分の仕事をレビュアーに委譲すること。相手だってエージェントを 回せるのだから、最初のレビューはこちらの責任。

各項目の結果を報告する。全部通るまで PR を開かない。

チェックリスト

1. 動くことが示せている。
   • uv run python tools/check.py all が緑（test-runner サブエージェント）
   • ファームに触ったなら実機ビルドが通っている
   • /device-check を実施した。「テストが通った」だけでは不十分
2. セルフレビュー済み。 code-reviewer サブエージェントを差分にかけ、 指摘を全部つぶすか、意識的に受け入れた。
3. 差分がレビュー可能。 重い認知負荷なしに読める大きさ。大きくなりすぎたら 分割する。小さい PR 複数のほうが大きい 1 本より良い。並べ替えは安い （/git-rescue）。
4. コミットが物語になっている。 fixup を潰し、曖昧なメッセージを書き直す。
5. 説明に文脈がある。 どのマイルストーン・仕様のどの節に対応するのか。 書いたら自分で読む。自分が読んでいない文章を人に読ませるのは失礼。
6. 証拠が付いている。 notes/ の記録、スクリーンショット、シリアルの 実出力。ファームの変更なら、使ったコア／ライブラリのバージョンも。
7. 仕様の禁止事項に触れていない（§13）。特に: WiFi を使っていない / set_config で NVS に書いていない / ハードコードした値で描画・送出していない / メインループを長時間止めていない。