claude-threads 1.22.0 → 1.23.0

package/CHANGELOG.md

@@ -5,6 +5,24 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.23.0] - 2026-08-08
+
+### Fixed
+- **A failed compaction no longer leaves a stale "🗜️ Compacting context..." post forever.** Captured against the real CLI (2.1.226, `real-cli-captures/compact-failed.jsonl`): a failed compact emits **no** `compact_boundary` — only a `status` event with `compact_result: "failed"` and a `compact_error` — so the in-progress post was never resolved. It now updates to "⚠️ Compaction failed (reason)". Long-lived bot threads auto-compact in production, so this state was reachable by simply keeping a session busy.
+
+### Added
+- **Compaction completion shows real token counts.** The completion post now renders `pre → post` ("Context compacted (manual, 31k → 3k tokens)") using the `post_tokens` field verified in a new reference capture (`real-cli-captures/compact.jsonl`, recorded via a manual `/compact` driven through stream-json).
+- **`auth_status` events are handled.** An auth error from the CLI mid-session (expired OAuth, revoked key) now posts a warning to the thread instead of vanishing; progress-only auth updates are logged. Shape taken from the Agent SDK's published types (`SDKAuthStatusMessage`, `@anthropic-ai/claude-agent-sdk` 0.3.226) — deliberately not capture-backed, since provoking a real auth failure requires a broken environment.
+- **The mock's `persistent-session` scenario now carries the telemetry noise real streams have** (`thinking_tokens`, `post_turn_summary`, `active_goal`), pinning deliberately that the bot tolerates unconsumed event types — previously that tolerance was only proven incidentally. New `compaction`/`compaction-failed` scenarios and an integration suite cover the compaction lifecycle end to end on both platforms.
+
+## [1.22.1] - 2026-08-08
+
+### Fixed
+- **Permission prompts work in source/dev mode (`bun run dev`).** The per-session MCP config pointed at `mcp/mcp-server.js`, which only exists in a dist build — running the bot from source gave every session an MCP server path that could never spawn, silently breaking interactive permissions, `send_file`, and the decision bridge in dev mode. The path resolution now falls back to the TypeScript source and runs it under the current runtime (bun) instead of node. Built installs are unaffected.
+
+### Changed
+- **The integration-test mock Claude CLI now speaks the modern CLI dialect, verified against verbatim real-CLI captures.** The old mock emitted an event protocol modern CLIs no longer use (top-level `tool_result` events, `TodoWrite` task lists, plan/question tools that never touched the permission system, a process that died after every turn) — so ~120 integration tests were exercising a dialect the real CLI abandoned, and the last three releases' bugs lived exactly in that blind spot. The mock was rewritten against 15 captured reference flows from the real CLI (2.1.225), committed under `tests/integration/fixtures/real-cli-captures/` with a re-capture harness (`tests/e2e-real-cli/capture-events.ts`): `system/init` and `rate_limit_event` at startup, tool results as `tool_result` blocks in `user` events, TaskCreate/TaskUpdate task tracking with result-text id resolution, one `result` per turn with the process staying alive, and SIGINT aborting in-flight tools before exiting — all shapes taken from the captures. **Interactive scenarios now spawn the real MCP permission server and block ExitPlanMode/AskUserQuestion on `permission_prompt`, so the integration suite exercises the production permission-prompt → decision-bridge → reaction-UI path end to end** — including new regression tests pinning that no duplicate generic permission prompt appears next to the plan/question UI (the 1.21.2 bug class). Captures also documented that bypass mode exposes neither ExitPlanMode nor AskUserQuestion on modern CLIs, so the plan/question suites now run with interactive permissions like production. All 21 integration suites pass against the new mock.
+
 ## [1.22.0] - 2026-08-08
 
 ### Changed

package/dist/index.js

@@ -55998,6 +55998,9 @@ function buildClaudeChildEnv(parentEnv, account, opts) {
   }
   return env;
 }
