概要
ベテランエンジニアのレビュー187件を定量分析した結果、バグ指摘は全体の2割に過ぎず、残りの多くは将来の保守性や「意図の保存」に関する指摘であったことを明らかにしています。
詳細内容
## 概要
著者がベテランエンジニアのレビューコメント187件をGitHub APIで収集し、11の領域に分類して分析した記事です。レビューが「上手い」とされる人の着眼点が、単なるバグ探しではなく「コードベースの劣化を防ぐこと」にあることをデータで示しています。
## 主な分析結果
- **バグ指摘は20%のみ**: 最多の指摘領域は「隣のコードとの整合性（17%）」や「契約と意図の保存（16%）」であり、現時点で動作に問題がない箇所への指摘が全体の3分の1を占めていました。
- **差分外の確認が鍵**: 指摘の69%は、PRの変更範囲（差分）以外のファイルやドキュメントを開いて確認することで導き出されていました。
- **失敗への気づきやすさ**: 「機能は動くが、壊れた時に検知できない」という『失敗に気づけるか』という観点が独立した品質項目として重視されていました。
## 実践的な学び
レビューの質を向上させるのはセンスではなく、「注意を向ける領域の広さ」と「もう一枚ファイルを開く作業量」です。記事内では「数値リテラルの根拠確認」や「対になる箇所の非対称性チェック」など、明日から使える具体的な中領域のチェック項目も提案されています。