claude-threads 0.51.0 → 0.52.1

package/CHANGELOG.md

@@ -7,6 +7,19 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.52.1] - 2026-01-09
+
+### Fixed
+- **Bug report image upload error** - Fixed `downloadFile` method losing `this` context when passed as callback, causing "undefined is not an object" error (#163)
+
+## [0.52.0] - 2026-01-09
+
+### Fixed
+- **Image attachments in bug reports** - Fixed image attachments not appearing in bug reports by uploading to Catbox before generating the report (#158)
+- **Sticky message install command** - Fixed npm/bun string issue in sticky message and added website link (#159)
+- **Paused sessions auto-resuming** - Fixed paused sessions incorrectly auto-resuming on bot restart by persisting paused state (#160)
+- **403 permission errors** - Fixed 403 errors when unpinning/updating stale task posts by handling channel post deletion gracefully (#161)
+
 ## [0.51.0] - 2026-01-09
 
 ### Added

package/dist/index.js

@@ -42852,9 +42852,42 @@ class MattermostClient extends EventEmitter {
       extension: mattermostFile.extension
     };
   }
+  async processAndEmitPost(post) {
+    const hasFileIds = post.file_ids && post.file_ids.length > 0;
+    const hasFileMetadata = post.metadata?.files && post.metadata.files.length > 0;
+    if (hasFileIds && !hasFileMetadata) {
+      log.debug(`Post ${post.id.substring(0, 8)} has ${post.file_ids.length} file(s), fetching metadata`);
+      try {
+        const files = [];
+        for (const fileId of post.file_ids) {
+          try {
+            const file = await this.api("GET", `/files/${fileId}/info`);
+            files.push(file);
+          } catch (err) {
+            log.warn(`Failed to fetch file info for ${fileId}: ${err}`);
+          }
+        }
+        if (files.length > 0) {
+          post.metadata = {
+            ...post.metadata,
+            files
+          };
+          log.debug(`Enriched post ${post.id.substring(0, 8)} with ${files.length} file(s)`);
+        }
+      } catch (err) {
+        log.warn(`Failed to fetch file metadata for post ${post.id.substring(0, 8)}: ${err}`);
+      }
+    }
+    const user = await this.getUser(post.user_id);
+    const normalizedPost = this.normalizePlatformPost(post);
+    this.emit("message", normalizedPost, user);
+    if (!post.root_id) {
+      this.emit("channel_post", normalizedPost, user);
+    }
+  }
   MAX_RETRIES = 3;
   RETRY_DELAY_MS = 500;
-  async api(method, path, body, retryCount = 0) {
+  async api(method, path, body, retryCount = 0, options) {
     const url = `${this.url}/api/v4${path}`;
     log.debug(`API ${method} ${path}`);
     const response = await fetch(url, {
@@ -42871,9 +42904,14 @@ class MattermostClient extends EventEmitter {
         const delay = this.RETRY_DELAY_MS * Math.pow(2, retryCount);
         log.warn(`API ${method} ${path} failed with 500, retrying in ${delay}ms (attempt ${retryCount + 1}/${this.MAX_RETRIES})`);
         await new Promise((resolve2) => setTimeout(resolve2, delay));
-        return this.api(method, path, body, retryCount + 1);
+        return this.api(method, path, body, retryCount + 1, options);
+      }
+      const isSilent = options?.silent?.includes(response.status);
+      if (isSilent) {
+        log.debug(`API ${method} ${path} failed: ${response.status} (expected)`);
+      } else {
+        log.warn(`API ${method} ${path} failed: ${response.status} ${text.substring(0, 100)}`);
       }
-      log.warn(`API ${method} ${path} failed: ${response.status} ${text.substring(0, 100)}`);
       throw new Error(`Mattermost API error ${response.status}: ${text}`);
     }
     log.debug(`API ${method} ${path} \u2192 ${response.status}`);
@@ -42992,7 +43030,14 @@ class MattermostClient extends EventEmitter {
   }
   async unpinPost(postId) {
     log.debug(`Unpinning post ${postId.substring(0, 8)}`);
-    await this.api("POST", `/posts/${postId}/unpin`);
+    try {
+      await this.api("POST", `/posts/${postId}/unpin`, undefined, 0, { silent: [403, 404] });
+    } catch (err) {
+      if (err instanceof Error && (err.message.includes("403") || err.message.includes("404"))) {
+        return;
+      }
+      throw err;
+    }
   }
   async getPinnedPosts() {
     const response = await this.api("GET", `/channels/${this.channelId}/pinned`);
@@ -43120,13 +43165,7 @@ class MattermostClient extends EventEmitter {
         if (post.channel_id !== this.channelId)
           return;
         this.lastProcessedPostId = post.id;
-        this.getUser(post.user_id).then((user) => {
-          const normalizedPost = this.normalizePlatformPost(post);
-          this.emit("message", normalizedPost, user);
-          if (!post.root_id) {
-            this.emit("channel_post", normalizedPost, user);
-          }
-        });
+        this.processAndEmitPost(post);
       } catch (err) {
         wsLogger.warn(`Failed to parse post: ${err}`);
       }
@@ -45330,6 +45369,7 @@ function resetSessionActivity(session) {
   session.lastActivityAt = new Date;
   session.timeoutWarningPosted = false;
   session.lifecyclePostId = undefined;
+  session.isPaused = undefined;
 }
 function updateLastMessage(session, post) {
   session.lastMessageId = post.id;
@@ -45545,8 +45585,18 @@ async function bumpTasksToBottomWithContent(session, newContent, registerPost) {
       sessionLog(session).debug(`Could not remove toggle emoji: ${err}`);
     }
     await session.platform.unpinPost(oldTasksPostId).catch(() => {});
-    await withErrorHandling(() => session.platform.updatePost(oldTasksPostId, contentToPost), { action: "Repurpose task post", session });
-    registerPost(oldTasksPostId, session.threadId);
+    let repurposedPostId = null;
+    try {
+      await session.platform.updatePost(oldTasksPostId, contentToPost);
+      repurposedPostId = oldTasksPostId;
+      registerPost(oldTasksPostId, session.threadId);
+    } catch (err) {
+      sessionLog(session).debug(`Could not repurpose task post (creating new): ${err}`);
+      const newPost = await session.platform.createPost(contentToPost, session.threadId);
+      repurposedPostId = newPost.id;
+      registerPost(newPost.id, session.threadId);
+      updateLastMessage(session, newPost);
+    }
     if (oldTasksContent) {
       const displayContent = getTaskDisplayContent(session);
       const newTasksPost = await session.platform.createInteractivePost(displayContent, [TASK_TOGGLE_EMOJIS[0]], session.threadId);
@@ -45558,7 +45608,7 @@ async function bumpTasksToBottomWithContent(session, newContent, registerPost) {
     } else {
       session.tasksPostId = null;
     }
-    return oldTasksPostId;
+    return repurposedPostId || oldTasksPostId;
   } finally {
     releaseLock();
   }
@@ -51548,10 +51598,10 @@ async function reportBug(session, description, username, ctx, errorContext, atta
   const context = await collectBugReportContext(session, errorContext);
   let imageUrls = [];
   let imageErrors = [];
-  const downloadFile = session.platform.downloadFile;
+  const downloadFile = session.platform.downloadFile?.bind(session.platform);
   if (attachedFiles && attachedFiles.length > 0 && downloadFile) {
     await postInfo(session, `\uD83D\uDCE4 Uploading ${attachedFiles.length} image(s)...`);
-    const uploadResults = await uploadImages(attachedFiles, (fileId) => downloadFile(fileId));
+    const uploadResults = await uploadImages(attachedFiles, downloadFile);
     imageUrls = uploadResults.filter((r) => r.success && typeof r.url === "string").map((r) => r.url);
     imageErrors = uploadResults.filter((r) => !r.success).map((r) => `${r.originalFile.name}: ${r.error}`);
   }
@@ -52377,12 +52427,16 @@ async function handleExitPlanMode(session, toolUseId, ctx) {
 }
 async function cleanupOrphanedTaskPosts(session, currentTaskPostId) {
   try {
+    const botUser = await session.platform.getBotUser();
+    const botUserId = botUser.id;
     const history = await session.platform.getThreadHistory(session.threadId, { limit: 50 });
     const taskPostPattern = /^(?:(?:---|___|\*\*\*|\u2014+)\s*\n)?\uD83D\uDCCB/;
     let cleanedCount = 0;
     for (const msg of history) {
       if (msg.id === currentTaskPostId)
         continue;
+      if (msg.userId !== botUserId)
+        continue;
       if (!taskPostPattern.test(msg.message))
         continue;
       sessionLog4(session).info(`Cleaning up orphaned task post ${msg.id.substring(0, 8)}`);
@@ -53326,6 +53380,7 @@ ${CHAT_PLATFORM_PROMPT}`;
 ${sessionFormatter.formatItalic("Reconnected to Claude session. You can continue where you left off.")}`;
       await withErrorHandling(() => session.platform.updatePost(postId, resumeMsg), { action: "Update timeout/shutdown post for resume", session });
       session.lifecyclePostId = undefined;
+      session.isPaused = undefined;
     } else {
       const restartMsg = `${sessionFormatter.formatBold("Session resumed")} after bot restart (v${VERSION})
 ${sessionFormatter.formatItalic("Reconnected to Claude session. You can continue where you left off.")}`;
@@ -53418,15 +53473,20 @@ async function handleExit(sessionId, code, ctx) {
     sessionLog6(session).debug(`Exited after interrupt, preserving for resume`);
     ctx.ops.stopTyping(session);
     cleanupSessionTimers(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.`;
+    const pausePost = await withErrorHandling(() => postInfo(session, message), { action: "Post session pause notification", session });
     if (session.hasClaudeResponded) {
+      session.isPaused = true;
+      if (pausePost) {
+        session.lifecyclePostId = pausePost.id;
+        ctx.ops.registerPost(pausePost.id, session.threadId);
+      }
       ctx.ops.persistSession(session);
     }
     ctx.ops.emitSessionRemove(session.sessionId);
     mutableSessions(ctx).delete(session.sessionId);
     cleanupPostIndex(ctx, session.threadId);
     keepAlive.sessionEnded();
-    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 });
     sessionLog6(session).info(`\u23F8 Session paused`);
     await ctx.ops.updateStickyMessage();
     return;
@@ -53565,6 +53625,7 @@ async function cleanupIdleSessions(timeoutMs, warningMs, ctx) {
           ctx.ops.registerPost(timeoutPost.id, session.threadId);
         }
       }
+      session.isPaused = true;
       ctx.ops.persistSession(session);
       await killSession(session, false, ctx);
       continue;
@@ -54051,7 +54112,7 @@ async function buildStickyMessage(sessions, platformId, config, formatter, getTh
       }
     }
     lines2.push("");
-    lines2.push(`${formatter.formatItalic("Mention me to start a session")} \xB7 ${formatter.formatCode("npm i -g claude-threads")}`);
+    lines2.push(`${formatter.formatItalic("Mention me to start a session")} \xB7 ${formatter.formatCode("bun install -g claude-threads")} \xB7 ${formatter.formatLink("claude-threads.run", "https://claude-threads.run/")}`);
     return lines2.join(`
 `);
   }
@@ -54103,7 +54164,7 @@ async function buildStickyMessage(sessions, platformId, config, formatter, getTh
     }
   }
   lines.push("");
-  lines.push(`${formatter.formatItalic("Mention me to start a session")} \xB7 ${formatter.formatCode("npm i -g claude-threads")}`);
+  lines.push(`${formatter.formatItalic("Mention me to start a session")} \xB7 ${formatter.formatCode("bun install -g claude-threads")} \xB7 ${formatter.formatLink("claude-threads.run", "https://claude-threads.run/")}`);
   return lines.join(`
 `);
 }
@@ -54651,6 +54712,7 @@ class SessionManager extends EventEmitter4 {
       pendingContextPrompt: persistedContextPrompt,
       needsContextPromptOnNextMessage: session.needsContextPromptOnNextMessage,
       lifecyclePostId: session.lifecyclePostId,
+      isPaused: session.isPaused,
       sessionTitle: session.sessionTitle,
       sessionDescription: session.sessionDescription,
       pullRequestUrl: session.pullRequestUrl,
@@ -54858,9 +54920,23 @@ class SessionManager extends EventEmitter4 {
     const persisted = this.sessionStore.load();
     log18.info(`\uD83D\uDCC2 Loaded ${persisted.size} session(s) from persistence`);
     if (persisted.size > 0) {
-      log18.info(`\uD83D\uDD04 Attempting to resume ${persisted.size} persisted session(s)...`);
+      const activeToResume = [];
+      const pausedToSkip = [];
       for (const state of persisted.values()) {
-        await resumeSession(state, this.getContext());
+        if (state.isPaused) {
+          pausedToSkip.push(state);
+        } else {
+          activeToResume.push(state);
+        }
+      }
+      if (pausedToSkip.length > 0) {
+        log18.info(`\u23F8\uFE0F ${pausedToSkip.length} session(s) remain paused (waiting for user message)`);
+      }
+      if (activeToResume.length > 0) {
+        log18.info(`\uD83D\uDD04 Attempting to resume ${activeToResume.length} active session(s)...`);
+        for (const state of activeToResume) {
+          await resumeSession(state, this.getContext());
+        }
       }
     }
     await this.updateStickyMessage();

package/package.json

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