レビューゲート: マージ前のチェックリスト
レビューゲートとは、変更が本番に出る前に満たされていなければならない事項の短いリストです。自分で確認するか、AIに各項目を確認させましょう。
- 差分を読み、何がなぜ変わったか理解した
- 変更は実際に頼んだ範囲に収まっている
- テストがパスする(新しい振る舞いにはテストがある)
- 型チェックとリンターがクリーン
- シークレット、デバッグログ、コメントアウトされたコードが残っていない
- 安全チェックがこっそり削除されていない
- 少なくとも正常系について、自分で変更を試した
チェックを入れられない項目があるなら、まだ完了ではありません。
このチェックリストが意図的に短いのは、あなたが飛ばすゲートはゲートではないからです。特に速く動いていて変更が「問題なさそう」に見えるとき、そのまま通したくなる誘惑があります。ユーザーに触れるものについては、それに抗いましょう。役立つコツは、AIにあなたの代わりに記入させることです——リストを貼り付けて、こう頼みます。「いま行った変更について、このレビューゲートを一つずつ通して。各項目について、PASS か FAIL を、一行の根拠とともに言って」。AIはテストを実行せずに正直に「テストはパス」とは書けませんし、行ごとに根拠を示させれば、曖昧な「よさそう」が監査可能な何かに変わります。