コンテンツにスキップ

How to Kill the Code Review

チェック

  • [ ] 本文を確認した
  • [ ] 概要を確認した
  • [ ] タグを確認した
  • [ ] inbox/ 直下へ移行した

概要

AIエージェントによって変更量が増え続ける状況では、人間がすべての差分を読むコードレビューはスケールしない、という主張の記事。 人間の確認ポイントをコード差分の後段レビューから、仕様・制約・受け入れ条件の前段レビューへ移すべきだと述べている。 信頼は単一のレビューゲートではなく、複数案の比較、決定的な検証、権限境界、敵対的検証などのレイヤーで作るものとして整理されている。

本文

問題意識

AI利用が進むチームでは、タスク完了数やPR数が増える一方で、レビュー対象の量も増える。 人間が全変更を読み切る前提のプロセスは、AIがコード生成を担う開発速度と合わなくなる。

記事の中心的な転換は、レビュー対象を「コード」から「意図」に移すこと。 人間が見るべきものは実装の全行ではなく、仕様、計画、制約、受け入れ条件、検証ルールである。

コードレビューの代替レイヤー

  • 複数案を作らせ、テスト通過率、差分サイズ、依存追加の有無などで比較する
  • テスト、型検査、カスタムリンタ、組織共通の不変条件、ドメイン契約を決定的な検証として積む
  • 受け入れ条件は実装後に作るのではなく、仕様側から先に定義する
  • BDDのように、自然言語の仕様を機械検証できる形へ落とす
  • エージェントが触れるファイル、コマンド、権限をタスク単位で狭める
  • 認証、DBスキーマ、依存追加などは自動的に人間確認へエスカレーションする
  • 実装エージェントと検証エージェントを分け、修正権限を持たない検証役を置く

実務での読み替え

この主張は「レビュー不要」というより、「レビューの主戦場を移す」という話として読むのが安全。 差分レビューを完全に捨てるより先に、次のような設計が必要になる。

  • チケットやIssueに検証可能な受け入れ条件を書く
  • 仕様からテストケースを導出する流れを標準化する
  • コード生成エージェントの権限をタスクごとに絞る
  • レビューで見る観点を、実装の読解からリスク検出と仕様妥当性に寄せる
  • 監視、段階リリース、ロールバックを前提にする

要点

  • AI時代のボトルネックは「コードを書くこと」より「正しさを定義し、検証すること」になる。
  • 仕様、制約、受け入れ条件が曖昧なままでは、コードレビューをAI化しても同じ問題が残る。
  • 人間の価値は、実装行の読解よりも、問題設定・制約・失敗条件の定義に移る。
  • 権限設計はAIエージェント開発におけるアーキテクチャ要素として扱う必要がある。

タグ

ai-coding #code-review #spec-driven-development #testing #software-engineering