throughline 0.12.5 → 0.12.6

package/CHANGELOG.md

@@ -10,6 +10,21 @@ shipped to npm but were not individually tagged on GitHub.
 
 ## [Unreleased]
 
+## [0.12.6] — 2026-10-04
+
+### 修正
+
+- Codex Stop hookで、会話の取り込みが済んだ後のL1要約backend（Codex CLI）の失敗を、`HOOK_CODEX_FAILED`（Codex hook processing failed・high）ではなく`L1_SUMMARIZER_BACKEND_FAILED`（L1 summarizer backend failed・warn）として数える（[ADR 0028](docs/adr/0028-l1-summarizer-backend-failure-code.md)）。Codex CLIが利用上限や認証切れで非0終了すると、取り込みは成功しているのにhook処理の失敗として記録されていた。hookの終了codeとstderrは変えない。要約は次のStopが同じturnからやり直す。
+- 0.9.0と0.12.5の両方で、20 turnを超えるスレッドのStopがCodex CLIの失敗で`HOOK_CODEX_FAILED`を記録することを再現した。Macに残っていた1151回の最後の1回は、そのスレッドが利用上限に当たった時刻と一致する。1151回すべてがこの形だったことを示す記録は無い。
+
+### 変更
+
+- `~/.throughline/logs/hook-failures.log`に、errorが持つ`reason`と、外部CLIの`stderr`の末尾（1000字まで）を足す。`Codex CLI summarizer failed: exit 1`だけでは、利用上限か認証切れかが分からないため。端末内にだけ残し、外へは送らない。
+
+### 試験
+
+- Observerのcursorがprojectのpath・session id・本文を持たないことを確かめる試験が、偶然落ちることがあった。sha256の16進に目印の`a257`が現れると失敗し、Windowsではpathをそのまま正規表現にしていた。目印を16進に現れない文字列にし、値をそのまま探す形に直す。製品の動作は変えていない。
+
 ## [0.12.5] — 2026-10-04
 
 ### 修正
@@ -1588,7 +1603,8 @@ two attempts, instrument first instead of patching again.
 
 ---
 
-[Unreleased]: https://github.com/kitepon/Throughline/compare/v0.12.5...HEAD
+[Unreleased]: https://github.com/kitepon/Throughline/compare/v0.12.6...HEAD
+[0.12.6]: https://github.com/kitepon/Throughline/compare/v0.12.5...v0.12.6
 [0.12.5]: https://github.com/kitepon/Throughline/compare/v0.12.4...v0.12.5
 [0.12.4]: https://github.com/kitepon/Throughline/compare/v0.12.3...v0.12.4
 [0.12.3]: https://github.com/kitepon/Throughline/compare/v0.12.2...v0.12.3

package/docs/adr/0015-l1-summarizer-model-effort-ratio.md

@@ -66,6 +66,8 @@ none/low/medium/high/xhigh）のため none を下端とした。
 4. claude-primary の backend 順序を **codex-sidecar → Codex CLI (luna@low) → Haiku →
    raw L2** にする。各段の失敗理由は結果 (`sidecarReason` / `codexCliReason`) に記録
    する宣言済み fallback。codex-primary は従来どおり Codex CLI 一本で explicit error。
+   （2026-10-04 [ADR 0028](0028-l1-summarizer-backend-failure-code.md)で、この explicit error を runtime error store では
+   `L1_SUMMARIZER_BACKEND_FAILED` として数えることにした。hook の終了 code と stderr は変えていない。）
    （実測で Haiku は timeout 完全失敗があり、luna は同等品質・半分以下のレイテンシ・
    失敗ゼロのため、Haiku より先に置く。）
 5. sidecar `summarize-l1` preset のモデルは codex-sidecar（別repo）の所有。本 ADR では

package/docs/adr/0028-l1-summarizer-backend-failure-code.md

