claude-spotter 0.5.0 → 0.5.2

package/CHANGELOG.md

@@ -1,5 +1,51 @@
 # Changelog
 
+## 0.5.2
+
+**Haiku 呼び出しのレイテンシ可視化 (観測性の改善のみ、機能変更なし)**。
+
+### 事の発端
+
+v0.5.1 の hot-fix で session-scoped Haiku がようやく生きた状態で動き始めた。実セッション観測で `--resume` 経路の Haiku 判定はエラーなく走っているものの、**そこに「cold-start が実際に消えたか」を読み取れる情報が daemon ログに出ていない**。プラン §5.5 が前提としていた「`--resume` で毎ターン 20–50 秒の claude -p spawn コストを消せる」が実効的に効いているかは、呼び出し時間が見えないと判断できない。
+
+### 変更点
+
+- **[src/daemon/haiku-caller.mjs](src/daemon/haiku-caller.mjs)**: `createHaikuCaller` の返り値に `isFirstCall` getter を追加。daemon 側が「次の呼び出しが初回かどうか」をスナップショットできる。
+- **[src/daemon/daemon.mjs](src/daemon/daemon.mjs)**: `callHaikuTracked` が raw 文字列に加えて `{ durationMs, mode }` を返す構造に変更。`runHaikuJudgment` / `handleUserInput` / `handleTurnEnd` の戻り値構造も合わせて `{ parsed, meta }` に統一。ログ出力に `mode=first|resumed, duration_ms=<N>` を追加。role collapse 回復パスも meta を引き継ぐため silent-pass ログにも duration が残る。
+- **[test/haiku-caller.test.mjs](test/haiku-caller.test.mjs)**: `isFirstCall` の初期値と `reset()` 後の挙動を検証するテストを追加。
+- **[test/daemon.test.mjs](test/daemon.test.mjs)**: `user_input` / `turn_end` のログ行に `mode=first` / `mode=resumed` と `duration_ms=<N>` が含まれることを検証するテストを 2 件追加。
+
+### 観測できるようになったこと
+
+これまでは `user_input: pass=true, missing=` だけだったログ行が `user_input: pass=true, missing=, mode=first, duration_ms=8432` のように出る。次のセッションから:
+
+- **`mode=resumed` の duration_ms が mode=first より有意に短い**なら `--resume` の効果が実測で確認できる
+- **role_collapse_reset が発生しても duration_ms が見える**ので、回復が高速なのか cold-start 待ちなのかが区別できる
+- **timeout 30s に対する余裕**が数値で見える (ギリギリなら延長、余裕なら短縮の判断材料)
+
+これらは v0.5.0 / v0.5.1 の「resume の実効 spawn 削減量未検証」「role collapse 実発生頻度の観測」という既知課題を **観測可能な状態に引き上げる**ための最小変更。判断材料が貯まるまで追加の構造変更は凍結。
+
+### 機能的には非変更
+
+ログフォーマット以外の挙動は一切変わらない。envelope 契約・hook の終了コード・Haiku プロンプト・catalog 形式・回復ロジックのいずれも手付かず。
+
+## 0.5.1
+
+**v0.5.0 の Haiku spawn が初呼び出し時点で落ちていたバグの hot-fix**。
+
+### 事の発端
+
+v0.5.0 リリース直後の実セッションで、Stop hook の Haiku 判定が毎回失敗して daemon ログに `--session-id cannot be used with --resume unless --fork-session is also passed` が出ていた。claude CLI は `--session-id` と `--resume` の併用を `--fork-session` なしでは拒否する仕様で、v0.5.0 の `buildSpawnArgs` が resume 時に両方渡していたのが原因。v0.5.0 は起動はするが監査は 1 度も成立していなかった (role collapse 回復も永遠に triggering しない状態)。
+
+### 変更点
+
+- **[src/daemon/haiku-caller.mjs](src/daemon/haiku-caller.mjs)**: resume 時は `--session-id` を外して `--resume <uuid>` のみを渡す形に修正。初回は従来どおり `--session-id <uuid>` で新規セッション確立。
+- **[test/haiku-caller.test.mjs](test/haiku-caller.test.mjs)**: `buildSpawnArgs: subsequent call` のアサーションを「`--session-id` が含まれない / `--resume <uuid>` が正しく渡る」に書き換え。
+
+### 残る既知課題
+
+v0.5.0 の既知課題 (resume の spawn 削減量未検証、カタログ毎ターン再送コスト、role collapse 実発生頻度の観測) は引き続き持ち越し。v0.5.1 でようやく session-scoped の効果測定が可能になる。
+
 ## 0.5.0
 
 **Session-scoped Haiku 復活 + role-collapse 回復機構 (UX 改善)**。

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-spotter",
-  "version": "0.5.0",
+  "version": "0.5.2",
   "description": "Audit agent running alongside Claude Code that catches missed tool calls — 気づく役と実行する役の分離",
   "type": "module",
   "bin": {

package/src/daemon/daemon.mjs

@@ -88,23 +88,26 @@ export async function startDaemon({
 
   const callHaikuTracked = async (prompt) => {
     lastHaikuCallAt = Date.now();
-    return callHaiku(prompt);
+    const mode = callHaiku.isFirstCall === false ? 'resumed' : 'first';
+    const start = Date.now();
+    const raw = await callHaiku(prompt);
+    return { raw, meta: { durationMs: Date.now() - start, mode } };
   };
 
   // v0.5.0: shared Haiku-invocation + parse helper. On E_HAIKU_SCHEMA (role collapse),
   // rotates the Haiku session-id and silent-passes the turn (reason: role_collapse_reset).
   // Other Haiku errors (timeout, spawn failure) still propagate — §14 unexpected → throw.
   const runHaikuJudgment = async (stage, prompt) => {
-    const raw = await callHaikuTracked(prompt);
+    const { raw, meta } = await callHaikuTracked(prompt);
     try {
-      return parseHaikuResponse(raw);
+      return { parsed: parseHaikuResponse(raw), meta };
     } catch (err) {
       if (err instanceof HaikuError && err.code === 'E_HAIKU_SCHEMA') {
         logFn(`${stage}: role collapse detected, session reset: ${err.message}`);
         if (typeof callHaiku.reset === 'function') {
           callHaiku.reset();
         }
-        return { pass: true, missing_tools: [], reason: 'role_collapse_reset' };
+        return { parsed: { pass: true, missing_tools: [], reason: 'role_collapse_reset' }, meta };
       }
       throw err;
     }
@@ -166,9 +169,9 @@ export async function startDaemon({
     state.lastUserInput = userInput;
     state.usedTools = []; // reset tools for this turn
 
-    const parsed = await runHaikuJudgment('user_input', buildFirstStagePrompt({ catalog, userInput }));
+    const { parsed, meta } = await runHaikuJudgment('user_input', buildFirstStagePrompt({ catalog, userInput }));
     logFn(
-      `user_input: pass=${parsed.pass}, missing=${parsed.missing_tools.map((m) => m.name).join(',')}${
+      `user_input: pass=${parsed.pass}, missing=${parsed.missing_tools.map((m) => m.name).join(',')}, mode=${meta.mode}, duration_ms=${meta.durationMs}${
         parsed.reason ? `, reason=${parsed.reason}` : ''
       }`
     );
@@ -207,7 +210,7 @@ export async function startDaemon({
 
     const savedUserInput = state.lastUserInput;
     const savedUsedTools = state.usedTools.slice();
-    const parsed = await runHaikuJudgment(
+    const { parsed, meta } = await runHaikuJudgment(
       'turn_end',
       buildFinalStagePrompt({
         catalog,
@@ -217,7 +220,7 @@ export async function startDaemon({
       })
     );
     logFn(
-      `turn_end: pass=${parsed.pass}, missing=${parsed.missing_tools.map((m) => m.name).join(',')}${
+      `turn_end: pass=${parsed.pass}, missing=${parsed.missing_tools.map((m) => m.name).join(',')}, mode=${meta.mode}, duration_ms=${meta.durationMs}${
         parsed.reason ? `, reason=${parsed.reason}` : ''
       }`
     );

package/src/daemon/haiku-caller.mjs

@@ -168,12 +168,14 @@ function truncate(s, n = 300) {
 // without going through the shell. We use cmd.exe /c explicitly rather than spawn({ shell:
 // true }) because the latter triggers DEP0190 on Node 24+.
 //
-// v0.5.0: session-scoped. First call of a given session id uses --session-id only; every
-// subsequent call adds --resume so claude -p re-attaches to that session, skipping cold
-// start. buildSpawnArgs is exported so tests can assert the flag wiring without spawning.
+// v0.5.1: claude CLI rejects `--session-id` together with `--resume` unless `--fork-session`
+// is present (fork would create a new id, defeating the point). So first call uses
+// `--session-id <uuid>` to pin the id; every subsequent call uses `--resume <uuid>` alone to
+// re-attach. buildSpawnArgs is exported so tests can assert flag wiring without spawning.
 export function buildSpawnArgs({ claudeBin, model, sessionId, resume }) {
-  const args = ['-p', '--session-id', sessionId, '--model', model];
-  if (resume) args.push('--resume', sessionId);
+  const args = resume
+    ? ['-p', '--resume', sessionId, '--model', model]
+    : ['-p', '--session-id', sessionId, '--model', model];
   if (process.platform === 'win32') {
     return { cmd: 'cmd.exe', cmdArgs: ['/c', claudeBin, ...args] };
   }
@@ -261,5 +263,10 @@ export function createHaikuCaller({ timeoutMs, claudeBin = 'claude', model = HAI
     enumerable: true,
   });
 
+  Object.defineProperty(callHaiku, 'isFirstCall', {
+    get: () => isFirstCall,
+    enumerable: true,
+  });
+
   return callHaiku;
 }

package/src/version.mjs

@@ -1 +1 @@
-export const version = '0.5.0';
+export const version = '0.5.2';