claude-spotter 0.4.3 → 0.4.4

package/CHANGELOG.md

@@ -1,5 +1,26 @@
 # Changelog
 
+## 0.4.4
+
+**Stop hook が Bell の最終応答を Haiku に正しく渡すよう修正**。
+
+### 事の発端
+
+「Stop hook で Haiku に渡されるのは Bell の最終応答だけか、それとも thinking も含む膨大なテキストか」というユーザーからの疑問で調査したところ、**実は何も渡していなかった**ことが判明。
+
+[src/hooks/stop.mjs](src/hooks/stop.mjs) は `input.final_response` フィールドを読んでいたが、Claude Code の Stop hook が stdin に渡す標準入力にそんなフィールドは存在しない。標準は `transcript_path` (セッションの JSONL ログファイルパス)。結果、`optionalString` が常に null を返し、daemon には **`'(no final response provided)'`** という sentinel 文字列だけが送られ続けていた。turn_end の Haiku 判定は Bell の応答を 1 文字も見ずに行われていた (したがって Stop 段階でのツール呼び忘れ検出は実質機能していなかった)。
+
+### 変更点
+
+- **[src/hooks/transcript-reader.mjs](src/hooks/transcript-reader.mjs) 新設**: transcript JSONL の末尾から assistant の text ブロックだけを抽出する `getLastAssistantText()`。Throughline (MIT, 同作者) から必要最小限を移植。**thinking ブロック / tool_use ブロックは除外**されるので、ユーザーに見えた最終応答テキストだけが Haiku に渡る。
+- **[src/hooks/stop.mjs](src/hooks/stop.mjs) 書き換え**: `input.final_response` (存在しないフィールド) を廃止、`input.transcript_path` (§0 に従い `requireString` で必須化) から `getLastAssistantText()` 経由で最終応答を取得。
+- **[test/transcript-reader.test.mjs](test/transcript-reader.test.mjs) 追加**: 8 ケース。thinking 除外、tool_use スキップ、複数 text ブロックの連結、欠損ファイル、partial JSONL 末尾 (書き込み途中) への耐性等。
+
+### 効果
+
+- Stop hook の Haiku 判定が Bell の最終応答を実際に読むようになる → ツール呼び忘れ検出 (Stop 段階) が設計通り動く。
+- thinking は渡らないので内部推論の漏洩リスクなし。
+
 ## 0.4.3
 
 **Haiku プロンプトの最小化**。

package/package.json

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

package/src/hooks/stop.mjs

@@ -5,7 +5,6 @@
 import {
   readStdinJson,
   requireString,
-  optionalString,
   exitCodeFor,
   die,
   formatTransparentBlockReason,
@@ -13,6 +12,7 @@ import {
   isSubagentCall,
   isOutsideSpotterProject,
 } from './lib.mjs';
+import { getLastAssistantText } from './transcript-reader.mjs';
 import { sendRequest } from '../daemon/transport.mjs';
 
 const TIMEOUT_MS = 15_000;
@@ -24,9 +24,13 @@ export async function runStop() {
   if (isOutsideSpotterProject(input)) return;
 
   const sessionId = requireString(input, 'session_id');
+  const transcriptPath = requireString(input, 'transcript_path');
   const stopHookActive = input.stop_hook_active === true;
-  // Claude Code passes the transcript path; the final response is read from there or provided inline.
-  const finalResponse = optionalString(input, 'final_response') ?? '(no final response provided)';
+
+  // Extract only the visible assistant text (thinking and tool_use blocks excluded).
+  // null when the transcript has no assistant text yet — pass a sentinel so the
+  // daemon's schema (final_response: string) is still satisfied.
+  const finalResponse = getLastAssistantText(transcriptPath) ?? '(no final response available)';
 
   let response;
   try {

package/src/hooks/transcript-reader.mjs

@@ -0,0 +1,57 @@
+// Transcript JSONL reader — extracts the final assistant text from Claude Code's
+// transcript file. Used by Stop hook to pass *only* the visible response
+// (not thinking, not tool_use) to the Haiku judge.
+//
+// JSONL format (verified against Claude Code transcripts):
+//   {type: "user",      message: {role: "user",      content: [{type:"text", text:"..."}]}, ...}
+//   {type: "assistant", message: {role: "assistant", content: [
+//     {type:"thinking", thinking:"..."},    // MUST be excluded
+//     {type:"tool_use", ...},               // MUST be excluded
+//     {type:"text", text:"..."},            // what we want
+//   ]}, ...}
+//
+// Ported from Throughline (MIT, same author) src/transcript-reader.mjs.
+
+import { readFileSync, existsSync } from 'node:fs';
+
+// Concatenate only text blocks — thinking / tool_use / image are dropped.
+function extractText(content) {
+  if (typeof content === 'string') return content;
+  if (!Array.isArray(content)) return '';
+  return content
+    .filter((b) => b && b.type === 'text' && typeof b.text === 'string')
+    .map((b) => b.text)
+    .join('');
+}
+
+// Walk the JSONL backwards and return the text of the last assistant entry
+// that has at least one non-empty text block. Returns null if no such entry
+// exists or the file is missing.
+export function getLastAssistantText(transcriptPath) {
+  if (typeof transcriptPath !== 'string' || transcriptPath.length === 0) return null;
+  if (!existsSync(transcriptPath)) return null;
+
+  const raw = readFileSync(transcriptPath, 'utf8');
+  const lines = raw.split('\n');
+
+  for (let i = lines.length - 1; i >= 0; i--) {
+    const trimmed = lines[i].trim();
+    if (!trimmed) continue;
+
+    let entry;
+    try {
+      entry = JSON.parse(trimmed);
+    } catch {
+      continue;
+    }
+
+    if (entry.type !== 'assistant') continue;
+    const msg = entry.message;
+    if (!msg || msg.role !== 'assistant') continue;
+
+    const text = extractText(msg.content);
+    if (text.length > 0) return text;
+  }
+
+  return null;
+}

package/src/version.mjs

@@ -1 +1 @@
-export const version = '0.4.3';
+export const version = '0.4.4';