throughline 0.12.8 → 0.12.9

package/CHANGELOG.md

@@ -10,6 +10,19 @@ shipped to npm but were not individually tagged on GitHub.
 
 ## [Unreleased]
 
+## [0.12.9] — 2026-10-04
+
+### 修正
+
+- 新しいDBを複数のhookが同時に開くと、どれかが`no such table: judgments`か`database is locked`で落ちる不具合を直す（[ADR 0031](docs/adr/0031-serialized-schema-migration.md)）。WindowsのCursorの会話では同じhookが2本ほぼ同時に走るので、新しい端末の最初の会話で起きる。schemaの移行は、書き込みlockを取ってから版を読み直し、1つのtransactionで終える。WALへの切り替えは、断られたら読み直して5秒まで待つ。既に現行schemaのDBを開く時の動作は変わらない。
+- 切り離した置き場で再現した。新しいDBを6 processが同時に開くと、Linuxで6回中3回、どれか1本が落ちた。Windowsでも、最初の1回で2本が落ちた。修理後は48回（6 process）で失敗0。
+
+### 変更
+
+- `~/.throughline/logs/hook-failures.log`に、失敗した位置（`stack`、先頭6行まで）を足す。`database is locked`のような文面だけでは、どの処理で落ちたかが分からないため。端末内にだけ残し、外へは送らない。
+- hookのstdinがJSONとして読めない時の文面を`hook stdin is not valid JSON (N chars): …`にする。hostがstdinを渡さずに閉じた時（0字）と、途中で切れた時を見分けられる。失敗として数えることと、終了codeは変えない。
+- 2026-10-04にWindows（0.12.4）で起きたSessionStartとUserPromptSubmitの失敗各1回は、原因が分かっていない。どちらもCursorの会話で2本ほぼ同時に走ったhookの片方で、もう片方は成功している。現行schemaのDBで起きたので、上の修正の形ではない。この2つの変更は、次に起きた時に理由を端末へ残すためのもの。
+
 ## [0.12.8] — 2026-10-04
 
 ### 変更
@@ -1621,7 +1634,8 @@ two attempts, instrument first instead of patching again.
 
 ---
 
-[Unreleased]: https://github.com/kitepon/Throughline/compare/v0.12.8...HEAD
+[Unreleased]: https://github.com/kitepon/Throughline/compare/v0.12.9...HEAD
+[0.12.9]: https://github.com/kitepon/Throughline/compare/v0.12.8...v0.12.9
 [0.12.8]: https://github.com/kitepon/Throughline/compare/v0.12.7...v0.12.8
 [0.12.7]: https://github.com/kitepon/Throughline/compare/v0.12.6...v0.12.7
 [0.12.6]: https://github.com/kitepon/Throughline/compare/v0.12.5...v0.12.6

package/docs/adr/0031-serialized-schema-migration.md