+function runtimeForScriptPath(scriptPath) {
+  return scriptPath.endsWith(".ts") ? process.execPath : "node";
+}
 function isErrorResultEvent(event) {
   const ev = event;
   if (typeof ev.subtype === "string" && ev.subtype.startsWith("error"))
@@ -56060,7 +56063,7 @@ function buildPermissionArgs(opts) {
     mcpServers: {
       "claude-threads-mcp": {
         type: "stdio",
-        command: "node",
+        command: runtimeForScriptPath(opts.mcpServerPath),
         args: [opts.mcpServerPath],
         env: mcpEnv
       }
@@ -56195,10 +56198,11 @@ class ClaudeCli extends EventEmitter2 {
     if (this.options.sessionId) {
       this.statusFilePath = join5(tmpdir(), `claude-threads-status-${this.options.sessionId}.json`);
       const statusLineWriterPath = this.getStatusLineWriterPath();
+      const runtime = runtimeForScriptPath(statusLineWriterPath);
       const statusLineSettings = {
         statusLine: {
           type: "command",
-          command: `node ${statusLineWriterPath} ${this.options.sessionId}`,
+          command: `${runtime} ${statusLineWriterPath} ${this.options.sessionId}`,
           padding: 0
         }
       };
@@ -56427,7 +56431,15 @@ class ClaudeCli extends EventEmitter2 {
     if (existsSync7(bundledPath)) {
       return bundledPath;
     }
-    return resolve3(__dirname4, "..", "mcp", "mcp-server.js");
+    const sourceLayoutPath = resolve3(__dirname4, "..", "mcp", "mcp-server.js");
+    if (existsSync7(sourceLayoutPath)) {
+      return sourceLayoutPath;
+    }
+    const tsPath = resolve3(__dirname4, "..", "mcp", "mcp-server.ts");
+    if (existsSync7(tsPath)) {
+      return tsPath;
+    }
+    return sourceLayoutPath;
   }
   getStatusLineWriterPath() {
     const __filename2 = fileURLToPath3(import.meta.url);
@@ -56436,7 +56448,15 @@ class ClaudeCli extends EventEmitter2 {
     if (existsSync7(bundledPath)) {
       return bundledPath;
     }
-    return resolve3(__dirname4, "..", "statusline", "writer.js");
+    const sourceLayoutPath = resolve3(__dirname4, "..", "statusline", "writer.js");
+    if (existsSync7(sourceLayoutPath)) {
+      return sourceLayoutPath;
+    }
+    const tsPath = resolve3(__dirname4, "..", "statusline", "writer.ts");
+    if (existsSync7(tsPath)) {
+      return tsPath;
+    }
+    return sourceLayoutPath;
   }
 }
 
@@ -68828,10 +68848,25 @@ function handleEventPreProcessing(session, event, ctx) {
     if (e.subtype === "status" && e.status === "compacting") {
       handleCompactionStart(session, ctx);
     }
+    if (e.subtype === "status" && e.compact_result === "failed") {
+      handleCompactionFailed(session, e.compact_error, ctx);
+    }
     if (e.subtype === "compact_boundary") {
       handleCompactionComplete(session, e.compact_metadata, ctx);
     }
   }
+  if (event.type === "auth_status") {
+    const e = event;
+    if (e.error) {
+      sessionLog3(session).warn(`\uD83D\uDD10 Claude CLI auth error: ${e.error}`);
+      if (session.lastAuthErrorPosted !== e.error) {
+        session.lastAuthErrorPosted = e.error;
+        withErrorHandling(() => post(session, "warning", `\uD83D\uDD10 Claude CLI authentication problem: ${e.error}`), { action: "Post auth status warning", session });
+      }
+    } else {
+      sessionLog3(session).info(`\uD83D\uDD10 Claude CLI auth status: authenticating=${e.isAuthenticating ?? false}${e.output?.length ? ` (${e.output[e.output.length - 1]})` : ""}`);
+    }
+  }
   if (event.type === "assistant" && !isSidechainEvent(event)) {
     const msg = event.message;
     if (Array.isArray(msg?.content)) {
@@ -68888,21 +68923,46 @@ function handleEventPostProcessing(session, event, ctx) {
     }
   }
 }
-async function handleCompactionStart(session, _ctx) {
-  await session.messageManager?.closeCurrentPost();
+function handleCompactionStart(session, _ctx) {
+  session.compactionPostPromise = (async () => {
+    await session.messageManager?.closeCurrentPost();
+    const formatter = session.platform.getFormatter();
+    const message = `\uD83D\uDDDC️ ${formatter.formatBold("Compacting context...")} ${formatter.formatItalic("(freeing up memory)")}`;
+    const compactionPost = await withErrorHandling(() => post(session, "info", message), { action: "Post compaction start", session });
+    if (compactionPost) {
+      session.compactionPostId = compactionPost.id;
+    }
+  })();
+}
+async function awaitCompactionStartPost(session) {
+  if (session.compactionPostPromise) {
+    await session.compactionPostPromise.catch(() => {});
+    session.compactionPostPromise = undefined;
+  }
+}
+async function handleCompactionFailed(session, compactError, _ctx) {
+  await awaitCompactionStartPost(session);
   const formatter = session.platform.getFormatter();
-  const message = `\uD83D\uDDDC️ ${formatter.formatBold("Compacting context...")} ${formatter.formatItalic("(freeing up memory)")}`;
-  const compactionPost = await withErrorHandling(() => post(session, "info", message), { action: "Post compaction start", session });
-  if (compactionPost) {
-    session.compactionPostId = compactionPost.id;
+  const reason = compactError || "unknown error";
+  const message = `⚠️ ${formatter.formatBold("Compaction failed")} ${formatter.formatItalic(`(${reason})`)}`;
+  const startPostId = session.compactionPostId;
+  if (startPostId) {
+    await withErrorHandling(() => updatePost(session, startPostId, message), { action: "Update compaction post (failed)", session });
+    session.compactionPostId = undefined;
+  } else {
+    await withErrorHandling(() => post(session, "info", message), { action: "Post compaction failure", session });
   }
 }
 async function handleCompactionComplete(session, compactMetadata, _ctx) {
+  await awaitCompactionStartPost(session);
   const metadata = compactMetadata;
   const trigger = metadata?.trigger || "auto";
   const preTokens = metadata?.pre_tokens;
+  const postTokens = metadata?.post_tokens;
   let info = trigger === "manual" ? "manual" : "auto";
-  if (preTokens && preTokens > 0) {
+  if (preTokens && preTokens > 0 && postTokens && postTokens > 0) {
+    info += `, ${Math.round(preTokens / 1000)}k → ${Math.max(1, Math.round(postTokens / 1000))}k tokens`;
+  } else if (preTokens && preTokens > 0) {
     info += `, ${Math.round(preTokens / 1000)}k tokens`;
   }
   const formatter = session.platform.getFormatter();

package/dist/mcp/mcp-server.js

@@ -56106,6 +56106,9 @@ function buildClaudeChildEnv(parentEnv, account, opts) {
   }
   return env;
 }
+function runtimeForScriptPath(scriptPath) {
+  return scriptPath.endsWith(".ts") ? process.execPath : "node";
+}
 function isErrorResultEvent(event) {
   const ev = event;
   if (typeof ev.subtype === "string" && ev.subtype.startsWith("error"))
@@ -56168,7 +56171,7 @@ function buildPermissionArgs(opts) {
     mcpServers: {
       "claude-threads-mcp": {
         type: "stdio",
-        command: "node",
+        command: runtimeForScriptPath(opts.mcpServerPath),
         args: [opts.mcpServerPath],
         env: mcpEnv
       }
@@ -56303,10 +56306,11 @@ class ClaudeCli extends EventEmitter2 {
     if (this.options.sessionId) {
       this.statusFilePath = join4(tmpdir2(), `claude-threads-status-${this.options.sessionId}.json`);
       const statusLineWriterPath = this.getStatusLineWriterPath();
+      const runtime = runtimeForScriptPath(statusLineWriterPath);
       const statusLineSettings = {
         statusLine: {
           type: "command",
-          command: `node ${statusLineWriterPath} ${this.options.sessionId}`,
+          command: `${runtime} ${statusLineWriterPath} ${this.options.sessionId}`,
           padding: 0
         }
       };
@@ -56535,7 +56539,15 @@ class ClaudeCli extends EventEmitter2 {
     if (existsSync4(bundledPath)) {
       return bundledPath;
     }
-    return resolve4(__dirname4, "..", "mcp", "mcp-server.js");
+    const sourceLayoutPath = resolve4(__dirname4, "..", "mcp", "mcp-server.js");
+    if (existsSync4(sourceLayoutPath)) {
+      return sourceLayoutPath;
+    }
+    const tsPath = resolve4(__dirname4, "..", "mcp", "mcp-server.ts");
+    if (existsSync4(tsPath)) {
+      return tsPath;
+    }
+    return sourceLayoutPath;
   }
   getStatusLineWriterPath() {
     const __filename2 = fileURLToPath3(import.meta.url);
@@ -56544,7 +56556,15 @@ class ClaudeCli extends EventEmitter2 {
     if (existsSync4(bundledPath)) {
       return bundledPath;
     }
-    return resolve4(__dirname4, "..", "statusline", "writer.js");
+    const sourceLayoutPath = resolve4(__dirname4, "..", "statusline", "writer.js");
+    if (existsSync4(sourceLayoutPath)) {
+      return sourceLayoutPath;
+    }
+    const tsPath = resolve4(__dirname4, "..", "statusline", "writer.ts");
+    if (existsSync4(tsPath)) {
+      return tsPath;
+    }
+    return sourceLayoutPath;
   }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-threads",
-  "version": "1.22.0",
+  "version": "1.23.0",
   "description": "Run Claude Code from Slack or Mattermost. Sessions stream live into threads where your whole team can watch and steer.",
   "main": "dist/index.js",
   "type": "module",