claude-threads 1.0.6 → 1.0.8

package/CHANGELOG.md

@@ -5,6 +5,16 @@ 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.0.8] - 2026-01-13
+
+### Fixed
+- **Maximize content per message when height-splitting** - Split algorithm now finds the optimal breakpoint to maximize content per message, instead of splitting at the first available breakpoint. Previously would split after Part 1 when Parts 1-4 could fit together.
+
+## [1.0.7] - 2026-01-13
+
+### Fixed
+- **Check combined content height for streaming split decisions** - Fixed bug where height check only evaluated new content instead of combined content (existing + new), causing "Show More" collapse when total exceeded threshold but new content alone didn't (#212)
+
 ## [1.0.6] - 2026-01-13
 
 ### Fixed

package/dist/index.js

@@ -50926,6 +50926,10 @@ class DefaultContentBreaker {
       return true;
     return false;
   }
+  exceedsHeightThreshold(content) {
+    const estimatedHeight = estimateRenderedHeight(content);
+    return estimatedHeight >= MAX_HEIGHT_THRESHOLD;
+  }
   endsAtBreakpoint(content) {
     const trimmed = content.trimEnd();
     if (/ {2}\u21B3 [\u2713\u274C][^\n]*$/.test(trimmed)) {
@@ -50947,16 +50951,29 @@ function splitContentForHeight(content, contentBreaker) {
   let remaining = content;
   const goodBreakpointTypes = new Set(["paragraph", "code_block_end", "heading", "tool_marker"]);
   while (remaining && contentBreaker.shouldFlushEarly(remaining)) {
-    const breakpoint = contentBreaker.findLogicalBreakpoint(remaining, 0, remaining.length);
-    if (!breakpoint || breakpoint.position <= 0 || breakpoint.position >= remaining.length) {
-      break;
+    let bestBreakpoint = null;
+    let searchStart = 0;
+    while (searchStart < remaining.length) {
+      const breakpoint = contentBreaker.findLogicalBreakpoint(remaining, searchStart, remaining.length - searchStart);
+      if (!breakpoint || breakpoint.position <= searchStart || breakpoint.position >= remaining.length) {
+        break;
+      }
+      if (!goodBreakpointTypes.has(breakpoint.type)) {
+        searchStart = breakpoint.position + 1;
+        continue;
+      }
+      const firstPart2 = remaining.substring(0, breakpoint.position).trim();
+      if (!contentBreaker.exceedsHeightThreshold(firstPart2)) {
+        bestBreakpoint = breakpoint;
+      }
+      searchStart = breakpoint.position + 1;
     }
-    if (!goodBreakpointTypes.has(breakpoint.type)) {
+    if (!bestBreakpoint) {
       break;
     }
-    const firstPart = remaining.substring(0, breakpoint.position).trim();
-    const secondPart = remaining.substring(breakpoint.position).trim();
-    if (!contentBreaker.shouldFlushEarly(firstPart) && secondPart.length > 0) {
+    const firstPart = remaining.substring(0, bestBreakpoint.position).trim();
+    const secondPart = remaining.substring(bestBreakpoint.position).trim();
+    if (secondPart.length > 0) {
       chunks.push(firstPart);
       remaining = secondPart;
     } else {
@@ -52418,7 +52435,7 @@ class ContentExecutor extends BaseExecutor {
     } else {
       combinedContent = content;
     }
-    const shouldBreakEarly = this.state.currentPostId && content.length > MIN_BREAK_THRESHOLD && ctx.contentBreaker.shouldFlushEarly(content);
+    const shouldBreakEarly = this.state.currentPostId && combinedContent.length > MIN_BREAK_THRESHOLD && ctx.contentBreaker.shouldFlushEarly(combinedContent);
     if (this.state.currentPostId && (combinedContent.length > HARD_CONTINUATION_THRESHOLD || shouldBreakEarly)) {
       await this.handleSplit(ctx, combinedContent, pendingAtFlushStart, HARD_CONTINUATION_THRESHOLD);
       return;
@@ -52507,9 +52524,26 @@ class ContentExecutor extends BaseExecutor {
         }
       }
     } else {
-      const breakInfo = ctx.contentBreaker.findLogicalBreakpoint(content, 2000);
-      if (breakInfo && breakInfo.position < content.length) {
-        breakPoint = breakInfo.position;
+      const goodBreakpointTypes = new Set(["paragraph", "code_block_end", "heading", "tool_marker"]);
+      let bestBreakPoint = null;
+      let searchStart = 0;
+      while (searchStart < content.length) {
+        const breakInfo = ctx.contentBreaker.findLogicalBreakpoint(content, searchStart, content.length - searchStart);
+        if (!breakInfo || breakInfo.position <= searchStart || breakInfo.position >= content.length) {
+          break;
+        }
+        if (!goodBreakpointTypes.has(breakInfo.type)) {
+          searchStart = breakInfo.position + 1;
+          continue;
+        }
+        const firstPart2 = content.substring(0, breakInfo.position).trim();
+        if (!ctx.contentBreaker.exceedsHeightThreshold(firstPart2)) {
+          bestBreakPoint = breakInfo.position;
+        }
+        searchStart = breakInfo.position + 1;
+      }
+      if (bestBreakPoint !== null && bestBreakPoint > 0) {
+        breakPoint = bestBreakPoint;
       } else {
         if (this.state.currentPostId) {
           try {

package/dist/mcp/permission-server.js

@@ -35635,6 +35635,10 @@ class DefaultContentBreaker {
       return true;
     return false;
   }
+  exceedsHeightThreshold(content) {
+    const estimatedHeight = estimateRenderedHeight(content);
+    return estimatedHeight >= MAX_HEIGHT_THRESHOLD;
+  }
   endsAtBreakpoint(content) {
     const trimmed = content.trimEnd();
     if (/ {2}\u21B3 [\u2713\u274C][^\n]*$/.test(trimmed)) {
@@ -35656,16 +35660,29 @@ function splitContentForHeight(content, contentBreaker) {
   let remaining = content;
   const goodBreakpointTypes = new Set(["paragraph", "code_block_end", "heading", "tool_marker"]);
   while (remaining && contentBreaker.shouldFlushEarly(remaining)) {
-    const breakpoint = contentBreaker.findLogicalBreakpoint(remaining, 0, remaining.length);
-    if (!breakpoint || breakpoint.position <= 0 || breakpoint.position >= remaining.length) {
-      break;
+    let bestBreakpoint = null;
+    let searchStart = 0;
+    while (searchStart < remaining.length) {
+      const breakpoint = contentBreaker.findLogicalBreakpoint(remaining, searchStart, remaining.length - searchStart);
+      if (!breakpoint || breakpoint.position <= searchStart || breakpoint.position >= remaining.length) {
+        break;
+      }
+      if (!goodBreakpointTypes.has(breakpoint.type)) {
+        searchStart = breakpoint.position + 1;
+        continue;
+      }
+      const firstPart2 = remaining.substring(0, breakpoint.position).trim();
+      if (!contentBreaker.exceedsHeightThreshold(firstPart2)) {
+        bestBreakpoint = breakpoint;
+      }
+      searchStart = breakpoint.position + 1;
     }
-    if (!goodBreakpointTypes.has(breakpoint.type)) {
+    if (!bestBreakpoint) {
       break;
     }
-    const firstPart = remaining.substring(0, breakpoint.position).trim();
-    const secondPart = remaining.substring(breakpoint.position).trim();
-    if (!contentBreaker.shouldFlushEarly(firstPart) && secondPart.length > 0) {
+    const firstPart = remaining.substring(0, bestBreakpoint.position).trim();
+    const secondPart = remaining.substring(bestBreakpoint.position).trim();
+    if (secondPart.length > 0) {
       chunks.push(firstPart);
       remaining = secondPart;
     } else {
@@ -37218,7 +37235,7 @@ class ContentExecutor extends BaseExecutor {
     } else {
       combinedContent = content;
     }
-    const shouldBreakEarly = this.state.currentPostId && content.length > MIN_BREAK_THRESHOLD && ctx.contentBreaker.shouldFlushEarly(content);
+    const shouldBreakEarly = this.state.currentPostId && combinedContent.length > MIN_BREAK_THRESHOLD && ctx.contentBreaker.shouldFlushEarly(combinedContent);
     if (this.state.currentPostId && (combinedContent.length > HARD_CONTINUATION_THRESHOLD || shouldBreakEarly)) {
       await this.handleSplit(ctx, combinedContent, pendingAtFlushStart, HARD_CONTINUATION_THRESHOLD);
       return;
@@ -37307,9 +37324,26 @@ class ContentExecutor extends BaseExecutor {
         }
       }
     } else {
-      const breakInfo = ctx.contentBreaker.findLogicalBreakpoint(content, 2000);
-      if (breakInfo && breakInfo.position < content.length) {
-        breakPoint = breakInfo.position;
+      const goodBreakpointTypes = new Set(["paragraph", "code_block_end", "heading", "tool_marker"]);
+      let bestBreakPoint = null;
+      let searchStart = 0;
+      while (searchStart < content.length) {
+        const breakInfo = ctx.contentBreaker.findLogicalBreakpoint(content, searchStart, content.length - searchStart);
+        if (!breakInfo || breakInfo.position <= searchStart || breakInfo.position >= content.length) {
+          break;
+        }
+        if (!goodBreakpointTypes.has(breakInfo.type)) {
+          searchStart = breakInfo.position + 1;
+          continue;
+        }
+        const firstPart2 = content.substring(0, breakInfo.position).trim();
+        if (!ctx.contentBreaker.exceedsHeightThreshold(firstPart2)) {
+          bestBreakPoint = breakInfo.position;
+        }
+        searchStart = breakInfo.position + 1;
+      }
+      if (bestBreakPoint !== null && bestBreakPoint > 0) {
+        breakPoint = bestBreakPoint;
       } else {
         if (this.state.currentPostId) {
           try {

package/package.json

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