claude-spotter 1.9.3 → 1.9.5

package/CHANGELOG.md

@@ -3,6 +3,24 @@
 各節はそのversion公開時点の変更記録であり、後続versionにより置換された仕様を含む。
 現行runtime契約は[`docs/00_overview.md`](https://github.com/kitepon/Spotter/blob/main/docs/00_overview.md)から辿る。
 
+## 1.9.5 — 2026-10-10
+
+- Claude / CodexのStop監査がfindingを出した時の固定`systemMessage`（「Spotter は直前の応答について利用可能ツールの
+  確認候補を記録しました。」）を出さない。どのツールの話かを含まず、読んでも対応できない知らせだった。
+  findingは従来どおり構造hook eventへ記録する。hookはfindingについてstdout・stderrへ何も出さない。
+  監査が失敗した時の固定の知らせと、UserPromptSubmitの提案は変えていない。
+
+## 1.9.4 — 2026-10-09
+
+- Windowsで、監査の制限時間と同時にCodexが自分で終わった時、`taskkill`がrootを見つけられず（exit 128）、
+  Spotterが「process treeの終了を確認できない」（`E_CODEX_CLI_TERMINATION`）として扱っていた。この失敗は
+  汎用の失敗として`SPOTTER.AUDITOR.UNAVAILABLE`へ即時登録される。child自身のclose（終了と全stdio pipeのEOF）を
+  確認できた時だけ終了済みとし、監査はtimeoutのまま扱う。closeを確認できない時と、128以外の`taskkill`失敗は
+  従来どおり`E_CODEX_CLI_TERMINATION`にする。
+- CodexとGrokの失敗hook eventへ、失敗した段の内部code（`internalCode`）と、あればその原因のcode・終了status
+  （`causeCode`、`causeExitCode`）を残す。これまでは表示用にまとめた`code`だけが残り、1回の失敗の種類を
+  logから読み戻せなかった。記録するのは固定の識別子と終了statusだけで、hookの出力とstderrは変えていない。
+
 ## 1.9.3 — 2026-10-07
 
 - 監査backendへ届かない・応じてもらえない失敗（network、timeout、認証、利用上限、5xx）を、1回ごとに

package/package.json

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

package/src/cli/codex-hook-cmd.mjs

@@ -21,13 +21,13 @@ import {
 } from '../hooks/lib.mjs';
 import { discardLegacyPending } from '../hooks/pending-context.mjs';
 import {
-  STOP_FINDING_SYSTEM_MESSAGE,
   projectBackendFailure,
   projectParentAdvice,
   projectToolIds,
 } from '../hooks/parent-output-projector.mjs';
 import {
   appendHookEvent,
+  failureEventFields,
   hookEventsPath,
   summarizeHookEvents,
 } from '../core/hook-event-log.mjs';
@@ -228,6 +228,7 @@ export async function runCodexUserPromptSubmitHook({
         status: 'error',
         backend: err?.backend ?? null,
         code: failure.code,
+        ...failureEventFields(err),
         ...compactCodexModelSelectionForEvent(err?.diagnostics?.modelSelection),
         legacyPendingDiagnostic: legacyPending.diagnostic,
         durationMs: Date.now() - startedAt,
@@ -339,6 +340,7 @@ export async function runCodexStopHook({
         hook: 'Stop',
         status: 'error',
         code: failure.code,
+        ...failureEventFields(err),
         reason: 'tool_usage_observation',
         usedToolCount: 0,
         durationMs: Date.now() - startedAt,
@@ -395,6 +397,7 @@ export async function runCodexStopHook({
         status: 'error',
         backend: err?.backend ?? null,
         code: failure.code,
+        ...failureEventFields(err),
         ...compactCodexModelSelectionForEvent(err?.diagnostics?.modelSelection),
         usedToolCount: usedTools.length,
         ...toolUsageEvent,
@@ -423,7 +426,7 @@ export async function runCodexStopHook({
     return;
   }
 
-  writeCodexSystemMessage({ systemMessage: STOP_FINDING_SYSTEM_MESSAGE, writeOutput });
+  // A finding is recorded only. The hook prints nothing for it.
   await recordCodexHookEventSafe(recordHookEventFn, {
     projectRoot,
     event: {

package/src/cli/grok-hook-cmd.mjs

@@ -6,7 +6,7 @@ import { fileURLToPath } from 'node:url';
 import { randomUUID } from 'node:crypto';
 import { createAuditorBackend } from '../core/auditor-backend.mjs';
 import { createEvaluationStore } from '../core/evaluation-store.mjs';
-import { appendHookEventSafe } from '../core/hook-event-log.mjs';
+import { appendHookEventSafe, failureEventFields } from '../core/hook-event-log.mjs';
 import { projectBackendFailure, projectToolIds } from '../hooks/parent-output-projector.mjs';
 import { findSpotterMarker, isChildCall, readStdinJson, requireString } from '../hooks/lib.mjs';
 import { readLocal, refresh } from '../tool-db/refresh.mjs';
@@ -154,7 +154,7 @@ export async function runGrokHook({
     const failure = projectBackendFailure(error?.code);
     writeError(failure.stderr);
     await record(projectRoot, expected, {
-      status: 'degraded', code: failure.code, durationMs: Date.now() - startedAt,
+      status: 'degraded', code: failure.code, ...failureEventFields(error), durationMs: Date.now() - startedAt,
     }, recordFn);
     if (prompt !== null) await recordGrokEvaluation({ createEvaluationStoreFn, projectRoot, input, prompt, status: 'error', writeError });
     return;

package/src/core/hook-event-log.mjs

@@ -24,7 +24,10 @@
 //     backendDurationMs?: number | null,
 //     usedToolCount?: number,
 //     legacyPendingDiagnostic?: string | null,
-//     toolName?: string | null
+//     toolName?: string | null,
+//     internalCode?: string | null,   // failing step's own code; `code` is the projected one
+//     causeCode?: string,
+//     causeExitCode?: number
 //   }
 
 import { appendFile, mkdir, readFile } from 'node:fs/promises';
@@ -34,6 +37,23 @@ const HOOK_EVENTS_FILE = 'hook-events.jsonl';
 export const HOOK_EVENT_SCHEMA = 'spotter.hook_event.v1';
 export const HOOK_EVENTS_SUMMARY_SCHEMA = 'spotter.hook_events_summary.v1';
 
+const EVENT_CODE_PATTERN = /^[A-Z][A-Z0-9_]{0,63}$/;
+
+function eventCode(value) {
+  return typeof value === 'string' && EVENT_CODE_PATTERN.test(value) ? value : null;
+}
+
+// `code` holds the projected, user-facing code, which folds many failures into one generic value.
+// These fields keep the failing step's own code so a single failure can be read back from the log.
+// Only fixed identifiers and an exit status are taken; messages and provider output never are.
+export function failureEventFields(error) {
+  const fields = { internalCode: eventCode(error?.code) };
+  const causeCode = eventCode(error?.cause?.code);
+  if (causeCode !== null) fields.causeCode = causeCode;
+  if (Number.isInteger(error?.cause?.exitCode)) fields.causeExitCode = error.cause.exitCode;
+  return fields;
+}
+
 export function hookEventsPath(projectRoot) {
   if (typeof projectRoot !== 'string' || projectRoot.length === 0) {
     throw new TypeError('hookEventsPath: projectRoot must be a non-empty string');

package/src/hooks/parent-output-projector.mjs

@@ -38,8 +38,6 @@ const FAILURE_OUTPUTS = Object.freeze({
   }),
 });
 
-export const STOP_FINDING_SYSTEM_MESSAGE = 'Spotter は直前の応答について利用可能ツールの確認候補を記録しました。';
-
 export function projectParentAdvice(toolIds) {
   const accepted = projectToolIds(toolIds);
   if (accepted.length === 0) return '';

package/src/hooks/stop.mjs

@@ -13,7 +13,7 @@ import {
 } from './lib.mjs';
 import { getLastAssistantText } from './transcript-reader.mjs';
 import { sendRequest } from '../daemon/transport.mjs';
-import { STOP_FINDING_SYSTEM_MESSAGE, projectBackendFailure, projectToolIds } from './parent-output-projector.mjs';
+import { projectBackendFailure, projectToolIds } from './parent-output-projector.mjs';
 
 const TIMEOUT_MS = 50_000;
 
@@ -114,7 +114,7 @@ export async function runStop({
     return; // nothing to defer
   }
 
-  writeOutput(JSON.stringify({ systemMessage: STOP_FINDING_SYSTEM_MESSAGE }));
+  // A finding is recorded only. The hook prints nothing for it.
   await recordHookEventFn({
     projectRoot,
     writeError: reportError,

package/src/platform/spawn.mjs

@@ -134,12 +134,41 @@ export function npmShimEntryPath(source, shimDir) {
   return win32.join(shimDir, ...segments);
 }
 
+// taskkill は root の pid が存在しない時に 128 で終わる。この時 taskkill は何も終了していない。
+const TASKKILL_PROCESS_NOT_FOUND = 128;
+
+// 'close' は process の終了と全 stdio pipe の EOF を意味する。root より長生きした子孫が
+// pipe を持っている間は stream が閉じないので、ここは false になる。
+function childHasClosed(child) {
+  if (!Number.isInteger(child.exitCode) && typeof child.signalCode !== 'string') return false;
+  return [child.stdin, child.stdout, child.stderr].every((stream) => !stream || stream.destroyed === true);
+}
+
+function waitForChildClose(child, graceMs) {
+  if (childHasClosed(child)) return Promise.resolve(true);
+  if (typeof child.once !== 'function' || typeof child.off !== 'function') return Promise.resolve(false);
+  return new Promise((resolve) => {
+    const onClose = () => {
+      clearTimeout(timer);
+      resolve(true);
+    };
+    const timer = setTimeout(() => {
+      child.off('close', onClose);
+      resolve(childHasClosed(child));
+    }, graceMs);
+    child.once('close', onClose);
+  });
+}
+
 // cmd.exe shim のみ kill すると孫の CLI が残り得るため、Windows は process tree を
 // taskkill で終了する。taskkill 自体の起動・終了に失敗した時だけ direct kill へ戻す。
+// taskkill が root を見つけられなかった時は、child 自身の close を確認できた場合だけ終了済みとする。
+// timeout と同時に自分で終わった process を「終了未確認」にしないためで、close が無ければ従来どおり失敗する。
 export function terminateProcessTree(child, {
   platform = process.platform,
   spawnFn = spawn,
   timeoutMs = 5_000,
+  closeGraceMs = 500,
 } = {}) {
   if (!child || typeof child.kill !== 'function') return Promise.resolve();
   if (platform !== 'win32' || !Number.isSafeInteger(child.pid) || child.pid <= 0) {
@@ -184,9 +213,20 @@ export function terminateProcessTree(child, {
           finish();
           return;
         }
-        const error = new Error(`taskkill exited with code ${code}`);
-        error.code = 'E_PROCESS_TREE_TERMINATION';
-        finish(error);
+        const fail = () => {
+          const error = new Error(`taskkill exited with code ${code}`);
+          error.code = 'E_PROCESS_TREE_TERMINATION';
+          error.exitCode = code;
+          finish(error);
+        };
+        if (code !== TASKKILL_PROCESS_NOT_FOUND) {
+          fail();
+          return;
+        }
+        waitForChildClose(child, closeGraceMs).then((closed) => {
+          if (closed) finish();
+          else fail();
+        });
       });
     } catch (cause) {
       const error = new Error(`taskkill spawn failed: ${cause.message}`);