claude-threads 1.9.0 → 1.9.1

package/CHANGELOG.md

@@ -5,6 +5,15 @@ 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.9.1] - 2026-04-24
+
+### Internals
+- **Unified `Executor<TState>` contract.** New interface in `src/operations/executors/types.ts` formalizes what `MessageManager` actually relies on: `getState` / `reset` required, `handleReaction` / `serialize` optional. `BaseExecutor<T>` implements it. A new `contract.test.ts` iterates every executor and asserts the shape — catches drift when someone adds an executor without the required members. (#346)
+- **Uniform `handleReaction` signature across all seven reaction executors** — `(postId, emoji, user, action, ctx) => Promise<boolean>`. Previously `TaskList`, `Subagent`, and `WorktreePrompt` had slightly different shapes. `MessageManager.handleReaction` now dispatches via a `reactionDispatchList()` table instead of an if/else chain. (#346)
+- **`MessageManager.serialize()` aggregates executor state** for `SessionManager.persistSession`. The writer no longer reaches into individual executors via named getters (`getTaskListState()`, `getPendingContextPrompt()`). Legacy getters kept as `@deprecated` shims — they still have non-persistence consumers. (#346)
+- **Byte-identical `sessions.json` guarantee.** New snapshot tests in `manager.test.ts` pin the full payload's field set and run the new and legacy (`CLAUDE_THREADS_SERIALIZE_V2=0`) paths through a parity assertion. No persisted-schema change on disk. (#346)
+- **Rollback hatch:** `CLAUDE_THREADS_SERIALIZE_V2=0` falls back to the pre-refactor per-getter writer for one release. Removed in the next minor. (#346)
+
 ## [1.9.0] - 2026-04-24
 
 ### Added

package/dist/index.js

@@ -58611,7 +58611,15 @@ class TaskListExecutor extends BaseExecutor {
       await ctx.platform.updatePost(this.state.tasksPostId, displayContent);
     } catch {}
   }
-  async handleReaction(postId, emoji, action, ctx) {
+  serialize() {
+    return {
+      postId: this.state.tasksPostId,
+      content: this.state.lastTasksContent,
+      isMinimized: this.state.tasksMinimized,
+      isCompleted: this.state.tasksCompleted
+    };
+  }
+  async handleReaction(postId, emoji, _user, action, ctx) {
     ctx.logger.debug(`TaskListExecutor.handleReaction: postId=${postId.substring(0, 8)}, emoji=${emoji}, action=${action}, tasksPostId=${this.state.tasksPostId?.substring(0, 8) ?? "none"}`);
     if (postId !== this.state.tasksPostId) {
       ctx.logger.debug(`TaskListExecutor: postId does not match tasksPostId, ignoring`);
@@ -58880,7 +58888,7 @@ class SubagentExecutor extends BaseExecutor {
     }
     return false;
   }
-  async handleReaction(postId, emoji, action, ctx) {
+  async handleReaction(postId, emoji, _user, action, ctx) {
     ctx.logger.debug(`SubagentExecutor.handleReaction: postId=${postId.substring(0, 8)}, emoji=${emoji}, action=${action}`);
     if (!isMinimizeToggleEmoji(emoji)) {
       ctx.logger.debug(`SubagentExecutor: emoji ${emoji} is not minimize toggle, ignoring`);
@@ -59420,6 +59428,9 @@ class PromptExecutor extends BaseExecutor {
   getPendingContextPrompt() {
     return this.state.pendingContextPrompt;
   }
+  serialize() {
+    return this.state.pendingContextPrompt;
+  }
   hasPendingContextPrompt() {
     return this.state.pendingContextPrompt !== null;
   }
@@ -60782,12 +60793,12 @@ class MessageManager {
     await this.taskListExecutor.bumpToBottom(this.getExecutorContext());
   }
   getTaskListState() {
-    const state = this.taskListExecutor.getState();
+    return this.taskListExecutor.serialize();
+  }
+  serialize() {
     return {
-      postId: state.tasksPostId,
-      content: state.lastTasksContent,
-      isMinimized: state.tasksMinimized,
-      isCompleted: state.tasksCompleted
+      taskList: this.taskListExecutor.serialize(),
+      contextPrompt: this.promptExecutor.serialize()
     };
   }
   hydrateTaskListState(persisted) {
@@ -60880,33 +60891,27 @@ class MessageManager {
   getSession() {
     return this.session;
   }
+  reactionDispatchList() {
+    return [
+      { name: "QuestionApprovalExecutor", executor: this.questionApprovalExecutor },
+      { name: "MessageApprovalExecutor", executor: this.messageApprovalExecutor },
+      { name: "PromptExecutor", executor: this.promptExecutor },
+      { name: "BugReportExecutor", executor: this.bugReportExecutor },
+      { name: "TaskListExecutor", executor: this.taskListExecutor },
+      { name: "SubagentExecutor", executor: this.subagentExecutor }
+    ];
+  }
   async handleReaction(postId, emoji, user, action) {
     const logger = log18.forSession(this.sessionId);
     const ctx = this.getExecutorContext();
     logger.debug(`Routing reaction: postId=${postId}, emoji=${emoji}, user=${user}, action=${action}`);
-    if (await this.questionApprovalExecutor.handleReaction(postId, emoji, user, action, ctx)) {
-      logger.debug("Reaction handled by QuestionApprovalExecutor");
-      return true;
-    }
-    if (await this.messageApprovalExecutor.handleReaction(postId, emoji, user, action, ctx)) {
-      logger.debug("Reaction handled by MessageApprovalExecutor");
-      return true;
-    }
-    if (await this.promptExecutor.handleReaction(postId, emoji, user, action, ctx)) {
-      logger.debug("Reaction handled by PromptExecutor");
-      return true;
-    }
-    if (await this.bugReportExecutor.handleReaction(postId, emoji, user, action, ctx)) {
-      logger.debug("Reaction handled by BugReportExecutor");
-      return true;
-    }
-    if (await this.taskListExecutor.handleReaction(postId, emoji, action, ctx)) {
-      logger.debug("Reaction handled by TaskListExecutor");
-      return true;
-    }
-    if (await this.subagentExecutor.handleReaction(postId, emoji, action, ctx)) {
-      logger.debug("Reaction handled by SubagentExecutor");
-      return true;
+    for (const { name, executor } of this.reactionDispatchList()) {
+      if (!executor.handleReaction)
+        continue;
+      if (await executor.handleReaction(postId, emoji, user, action, ctx)) {
+        logger.debug(`Reaction handled by ${name}`);
+        return true;
+      }
     }
     logger.debug("Reaction not handled by any executor");
     return false;
@@ -68910,19 +68915,29 @@ class SessionManager extends EventEmitter4 {
     this.stopTyping(session);
   }
   persistSession(session) {
-    let persistedContextPrompt;
-    const contextPromptState = session.messageManager?.getPendingContextPrompt();
-    if (contextPromptState) {
-      persistedContextPrompt = {
-        postId: contextPromptState.postId,
-        queuedPrompt: contextPromptState.queuedPrompt,
-        queuedFiles: contextPromptState.queuedFiles,
-        threadMessageCount: contextPromptState.threadMessageCount,
-        createdAt: contextPromptState.createdAt,
-        availableOptions: contextPromptState.availableOptions
-      };
+    const useSerializeV2 = process.env.CLAUDE_THREADS_SERIALIZE_V2 !== "0";
+    let taskListSnapshot;
+    let contextPromptSnapshot;
+    if (useSerializeV2 && session.messageManager) {
+      const serialized = session.messageManager.serialize();
+      taskListSnapshot = serialized.taskList;
+      if (serialized.contextPrompt) {
+        contextPromptSnapshot = serialized.contextPrompt;
+      }
+    } else {
+      const legacyPrompt = session.messageManager?.getPendingContextPrompt();
+      if (legacyPrompt) {
+        contextPromptSnapshot = {
+          postId: legacyPrompt.postId,
+          queuedPrompt: legacyPrompt.queuedPrompt,
+          queuedFiles: legacyPrompt.queuedFiles,
+          threadMessageCount: legacyPrompt.threadMessageCount,
+          createdAt: legacyPrompt.createdAt,
+          availableOptions: legacyPrompt.availableOptions
+        };
+      }
+      taskListSnapshot = session.messageManager?.getTaskListState();
     }
-    const taskState = session.messageManager?.getTaskListState();
     const state = {
       platformId: session.platformId,
       threadId: session.threadId,
@@ -68937,10 +68952,10 @@ class SessionManager extends EventEmitter4 {
       sessionAllowedUsers: [...session.sessionAllowedUsers],
       forceInteractivePermissions: session.forceInteractivePermissions,
       sessionStartPostId: session.sessionStartPostId,
-      tasksPostId: taskState?.postId ?? null,
-      lastTasksContent: taskState?.content ?? null,
-      tasksCompleted: taskState?.isCompleted ?? false,
-      tasksMinimized: taskState?.isMinimized ?? false,
+      tasksPostId: taskListSnapshot?.postId ?? null,
+      lastTasksContent: taskListSnapshot?.content ?? null,
+      tasksCompleted: taskListSnapshot?.isCompleted ?? false,
+      tasksMinimized: taskListSnapshot?.isMinimized ?? false,
       worktreeInfo: session.worktreeInfo,
       isWorktreeOwner: session.isWorktreeOwner,
       pendingWorktreePrompt: session.pendingWorktreePrompt,
@@ -68948,7 +68963,7 @@ class SessionManager extends EventEmitter4 {
       queuedPrompt: session.queuedPrompt,
       queuedFiles: session.queuedFiles,
       firstPrompt: session.firstPrompt,
-      pendingContextPrompt: persistedContextPrompt,
+      pendingContextPrompt: contextPromptSnapshot,
       needsContextPromptOnNextMessage: session.needsContextPromptOnNextMessage,
       lifecyclePostId: session.lifecyclePostId,
       isPaused: session.lifecycle.state === "paused" || session.lifecycle.state === "interrupted",

package/dist/mcp/permission-server.js

@@ -51175,7 +51175,15 @@ class TaskListExecutor extends BaseExecutor {
       await ctx.platform.updatePost(this.state.tasksPostId, displayContent);
     } catch {}
   }
-  async handleReaction(postId, emoji4, action, ctx) {
+  serialize() {
+    return {
+      postId: this.state.tasksPostId,
+      content: this.state.lastTasksContent,
+      isMinimized: this.state.tasksMinimized,
+      isCompleted: this.state.tasksCompleted
+    };
+  }
+  async handleReaction(postId, emoji4, _user, action, ctx) {
     ctx.logger.debug(`TaskListExecutor.handleReaction: postId=${postId.substring(0, 8)}, emoji=${emoji4}, action=${action}, tasksPostId=${this.state.tasksPostId?.substring(0, 8) ?? "none"}`);
     if (postId !== this.state.tasksPostId) {
       ctx.logger.debug(`TaskListExecutor: postId does not match tasksPostId, ignoring`);
@@ -51444,7 +51452,7 @@ class SubagentExecutor extends BaseExecutor {
     }
     return false;
   }
-  async handleReaction(postId, emoji4, action, ctx) {
+  async handleReaction(postId, emoji4, _user, action, ctx) {
     ctx.logger.debug(`SubagentExecutor.handleReaction: postId=${postId.substring(0, 8)}, emoji=${emoji4}, action=${action}`);
     if (!isMinimizeToggleEmoji(emoji4)) {
       ctx.logger.debug(`SubagentExecutor: emoji ${emoji4} is not minimize toggle, ignoring`);
@@ -51984,6 +51992,9 @@ class PromptExecutor extends BaseExecutor {
   getPendingContextPrompt() {
     return this.state.pendingContextPrompt;
   }
+  serialize() {
+    return this.state.pendingContextPrompt;
+  }
   hasPendingContextPrompt() {
     return this.state.pendingContextPrompt !== null;
   }
@@ -52734,12 +52745,12 @@ class MessageManager {
     await this.taskListExecutor.bumpToBottom(this.getExecutorContext());
   }
   getTaskListState() {
-    const state = this.taskListExecutor.getState();
+    return this.taskListExecutor.serialize();
+  }
+  serialize() {
     return {
-      postId: state.tasksPostId,
-      content: state.lastTasksContent,
-      isMinimized: state.tasksMinimized,
-      isCompleted: state.tasksCompleted
+      taskList: this.taskListExecutor.serialize(),
+      contextPrompt: this.promptExecutor.serialize()
     };
   }
   hydrateTaskListState(persisted) {
@@ -52832,33 +52843,27 @@ class MessageManager {
   getSession() {
     return this.session;
   }
+  reactionDispatchList() {
+    return [
+      { name: "QuestionApprovalExecutor", executor: this.questionApprovalExecutor },
+      { name: "MessageApprovalExecutor", executor: this.messageApprovalExecutor },
+      { name: "PromptExecutor", executor: this.promptExecutor },
+      { name: "BugReportExecutor", executor: this.bugReportExecutor },
+      { name: "TaskListExecutor", executor: this.taskListExecutor },
+      { name: "SubagentExecutor", executor: this.subagentExecutor }
+    ];
+  }
   async handleReaction(postId, emoji4, user, action) {
     const logger = log3.forSession(this.sessionId);
     const ctx = this.getExecutorContext();
     logger.debug(`Routing reaction: postId=${postId}, emoji=${emoji4}, user=${user}, action=${action}`);
-    if (await this.questionApprovalExecutor.handleReaction(postId, emoji4, user, action, ctx)) {
-      logger.debug("Reaction handled by QuestionApprovalExecutor");
-      return true;
-    }
-    if (await this.messageApprovalExecutor.handleReaction(postId, emoji4, user, action, ctx)) {
-      logger.debug("Reaction handled by MessageApprovalExecutor");
-      return true;
-    }
-    if (await this.promptExecutor.handleReaction(postId, emoji4, user, action, ctx)) {
-      logger.debug("Reaction handled by PromptExecutor");
-      return true;
-    }
-    if (await this.bugReportExecutor.handleReaction(postId, emoji4, user, action, ctx)) {
-      logger.debug("Reaction handled by BugReportExecutor");
-      return true;
-    }
-    if (await this.taskListExecutor.handleReaction(postId, emoji4, action, ctx)) {
-      logger.debug("Reaction handled by TaskListExecutor");
-      return true;
-    }
-    if (await this.subagentExecutor.handleReaction(postId, emoji4, action, ctx)) {
-      logger.debug("Reaction handled by SubagentExecutor");
-      return true;
+    for (const { name, executor } of this.reactionDispatchList()) {
+      if (!executor.handleReaction)
+        continue;
+      if (await executor.handleReaction(postId, emoji4, user, action, ctx)) {
+        logger.debug(`Reaction handled by ${name}`);
+        return true;
+      }
     }
     logger.debug("Reaction not handled by any executor");
     return false;

package/package.json

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