@@ -0,0 +1,43 @@
+# ADR 0031: schema の移行は書き込み lock の中で版を読み直して行い、WAL への切り替えは読み直して待つ
+
+日付: 2026-10-04
+
+## Context
+
+`getDb()` は DB を開くたびに `PRAGMA user_version` を読み、現行版より古ければ移行を走らせていた。
+版を読むのも、移行の各 statement も、transaction の外だった。
+
+Windows の Cursor の会話では、同じ hook が2本ほぼ同時に走る（2026-10-04 の記録では、1会話の SessionStart が
+数ms〜320ms の間隔で2行残る。`~/.cursor/hooks.json` と `~/.claude/settings.json` の両方に Throughline の登録がある。
+2本目がどちらの登録から来るかは確かめていない）。新しい端末の最初の会話では、2本とも版 0 を読む。片方が先に移行を終えると、もう片方は版 0 のつもりで
+v2 の移行（`DELETE FROM judgments ...`）を走らせ、v4 で消えた表へ触って `no such table: judgments` で落ちる。
+
+もう1つ、新しい DB の journal mode を WAL へ切り替える `PRAGMA journal_mode = WAL` は排他 lock を要る。
+他の process が同じ DB を開いている間は、`busy_timeout` を待たずに `database is locked` で断られる。
+
+2026-10-04 に、切り離した置き場で再現した。新しい DB を 6 process が同時に開くと、Linux（Node 26）で
+6回中3回、どれか1本が上のどちらかで落ちた。Windows（Node 24）でも、SessionStart と UserPromptSubmit を
+2本ずつ同時に動かした最初の1回で2本が `no such table: judgments` で落ちた。
+既に現行 schema の DB では、どちらも起きない（版が現行なら移行は何も書かず、WAL なら切り替えない）。
+
+## Decision
+
+1. 移行は `BEGIN IMMEDIATE` で書き込み lock を取り、その中で `PRAGMA user_version` を読み直してから走らせ、
+   `user_version` の更新まで1つの transaction で commit する。失敗した時は rollback し、途中の schema を残さない。
+2. lock の外で読んだ版が現行版以上なら、lock を取らない。現行 schema の DB を開く hook は、今までどおり何も書かない。
+3. 他の process が移行している間に開いた process は、`busy_timeout`（5秒）まで lock を待ち、
+   取れた後に読み直した版が現行なら何もしない。
+4. WAL への切り替えは、`database is locked` で断られたら 25ms 待って journal mode を読み直し、
+   `busy_timeout` と同じ時間まで繰り返す。他の process が先に切り替えれば、読み直しで WAL が見えて終わる。
+5. `migrate --json`（ADR 0018 の正規入口）も同じ関数を通る。結果の形は変えない。
+
+## Consequences
+
+- 新しい端末で複数の hook が同時に最初の DB を開いても、全員が現行 schema で開ける。
+- 古い schema からの移行中、他の hook は最長5秒待つ。移行が5秒を超えると、待った hook は
+  `database is locked` で失敗する。今までは、移行の途中の schema で走るか、同じ移行をもう一度走らせていた。
+- 移行が途中で失敗した DB は、移行前の schema のまま残る。次に開いた時に最初からやり直す。
+- 2026-10-04 に Windows（0.12.4）で起きた SessionStart と UserPromptSubmit の失敗各1回は、
+  現行 schema の DB で起きている。この ADR の2つの形ではなく、原因は分かっていない。
+  理由の文面は 0.12.4 では端末に残らない。0.12.9 は `hook-failures.log` に失敗した位置（stack の先頭）を足し、
+  hook の stdin が JSON として読めない時の文面に字数を入れる。

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "throughline",
-  "version": "0.12.8",
+  "version": "0.12.9",
   "type": "module",
   "description": "Persistent memory hooks for Claude Code, Codex, Grok, and Cursor (/clear-safe context compression)",
   "keywords": [

package/src/db-contention.test.mjs

@@ -93,3 +93,30 @@ test('getDb reuses an existing WAL database while another process owns the write
     rmSync(home, { recursive: true, force: true });
   }
 });
+
+test('getDbは、新しいDBを複数のprocessが同時に開いても、全員が現行schemaで開ける', async () => {
+  // WindowsのCursorの会話では同じhookが2本ほぼ同時に走る。初回はどのprocessも版0を読むので、lockの外で読んだ版から
+  // 移行をやり直すと、先に進んだprocessが消した表へ触って落ちた（修理前は6本で半分ほどの確率）。
+  const rounds = 6;
+  const processes = 6;
+  for (let round = 0; round < rounds; round += 1) {
+    const home = mkdtempSync(join(tmpdir(), 'throughline-db-first-open-'));
+    const env = { ...process.env, HOME: home, USERPROFILE: home };
+    try {
+      const results = await Promise.all(Array.from({ length: processes }, () => {
+        const child = spawn(process.execPath, ['--input-type=module', '-e', `
+          import { getDb, CURRENT_VERSION } from ${JSON.stringify(DB_MODULE_URL)};
+          const db = getDb();
+          const version = db.prepare('PRAGMA user_version').get().user_version;
+          db.prepare('SELECT count(*) AS n FROM sessions').get();
+          db.close();
+          if (version !== CURRENT_VERSION) process.exit(2);
+        `], { env, stdio: ['ignore', 'ignore', 'pipe'] });
+        return waitForExit(child);
+      }));
+      assert.deepEqual(results, Array.from({ length: processes }, () => ({ code: 0, stderr: '' })));
+    } finally {
+      rmSync(home, { recursive: true, force: true });
+    }
+  }
+});

package/src/db.mjs

@@ -25,10 +25,68 @@ export function openReadOnlyDb(path = DB_PATH) {
   }
 }
 
