@bobfrankston/mailx 1.0.200 → 1.0.202

package/client/.msger-window.json

@@ -1 +1 @@
-{"height":1344,"width":2151,"x":587,"y":80}
+{"height":1344,"width":2151,"x":469,"y":64}

package/client/app.js

@@ -310,12 +310,15 @@ async function openCompose(mode) {
         if (!msg || identityDomains.length === 0)
             return undefined;
         const candidates = [msg.deliveredTo, ...(msg.to || []).map((a) => a.address)].filter(Boolean);
+        console.log(`[compose] detectReplyFrom: deliveredTo=${msg.deliveredTo}, to=${msg.to?.map((a) => a.address)}, domains=${identityDomains}`);
         for (const addr of candidates) {
             const domain = addr.split("@")[1]?.toLowerCase();
             if (domain && identityDomains.some((d) => domain === d.toLowerCase())) {
+                console.log(`[compose] matched: ${addr}`);
                 return addr;
             }
         }
+        console.log(`[compose] no identity match`);
         return undefined;
     }
     if (msg && mode === "reply") {
@@ -415,7 +418,16 @@ function showComposeOverlay() {
 function quoteBody(msg) {
     const date = new Date(msg.date).toLocaleString();
     const from = msg.from.name ? `${msg.from.name} <${msg.from.address}>` : msg.from.address;
-    const body = msg.bodyHtml || `<pre>${msg.bodyText || ""}</pre>`;
+    // Simplify complex HTML (tables, inline styles) for Quill compatibility
+    let body = msg.bodyHtml || `<pre>${msg.bodyText || ""}</pre>`;
+    // Strip style tags and attributes
+    body = body.replace(/<style[^>]*>[\s\S]*?<\/style>/gi, "");
+    body = body.replace(/\s+style="[^"]*"/gi, "");
+    body = body.replace(/\s+class="[^"]*"/gi, "");
+    // Convert tables to simple divs
+    body = body.replace(/<table[^>]*>/gi, "<div>").replace(/<\/table>/gi, "</div>");
+    body = body.replace(/<t[rdh][^>]*>/gi, "").replace(/<\/t[rdh]>/gi, " ");
+    body = body.replace(/<thead[^>]*>|<\/thead>|<tbody[^>]*>|<\/tbody>/gi, "");
     return `<br><div class="reply"><p>On ${date}, ${from} wrote:</p><blockquote>${body}</blockquote></div>`;
 }
 function forwardBody(msg) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx",
-  "version": "1.0.200",
+  "version": "1.0.202",
   "description": "Local-first email client with IMAP sync and standalone native app",
   "type": "module",
   "main": "bin/mailx.js",
@@ -24,7 +24,7 @@
     "@bobfrankston/iflow-node": "^0.1.2",
     "@bobfrankston/miscinfo": "^1.0.8",
     "@bobfrankston/oauthsupport": "^1.0.22",
-    "@bobfrankston/msger": "^0.1.252",
+    "@bobfrankston/msger": "^0.1.254",
     "@capacitor/android": "^8.3.0",
     "@capacitor/cli": "^8.3.0",
     "@capacitor/core": "^8.3.0",

package/packages/mailx-imap/index.d.ts

@@ -128,7 +128,8 @@ export declare class ImapManager extends EventEmitter {
     private fetchQueues;
     /** Serialize body fetch operations per account — prevents concurrent IMAP commands on same connection */
     private enqueueFetch;
-    /** Fetch a single message body on demand, caching in the store */
+    /** Fetch a single message body on demand, caching in the store.
+     *  Uses its own fresh connection — never blocked by background prefetch. */
     fetchMessageBody(accountId: string, folderId: number, uid: number): Promise<Buffer | null>;
     /** Fetch message body via Gmail/Outlook API */
     private fetchMessageBodyViaApi;

package/packages/mailx-imap/index.js

@@ -1218,7 +1218,8 @@ export class ImapManager extends EventEmitter {
         return next;
     }
     // Body fetch uses withConnection — no separate client needed
-    /** Fetch a single message body on demand, caching in the store */
+    /** Fetch a single message body on demand, caching in the store.
+     *  Uses its own fresh connection — never blocked by background prefetch. */
     async fetchMessageBody(accountId, folderId, uid) {
         // Already cached?
         if (await this.bodyStore.hasMessage(accountId, folderId, uid)) {
@@ -1233,36 +1234,30 @@ export class ImapManager extends EventEmitter {
         if (this.isGmailAccount(accountId)) {
             return this.fetchMessageBodyViaApi(accountId, folderId, uid, folder.path);
         }
-        // IMAP: serialize — only one body fetch per account at a time
-        return this.enqueueFetch(accountId, async () => {
-            // Re-check cache — may have been fetched while queued
-            if (await this.bodyStore.hasMessage(accountId, folderId, uid)) {
-                return this.bodyStore.getMessage(accountId, folderId, uid);
-            }
-            let client = null;
-            try {
-                client = this.newClient(accountId);
-                const msg = await client.fetchMessageByUid(folder.path, uid, { source: true });
-                await client.logout();
-                client = null;
-                if (!msg?.source)
-                    return null;
-                const raw = Buffer.from(msg.source, "utf-8");
-                const bodyPath = await this.bodyStore.putMessage(accountId, folderId, uid, raw);
-                this.db.updateBodyPath(accountId, uid, bodyPath);
-                return raw;
-            }
-            catch (e) {
-                console.error(`  Body fetch error (${accountId}/${uid}): ${e.message}`);
-                if (client) {
-                    try {
-                        await client.logout();
-                    }
-                    catch { /* */ }
-                }
+        // IMAP: fresh connection per on-demand fetch — never queued behind prefetch
+        let client = null;
+        try {
+            client = this.newClient(accountId);
+            const msg = await client.fetchMessageByUid(folder.path, uid, { source: true });
+            await client.logout();
+            client = null;
+            if (!msg?.source)
                 return null;
+            const raw = Buffer.from(msg.source, "utf-8");
+            const bodyPath = await this.bodyStore.putMessage(accountId, folderId, uid, raw);
+            this.db.updateBodyPath(accountId, uid, bodyPath);
+            return raw;
+        }
+        catch (e) {
+            console.error(`  Body fetch error (${accountId}/${uid}): ${e.message}`);
+            if (client) {
+                try {
+                    await client.logout();
+                }
+                catch { /* */ }
             }
-        });
+            return null;
+        }
     }
     /** Fetch message body via Gmail/Outlook API */
     async fetchMessageBodyViaApi(accountId, folderId, uid, folderPath) {

package/packages/mailx-settings/index.js

@@ -114,9 +114,7 @@ export async function cloudRead(filename) {
         }
         catch { /* ignore cache write failure */ }
     }
-    else {
-        lastCloudError = `Could not read ${filename} from ${pendingCloudConfig.provider} (check credentials)`;
-    }
+    // Don't set error for missing files — they may not exist yet (e.g., clients.jsonc on first run)
     return content;
 }
 /** Write a file via cloud API */