@@ -0,0 +1,59 @@
+# ADR 0028: Codex Stop の L1 要約 backend の失敗は、hook 処理の失敗と別の code で数える
+
+日付: 2026-10-04
+
+## Context
+
+Codex Stop hook は、rollout を DB へ取り込んで commit した後、bodies が L2 window（20 turn）を超えていれば、
+最古の未要約 turn を Codex CLI（`codex exec`）で L1 に要約する。codex-primary の要約は Codex CLI 一本で、
+CLI の失敗は explicit error にする（ADR 0015 Decision 4）。
+
+この error は hook の一番外の catch まで届き、`HOOK_CODEX_FAILED`（`Throughline Codex hook processing failed`、
+severity `high`）として runtime error store に数えられていた。
+
+Mac の runtime error store には `HOOK_CODEX_FAILED` が 1151 回あった（2026-07-13 〜 2026-08-07、最後の版は 0.9.0）。
+最後の1回は 2026-08-07T06:23:58.298Z で、その Codex スレッドは同じ秒に次の turn が
+`You've hit your usage limit` で終わっている。当時の stderr はどこにも残っていない。
+
+0.9.0 と 0.12.5 の両方で、次を再現した。20 turn を超えるスレッドに Stop を送り、Codex CLI が非0で終わると、
+取り込みは成功したまま hook が `Codex CLI summarizer failed: exit N`（reason `codex_cli_failed`）で失敗し、
+`HOOK_CODEX_FAILED` が1回増える。UserPromptSubmit と PostToolUse は要約をしないので失敗しない。
+
+`HOOK_CODEX_FAILED` という code からは、取り込みが壊れたのか、外部 CLI が使えなかっただけなのかを区別できない。
+後者は Codex 側の利用上限・認証切れ・model の失敗で起き、Throughline の修理では直らない。
+
+## Decision
+
+1. Codex hook の失敗のうち、L1 要約 backend の失敗だけを `L1_SUMMARIZER_BACKEND_FAILED` として数える。
+   component は `codex_l1_summarizer`、template は `Throughline L1 summarizer backend (Codex CLI) failed`、
+   severity は `warn`。
+2. 対象は `source` が `codex-cli` で、`reason` が `codex_cli_failed`（非0終了・timeout・起動失敗）か
+   `empty_output` の error だけ。`missing_project_path` と `recursion_guard` は製品側の誤りなので、
+   `HOOK_CODEX_FAILED` のままにする。要約以外の失敗も `HOOK_CODEX_FAILED` のまま。
+3. hook の終了 code（1）と stderr（`[codex-hook] Codex CLI summarizer failed: exit N`）は変えない。
+   ADR 0015 の explicit error 契約と、失敗を隠さない原則はそのまま。変えるのは数える code だけ。
+4. `hook-failures.log`（ADR 0027）に、error が持つ `reason` と、外部 CLI の `stderr` の末尾（1000字まで）を足す。
+   `Codex CLI summarizer failed: exit 1` だけでは、利用上限なのか認証切れなのかが分からないため。
+   このログは端末内にだけ残し、外へは送らない。runtime error store には今までどおり定型 code と回数だけを入れる。
+5. severity を `warn` にする理由: 会話の取り込みは commit 済みで、記憶は失われていない。要約は次の Stop が
+   同じ turn からやり直す。L1 が無い間、recall は L2 の本文を返す（ADR 0016）。
+   BugHub が受ける severity は `fatal`・`high`・`warn`・`info` の4つで、他の値は report 全体が 422 になる
+   （2026-10-04、BugHub の `PRODUCT_REPORTING.md` と `factory-report-v1.schema.json` をBugHubの担当が確認）。
+
+## Consequences
+
+- 外部 CLI が使えない端末は、BugHub で「L1 summarizer backend failed」（warn、日次のまとめ）として見える。
+  「Codex hook processing failed」（high、即時通知）は、取り込みや hook 処理そのものの失敗だけになる。
+- fingerprint は code ごとに決まるので、新しい code は BugHub で新しい issue になる。過去の `HOOK_CODEX_FAILED` の
+  回数は移らない。
+- runtime error store は、知らない code を持つ記録を schema invalid として拒否する。この版で
+  `L1_SUMMARIZER_BACKEND_FAILED` を記録した端末を 0.12.5 以前へ戻すと、store を読めなくなる。
+- Claude Stop は対象外。claude-primary の要約は Codex CLI → Haiku → raw L2 の宣言済み fallback で、
+  backend の失敗では throw しない。
+
+## 確認できていないこと
+
+- Mac の 1151 回がすべてこの形だったことを示す記録は無い。確認できたのは、同じ code を同じ版で再現できたことと、
+  最後の1回が利用上限に当たった時刻と一致することだけ。外した仮説は、製品の試験からの漏れ、rollout の大きさ、
+  入力の形、同時実行、スレッド全体の順次再生。
+- 要約が失敗し続ける端末では、Codex の Stop hook が毎回 exit 1 になる。Codex 側の表示への影響は調べていない。

