claude-threads 1.39.2 → 1.39.3

package/CHANGELOG.md

@@ -5,6 +5,12 @@ 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.39.3] - 2026-10-01
+
+### Fixed
+- **Mattermost: a skin-toned 👍 no longer denies** (#629). Mattermost names an emoji picked with a skin tone `+1_medium_skin_tone` (verified against Mattermost v11), and the permission prompt treats any emoji it does not recognize as a denial. 1.39.2 fixed Slack's `+1::skin-tone-3` form; the Mattermost form is now matched on its base name too, for permission prompts, plan approvals and the other reaction controls.
+- **One failed update no longer turns a code block into a post per flush** (#620). When the platform refused an update of the post Claude was writing a code block into, the bot gave up on that post at once. The continuation started in the middle of the block without its opening fence, so the rest rendered as prose, the height check split it every few lines (a new post per flush, close to 200 posts for a 2000-line file), and the closing fence opened a new block. A failed update is now retried on the next flush, which recovers the post completely for a passing error; only after three failures in a row is the post given up, with its code block reopened at the start of the next one.
+
 ## [1.39.2] - 2026-10-01
 
 ### Fixed

package/dist/index.js

@@ -6033,7 +6033,7 @@ var init_usage = __esm(() => {
 
 // src/utils/emoji.ts
 function withoutSkinTone(emoji) {
-  return emoji.replace(/::skin-tone-\d$/, "");
+  return emoji.replace(/::skin-tone-\d$/, "").replace(/(?<=.)_(?:light|medium_light|medium|medium_dark|dark)_skin_tone$/, "");
 }
 function isApprovalEmoji(emoji) {
   return APPROVAL_EMOJIS.includes(withoutSkinTone(emoji));
@@ -60297,7 +60297,8 @@ class ContentExecutor extends BaseExecutor {
       headerPostId: null,
       headerBody: "",
       turnOpen: false,
-      paragraphBreak: false
+      paragraphBreak: false,
+      updateFailures: 0
     };
   }
   setHeader(line) {
@@ -60318,6 +60319,7 @@ class ContentExecutor extends BaseExecutor {
     this.state.headerPostId = null;
     this.state.headerBody = "";
     this.state.turnOpen = false;
+    this.state.updateFailures = 0;
   }
   getRenderedCurrentPost() {
     const postId = this.state.currentPostId;
@@ -60344,7 +60346,9 @@ ${body}` : this.state.header;
     this.state.headerDirty = false;
     if (this.state.headerPostId) {
       const postId = this.state.headerPostId;
+      const failures = this.state.updateFailures;
       await this.tryUpdatePost(ctx, postId, this.state.headerBody, "header", { reason: "header_update", headerLength: this.state.header?.length ?? 0 }, { reason: "header_update_failed" }, () => {}, () => {});
+      this.state.updateFailures = failures;
       return;
     }
     if (this.state.header && this.state.turnOpen) {
@@ -60367,6 +60371,7 @@ ${body}` : this.state.header;
         this.state.headerBody = content;
         this.state.headerDirty = false;
       }
+      this.state.updateFailures = 0;
       onSuccess();
       ctx.threadLogger?.logExecutor("content", "update", postId, successDetails, logTag);
     } catch (err) {
@@ -60379,8 +60384,11 @@ ${body}` : this.state.header;
   closeCurrentPost(ctx) {
     const oldPostId = this.state.currentPostId;
     const contentLength = this.state.currentPostContent.length;
+    if (this.state.updateFailures > 0 && this.state.pendingContent.trim())
+      this.giveUpCurrentPost();
     this.state.currentPostId = null;
     this.state.currentPostContent = "";
+    this.state.updateFailures = 0;
     if (ctx?.threadLogger && oldPostId) {
       ctx.threadLogger.logExecutor("content", "close", oldPostId, {
         contentLength,
@@ -60436,13 +60444,22 @@ ${body}` : this.state.header;
         this.state.turnOpen = false;
       return;
     }
-    await this.flushPending(ctx);
-    if (this.state.headerDirty)
+    const postBefore = this.state.currentPostId;
+    await this.flushPending(ctx, reason);
+    if (reason === "result") {
+      if (this.state.updateFailures > 0 && this.state.currentPostId !== null && this.state.pendingContent.trim()) {
+        await this.flushPending(ctx, reason);
+      }
+      if (postBefore !== null && this.state.currentPostId === null && this.state.pendingContent.trim()) {
+        await this.flushPending(ctx, reason);
+      }
+    }
+    if (this.state.headerDirty && this.state.updateFailures === 0)
       await this.renderHeaderOnly(ctx);
     if (reason === "result")
       this.state.turnOpen = false;
   }
-  async flushPending(ctx) {
+  async flushPending(ctx, reason) {
     const pendingAtFlushStart = this.state.pendingContent;
     let content = ctx.formatter.formatMarkdown(pendingAtFlushStart).trim();
     const { maxLength: MAX_POST_LENGTH, hardThreshold: HARD_CONTINUATION_THRESHOLD } = ctx.platform.getMessageLimits();
@@ -60460,7 +60477,7 @@ ${body}` : this.state.header;
     const shouldBreakEarly = this.state.currentPostId && combinedContent.length > MIN_BREAK_THRESHOLD && ctx.contentBreaker.shouldFlushEarly(combinedContent);
     const reserve = this.headerReserve(this.state.currentPostId);
     if (this.state.currentPostId && (combinedContent.length + reserve > HARD_CONTINUATION_THRESHOLD || shouldBreakEarly)) {
-      await this.handleSplit(ctx, combinedContent, pendingAtFlushStart, HARD_CONTINUATION_THRESHOLD);
+      await this.handleSplit(ctx, combinedContent, pendingAtFlushStart, HARD_CONTINUATION_THRESHOLD, reason);
       return;
     }
     if (this.state.currentPostId && content.length + reserve > MAX_POST_LENGTH) {
@@ -60495,10 +60512,7 @@ ${body}` : this.state.header;
       await this.tryUpdatePost(ctx, postId, combinedContent, "flush", { newContentLength: content.length, combinedLength: combinedContent.length }, (err) => ({ failedOp: "updatePost", error: String(err) }), () => {
         this.state.currentPostContent = combinedContent;
         this.clearFlushedContent(pendingAtFlushStart);
-      }, () => {
-        this.state.currentPostId = null;
-        this.state.currentPostContent = "";
-      });
+      }, () => this.onInPlaceUpdateFailed(reason));
     } else {
       const chunks = splitContentForHeight(content, ctx.contentBreaker).flatMap((chunk) => chunk.length + reserve > MAX_POST_LENGTH ? splitByLength(chunk, HARD_CONTINUATION_THRESHOLD - reserve) : [chunk]);
       ctx.threadLogger?.logExecutor("content", "create_start", "none", {
@@ -60509,7 +60523,7 @@ ${body}` : this.state.header;
       await this.postChunks(ctx, chunks, pendingAtFlushStart, false);
     }
   }
-  async splitOversized(ctx, content, pendingAtFlushStart, hardThreshold) {
+  async splitOversized(ctx, content, pendingAtFlushStart, hardThreshold, reason) {
     const postId = this.state.currentPostId;
     if (!postId)
       return false;
@@ -60520,17 +60534,35 @@ ${body}` : this.state.header;
     let landed = false;
     await this.tryUpdatePost(ctx, postId, firstPiece, "handleSplit", { reason: "oversized_code_block", contentLength: content.length, pieces: rest.length + 1 }, { reason: "oversized_code_block_failed" }, () => {
       landed = true;
-    }, () => {
-      if (postId === this.state.headerPostId)
-        this.state.headerBody = firstPiece;
-    });
+    }, () => {});
+    if (!landed) {
+      this.onInPlaceUpdateFailed(reason);
+      return true;
+    }
     this.state.currentPostId = null;
     this.state.currentPostContent = "";
-    if (landed)
-      this.clearFlushedContent(pendingAtFlushStart);
-    await this.postChunks(ctx, rest, pendingAtFlushStart, landed);
+    this.clearFlushedContent(pendingAtFlushStart);
+    await this.postChunks(ctx, rest, pendingAtFlushStart, true);
     return true;
   }
+  onInPlaceUpdateFailed(reason) {
+    this.state.updateFailures++;
+    const limit = reason === "result" ? 2 : MAX_IN_PLACE_UPDATE_FAILURES;
+    if (this.state.updateFailures < limit)
+      return;
+    this.giveUpCurrentPost();
+  }
+  giveUpCurrentPost() {
+    const opener = openFenceOf(this.state.currentPostContent);
+    this.state.currentPostId = null;
+    this.state.currentPostContent = "";
+    this.state.updateFailures = 0;
+    if (!opener)
+      return;
+    const rest = this.state.pendingContent.replace(/^\n+/, "");
+    this.state.pendingContent = /^```[ \t]*(\n|$)/.test(rest) ? rest.replace(/^```[ \t]*\n?/, "") : `${opener}
+${rest}`;
+  }
   async postChunks(ctx, chunks, pendingAtFlushStart, flushCleared) {
     let cleared = flushCleared;
     for (let i = 0;i < chunks.length; i++) {
@@ -60554,7 +60586,7 @@ ${this.state.pendingContent}` : "");
       }
     }
   }
-  async handleSplit(ctx, content, pendingAtFlushStart, hardThreshold) {
+  async handleSplit(ctx, content, pendingAtFlushStart, hardThreshold, reason) {
     let breakPoint;
     let codeBlockOpenPosition;
     if (content.length > hardThreshold) {
@@ -60602,26 +60634,21 @@ ${this.state.pendingContent}` : "");
           await this.tryUpdatePost(ctx, postId, content, "handleSplit", { reason: "soft_break_no_breakpoint", contentLength: content.length }, { reason: "soft_break_no_breakpoint_failed" }, () => {
             this.state.currentPostContent = content;
             this.clearFlushedContent(pendingAtFlushStart);
-          }, () => {
-            this.state.currentPostId = null;
-          });
+          }, () => this.onInPlaceUpdateFailed(reason));
         }
         return;
       }
     }
     if (codeBlockOpenPosition !== undefined) {
       if (codeBlockOpenPosition === 0) {
-        if (await this.splitOversized(ctx, content, pendingAtFlushStart, hardThreshold))
+        if (await this.splitOversized(ctx, content, pendingAtFlushStart, hardThreshold, reason))
           return;
         if (this.state.currentPostId) {
           const postId = this.state.currentPostId;
           await this.tryUpdatePost(ctx, postId, content, "handleSplit", { reason: "code_block_at_start", contentLength: content.length }, { reason: "code_block_at_start_failed" }, () => {
             this.state.currentPostContent = content;
             this.clearFlushedContent(pendingAtFlushStart);
-          }, () => {
-            this.state.currentPostId = null;
-            this.state.currentPostContent = "";
-          });
+          }, () => this.onInPlaceUpdateFailed(reason));
         }
         return;
       }
