claude-threads 1.0.10 → 1.0.12

package/CHANGELOG.md

@@ -5,6 +5,22 @@ 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.12] - 2026-01-15
+
+### Fixed
+- **Skipped file feedback posting** - Fixed swapped arguments in createPost call that caused "Invalid RootId parameter" errors when posting feedback for skipped files (#221)
+
+## [1.0.11] - 2026-01-15
+
+### Added
+- **Improved gzip error handling** - Uses streaming decompression for better error messages when gzip files are corrupt or truncated (#219)
+
+### Changed
+- **Shorter initial session message** - Session start message is now more concise for popup-friendly display (#218)
+
+### Dependencies
+- **Bump diff from 8.0.2 to 8.0.3** (#220)
+
 ## [1.0.10] - 2026-01-14
 
 ### Added

package/dist/index.js

@@ -51257,15 +51257,6 @@ class ClaudeCli extends EventEmitter2 {
   }
 }
 
-// src/logo.ts
-function getLogo(version) {
-  return `\`\`\`
- \u2734 \u2584\u2588\u2580 \u2588\u2588\u2588 \u2734   claude-threads v${version}
-\u2734  \u2588\u2580   \u2588   \u2734  Chat \xD7 Claude Code
- \u2734 \u2580\u2588\u2584  \u2588  \u2734
-\`\`\``;
-}
-
 // src/version.ts
 import { readFileSync as readFileSync6, existsSync as existsSync7 } from "fs";
 import { dirname as dirname6, resolve as resolve3 } from "path";
@@ -55471,12 +55462,15 @@ function createMessageManagerEvents() {
 
 // src/operations/streaming/handler.ts
 var import_yauzl = __toESM(require_yauzl(), 1);
-import { gunzipSync } from "zlib";
+import { createGunzip } from "zlib";
+import { pipeline } from "stream/promises";
+import { Readable, Writable } from "stream";
 var log17 = createLogger("streaming");
 var MAX_PDF_SIZE = 32 * 1024 * 1024;
 var MAX_TEXT_FILE_SIZE = 1 * 1024 * 1024;
 var MAX_DECOMPRESSED_SIZE = 10 * 1024 * 1024;
 var MAX_ZIP_SIZE = 50 * 1024 * 1024;
+var MAX_GZIP_SIZE = 50 * 1024 * 1024;
 var MAX_ZIP_FILES = 20;
 var SUPPORTED_IMAGE_TYPES = [
   "image/jpeg",
@@ -55709,6 +55703,25 @@ function formatTextFileContent(filename, content) {
 ${content}
 \`\`\``;
 }
+async function decompressGzipStream(compressedBuffer) {
+  const chunks = [];
+  let totalSize = 0;
+  const gunzip = createGunzip();
+  const source = Readable.from(compressedBuffer);
+  const collector = new Writable({
+    write(chunk, _encoding, callback) {
+      totalSize += chunk.length;
+      if (totalSize > MAX_DECOMPRESSED_SIZE) {
+        callback(new Error(`Decompressed size exceeds ${Math.round(MAX_DECOMPRESSED_SIZE / 1024 / 1024)}MB limit`));
+        return;
+      }
+      chunks.push(chunk);
+      callback();
+    }
+  });
+  await pipeline(source, gunzip, collector);
+  return Buffer.concat(chunks);
+}
 async function processGzipFile(file, platform, debug = false) {
   try {
     if (!platform.downloadFile) {
@@ -55719,24 +55732,60 @@ async function processGzipFile(file, platform, debug = false) {
         }
       };
     }
-    const compressedBuffer = await platform.downloadFile(file.id);
-    let decompressedBuffer;
+    if (file.size && file.size > MAX_GZIP_SIZE) {
+      return {
+        skipped: {
+          name: file.name,
+          reason: `Gzip file exceeds ${Math.round(MAX_GZIP_SIZE / 1024 / 1024)}MB limit (${Math.round(file.size / 1024 / 1024)}MB)`,
+          suggestion: "Try compressing a smaller file or splitting the content"
+        }
+      };
+    }
+    let compressedBuffer;
     try {
-      decompressedBuffer = gunzipSync(compressedBuffer);
+      compressedBuffer = await platform.downloadFile(file.id);
     } catch (err) {
+      const errorMessage = err instanceof Error ? err.message : String(err);
+      log17.error(`Failed to download gzip file ${file.name}: ${errorMessage}`);
       return {
         skipped: {
           name: file.name,
-          reason: `Decompression failed: ${err instanceof Error ? err.message : String(err)}`
+          reason: `Download failed: ${errorMessage}`,
+          suggestion: "Check if the file is still available and try again"
         }
       };
     }
-    if (decompressedBuffer.length > MAX_DECOMPRESSED_SIZE) {
+    if (file.size && compressedBuffer.length !== file.size) {
+      log17.warn(`Downloaded size mismatch for ${file.name}: expected ${file.size}, got ${compressedBuffer.length}`);
+    }
+    let decompressedBuffer;
+    try {
+      decompressedBuffer = await decompressGzipStream(compressedBuffer);
+    } catch (err) {
+      const errorMessage = err instanceof Error ? err.message : String(err);
+      let reason;
+      let suggestion;
+      if (errorMessage.includes("incorrect header check")) {
+        reason = "Invalid gzip file: the file header is corrupted or this is not a gzip file";
+        suggestion = "Verify the file is a valid gzip archive";
+      } else if (errorMessage.includes("unexpected end of file")) {
+        reason = "Incomplete gzip file: the file appears to be truncated";
+        suggestion = "Re-download the file or check if the upload completed";
+      } else if (errorMessage.includes("invalid stored block lengths")) {
+        reason = "Corrupted gzip file: the compressed data is damaged";
+        suggestion = "Try re-compressing the original file";
+      } else if (errorMessage.includes("exceeds") && errorMessage.includes("limit")) {
+        reason = errorMessage;
+        suggestion = "Try extracting only the relevant portions of the file";
+      } else {
+        reason = `Decompression failed: ${errorMessage}`;
+        suggestion = "Verify the file is a valid gzip archive";
+      }
       return {
         skipped: {
           name: file.name,
-          reason: `Decompressed size exceeds ${Math.round(MAX_DECOMPRESSED_SIZE / 1024 / 1024)}MB limit (${Math.round(decompressedBuffer.length / 1024 / 1024)}MB)`,
-          suggestion: "Try extracting only the relevant portions"
+          reason,
+          suggestion
         }
       };
     }
@@ -55777,11 +55826,13 @@ async function processGzipFile(file, platform, debug = false) {
       };
     }
   } catch (err) {
-    log17.error(`Failed to process gzip file ${file.name}: ${err}`);
+    const errorMessage = err instanceof Error ? err.message : String(err);
+    log17.error(`Failed to process gzip file ${file.name}: ${errorMessage}`);
     return {
       skipped: {
         name: file.name,
-        reason: `Processing failed: ${err instanceof Error ? err.message : String(err)}`
+        reason: `Processing failed: ${errorMessage}`,
+        suggestion: "An unexpected error occurred. Please try again or contact support if the issue persists"
       }
     };
   }
@@ -56538,7 +56589,7 @@ class MessageManager {
     this.session.claude.sendMessage(content);
     if (skippedFiles.length > 0) {
       const feedback = this.formatSkippedFilesFeedback(skippedFiles);
-      await this.platform.createPost(this.threadId, feedback);
+      await this.platform.createPost(feedback, this.threadId);
     }
     this.session.lastActivityAt = new Date;
     this.session.isProcessing = true;
@@ -63294,9 +63345,7 @@ async function startSession(options2, username, displayName, replyToPostId, plat
     return;
   }
   const startFormatter = platform.getFormatter();
-  const startPost = await withErrorHandling(() => platform.createPost(`${getLogo(VERSION)}
-
-${startFormatter.formatItalic("Starting session...")}`, replyToPostId), { action: "Create session post" });
+  const startPost = await withErrorHandling(() => platform.createPost(startFormatter.formatItalic("Claude Threads session starting..."), replyToPostId), { action: "Create session post" });
   if (!startPost)
     return;
   const actualThreadId = replyToPostId || startPost.id;

package/dist/mcp/permission-server.js

@@ -40344,12 +40344,15 @@ function createMessageManagerEvents() {
 
 // src/operations/streaming/handler.ts
 var import_yauzl = __toESM(require_yauzl(), 1);
-import { gunzipSync } from "zlib";
+import { createGunzip } from "zlib";
+import { pipeline as pipeline2 } from "stream/promises";
+import { Readable, Writable } from "stream";
 var log2 = createLogger("streaming");
 var MAX_PDF_SIZE = 32 * 1024 * 1024;
 var MAX_TEXT_FILE_SIZE = 1 * 1024 * 1024;
 var MAX_DECOMPRESSED_SIZE = 10 * 1024 * 1024;
 var MAX_ZIP_SIZE = 50 * 1024 * 1024;
+var MAX_GZIP_SIZE = 50 * 1024 * 1024;
 var MAX_ZIP_FILES = 20;
 var SUPPORTED_IMAGE_TYPES = [
   "image/jpeg",
@@ -40582,6 +40585,25 @@ function formatTextFileContent(filename, content) {
 ${content}
 \`\`\``;
 }
+async function decompressGzipStream(compressedBuffer) {
+  const chunks = [];
+  let totalSize = 0;
+  const gunzip = createGunzip();
+  const source = Readable.from(compressedBuffer);
+  const collector = new Writable({
+    write(chunk, _encoding, callback) {
+      totalSize += chunk.length;
+      if (totalSize > MAX_DECOMPRESSED_SIZE) {
+        callback(new Error(`Decompressed size exceeds ${Math.round(MAX_DECOMPRESSED_SIZE / 1024 / 1024)}MB limit`));
+        return;
+      }
+      chunks.push(chunk);
+      callback();
+    }
+  });
+  await pipeline2(source, gunzip, collector);
+  return Buffer.concat(chunks);
+}
 async function processGzipFile(file2, platform, debug = false) {
   try {
     if (!platform.downloadFile) {
@@ -40592,24 +40614,60 @@ async function processGzipFile(file2, platform, debug = false) {
         }
       };
     }
-    const compressedBuffer = await platform.downloadFile(file2.id);
-    let decompressedBuffer;
+    if (file2.size && file2.size > MAX_GZIP_SIZE) {
+      return {
+        skipped: {
+          name: file2.name,
+          reason: `Gzip file exceeds ${Math.round(MAX_GZIP_SIZE / 1024 / 1024)}MB limit (${Math.round(file2.size / 1024 / 1024)}MB)`,
+          suggestion: "Try compressing a smaller file or splitting the content"
+        }
+      };
+    }
+    let compressedBuffer;
     try {
-      decompressedBuffer = gunzipSync(compressedBuffer);
+      compressedBuffer = await platform.downloadFile(file2.id);
     } catch (err) {
+      const errorMessage = err instanceof Error ? err.message : String(err);
+      log2.error(`Failed to download gzip file ${file2.name}: ${errorMessage}`);
       return {
         skipped: {
           name: file2.name,
-          reason: `Decompression failed: ${err instanceof Error ? err.message : String(err)}`
+          reason: `Download failed: ${errorMessage}`,
+          suggestion: "Check if the file is still available and try again"
         }
       };
     }
-    if (decompressedBuffer.length > MAX_DECOMPRESSED_SIZE) {
+    if (file2.size && compressedBuffer.length !== file2.size) {
+      log2.warn(`Downloaded size mismatch for ${file2.name}: expected ${file2.size}, got ${compressedBuffer.length}`);
+    }
+    let decompressedBuffer;
+    try {
+      decompressedBuffer = await decompressGzipStream(compressedBuffer);
+    } catch (err) {
+      const errorMessage = err instanceof Error ? err.message : String(err);
+      let reason;
+      let suggestion;
+      if (errorMessage.includes("incorrect header check")) {
+        reason = "Invalid gzip file: the file header is corrupted or this is not a gzip file";
+        suggestion = "Verify the file is a valid gzip archive";
+      } else if (errorMessage.includes("unexpected end of file")) {
+        reason = "Incomplete gzip file: the file appears to be truncated";
+        suggestion = "Re-download the file or check if the upload completed";
+      } else if (errorMessage.includes("invalid stored block lengths")) {
+        reason = "Corrupted gzip file: the compressed data is damaged";
+        suggestion = "Try re-compressing the original file";
+      } else if (errorMessage.includes("exceeds") && errorMessage.includes("limit")) {
+        reason = errorMessage;
+        suggestion = "Try extracting only the relevant portions of the file";
+      } else {
+        reason = `Decompression failed: ${errorMessage}`;
+        suggestion = "Verify the file is a valid gzip archive";
+      }
       return {
         skipped: {
           name: file2.name,
-          reason: `Decompressed size exceeds ${Math.round(MAX_DECOMPRESSED_SIZE / 1024 / 1024)}MB limit (${Math.round(decompressedBuffer.length / 1024 / 1024)}MB)`,
-          suggestion: "Try extracting only the relevant portions"
+          reason,
+          suggestion
         }
       };
     }
@@ -40650,11 +40708,13 @@ async function processGzipFile(file2, platform, debug = false) {
       };
     }
   } catch (err) {
-    log2.error(`Failed to process gzip file ${file2.name}: ${err}`);
+    const errorMessage = err instanceof Error ? err.message : String(err);
+    log2.error(`Failed to process gzip file ${file2.name}: ${errorMessage}`);
     return {
       skipped: {
         name: file2.name,
-        reason: `Processing failed: ${err instanceof Error ? err.message : String(err)}`
+        reason: `Processing failed: ${errorMessage}`,
+        suggestion: "An unexpected error occurred. Please try again or contact support if the issue persists"
       }
     };
   }
@@ -41384,7 +41444,7 @@ class MessageManager {
     this.session.claude.sendMessage(content);
     if (skippedFiles.length > 0) {
       const feedback = this.formatSkippedFilesFeedback(skippedFiles);
-      await this.platform.createPost(this.threadId, feedback);
+      await this.platform.createPost(feedback, this.threadId);
     }
     this.session.lastActivityAt = new Date;
     this.session.isProcessing = true;

package/package.json

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