package/package.json

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

package/src/cli/codex-hook.mjs

@@ -360,6 +360,22 @@ async function runCodexContextRefreshInstructionHook({
   };
 }
 
+const L1_SUMMARIZER_BACKEND_REASONS = new Set(['codex_cli_failed', 'empty_output']);
+
+/**
+ * Codex hook が失敗した時に記録する runtime error の code を決める（ADR 0028）。
+ * L1 要約の backend（Codex CLI）が非0で終わった、または空を返した失敗は、取り込みの後に起きる
+ * 外部 CLI の失敗なので、hook 処理の失敗とは別の code で数える。hook の終了 code と stderr は変えない。
+ * @param {unknown} error
+ * @returns {'HOOK_CODEX_FAILED' | 'L1_SUMMARIZER_BACKEND_FAILED'}
+ */
+export function codexHookFailureCode(error) {
+  if (error?.source === 'codex-cli' && L1_SUMMARIZER_BACKEND_REASONS.has(error?.reason)) {
+    return 'L1_SUMMARIZER_BACKEND_FAILED';
+  }
+  return 'HOOK_CODEX_FAILED';
+}
+
 export async function run(argv = []) {
   suppressExperimentalWarnings();
   let parsed;
@@ -417,8 +433,9 @@ export async function run(argv = []) {
     // continue=falseは、明示した失敗理由を含めてhostへ停止を届けた返答。背景処理の成否は引き継ぎ状態へ記録する。
     process.exit(result.continue === false || result.status === 'ok' || result.status === 'skipped' ? 0 : 1);
   } catch (err) {
-    recordRuntimeErrorBestEffort('HOOK_CODEX_FAILED', { env: process.env });
-    logHookFailure('HOOK_CODEX_FAILED', err);
+    const code = codexHookFailureCode(err);
+    recordRuntimeErrorBestEffort(code, { env: process.env });
+    logHookFailure(code, err);
     const msg = err instanceof Error ? err.message : 'unknown';
     if (parsed.json) {
       process.stdout.write(

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

@@ -6,6 +6,7 @@ import { test } from 'node:test';
 import { DatabaseSync } from 'node:sqlite';
 
 import {
+  codexHookFailureCode,
   runCodexPostToolUseHook,
   runCodexStopHook,
   runCodexUserPromptSubmitHook,
@@ -497,6 +498,19 @@ test('codex-hook stop skips cleanly when Codex thread id is unavailable', async
   }
 });
 
+test('codexHookFailureCodeは、L1要約backendの失敗だけを別のcodeにする', () => {
+  const backend = (reason) => Object.assign(new Error('x'), { source: 'codex-cli', reason });
+  assert.equal(codexHookFailureCode(backend('codex_cli_failed')), 'L1_SUMMARIZER_BACKEND_FAILED');
+  assert.equal(codexHookFailureCode(backend('empty_output')), 'L1_SUMMARIZER_BACKEND_FAILED');
+  // 製品側の誤り（projectPath 欠落・再帰）と、要約以外の失敗は hook 処理の失敗のまま。
+  assert.equal(codexHookFailureCode(backend('missing_project_path')), 'HOOK_CODEX_FAILED');
+  assert.equal(codexHookFailureCode(backend('recursion_guard')), 'HOOK_CODEX_FAILED');
+  assert.equal(codexHookFailureCode(Object.assign(new Error('x'), { reason: 'codex_cli_failed' })), 'HOOK_CODEX_FAILED');
+  assert.equal(codexHookFailureCode(new Error('database is locked')), 'HOOK_CODEX_FAILED');
+  assert.equal(codexHookFailureCode('thrown string'), 'HOOK_CODEX_FAILED');
+  assert.equal(codexHookFailureCode(null), 'HOOK_CODEX_FAILED');
+});
+
 test('codexHomeFromTranscriptPath infers CODEX_HOME from rollout path', () => {
   const path = '/tmp/codex-home/sessions/2026/05/06/rollout-2026-05-06T09-40-50-id.jsonl';
   assert.equal(_internal.codexHomeFromTranscriptPath(path), '/tmp/codex-home');

package/src/hook-failure-log.mjs

@@ -36,6 +36,12 @@ 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),
   };
+  // 外部 CLI の失敗は、message が終了 code だけになる。理由（reason）と CLI の stderr の末尾も残す
+  // （CLI は先頭に起動情報を出し、失敗の理由は末尾に出す）。
+  if (typeof error?.reason === 'string' && error.reason) entry.reason = error.reason;
+  if (typeof error?.stderr === 'string' && error.stderr) {
+    entry.stderr = error.stderr.slice(-HOOK_FAILURE_MESSAGE_LIMIT);
+  }
   const path = hookFailureLogPath(home === undefined ? {} : { home });
   try {
     mkdirSync(dirname(path), { recursive: true });

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

@@ -23,6 +23,24 @@ test('logHookFailureは、失敗の理由を1行のJSONで追記する', () => {
   }
 });
 
+test('logHookFailureは、外部CLIの失敗の理由とstderrの末尾を残す', () => {
+  const home = mkdtempSync(join(tmpdir(), 'tl-hook-failure-log-'));
+  try {
+    const err = new Error('Codex CLI summarizer failed: exit 1');
+    err.reason = 'codex_cli_failed';
+    err.stderr = `${'banner '.repeat(300)}ERROR: usage limit`;
+    logHookFailure('L1_SUMMARIZER_BACKEND_FAILED', err, { home });
+    const [entry] = readFileSync(hookFailureLogPath({ home }), 'utf8').trim().split('\n').map((line) => JSON.parse(line));
+    assert.equal(entry.code, 'L1_SUMMARIZER_BACKEND_FAILED');
+    assert.equal(entry.message, 'Codex CLI summarizer failed: exit 1');
+    assert.equal(entry.reason, 'codex_cli_failed');
+    assert.equal(entry.stderr.length, HOOK_FAILURE_MESSAGE_LIMIT);
+    assert.match(entry.stderr, /ERROR: usage limit$/);
+  } finally {
+    rmSync(home, { recursive: true, force: true });
+  }
+});
+
 test('logHookFailureは、長い文面を上限で切る', () => {
   const home = mkdtempSync(join(tmpdir(), 'tl-hook-failure-log-'));
   try {

package/src/observer-turn-feed.test.mjs

@@ -126,7 +126,7 @@ test('observer feed: Claude history floor, host/thread switch, cross-host tie, a
   try {
     const options = box.receiptOptions;
     seedCompletedTurnReceiptStore({ projectPath: box.project, receiptOptions: options, targetSessionId: 'claude-session' });
-    writeCompletedTurnReceipt({ projectPath: box.project, targetSessionId: 'claude-session', originSessionId: 'o257', userBody: 'u257', assistantBody: 'a257', completedAt: 257 }, options);
+    writeCompletedTurnReceipt({ projectPath: box.project, targetSessionId: 'claude-session', originSessionId: 'o257', userBody: 'user-body-257', assistantBody: 'assistant-body-257', completedAt: 257 }, options);
     const latest = resolveObserverTurnFeed({ projectPath: box.project, receiptOptions: options, codexHome: box.home });
     const decoded = decodeObserverCursor(latest.throughCursor);
     const beforeFloor = encodeObserverCursor({ ...decoded, history_floor: 1 });
@@ -137,7 +137,12 @@ test('observer feed: Claude history floor, host/thread switch, cross-host tie, a
     const changedPrefix = encodeObserverCursor({ ...decoded, prefix_sha256: 'f'.repeat(64) });
     assert.equal(resolveObserverTurnFeed({ projectPath: box.project, cursor: changedPrefix, receiptOptions: options, codexHome: box.home }).status, 'resync_required');
     assert.throws(() => encodeObserverCursor({ ...decoded, length: 0 }), /cursor invalid/);
-    assert.doesNotMatch(JSON.stringify(decoded), new RegExp(`${box.project}|claude-session|u257|a257`));
+    // cursor は project の path・session id・本文を持たない。目印は16進に現れない文字列にし、正規表現を使わず
+    // 値をそのまま探す（sha256 の16進に `a257` が偶然現れる、Windows の path が正規表現として壊れる、の2つを避ける）。
+    const cursorValues = Object.values(decoded).map(String).join('\n');
+    for (const secret of [box.project, 'claude-session', 'user-body-257', 'assistant-body-257']) {
+      assert.equal(cursorValues.includes(secret), false, secret);
+    }
     const id = '019dfaba-f87e-7f41-a144-d5ca7c6dd7f9';
     writeRollout(box.home, box.project, id, completeEvents('2026-07-15T00:03:00.000Z'));
     const switched = resolveObserverTurnFeed({ projectPath: box.project, cursor: latest.throughCursor, receiptOptions: options, codexHome: box.home });

package/src/runtime-error-hook.test.mjs

@@ -1,7 +1,8 @@
 import { test } from 'node:test';
 import assert from 'node:assert/strict';
 import { execFileSync, spawnSync } from 'node:child_process';
-import { mkdirSync, mkdtempSync, readFileSync, writeFileSync } from 'node:fs';
+import { chmodSync, mkdirSync, mkdtempSync, readFileSync, writeFileSync } from 'node:fs';
+import { DatabaseSync } from 'node:sqlite';
 import { tmpdir } from 'node:os';
 import { dirname, join } from 'node:path';
 import { fileURLToPath } from 'node:url';
@@ -69,6 +70,77 @@ test('top-level hook owners record one fixed aggregate per failure without repla
   assert.equal(store.records.find((record) => record.error_code === 'HOOK_PROCESS_TURN_FAILED').count, 2);
 });
 
+function makeBin(dir, name, body) {
+  const script = join(dir, `${name}.mjs`);
+  writeFileSync(script, body);
+  if (process.platform === 'win32') {
+    const command = join(dir, `${name}.cmd`);
+    writeFileSync(command, `@echo off\r\n${JSON.stringify(process.execPath)} ${JSON.stringify(script)} %*\r\n`);
+    writeFileSync(join(dir, `${name}.ps1`), `& ${JSON.stringify(process.execPath)} ${JSON.stringify(script)} @args\nexit $LASTEXITCODE\n`);
+    return command;
+  }
+  const command = join(dir, name);
+  writeFileSync(command, `#!/bin/sh\nexec ${JSON.stringify(process.execPath)} ${JSON.stringify(script)} "$@"\n`);
+  chmodSync(command, 0o755);
+  return command;
+}
+
+test('Codex StopでL1要約backendが失敗した時、取り込みは残り、別のcodeで1回数える', () => {
+  const { root, env } = createEnabledEnvironment('throughline-runtime-hook-summarizer-');
+  const threadId = '019dfaba-f87e-7f41-a144-d5ca7c6dd7f9';
+  const project = join(root, 'project');
+  const codexHome = join(root, 'codex');
+  const rolloutDir = join(codexHome, 'sessions', '2026', '05', '06');
+  mkdirSync(project, { recursive: true });
+  mkdirSync(rolloutDir, { recursive: true });
+  const at = '2026-05-06T00:41:00.000Z';
+  const event = (type, payload = {}) => ({ timestamp: at, type: 'event_msg', payload: { type, ...payload } });
+  const rows = [{ timestamp: at, type: 'session_meta',
+    payload: { id: threadId, timestamp: at, cwd: project, source: 'vscode', cli_version: '0.128.0-alpha.1' } }];
+  // L2 window（20 turn）を超えると、Stop は最古の turn を Codex CLI で要約する。
+  for (let turn = 0; turn < 22; turn += 1) {
+    rows.push(event('user_message', { message: `request ${turn}` }), event('task_started'),
+      { timestamp: at, type: 'turn_context', payload: { model: 'gpt-5.5', cwd: project } },
+      event('agent_message', { message: `answer ${turn}` }), event('task_complete'));
+  }
+  const rolloutPath = join(rolloutDir, `rollout-2026-05-06T09-40-50-${threadId}.jsonl`);
+  writeFileSync(rolloutPath, rows.map((row) => JSON.stringify(row)).join('\n') + '\n');
+  const binDir = join(root, 'bin');
+  mkdirSync(binDir, { recursive: true });
+  const codex = makeBin(binDir, 'codex', "process.stderr.write('ERROR: usage limit reached\\n');\nprocess.exit(42);\n");
+
+  const result = spawnSync(process.execPath, [BIN, 'codex-hook', 'stop'], {
+    env: { ...env, CODEX_HOME: codexHome, THROUGHLINE_CODEX_CLI_BIN: codex },
+    input: JSON.stringify({ session_id: threadId, transcript_path: rolloutPath, cwd: project }),
+    encoding: 'utf8',
+  });
+
+  // hook は明示して失敗する（終了 code と stderr は変えない）。
+  assert.equal(result.status, 1, result.stderr);
+  assert.match(result.stderr, /\[codex-hook\] Codex CLI summarizer failed: exit 42/);
+
+  const store = JSON.parse(readFileSync(defaultRuntimeErrorStorePath(env), 'utf8'));
+  assert.deepEqual(store.records.map((record) => [record.error_code, record.component, record.severity, record.count]), [
+    ['L1_SUMMARIZER_BACKEND_FAILED', 'codex_l1_summarizer', 'warn', 1],
+  ]);
+
+  // 取り込みは要約の前に commit 済み。
+  const db = new DatabaseSync(join(root, '.throughline', 'throughline.db'));
+  try {
+    const { n } = db.prepare("SELECT COUNT(*) AS n FROM bodies WHERE session_id = ? AND role = 'assistant'")
+      .get(`codex:${threadId}`);
+    assert.equal(n, 22);
+  } finally {
+    db.close();
+  }
+
+  const [entry] = readFileSync(join(root, '.throughline', 'logs', 'hook-failures.log'), 'utf8')
+    .trim().split('\n').map((line) => JSON.parse(line));
+  assert.equal(entry.code, 'L1_SUMMARIZER_BACKEND_FAILED');
+  assert.equal(entry.reason, 'codex_cli_failed');
+  assert.match(entry.stderr, /usage limit reached/);
+});
+
 test('store failure preserves product failure and emits only fixed storage diagnostic', () => {
   const { env } = createEnabledEnvironment('throughline-runtime-hook-store-fail-');
   const storePath = defaultRuntimeErrorStorePath(env);

package/src/runtime-error-store.mjs

@@ -52,6 +52,13 @@ const DEFINITIONS = Object.freeze({
     severity: 'high',
     template: 'Throughline Codex hook processing failed',
   }),
+  // Codex Stop の取り込みは済み、L1 要約の backend（Codex CLI）だけが失敗した（ADR 0028）。
+  L1_SUMMARIZER_BACKEND_FAILED: Object.freeze({
+    component: 'codex_l1_summarizer',
+    // BugHub が受ける severity は fatal・high・warn・info だけ。他の値は report 全体が 422 になる。
+    severity: 'warn',
+    template: 'Throughline L1 summarizer backend (Codex CLI) failed',
+  }),
 });
 
 export function defaultRuntimeErrorConfigPath(env = process.env) {