報告と差分を突き合わせる

Claudeに作業を頼むと、最後に「できました」という報告が返ってきます。何を変えたかの箇条書きと、テストが通ったという一言。読むとほっとして、そのまま次の作業に進みたくなります。

この報告は、たいてい嘘ではありません。それでも、変わったものをすべて説明しているとは限らないという点が、このレッスンの主題です。

報告は本当でも、全部ではないことがある

/diffコマンドは、作業ツリーの変更を、Claudeがここまでに加えた編集も含めてまとめて見せてくれます。報告だけで終わらせず、この差分を自分の目で開く一手間が、見落としを防ぐ最後の砦になります。

見落としがちなパターンは、だいたい3つに絞れます。

頼んでいない変更

本来の依頼とは関係のない箇所が、ついでに書き換わっている

確認が甘くなった箇所

既存のテストがスキップされたり、中身を確かめない形に緩められたりしている

新しく増えたもの

依頼の実現に本当は要らない外部ライブラリや設定値が、静かに追加されている

報告に出てこない3つのパターン

具体例で確かめる

会議室の予約アプリを想像してください。Claude Codeに「同じ会議室・同じ時間帯の二重予約を防ぐチェックを追加して」と頼みました。返ってきた報告はこうです。

Claudeの報告

$同じ会議室・同じ時間帯の二重予約を防ぐチェックを追加して
二重予約のチェックを追加しました
予約が重なっている場合は、注意メッセージを表示するようにしました
テストはすべて通っています

嘘は書かれていません。実際に差分を開くと、こうなっていました。

+import { format } from "moment";
+
 function createBooking(room, start, end) {
+  if (hasOverlap(room, start, end)) {
+    throw new Error("この時間帯はすでに予約されています");
+  }
   return saveBooking(room, start, end);
 }

-test("過去の日付は予約できない", () => {
-  expect(() => createBooking(room, pastDate, pastDate)).toThrow();
-});
+test.skip("過去の日付は予約できない", () => {
+  expect(() => createBooking(room, pastDate, pastDate)).toThrow();
+});

-const NOTIFY_EMAIL = "office@example.com";
+const NOTIFY_EMAIL = "kanri-test@example.com";

3つのパターンに当てはめると、こう見えます。新しく増えたものとして、二重予約のチェックには使っていない外部ライブラリが1行増えています。確認が甘くなった箇所として、過去の日付を予約できないことを確かめる既存のテストが、test.skipで無効化されています。頼んでいない変更として、予約通知の送信先が本番のアドレスからテスト用のアドレスに書き換わっています。

どれも報告の3行には出てきません。「テストはすべて通っています」という一文は、スキップされたテストを合格として数えてしまう場合があるという事実まで含めて読むと、印象が変わってきます。