@@ -60630,17 +60657,14 @@ ${this.state.pendingContent}` : "");
       if (breakBeforeCodeBlock > 0) {
         breakPoint = breakBeforeCodeBlock;
       } else {
-        if (await this.splitOversized(ctx, content, pendingAtFlushStart, hardThreshold))
+        if (await this.splitOversized(ctx, content, pendingAtFlushStart, hardThreshold, reason))
           return;
         if (this.state.currentPostId) {
           const postId = this.state.currentPostId;
           await this.tryUpdatePost(ctx, postId, content, "handleSplit", { reason: "no_break_before_code_block", contentLength: content.length }, { reason: "no_break_before_code_block_failed" }, () => {
             this.state.currentPostContent = content;
             this.clearFlushedContent(pendingAtFlushStart);
-          }, () => {
-            this.state.currentPostId = null;
-            this.state.currentPostContent = "";
-          });
+          }, () => this.onInPlaceUpdateFailed(reason));
         }
         return;
       }
@@ -60652,11 +60676,11 @@ ${this.state.pendingContent}` : "");
       const postId = this.state.currentPostId;
       await this.tryUpdatePost(ctx, postId, firstPart, "handleSplit", { reason: "split_first_part", firstPartLength: firstPart.length, remainderLength: remainder.length }, { reason: "split_first_part_failed" }, () => {
         firstPartLanded = true;
-      }, () => {
-        if (postId === this.state.headerPostId) {
-          this.state.headerBody = firstPart;
-        }
-      });
+      }, () => {});
+      if (!firstPartLanded) {
+        this.onInPlaceUpdateFailed(reason);
+        return;
+      }
     }
     this.state.currentPostId = null;
     this.state.currentPostContent = "";
