この記事で言いたいこと
レビューで疲れる原因は、コード量でも、知識量でもなくて、自分がどこまで背負ってしまうかにある。
私は以前、レビューで「将来困る点（L2）」を見つけるたびに、
- 代替案を考え
- 相手と合意形成し
- 最適解まで一緒に決めに行く …という動きをしていて、めちゃくちゃ消耗していた。
でもある時、気づいた。
レビューの指摘はレイヤーで扱いを変えるべき 特に L2 は「決めない」ほうが現場が回る
この記事では、レビューを L1 / L2 / L3 の三段階に分けて、 疲れずに価値を出し続けるための整理を書きます。
三つのレイヤー（全体像）
レビューの指摘は、ざっくりこの3つに分類できる。
| レイヤー | 内容 |
|---|---|
| L1 | 今壊れる（安全・事故） |
| L2 | 将来困る（運用・設計） |
| L3 | 美しい（理想・好み） |
重要なのは、全部を同じ熱量で扱わないこと。
L1：今壊れる（安全・事故）
L1の例
- 本番障害に直結する
- データ破壊につながる
- セキュリティ事故になりうる
- 金額・決済・請求がズレる
- 例外を握りつぶして検知できない
- トランザクションが壊れて整合性が崩れる
L1のスタンス
ここは必ず止める。妥協しない。
L1は「好み」ではなく「事故」なので、議論も短い。 疲れにくい。
L2：将来困る（運用・設計）
L2の例
- ログが足りず調査コストが上がる
- 依存の方向が逆で差し替えが難しい
- 責務が混ざって拡張がつらい
- 命名や構造が原因で読みづらい（ただしL3と混同注意）
- 例外の握り方が曖昧で運用の判断ができない
L2が一番消耗する理由
L2は「今は動く」。だから正解が1つじゃない。
ここでレビューアーがやりがちなのが、
- 最適解まで一緒に決めに行く
- 合意形成に時間を使う
- 実装方針まで背負う
- 結果として責任も背負う
という動き。
これがレビュー疲れの正体だった。
L2の正しい扱い：「材料を出して、判断を返す」
L2は、次の3ステップだけにする。
- 事実（リスク）を言う
- 選択肢を出す（2択以上）
- 判断を返す（決めない）
使えるテンプレ
このままだと将来◯◯のときに困る可能性があります。 今回は A：このまま進める（早い） B：ここだけ直す（将来が楽） が選べます。今回はどちらにしますか？
ポイントは、レビューアーが「決定者」にならないこと。 判断するのは、スコープや優先順位を持っている側（実装者・担当者・責任者）でいい。
レビューアーは、判断材料を出す役に徹する。
L3：美しい（理想・好み）
L3の例
- もっと綺麗に抽象化できる
- もっと理想的な設計がある
- もっと好きな書き方がある
- もっと将来に備えた作り方ができる
L3のスタンス
原則、言わない。
L3は議論が終わらない。 そして多くの場合、今それをやっても報われない。
言うなら、レビューコメントではなく、別の場所に「改善メモ」として残すくらいでいい。
まとめ：疲れない人は「背負うレイヤー」を選んでいる
- L1：止める（妥協しない）
- L2：材料を出す（決めない）
- L3：触らない（言わない）
これだけで、レビューの消耗は激減する。
成長とは、すべてを背負えるようになることではない。 背負わなくていいものを見極めることだと思っている。
おまけ：迷ったら自分に聞く4つ
レビュー中に迷ったら、これだけ自問する。
- これは L1（事故） か？
- これは L2（将来の運用） か？
- これは L3（好み・理想） ではないか？
- 自分が 決めに行って いないか？
これで、自分のリソースを残したまま仕事ができる。