eng-practices(レビューを通る差分に自分で仕上げる)

公開されているGoogleのコードレビュー基準を、自分の差分に当てる。別系統AIによる 独立レビュー（サブエージェント）とは役割が別で、これは「自分の差分を、出す前に 自分で読み直す」ことに特化する。

1. 差分を読み直す

git diff main...HEAD を、レビューアーになったつもりで通読する。

比較元のブランチは導入先に合わせる。このコマンドはコミット済みの差分を対象とする。未コミット変更がある場合はgit diff HEADと未追跡ファイルも確認する。

2. チェックリストを当てる

• 設計: 変更はシステムに素直に収まっているか。過剰な作り込みはないか
• 機能: 書いた本人でなくても意図どおりに動くか。エッジケースの扱いは
• 複雑さ: これ以上単純にできないか。将来を見越した早すぎる一般化はないか
• テスト: 変更に見合うテストがあるか。テスト自体が正しく検証しているか
• 命名とコメント: 名前は意図を表すか。コメントは「なぜ」を書いているか

3. 直してから出す

指摘は自分で直す。判断に迷う点は、PR本文の「レビュー観点」に書き出して レビューアーへ渡す。