+const WAL_SWITCH_RETRY_MS = 25;
+
+function isSqliteBusy(error) {
+  if (typeof error?.errcode === 'number') return (error.errcode & 0xff) === 5;
+  return /database is locked/u.test(String(error?.message ?? ''));
+}
+
+function sleepSync(ms) {
+  Atomics.wait(new Int32Array(new SharedArrayBuffer(4)), 0, 0, ms);
+}
+
+/**
+ * journal mode を WAL にする。既に WAL なら何も書かない。
+ *
+ * WAL への切り替えは排他 lock を要り、他の process が同じ DB を開いている間は busy_timeout を待たずに
+ * `database is locked` で断られる。新しい DB を複数の hook が同時に開く時に起きるので、
+ * busy_timeout と同じ時間まで読み直す (ADR 0031)。他の process が先に切り替えれば、読み直しで WAL が見える。
+ */
+function ensureWalJournalMode(db) {
+  const deadline = Date.now() + DB_BUSY_TIMEOUT_MS;
+  for (;;) {
+    try {
+      const journalMode = db.prepare('PRAGMA journal_mode').get().journal_mode;
+      if (String(journalMode).toLowerCase() === 'wal') return;
+      db.exec('PRAGMA journal_mode = WAL');
+      return;
+    } catch (error) {
+      if (!isSqliteBusy(error) || Date.now() >= deadline) throw error;
+      sleepSync(WAL_SWITCH_RETRY_MS);
+    }
+  }
+}
+
+function readSchemaVersion(db) {
+  return db.prepare('PRAGMA user_version').get().user_version ?? 0;
+}
+
+/**
+ * schema を現行版へ上げる。現行版なら何も書かない。
+ *
+ * 移行は書き込み lock を取ってから版を読み直し、1つの transaction で終える (ADR 0031)。
+ * lock の外で読んだ版は、他の process が移行を進めた後では古い。古い版から移行をやり直すと、
+ * 途中の版で消した表（v4 の judgments）へ触って `no such table` で落ちる。
+ */
 function initSchema(db) {
-  const row = db.prepare('PRAGMA user_version').get();
-  const version = row.user_version ?? 0;
+  if (readSchemaVersion(db) >= CURRENT_VERSION) return;
+  db.exec('BEGIN IMMEDIATE');
+  try {
+    const version = readSchemaVersion(db);
+    if (version < CURRENT_VERSION) applySchemaMigrations(db, version);
+    db.exec('COMMIT');
+  } catch (error) {
+    try {
+      db.exec('ROLLBACK');
+    } catch {
+      // ignore
+    }
+    throw error;
+  }
+}
 
