claude-threads 0.33.8 → 0.33.9

package/CHANGELOG.md

@@ -11,6 +11,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### 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
 

package/dist/index.js

@@ -20708,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);
@@ -20756,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,
@@ -20768,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,
@@ -20782,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,
@@ -20797,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);
@@ -20812,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 {
@@ -21602,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";
@@ -21990,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) {
@@ -22401,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;
     }
@@ -22693,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.8",
+  "version": "0.33.9",
   "description": "Share Claude Code sessions live in a Mattermost channel with interactive features",
   "main": "dist/index.js",
   "type": "module",