claude-spotter 1.6.0 → 1.6.2

package/CHANGELOG.md

@@ -3,6 +3,22 @@
 各節はそのversion公開時点の変更記録であり、後続versionにより置換された仕様を含む。
 現行runtime契約は[`docs/00_overview.md`](docs/00_overview.md)から辿る。
 
+## 1.6.2 — 2026-08-29
+
+- **Windowsの`spotter install -y`がMCP調査後に終了する。** 調査対象のprocess treeを
+  Windows共通経路で停止し、stdin / stdout / stderrのpipeを停止前後に破棄する。
+  カタログ生成後もstdin pipeがNodeのイベントループを保持し、工場セットアップが
+  完了しなかった問題を修正した。
+- **隔離runtime error workerが指定期限内に呼び出し元へ戻る。** timeout時は結果を先に
+  確定し、子process treeのbest-effort掃除を並行して続ける。Windowsで掃除の待機時間が
+  絶対期限へ上乗せされていた問題を修正した。
+- Cursor hookの安定Node path試験を、空白を含むWindows pathの正しい引用形式へ対応させた。
+
+## 1.6.1 — 2026-08-24
+
+- **Cursor hook が Homebrew Cellar の版付き Node を焼かない。** `spotter cursor-hook install` は
+  Codex hook と同じ安定 Node path を使い、版更新で sessionStart が消えるのを防ぐ。
+
 ## 1.6.0 — 2026-08-24
 
 - **Cursor を tool-db host として足した。** `src/host/adapters.mjs` に

package/docs/11_dashboard-operations.md

@@ -1,6 +1,6 @@
 # Spotter評価dashboard運用
 
-現行npm配布版: **v1.6.0**（2026-08-24）。v1.5.13はOS依存・ベンダー依存の内部配置だけを
+現行npm配布版: **v1.6.2**（2026-08-29）。v1.5.13はOS依存・ベンダー依存の内部配置だけを
 変更した挙動同一リファクタで、評価・dashboardのrouting構成は変更していない。
 
 この文書はservice設定の正本であり、各端末に現在installされているnpm versionの台帳ではない。

package/package.json

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

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

@@ -4,6 +4,7 @@ import { homedir } from 'node:os';
 import { dirname, join, resolve } from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { spawnRefreshDetached } from '../hooks/spawn-daemon.mjs';
+import { resolveCodexHookNodePath } from './codex-hook-cmd.mjs';
 import {
   die,
   findSpotterMarker,
@@ -90,7 +91,7 @@ export function cursorCwd(input) {
 
 export async function installCursorHooks({
   cursorHome = defaultCursorHome(),
-  nodePath = process.execPath,
+  nodePath = resolveCodexHookNodePath(),
   spotterBin = SPOTTER_BIN,
 } = {}) {
   const hooksPath = join(cursorHome, 'hooks.json');

package/src/core/runtime-error-store.mjs

@@ -270,10 +270,14 @@ function runRuntimeWorker(workerPath, args, timeoutMs, signal) {
       signal?.removeEventListener('abort', abort);
       resolve(result);
     };
-    const stop = (kind) => { if (stopping || settled) return; stopping = true; void killWorkerTree(child).then(() => finish({ kind })); };
+    const stop = (kind) => {
+      if (stopping || settled) return;
+      stopping = true;
+      finish({ kind });
+      void killWorkerTree(child);
+    };
     const abort = () => stop('cancelled');
-    const killGraceMs = process.platform === 'win32' ? 250 : 0;
-    const timer = setTimeout(() => stop('timeout'), Math.max(1, timeoutMs - killGraceMs));
+    const timer = setTimeout(() => stop('timeout'), timeoutMs);
     try {
       child = spawn(process.execPath, [workerPath, ...args], {
         stdio: 'ignore',

package/src/tool-db/investigate-mcp.mjs

@@ -11,7 +11,11 @@
 
 import { spawn } from 'node:child_process';
 import { isWindowsAbsolutePath } from '../platform/paths.mjs';
-import { execFileWindowsSafe, windowsCompatibleCommand } from '../platform/spawn.mjs';
+import {
+  execFileWindowsSafe,
+  terminateProcessTree,
+  windowsCompatibleCommand,
+} from '../platform/spawn.mjs';
 import { listToolsHttp } from './investigate-mcp-http.mjs';
 import { readMcpServers, describeServer } from './mcp-config.mjs';
 import { version as SPOTTER_VERSION } from '../version.mjs';
@@ -253,6 +257,17 @@ export function buildStdioSpawn(command, args) {
   return { cmd: invocation.command, cmdArgs: invocation.args };
 }
 
+export async function closeMcpTransport(child, { terminateChildFn = terminateProcessTree } = {}) {
+  const destroyStreams = () => {
+    for (const stream of [child.stdin, child.stdout, child.stderr]) {
+      try { stream.destroy(); } catch { /* ignore */ }
+    }
+  };
+  destroyStreams();
+  try { await terminateChildFn(child); } catch { /* direct kill fallback already ran */ }
+  destroyStreams();
+}
+
 async function spawnAndQuery({ command, args, env = {} }, serverName) {
   return new Promise((resolve, reject) => {
     const { cmd, cmdArgs } = buildStdioSpawn(command, args);
@@ -267,16 +282,14 @@ async function spawnAndQuery({ command, args, env = {} }, serverName) {
     let settled = false;
     let initializedSent = false;
 
-    const cleanup = () => {
-      try { child.stdin.end(); } catch { /* ignore */ }
-      try { child.kill(); } catch { /* ignore */ }
-    };
+    const cleanup = () => closeMcpTransport(child);
 
     const timer = setTimeout(() => {
       if (settled) return;
       settled = true;
-      cleanup();
-      reject(new McpInvestigationError(`handshake/list timed out after ${HANDSHAKE_TIMEOUT_MS}ms`, serverName));
+      void cleanup().then(() => {
+        reject(new McpInvestigationError(`handshake/list timed out after ${HANDSHAKE_TIMEOUT_MS}ms`, serverName));
+      });
     }, HANDSHAKE_TIMEOUT_MS);
 
     const send = (msg) => {
@@ -347,13 +360,13 @@ async function spawnAndQuery({ command, args, env = {} }, serverName) {
         if (settled) return;
         settled = true;
         clearTimeout(timer);
-        cleanup();
+        await cleanup();
         resolve(tools);
       } catch (err) {
         if (settled) return;
         settled = true;
         clearTimeout(timer);
-        cleanup();
+        await cleanup();
         reject(err);
       }
     })();