+function applySchemaMigrations(db, version) {
   // v0 → v1: 全テーブル作成
   if (version < 1) {
     db.exec(`
@@ -353,10 +411,7 @@ export function migrateDefaultDb() {
       };
     }
 
-    const journalMode = db.prepare('PRAGMA journal_mode').get().journal_mode;
-    if (String(journalMode).toLowerCase() !== 'wal') {
-      db.exec('PRAGMA journal_mode = WAL');
-    }
+    ensureWalJournalMode(db);
     initSchema(db);
     const afterSchemaVersion = Number(db.prepare('PRAGMA user_version').get().user_version ?? 0);
     if (afterSchemaVersion !== CURRENT_VERSION) {
@@ -397,10 +452,7 @@ export function getDb() {
   const db = new DatabaseSync(DB_PATH);
   try {
     db.exec(`PRAGMA busy_timeout = ${DB_BUSY_TIMEOUT_MS}`);
-    const journalMode = db.prepare('PRAGMA journal_mode').get().journal_mode;
-    if (String(journalMode).toLowerCase() !== 'wal') {
-      db.exec('PRAGMA journal_mode = WAL');
-    }
+    ensureWalJournalMode(db);
     db.exec('PRAGMA foreign_keys = ON');
     initSchema(db);
     _db = db;

package/src/hook-entrypoints.test.mjs

@@ -976,7 +976,9 @@ test('a failed hook leaves its reason in the local hook failure log', () => {
       ['HOOK_SESSION_START_FAILED', 'Error', 'Missing session_id in SessionStart payload'],
     ]);
     for (const entry of entries) {
-      assert.deepEqual(Object.keys(entry), ['ts', 'code', 'version', 'name', 'message']);
+      assert.deepEqual(Object.keys(entry), ['ts', 'code', 'version', 'name', 'message', 'stack']);
+      // 失敗した位置は、payloadを検査して投げたhookの入口を指す。
+      assert.match(entry.stack[0], /^at .*(turn-processor|session-start)\.mjs:\d+:\d+\)?$/);
       assert.match(entry.ts, /^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}\.\d{3}Z$/);
       assert.match(entry.version, /^\d+\.\d+\.\d+$/);
     }

package/src/hook-failure-log.mjs

@@ -3,7 +3,7 @@
  *
  * runtime error store は外へ送れる形（定型 code と回数）だけを持つ (ADR 0025)。失敗した時の
  * 例外の文面はどこにも残らず、後から原因を追えなかった（2026-10-04、Mac と main-server の
- * 7〜8月の失敗）。このログは理由を端末内にだけ残す。runtime error の収集・送信の設定とは
+ * 7〜8月の失敗）。このログは理由と失敗した位置（stack の先頭）を端末内にだけ残す。runtime error の収集・送信の設定とは
  * 独立で、外へは送らない。
  * 書き込み失敗は stderr に出す（黙って握りつぶさない）。
  */
@@ -17,11 +17,23 @@ const require = createRequire(import.meta.url);
 const PACKAGE_VERSION = require('../package.json').version;
 
 export const HOOK_FAILURE_MESSAGE_LIMIT = 1000;
+export const HOOK_FAILURE_STACK_FRAMES = 6;
+export const HOOK_FAILURE_STACK_FRAME_LIMIT = 300;
 
 export function hookFailureLogPath({ home = homedir() } = {}) {
   return join(home, '.throughline', 'logs', 'hook-failures.log');
 }
 
+function stackFrames(error) {
+  if (!(error instanceof Error) || typeof error.stack !== 'string') return [];
+  return error.stack
+    .split('\n')
+    .map((line) => line.trim())
+    .filter((line) => line.startsWith('at '))
+    .slice(0, HOOK_FAILURE_STACK_FRAMES)
+    .map((line) => line.slice(0, HOOK_FAILURE_STACK_FRAME_LIMIT));
+}
+
 /**
  * @param {string} code runtime error の定型 code（例: HOOK_PROCESS_TURN_FAILED）
  * @param {unknown} error 失敗の原因
@@ -36,6 +48,10 @@ export function logHookFailure(code, error, { home, now = Date.now(), stderr = p
     name: error instanceof Error ? error.name : typeof error,
     message: message.slice(0, HOOK_FAILURE_MESSAGE_LIMIT),
   };
+  // 失敗した位置を残す。`database is locked` のような文面だけでは、DB を開く時か書く時か、
+  // どの処理で落ちたかが分からない（2026-10-04、Windows の SessionStart と UserPromptSubmit の失敗）。
+  const frames = stackFrames(error);
+  if (frames.length > 0) entry.stack = frames;
   // どの会話の失敗かを残す（errorが `hookContext` を持つ時）。回数と文面だけでは、失敗した会話を
   // 端末の他の記録から探すことになる。
   const context = error?.hookContext;

package/src/hook-failure-log.test.mjs

@@ -3,7 +3,13 @@ import assert from 'node:assert/strict';
 import { mkdtempSync, readFileSync, rmSync, writeFileSync } from 'node:fs';
 import { tmpdir } from 'node:os';
 import { join } from 'node:path';
-import { HOOK_FAILURE_MESSAGE_LIMIT, hookFailureLogPath, logHookFailure } from './hook-failure-log.mjs';
+import {
+  HOOK_FAILURE_MESSAGE_LIMIT,
+  HOOK_FAILURE_STACK_FRAMES,
+  HOOK_FAILURE_STACK_FRAME_LIMIT,
+  hookFailureLogPath,
+  logHookFailure,
+} from './hook-failure-log.mjs';
 
 test('logHookFailureは、失敗の理由を1行のJSONで追記する', () => {
   const home = mkdtempSync(join(tmpdir(), 'tl-hook-failure-log-'));
@@ -12,9 +18,12 @@ test('logHookFailureは、失敗の理由を1行のJSONで追記する', () => {
     logHookFailure('HOOK_CODEX_FAILED', 'thrown string', { home, now: Date.UTC(2026, 9, 4, 0, 0, 1) });
     const lines = readFileSync(hookFailureLogPath({ home }), 'utf8').trim().split('\n').map((line) => JSON.parse(line));
     assert.equal(lines.length, 2);
-    assert.deepEqual({ ...lines[0], version: 'x' }, {
+    const { stack, ...first } = lines[0];
+    assert.deepEqual({ ...first, version: 'x' }, {
       ts: '2026-10-04T00:00:00.005Z', code: 'HOOK_PROCESS_TURN_FAILED', version: 'x', name: 'TypeError', message: 'first reason',
     });
+    // Errorは失敗した位置を持つ。文字列を投げた時はstackが無いので、項目ごと残さない。
+    assert.equal(Array.isArray(stack) && stack.length > 0, true);
     assert.deepEqual({ ...lines[1], version: 'x' }, {
       ts: '2026-10-04T00:00:01.000Z', code: 'HOOK_CODEX_FAILED', version: 'x', name: 'string', message: 'thrown string',
     });
@@ -61,6 +70,31 @@ test('logHookFailureは、どの会話の失敗かを残す', () => {
   }
 });
 
+test('logHookFailureは、失敗した位置をstackの先頭から残す', () => {
+  const home = mkdtempSync(join(tmpdir(), 'tl-hook-failure-log-'));
+  try {
+    const err = new Error('database is locked');
+    err.stack = [
+      'Error: database is locked',
+      '    at getDb (file:///throughline/src/db.mjs:455:5)',
+      `    at ${'x'.repeat(HOOK_FAILURE_STACK_FRAME_LIMIT + 40)}`,
+      ...Array.from({ length: HOOK_FAILURE_STACK_FRAMES + 3 }, (_, index) => `    at frame${index} (file:///throughline/src/a.mjs:${index}:1)`),
+    ].join('\n');
+    logHookFailure('HOOK_SESSION_START_FAILED', err, { home });
+    const noFrames = new Error('no frames');
+    noFrames.stack = 'Error: no frames';
+    logHookFailure('HOOK_SESSION_START_FAILED', noFrames, { home });
+    const [entry, second] = readFileSync(hookFailureLogPath({ home }), 'utf8').trim().split('\n').map((line) => JSON.parse(line));
+    assert.equal(entry.stack.length, HOOK_FAILURE_STACK_FRAMES);
+    assert.equal(entry.stack[0], 'at getDb (file:///throughline/src/db.mjs:455:5)');
+    assert.equal(entry.stack[1].length, HOOK_FAILURE_STACK_FRAME_LIMIT);
+    assert.equal(entry.stack[2], 'at frame0 (file:///throughline/src/a.mjs:0:1)');
+    assert.equal('stack' in second, false);
+  } finally {
+    rmSync(home, { recursive: true, force: true });
+  }
+});
+
 test('logHookFailureは、長い文面を上限で切る', () => {
   const home = mkdtempSync(join(tmpdir(), 'tl-hook-failure-log-'));
   try {

package/src/hosts/cursor.test.mjs

@@ -25,6 +25,16 @@ test('parseHookPayload accepts one leading BOM and preserves BOM inside prompt t
   assert.throws(() => parseHookPayload('\uFEFF{"conversation_id":'), SyntaxError);
 });
 
+test('parseHookPayloadは、stdinが空の時と途中で切れた時を、文面の字数で見分けられるようにする', () => {
+  assert.throws(() => parseHookPayload(''), (error) => {
+    assert.equal(error instanceof SyntaxError, true);
+    assert.match(error.message, /^hook stdin is not valid JSON \(0 chars\): /u);
+    assert.equal(error.cause instanceof SyntaxError, true);
+    return true;
+  });
+  assert.throws(() => parseHookPayload('\uFEFF{"conversation_id":'), /^SyntaxError: hook stdin is not valid JSON \(20 chars\): /u);
+});
+
 test('isCursorEnvelope detects Cursor events and cursor_version', () => {
   assert.equal(
     isCursorEnvelope({

package/src/hosts/index.mjs

@@ -29,7 +29,15 @@ const ADAPTERS = Object.freeze({
 
 /** Parse hook stdin, including the leading UTF-8 BOM emitted by Windows Cursor. */
 export function parseHookPayload(raw, options = {}) {
-  return normalizeHookPayload(JSON.parse(raw.replace(/^\uFEFF/, '')), options);
+  let parsed;
+  try {
+    parsed = JSON.parse(raw.replace(/^\uFEFF/, ''));
+  } catch (error) {
+    // host が stdin を渡さずに閉じた時（0 字）と、途中で切れた時を、理由の文面で見分けられるようにする。
+    const detail = error instanceof Error ? error.message : String(error);
+    throw new SyntaxError(`hook stdin is not valid JSON (${raw.length} chars): ${detail}`, { cause: error });
+  }
+  return normalizeHookPayload(parsed, options);
 }
 
 /**