claude-threads 0.33.7 → 0.33.9

package/CHANGELOG.md

@@ -7,6 +7,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.33.8] - 2026-01-04
+
+### Fixed
+- **Session resume broken after v0.33.7** - Fixed migration issue where sessions persisted with the old `timeoutPostId` field name couldn't be resumed after upgrading to v0.33.7. The `timeoutPostId` → `lifecyclePostId` rename now includes a proper migration that converts existing sessions on first load.
+- **Defensive defaults for persisted session fields** - Session resume now uses safe defaults for all optional fields, preventing crashes when loading sessions from older versions that may have missing fields. Fields like `sessionAllowedUsers`, `planApproved`, `forceInteractivePermissions`, etc. now gracefully default instead of potentially causing undefined errors.
+- **Validate required fields before resume** - Sessions with missing critical fields (`threadId`, `platformId`, `claudeSessionId`, `workingDir`) are now skipped gracefully with a warning instead of crashing.
+
 ## [0.33.7] - 2026-01-04
 
 ### Changed

package/dist/index.js

@@ -13792,6 +13792,20 @@ class SessionStore {
         log3.warn(`Sessions file version ${data.version} not supported, starting fresh`);
         return sessions;
       }
+      let needsSave = false;
+      for (const session of Object.values(data.sessions)) {
+        const legacySession = session;
+        if (legacySession.timeoutPostId && !session.lifecyclePostId) {
+          session.lifecyclePostId = legacySession.timeoutPostId;
+          delete legacySession.timeoutPostId;
+          needsSave = true;
+          log3.debug(`Migrated timeoutPostId to lifecyclePostId for session ${session.threadId.substring(0, 8)}...`);
+        }
+      }
+      if (needsSave) {
+        log3.info("Migrated session(s) from timeoutPostId to lifecyclePostId");
+        this.writeAtomic(data);
+      }
       for (const session of Object.values(data.sessions)) {
         if (session.cleanedAt)
           continue;
@@ -13877,7 +13891,9 @@ class SessionStore {
         historySessions.push(session);
         continue;
       }
-      if (session.lifecyclePostId && activeSessions && !activeSessions.has(sessionId)) {
+      const legacySession = session;
+      const hasLifecyclePost = session.lifecyclePostId || legacySession.timeoutPostId;
+      if (hasLifecyclePost && activeSessions && !activeSessions.has(sessionId)) {
         historySessions.push(session);
       }
     }
@@ -13923,7 +13939,12 @@ class SessionStore {
     for (const session of Object.values(data.sessions)) {
       if (session.platformId !== platformId)
         continue;
-      if (session.lifecyclePostId === postId || session.sessionStartPostId === postId) {
+      const legacySession = session;
+      const lifecycleId = session.lifecyclePostId || legacySession.timeoutPostId;
+      if (lifecycleId === postId || session.sessionStartPostId === postId) {
+        if (legacySession.timeoutPostId && !session.lifecyclePostId) {
+          session.lifecyclePostId = legacySession.timeoutPostId;
+        }
         return session;
       }
     }
@@ -20687,6 +20708,16 @@ async function startSession(options, username, displayName, replyToPostId, platf
   claude.sendMessage(content);
 }
 async function resumeSession(state, ctx) {
+  if (!state.threadId || !state.platformId || !state.claudeSessionId || !state.workingDir) {
+    const missing = [
+      !state.threadId && "threadId",
+      !state.platformId && "platformId",
+      !state.claudeSessionId && "claudeSessionId",
+      !state.workingDir && "workingDir"
+    ].filter(Boolean).join(", ");
+    log10.warn(`Skipping session with missing required fields: ${missing}`);
+    return;
+  }
   const shortId = state.threadId.substring(0, 8);
   const platforms = ctx.state.platforms;
   const platform = platforms.get(state.platformId);
@@ -20735,11 +20766,11 @@ Please start a new session.`, state.threadId), { action: "Post resume failure no
     sessionId,
     platform,
     claudeSessionId: state.claudeSessionId,
-    startedBy: state.startedBy,
+    startedBy: state.startedBy || "unknown",
     startedByDisplayName: state.startedByDisplayName,
-    startedAt: new Date(state.startedAt),
+    startedAt: state.startedAt ? new Date(state.startedAt) : new Date,
     lastActivityAt: new Date,
-    sessionNumber: state.sessionNumber,
+    sessionNumber: state.sessionNumber ?? 1,
     workingDir: state.workingDir,
     claude,
     currentPostId: null,
@@ -20747,11 +20778,11 @@ Please start a new session.`, state.threadId), { action: "Post resume failure no
     pendingApproval: null,
     pendingQuestionSet: null,
     pendingMessageApproval: null,
-    planApproved: state.planApproved,
-    sessionAllowedUsers: new Set(state.sessionAllowedUsers),
-    forceInteractivePermissions: state.forceInteractivePermissions,
-    sessionStartPostId: state.sessionStartPostId,
-    tasksPostId: state.tasksPostId,
+    planApproved: state.planApproved ?? false,
+    sessionAllowedUsers: new Set(state.sessionAllowedUsers || [state.startedBy].filter(Boolean)),
+    forceInteractivePermissions: state.forceInteractivePermissions ?? false,
+    sessionStartPostId: state.sessionStartPostId ?? null,
+    tasksPostId: state.tasksPostId ?? null,
     lastTasksContent: state.lastTasksContent ?? null,
     tasksCompleted: state.tasksCompleted ?? false,
     tasksMinimized: state.tasksMinimized ?? false,
@@ -20761,7 +20792,7 @@ Please start a new session.`, state.threadId), { action: "Post resume failure no
     timeoutWarningPosted: false,
     isRestarting: false,
     isResumed: true,
-    resumeFailCount: state.resumeFailCount || 0,
+    resumeFailCount: state.resumeFailCount ?? 0,
     wasInterrupted: false,
     hasClaudeResponded: true,
     inProgressTaskStart: null,
@@ -20776,6 +20807,7 @@ Please start a new session.`, state.threadId), { action: "Post resume failure no
     sessionDescription: state.sessionDescription,
     pullRequestUrl: state.pullRequestUrl,
     messageCount: state.messageCount ?? 0,
+    lifecyclePostId: state.lifecyclePostId,
     statusBarTimer: null
   };
   mutableSessions(ctx).set(sessionId, session);
@@ -20791,8 +20823,8 @@ Please start a new session.`, state.threadId), { action: "Post resume failure no
   try {
     claude.start();
     log10.info(`Resumed session ${shortId}... (@${state.startedBy})`);
-    if (state.lifecyclePostId) {
-      await withErrorHandling(() => session.platform.updatePost(state.lifecyclePostId, `\uD83D\uDD04 **Session resumed** by @${state.startedBy}
+    if (session.lifecyclePostId) {
+      await withErrorHandling(() => session.platform.updatePost(session.lifecyclePostId, `\uD83D\uDD04 **Session resumed** by @${session.startedBy}
 *Reconnected to Claude session. You can continue where you left off.*`), { action: "Update timeout/shutdown post for resume", session });
       session.lifecyclePostId = undefined;
     } else {
@@ -21581,8 +21613,9 @@ function formatHistoryEntry(session) {
   const topic = getHistorySessionTopic(session);
   const threadLink = `[${topic}](/_redirect/pl/${session.threadId})`;
   const displayName = session.startedByDisplayName || session.startedBy;
-  const isTimedOut = !session.cleanedAt && session.lifecyclePostId;
-  const lastActivity = new Date(session.lastActivityAt);
+  const legacySession = session;
+  const isTimedOut = !session.cleanedAt && (session.lifecyclePostId || legacySession.timeoutPostId);
+  const lastActivity = session.lastActivityAt ? new Date(session.lastActivityAt) : new Date;
   const time = formatRelativeTimeShort(lastActivity);
   const prStr = session.pullRequestUrl ? ` \xB7 ${formatPullRequestLink(session.pullRequestUrl)}` : "";
   const indicator = isTimedOut ? "\u23F8\uFE0F" : "\u2713";
@@ -21969,7 +22002,7 @@ class SessionManager {
       }
       return false;
     }
-    const allowedUsers = new Set(persistedSession.sessionAllowedUsers);
+    const allowedUsers = new Set(persistedSession.sessionAllowedUsers || []);
     const platform = this.platforms.get(platformId);
     if (!allowedUsers.has(username) && !platform?.isUserAllowed(username)) {
       if (platform) {
@@ -22380,7 +22413,7 @@ class SessionManager {
     if (!session) {
       const persisted = this.getPersistedSession(threadId);
       if (persisted) {
-        return persisted.sessionAllowedUsers.includes(username) || this.platforms.get(persisted.platformId)?.isUserAllowed(username) || false;
+        return (persisted.sessionAllowedUsers || []).includes(username) || this.platforms.get(persisted.platformId)?.isUserAllowed(username) || false;
       }
       return false;
     }
@@ -22672,7 +22705,7 @@ Release notes not available. See [GitHub releases](https://github.com/anneschuth
         const content = mattermost.isBotMentioned(message) ? mattermost.extractPrompt(message) : message.trim();
         const persistedSession = session.getPersistedSession(threadRoot);
         if (persistedSession) {
-          const allowedUsers = new Set(persistedSession.sessionAllowedUsers);
+          const allowedUsers = new Set(persistedSession.sessionAllowedUsers || []);
           if (!allowedUsers.has(username) && !mattermost.isUserAllowed(username)) {
             await mattermost.createPost(`\u26A0\uFE0F @${username} is not authorized to resume this session`, threadRoot);
             return;

package/package.json

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