throughline 0.4.1 → 0.4.2

package/CHANGELOG.md

@@ -10,6 +10,16 @@ shipped to npm but were not individually tagged on GitHub.
 
 ## [Unreleased]
 
+## [0.4.2] — 2026-05-09
+
+### Fixed
+
+- Codex trim no longer falls back to the latest project session when `--session`
+  is omitted. For `--host codex`, the default memory session is now the current
+  Codex thread (`codex:<thread_id>` from `--codex-thread-id`,
+  `CODEX_THREAD_ID`, or `THROUGHLINE_CODEX_THREAD_ID`), so Claude-side work
+  cannot accidentally become the injected memory for a Codex rollback.
+
 ## [0.4.1] — 2026-05-09
 
 ### Changed

package/README.md

@@ -356,6 +356,12 @@ CODEX_THREAD_ID=<id> throughline trim --preflight --host codex
 throughline trim --execute --host codex --all --codex-thread-id <id>
 ```
 
+For Codex trim, the default memory session is the current Codex thread:
+`codex:<CODEX_THREAD_ID>` / `codex:<THROUGHLINE_CODEX_THREAD_ID>`. Throughline
+does not fall back to the latest project session for Codex injection, because
+that can mix Claude-side memory into a Codex rollback. Pass `--session` only
+when deliberately injecting a different captured session.
+
 That current-work framing matters: the original `/tl` design learned that L1/L2
 memory alone can read like past logs rather than "the work in progress". The
 memo is one strong signal, but the broader mechanism is explicit structure:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "throughline",
-  "version": "0.4.1",
+  "version": "0.4.2",
   "type": "module",
   "description": "Claude Code hooks plugin for structured context compression (/clear-safe persistent memory)",
   "keywords": [

package/src/trim-cli.test.mjs

@@ -191,12 +191,24 @@ async function seedDb(home, project) {
       `INSERT INTO sessions (session_id, project_path, status, created_at, updated_at)
        VALUES ('sess-trim-cli', ?, 'active', 1, 2)`,
     ).run(project);
-    for (let turn = 1; turn <= 22; turn++) {
-      db.prepare(
-        `INSERT INTO bodies
-           (session_id, origin_session_id, turn_number, role, text, token_count, created_at)
-         VALUES ('sess-trim-cli', 'sess-trim-cli', ?, 'assistant', ?, 1, ?)`,
-      ).run(turn, `assistant body ${turn}`, turn * 1000);
+    for (const sessionId of [
+      'sess-trim-cli',
+      'codex:019dfabf-thread',
+      'codex:019dfaba-f87e-7f41-a144-d5ca7c6dd7f9',
+    ]) {
+      if (sessionId !== 'sess-trim-cli') {
+        db.prepare(
+          `INSERT INTO sessions (session_id, project_path, status, created_at, updated_at)
+           VALUES (?, ?, 'active', 1, 1)`,
+        ).run(sessionId, project);
+      }
+      for (let turn = 1; turn <= 22; turn++) {
+        db.prepare(
+          `INSERT INTO bodies
+             (session_id, origin_session_id, turn_number, role, text, token_count, created_at)
+           VALUES (?, ?, ?, 'assistant', ?, 1, ?)`,
+        ).run(sessionId, sessionId, turn, `assistant body ${turn}`, turn * 1000);
+      }
     }
     db.close();
   } finally {
@@ -284,6 +296,7 @@ test('trim CLI carries explicit Codex thread id in dry-run JSON', async () => {
       explicit: true,
       reason: 'explicit_codex_thread_id',
     });
+    assert.equal(plan.session.id, 'codex:019dfabf-thread');
     assert.equal(plan.trim.automaticExecutionAllowed, true);
   } finally {
     rmSync(project, { recursive: true, force: true });
@@ -361,7 +374,8 @@ test('trim CLI uses explicit Codex rollout source when DB has no captured turns'
 
     assert.equal(result.status, 0, result.stderr);
     const plan = JSON.parse(result.stdout);
-    assert.equal(plan.session.id, 'sess-empty-codex');
+    assert.equal(plan.session.id, 'codex:019dfaba-f87e-7f41-a144-d5ca7c6dd7f9');
+    assert.equal(plan.session.source, 'codex-rollout');
     assert.equal(plan.trim.source, 'codex-rollout');
     assert.equal(plan.trim.sourceReason, 'explicit_codex_thread_rollout');
     assert.equal(plan.trim.capturedTurns, 22);

package/src/trim-model.mjs

@@ -46,6 +46,14 @@ export function findLatestSessionIdForProject(db, projectPath) {
   }
 }
 
+function resolveDefaultSessionId({ sessionId, host, codexThreadId, db, projectPath }) {
+  if (sessionId) return sessionId;
+  if (host === 'codex') {
+    return codexThreadId ? `codex:${codexThreadId}` : null;
+  }
+  return findLatestSessionIdForProject(db, projectPath);
+}
+
 function countDistinctCapturedTurns(db, sessionId) {
   try {
     const row = db
@@ -244,7 +252,13 @@ export function buildTrimPlan(
 ) {
   const normalizedHost = TRIM_HOSTS.includes(host) ? host : 'unknown';
   const normalizedTrimSource = normalizeTrimSource(trimSource);
-  const resolvedSessionId = sessionId ?? findLatestSessionIdForProject(db, projectPath);
+  const resolvedSessionId = resolveDefaultSessionId({
+    sessionId,
+    host: normalizedHost,
+    codexThreadId,
+    db,
+    projectPath,
+  });
   if (!resolvedSessionId && !normalizedTrimSource) {
     return {
       status: 'unavailable',
@@ -538,7 +552,7 @@ function buildPlanSession({ resolvedSessionId, session, trimSource, projectPath
   }
 
   return {
-    id: trimSource?.threadId ?? resolvedSessionId ?? null,
+    id: resolvedSessionId ?? trimSource?.threadId ?? null,
     projectPath: trimSource?.projectPath ?? projectPath ?? null,
     status: 'external',
     mergedInto: null,

package/src/trim-model.test.mjs

@@ -89,6 +89,26 @@ function seedSkeleton(db, { turn = 1, summary = 'old L1 summary' } = {}) {
   ).run(turn, summary, turn * 1000 + 500);
 }
 
+function seedSessionTurns(db, { sessionId, projectPath = '/repo', updatedAt = 2, textPrefix, count = 2 }) {
+  db.prepare(
+    `INSERT INTO sessions (session_id, project_path, status, created_at, updated_at)
+     VALUES (?, ?, 'active', 1, ?)`,
+  ).run(sessionId, projectPath, updatedAt);
+
+  for (let turn = 1; turn <= count; turn++) {
+    db.prepare(
+      `INSERT INTO bodies
+         (session_id, origin_session_id, turn_number, role, text, token_count, created_at)
+       VALUES (?, ?, ?, 'user', ?, 1, ?)`,
+    ).run(sessionId, sessionId, turn, `${textPrefix} user ${turn}`, turn * 1000);
+    db.prepare(
+      `INSERT INTO bodies
+         (session_id, origin_session_id, turn_number, role, text, token_count, created_at)
+       VALUES (?, ?, ?, 'assistant', ?, 1, ?)`,
+    ).run(sessionId, sessionId, turn, `${textPrefix} assistant ${turn}`, turn * 1000 + 100);
+  }
+}
+
 test('buildTrimPlan: default dry-run keeps recent 20 and marks Claude as manual-only', () => {
   const db = makeDb();
   seedTurns(db);
@@ -220,6 +240,54 @@ test('buildTrimPlan: env Codex thread id is marked non-explicit', () => {
   });
 });
 
+test('buildTrimPlan: Codex defaults memory session to the current Codex thread, not latest project session', () => {
+  const db = makeDb();
+  const threadId = '019dfabf-thread';
+  seedSessionTurns(db, {
+    sessionId: 'claude-latest',
+    updatedAt: 999,
+    textPrefix: 'claude latest memory',
+  });
+  seedSessionTurns(db, {
+    sessionId: `codex:${threadId}`,
+    updatedAt: 100,
+    textPrefix: 'codex current memory',
+  });
+
+  const plan = buildTrimPlan(db, {
+    projectPath: '/repo',
+    host: 'codex',
+    codexThreadId: threadId,
+    trimAll: true,
+  });
+
+  assert.equal(plan.status, 'ready');
+  assert.equal(plan.session.id, `codex:${threadId}`);
+  assert.equal(plan.session.source, 'throughline-db');
+  assert.equal(plan.trim.capturedTurns, 2);
+  assert.match(plan.memoryPreview.text, /codex current memory assistant 2/);
+  assert.doesNotMatch(plan.memoryPreview.text, /claude latest memory/);
+});
+
+test('buildTrimPlan: Codex without a thread id does not fall back to latest project session', () => {
+  const db = makeDb();
+  seedSessionTurns(db, {
+    sessionId: 'claude-latest',
+    updatedAt: 999,
+    textPrefix: 'claude latest memory',
+  });
+
+  const plan = buildTrimPlan(db, {
+    projectPath: '/repo',
+    host: 'codex',
+    trimAll: true,
+  });
+
+  assert.equal(plan.status, 'unavailable');
+  assert.equal(plan.reason, 'no_session');
+  assert.equal(plan.session, null);
+});
+
 test('buildTrimPlan: current-work memo is placed in curated memory preview', () => {
   const db = makeDb();
   seedTurns(db, { count: 3 });
@@ -345,7 +413,7 @@ test('buildTrimPlan: external Codex rollout source can stand in when DB session
   });
 
   assert.equal(plan.status, 'ready');
-  assert.equal(plan.session.id, '019dfabf-thread');
+  assert.equal(plan.session.id, 'codex:019dfabf-thread');
   assert.equal(plan.session.status, 'external');
   assert.equal(plan.session.source, 'codex-rollout');
   assert.equal(plan.trim.rollbackTurns, 2);