claude-threads 1.9.3 → 1.10.0

package/CHANGELOG.md

@@ -5,6 +5,19 @@ 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.10.0] - 2026-04-29
+
+### Added
+- **Thread permalink in Claude session context.** The system-prompt context block now includes `**Thread:** <permalink>` alongside the existing platform and working-directory entries. When Claude opens a PR, files a ticket, or otherwise produces an artifact for someone to review later, it can paste the link back into the description so reviewers can trace the change to the discussion it came from. Mattermost and Slack both already exposed `getThreadLink()`; pure plumbing change. Also restores the platform/working-directory context on the worktree-restart path, which had been silently dropped pre-existing. (#352)
+
+### Fixed
+- **Sticky-by-thread Claude account binding (multi-account mode).** In multi-account configurations the `claudeAccountId` written to `sessions.json` and the `$HOME` Claude was actually spawned under could drift apart under concurrent acquisitions, leaving sessions unresumable after a bot restart with `Detected permanent failure: The conversation history for this session no longer exists`. Real-world incident: 14 sessions soft-deleted in one restart cycle, 3 of them genuine victims of this drift. `AccountPool.acquire()` now binds threads to accounts deterministically via `accounts[hash(threadId) % n]` (FNV-1a, dependency-free), so the spawn-time `$HOME` and the persisted id both derive from the same hash and cannot drift. `preferredId` still wins over the sticky binding (resume invariant: OAuth history can't move), and the sticky path falls through to round-robin when the chosen account is in cooldown. (#350)
+
+## [1.9.4] - 2026-04-29
+
+### Fixed
+- **Memory leak across session lifecycles.** `PostTracker` registered every post created during a session but its `clearSession(sessionId)` was only invoked on full bot shutdown. Across many sessions the in-memory map grew without bound and contributed to V8 mark-compact becoming ineffective and an eventual OOM abort (observed at ~14 h uptime). `MessageManager.dispose()` now clears its session's `PostTracker` bucket, and dispose is wired into all five session-removal paths in `src/session/lifecycle.ts`: normal exit, kill, idle timeout, pause, shutdown, early exit, resume-fail, plus the start-failure and resume-failure error branches that bypass the shared cleanup helpers. (#356, fixes #351)
+
 ## [1.9.3] - 2026-04-24
 
 ### Internals

package/dist/index.js

@@ -53655,6 +53655,14 @@ class SessionStore {
 // src/claude/account-pool.ts
 init_logger();
 var log5 = createLogger("account-pool");
+function hashThreadId(threadId) {
+  let h = 2166136261;
+  for (let i2 = 0;i2 < threadId.length; i2++) {
+    h ^= threadId.charCodeAt(i2);
+    h = Math.imul(h, 16777619);
+  }
+  return h >>> 0;
+}
 
 class AccountPool {
   accounts;
@@ -53686,7 +53694,7 @@ class AccountPool {
   get size() {
     return this.accounts.length;
   }
-  acquire(preferredId) {
+  acquire(preferredId, threadId) {
     if (this.isEmpty)
       return null;
     if (preferredId) {
@@ -53699,6 +53707,14 @@ class AccountPool {
     }
     const now = Date.now();
     const n = this.accounts.length;
+    if (threadId) {
+      const sticky = this.accounts[hashThreadId(threadId) % n];
+      const cooling = this.coolingUntil.get(sticky.id) ?? 0;
+      if (cooling <= now) {
+        this.incrementActive(sticky.id);
+        return sticky;
+      }
+    }
     for (let i2 = 0;i2 < n; i2++) {
       const idx = (this.roundRobinIndex + i2) % n;
       const candidate = this.accounts[idx];
@@ -55582,9 +55598,10 @@ function formatClaudeCommand(cmd) {
   }
   return line;
 }
-function buildSessionContext(platform, workingDir) {
+function buildSessionContext(platform, workingDir, threadId) {
   const platformName = platform.platformType.charAt(0).toUpperCase() + platform.platformType.slice(1);
-  return `**Platform:** ${platformName} (${platform.displayName}) | **Working Directory:** ${workingDir}`;
+  const threadUrl = platform.getThreadLink(threadId);
+  return `**Platform:** ${platformName} (${platform.displayName}) | **Working Directory:** ${workingDir} | **Thread:** ${threadUrl}`;
 }
 function generateChatPlatformPrompt() {
   const userCommands = COMMAND_REGISTRY.filter((cmd) => cmd.category !== "passthrough" && ["stop", "escape", "approve", "invite", "kick", "cd", "permissions", "update"].includes(cmd.command));
@@ -60991,6 +61008,7 @@ class MessageManager {
   }
   dispose() {
     this.cancelScheduledFlush();
+    this.postTracker.clearSession(this.sessionId);
     this.reset();
   }
 }
@@ -65911,7 +65929,7 @@ async function changeDirectory(session, newDir, username, ctx) {
   session.workingDir = absoluteDir;
   const newSessionId = randomUUID2();
   session.claudeSessionId = newSessionId;
-  const sessionContext = buildSessionContext(session.platform, absoluteDir);
+  const sessionContext = buildSessionContext(session.platform, absoluteDir, session.threadId);
   const appendSystemPrompt = `${sessionContext}
 
 ${CHAT_PLATFORM_PROMPT}`;
@@ -66614,6 +66632,7 @@ async function createAndSwitchToWorktree(session, branch, username, options2) {
         const newSessionId = randomUUID3();
         session.claudeSessionId = newSessionId;
         const needsTitlePrompt = !session.sessionTitle;
+        const sessionContext = needsTitlePrompt ? buildSessionContext(session.platform, existing.path, session.threadId) : null;
         const cliOptions = {
           workingDir: existing.path,
           threadId: session.threadId,
@@ -66626,7 +66645,9 @@ async function createAndSwitchToWorktree(session, branch, username, options2) {
           resume: false,
           chrome: options2.chromeEnabled,
           platformConfig: session.platform.getMcpConfig(),
-          appendSystemPrompt: needsTitlePrompt ? options2.appendSystemPrompt : undefined,
+          appendSystemPrompt: sessionContext ? `${sessionContext}
+
+${options2.appendSystemPrompt}` : undefined,
           logSessionId: session.sessionId,
           permissionTimeoutMs: options2.permissionTimeoutMs
         };
@@ -66707,6 +66728,7 @@ ${fmt.formatItalic("Claude Code restarted in the worktree")}`);
       const newSessionId = randomUUID3();
       session.claudeSessionId = newSessionId;
       const needsTitlePrompt = !session.sessionTitle;
+      const sessionContext = needsTitlePrompt ? buildSessionContext(session.platform, worktreePath, session.threadId) : null;
       const cliOptions = {
         workingDir: worktreePath,
         threadId: session.threadId,
@@ -66719,7 +66741,9 @@ ${fmt.formatItalic("Claude Code restarted in the worktree")}`);
         resume: false,
         chrome: options2.chromeEnabled,
         platformConfig: session.platform.getMcpConfig(),
-        appendSystemPrompt: needsTitlePrompt ? options2.appendSystemPrompt : undefined,
+        appendSystemPrompt: sessionContext ? `${sessionContext}
+
+${options2.appendSystemPrompt}` : undefined,
         logSessionId: session.sessionId,
         permissionTimeoutMs: options2.permissionTimeoutMs
       };
@@ -67485,6 +67509,7 @@ async function cleanupSession(session, ctx, options2 = {}) {
   if (doCloseLogger) {
     await closeThreadLogger(session, action, details);
   }
+  session.messageManager?.dispose();
   ctx.ops.emitSessionRemove(session.sessionId);
   mutableSessions(ctx).delete(session.sessionId);
   if (doCleanupPostIndex) {
@@ -67500,6 +67525,7 @@ function releaseAccountIfHeld(session, ctx) {
   }
 }
 function removeFromRegistry(session, ctx) {
+  session.messageManager?.dispose();
   ctx.ops.emitSessionRemove(session.sessionId);
   mutableSessions(ctx).delete(session.sessionId);
   cleanupPostIndex(ctx, session.threadId);
@@ -67835,12 +67861,12 @@ async function startSession(options2, username, displayName, replyToPostId, plat
     permissionMode = "default";
     log26.info(`Starting session with interactive permissions (from !permissions command)`);
   }
-  const sessionContext = buildSessionContext(platform, workingDir);
+  const sessionContext = buildSessionContext(platform, workingDir, actualThreadId);
   const systemPrompt = `${sessionContext}
 
 ${CHAT_PLATFORM_PROMPT}`;
   const platformMcpConfig = platform.getMcpConfig();
-  const claudeAccount = ctx.ops.acquireClaudeAccount();
+  const claudeAccount = ctx.ops.acquireClaudeAccount(undefined, actualThreadId);
   if (claudeAccount) {
     log26.info(`Session ${sessionId.substring(0, 20)} reserved Claude account "${claudeAccount.id}"`);
   }
@@ -67911,6 +67937,7 @@ ${CHAT_PLATFORM_PROMPT}`;
   } catch (err) {
     await logAndNotify(err, { action: "Start Claude", session });
     ctx.ops.stopTyping(session);
+    session.messageManager?.dispose();
     ctx.ops.emitSessionRemove(session.sessionId);
     mutableSessions(ctx).delete(session.sessionId);
     releaseAccountIfHeld(session, ctx);
@@ -67986,11 +68013,11 @@ Please start a new session.`), { action: "Post resume failure notification" });
   const sessionId = ctx.ops.getSessionId(platformId, state.threadId);
   const resumePermissionMode = state.forceInteractivePermissions ? "default" : ctx.config.permissionMode;
   const platformMcpConfig = platform.getMcpConfig();
-  const sessionContext = buildSessionContext(platform, state.workingDir);
+  const sessionContext = buildSessionContext(platform, state.workingDir, state.threadId);
   const appendSystemPrompt = `${sessionContext}
 
 ${CHAT_PLATFORM_PROMPT}`;
-  const claudeAccount = ctx.ops.acquireClaudeAccount(state.claudeAccountId);
+  const claudeAccount = ctx.ops.acquireClaudeAccount(state.claudeAccountId, state.threadId);
   if (state.claudeAccountId && !claudeAccount) {
     log26.warn(`Persisted session referenced Claude account "${state.claudeAccountId}" ` + `which is no longer configured — resuming under default env`);
   }
@@ -68114,6 +68141,7 @@ ${sessionFormatter.formatItalic("Reconnected to Claude session. You can continue
     ctx.ops.persistSession(session);
   } catch (err) {
     log26.error(`Failed to resume session ${shortId}`, err instanceof Error ? err : undefined);
+    session.messageManager?.dispose();
     ctx.ops.emitSessionRemove(sessionId);
     mutableSessions(ctx).delete(sessionId);
     ctx.state.sessionStore.remove(sessionId);
@@ -68888,7 +68916,7 @@ class SessionManager extends EventEmitter4 {
       emitSessionAdd: (s) => this.emitSessionAdd(s),
       emitSessionUpdate: (sid, u) => this.emitSessionUpdate(sid, u),
       emitSessionRemove: (sid) => this.emitSessionRemove(sid),
-      acquireClaudeAccount: (preferredId) => this.accountPool.acquire(preferredId),
+      acquireClaudeAccount: (preferredId, threadId) => this.accountPool.acquire(preferredId, threadId),
       getClaudeAccount: (id) => this.accountPool.get(id),
       releaseClaudeAccount: (id) => this.accountPool.release(id),
       markClaudeAccountCooling: (id, untilMs) => this.accountPool.markCooling(id, untilMs),

package/dist/mcp/permission-server.js

@@ -52899,6 +52899,7 @@ class MessageManager {
   }
   dispose() {
     this.cancelScheduledFlush();
+    this.postTracker.clearSession(this.sessionId);
     this.reset();
   }
 }

package/package.json

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