claude-threads 0.33.1 → 0.33.3

package/CHANGELOG.md

@@ -7,6 +7,22 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.33.3] - 2026-01-04
+
+### Fixed
+- **Graceful shutdown sends two SIGINTs** - Claude CLI requires two Ctrl+C presses to exit in interactive mode. Updated kill() to send two SIGINTs (100ms apart) before falling back to SIGTERM after 2 seconds.
+
+## [0.33.2] - 2026-01-04
+
+### Fixed
+- **Session resume "No conversation found" errors** - Fixed issue where cancelled sessions would fail to resume with "No conversation found with session ID" error. Root cause: sessions were persisted before Claude had a chance to save the conversation.
+- **Graceful session termination** - When killing a session (cancel, !stop, etc.), Claude now gets 2 seconds to save the conversation (SIGINT then SIGTERM) instead of being killed immediately.
+- **Detect invalid session IDs immediately** - Sessions with "No conversation found" errors are now recognized as permanent failures and removed from persistence immediately, instead of retrying 3 times.
+- **User notification for early exits** - When a session ends before Claude responds, the user is now notified: "Session ended before Claude could respond. Please start a new session."
+
+### Changed
+- **Delayed session persistence** - Sessions are only persisted after Claude has actually responded (first `assistant` or `tool_use` event), preventing dangling session records that can't be resumed.
+
 ## [0.33.1] - 2026-01-04
 
 ### Fixed

package/dist/index.js

@@ -15282,6 +15282,10 @@ function extractAndUpdatePullRequest(text, session, ctx) {
 function handleEvent(session, event, ctx) {
   session.lastActivityAt = new Date;
   session.timeoutWarningPosted = false;
+  if (!session.hasClaudeResponded && (event.type === "assistant" || event.type === "tool_use")) {
+    session.hasClaudeResponded = true;
+    ctx.ops.persistSession(session);
+  }
   if (event.type === "assistant") {
     const msg = event.message;
     let hasSpecialTool = false;
@@ -16084,6 +16088,9 @@ class ClaudeCli extends EventEmitter2 {
     if (stderr.includes("claude-mcp-browser-bridge") && (stderr.includes("EOPNOTSUPP") || stderr.includes("ENOENT"))) {
       return true;
     }
+    if (stderr.includes("No conversation found with session ID")) {
+      return true;
+    }
     return false;
   }
   getPermanentFailureReason() {
@@ -16091,12 +16098,32 @@ class ClaudeCli extends EventEmitter2 {
     if (stderr.includes("claude-mcp-browser-bridge") && (stderr.includes("EOPNOTSUPP") || stderr.includes("ENOENT"))) {
       return "Claude browser bridge state from a previous session is no longer accessible. This typically happens when a session with Chrome integration is resumed after a restart.";
     }
+    if (stderr.includes("No conversation found with session ID")) {
+      return "The conversation history for this session no longer exists. This can happen if Claude's history was cleared or if the session was created on a different machine.";
+    }
     return null;
   }
   kill() {
     this.stopStatusWatch();
-    this.process?.kill("SIGTERM");
+    if (!this.process)
+      return;
+    const proc = this.process;
     this.process = null;
+    proc.kill("SIGINT");
+    const secondSigint = setTimeout(() => {
+      try {
+        proc.kill("SIGINT");
+      } catch {}
+    }, 100);
+    const forceKillTimeout = setTimeout(() => {
+      try {
+        proc.kill("SIGTERM");
+      } catch {}
+    }, 2000);
+    proc.once("exit", () => {
+      clearTimeout(secondSigint);
+      clearTimeout(forceKillTimeout);
+    });
   }
   interrupt() {
     if (!this.process)
@@ -20578,6 +20605,7 @@ async function startSession(options, username, displayName, replyToPostId, platf
     isResumed: false,
     resumeFailCount: 0,
     wasInterrupted: false,
+    hasClaudeResponded: false,
     inProgressTaskStart: null,
     activeToolStarts: new Map,
     firstPrompt: options.prompt,
@@ -20621,7 +20649,6 @@ async function startSession(options, username, displayName, replyToPostId, platf
   }
   session.messageCount++;
   claude.sendMessage(content);
-  ctx.ops.persistSession(session);
 }
 async function resumeSession(state, ctx) {
   const shortId = state.threadId.substring(0, 8);
@@ -20700,6 +20727,7 @@ Please start a new session.`, state.threadId), { action: "Post resume failure no
     isResumed: true,
     resumeFailCount: state.resumeFailCount || 0,
     wasInterrupted: false,
+    hasClaudeResponded: true,
     inProgressTaskStart: null,
     activeToolStarts: new Map,
     worktreeInfo: state.worktreeInfo,
@@ -20808,15 +20836,30 @@ async function handleExit(sessionId, code, ctx) {
     log10.debug(`Session ${shortId}... exited after interrupt, preserving for resume`);
     ctx.ops.stopTyping(session);
     cleanupSessionTimers(session);
-    ctx.ops.persistSession(session);
+    if (session.hasClaudeResponded) {
+      ctx.ops.persistSession(session);
+    }
     mutableSessions(ctx).delete(session.sessionId);
     cleanupPostIndex(ctx, session.threadId);
     keepAlive.sessionEnded();
-    await withErrorHandling(() => postInfo(session, `\u2139\uFE0F Session paused. Send a new message to continue.`), { action: "Post session pause notification", session });
+    const message = session.hasClaudeResponded ? `\u2139\uFE0F Session paused. Send a new message to continue.` : `\u2139\uFE0F Session ended before Claude could respond. Send a new message to start fresh.`;
+    await withErrorHandling(() => postInfo(session, message), { action: "Post session pause notification", session });
     log10.info(`Session paused (${shortId}\u2026) \u2014 ${ctx.state.sessions.size} active`);
     await ctx.ops.updateStickyMessage();
     return;
   }
+  if (!session.hasClaudeResponded && !session.isResumed) {
+    log10.debug(`Session ${shortId}... exited before Claude responded, not persisting`);
+    ctx.ops.stopTyping(session);
+    cleanupSessionTimers(session);
+    mutableSessions(ctx).delete(session.sessionId);
+    cleanupPostIndex(ctx, session.threadId);
+    keepAlive.sessionEnded();
+    await withErrorHandling(() => postWarning(session, `**Session ended** before Claude could respond (exit code ${code}). Please start a new session.`), { action: "Post early exit notification", session });
+    log10.info(`Session ended early (${shortId}\u2026) \u2014 ${ctx.state.sessions.size} active`);
+    await ctx.ops.updateStickyMessage();
+    return;
+  }
   if (session.isResumed && code !== 0) {
     const MAX_RESUME_FAILURES = 3;
     session.resumeFailCount = (session.resumeFailCount || 0) + 1;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-threads",
-  "version": "0.33.1",
+  "version": "0.33.3",
   "description": "Share Claude Code sessions live in a Mattermost channel with interactive features",
   "main": "dist/index.js",
   "type": "module",