diff に隠れた挙動変化をレビューする
収録 2026-10-30
diff だけを読ませ、無害に見えて挙動を変える箇所を影響度順に挙げさせます。
あなたはコードレビュアーです。diff(前後文脈つき)のみ渡します。次を挙げてください:1)テスト未カバーで実行挙動が変わる変更(順序・既定値・タイムアウト・握りつぶした例外・ログレベル・並行順序);2)呼び出し側に影響するのに説明のないインターフェース変更;3)この変更で壊れうる旧データや旧クライアント。各項目にファイル・行、なぜ挙動変化か、最悪の結果、追加すべきテスト。スタイルの指摘は不要。
当サイトオリジナル
中国語の原文
使い方
大 PR 先跑一遍,能提前发现线上才会暴露的问题。
こう変えてみる
- コードを書く前に要件と制約を復唱させる。
- 全体書き換えではなく、最小の変更とロールバック方法を出させる。
得られる成果物
- そのまま使える最小変更(コピー可能なコード)
- 影響範囲:変更ファイルと追従が必要な呼び出し元
よくある失敗
- 一度に十ファイル変更させ、何が原因か分からなくなる。
- エラーだけ貼ってコードとバージョンを省くと、使えない修正が返る。
出力のチェック
- 変更は動いて戻せて、無関係なコードに手を入れていない。
- 境界条件(空・巨大・同時実行)が説明かテストで押さえられている。
使いどころと注意
- 向いている用途:要件と制約を復唱させてから、最小限の動く変更を出させる。
- 注意:エラーにはバージョンと最小再現を添える。無いと一見もっともらしいが合わない修正が返る。