@bobfrankston/rmfmail 1.2.227 → 1.2.228

package/packages/mailx-service/jsonrpc.ts

@@ -59,24 +59,34 @@ function capBodyForIpc(msg: any): any {
     if (!msg || typeof msg !== "object") return msg;
     const html = typeof msg.bodyHtml === "string" ? msg.bodyHtml : "";
     const text = typeof msg.bodyText === "string" ? msg.bodyText : "";
-    const total = html.length + text.length;
-    if (total <= IPC_BODY_MAX_BYTES) return msg;
 
-    // Prefer the plain text when we have to truncate: a half-sent HTML
-    // document renders as garbage (unclosed tags swallow the rest), whereas
-    // half of a text body is simply the first half of the text.
-    const keptText = text ? headOf(text, IPC_BODY_MAX_BYTES) : "";
-    const keptHtml = !text && html ? "" : "";     // never ship partial HTML
-    console.log(`  [ipc] body too large for evaluate_script — acct=${msg.accountId} uid=${msg.uid} `
-        + `html=${html.length} text=${text.length} → sending ${keptText.length} bytes of text + truncation notice`);
+    // A message with an HTML part is NEVER touched, at any size.
+    //
+    // The first cut of this guard truncated on total size and refused to send
+    // partial HTML, so a legitimate 9.8 MB message — one Bob had just written,
+    // large because of a pasted inline image — arrived with its HTML dropped
+    // and rendered as "Showing the first 0.0 MB" (Bob 2026-08-08: "I pasted in
+    // an image and you blew it away?"). Destroying real content to protect the
+    // renderer is a worse failure than the one being protected against.
+    //
+    // What remains guarded is the genuinely pathological shape: NO MIME
+    // structure at all, so the entire multi-megabyte payload is one implicit
+    // text/plain part (a qmail bounce that inlines the whole original). There,
+    // the tail is base64 of a returned attachment — truncating it costs the
+    // user nothing, while the head carries every diagnostic that matters.
+    if (html) return msg;
+    if (text.length <= IPC_BODY_MAX_BYTES) return msg;
+
+    const keptText = headOf(text, IPC_BODY_MAX_BYTES);
+    console.log(`  [ipc] MIME-less body of ${text.length} bytes — acct=${msg.accountId} uid=${msg.uid} `
+        + `→ sending first ${keptText.length} bytes with a truncation notice`);
     return {
         ...msg,
-        bodyHtml: keptHtml,
         bodyText: keptText,
         bodyTruncated: {
-            totalBytes: total,
+            totalBytes: text.length,
             sentBytes: keptText.length,
-            hadHtml: html.length > 0,
+            hadHtml: false,
             emlPath: msg.emlPath || "",
         },
     };

package/packages/mailx-service/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-service",
-  "version": "0.1.22",
+  "version": "0.1.23",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",