@@ -60689,6 +60713,7 @@ ${content}` : header : content;
       if (bumpedPostId) {
         adoptAsHeaderPost(bumpedPostId);
         this.state.currentPostId = bumpedPostId;
+        this.state.updateFailures = 0;
         this.state.currentPostContent = content;
         this.clearFlushedContent(pendingAtFlushStart);
         ctx.threadLogger?.logExecutor("content", "create", bumpedPostId, {
@@ -60705,6 +60730,7 @@ ${content}` : header : content;
       const post = await ctx.createPost(rendered, { type: "content" });
       adoptAsHeaderPost(post.id);
       this.state.currentPostId = post.id;
+      this.state.updateFailures = 0;
       this.state.currentPostContent = content;
       this.clearFlushedContent(pendingAtFlushStart);
       ctx.logger.debug(`Created post ${formatShortId(post.id)}`);
@@ -60730,6 +60756,11 @@ ${content}` : header : content;
   }
 }
 var FENCE_LINE = /^```.*$/gm;
+var MAX_IN_PLACE_UPDATE_FAILURES = 3;
+function openFenceOf(text) {
+  const fences = text.match(FENCE_LINE) ?? [];
+  return fences.length % 2 === 1 ? fences[fences.length - 1] : null;
+}
 var FENCE_CLOSE = "\n```";
 function splitByLength(text, max) {
   const room = max - FENCE_CLOSE.length;
@@ -71905,7 +71936,6 @@ class SlackClient extends BasePlatformClient {
       token: this.botToken,
       channelId: this.channelId,
       allowedUsers: this.allowedUsers,
-      appToken: this.appToken,
       apiUrl: this.apiUrl,
       outboundFiles: this.outboundFiles,
       mcpServers: this.mcpServers,

package/dist/mcp/mcp-server.js

@@ -12723,7 +12723,7 @@ var require_dist = __commonJS(function(exports, module) {
 
 // src/utils/emoji.ts
 function withoutSkinTone(emoji) {
-  return emoji.replace(/::skin-tone-\d$/, "");
+  return emoji.replace(/::skin-tone-\d$/, "").replace(/(?<=.)_(?:light|medium_light|medium|medium_dark|dark)_skin_tone$/, "");
 }
 function isApprovalEmoji(emoji) {
   return APPROVAL_EMOJIS.includes(withoutSkinTone(emoji));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-threads",
-  "version": "1.39.2",
+  "version": "1.39.3",
   "description": "Run Claude Code from Slack or Mattermost. Sessions stream live into threads where your whole team can watch and steer.",
   "main": "dist/index.js",
   "type": "module",