@bobfrankston/mailx 1.0.132 → 1.0.133

package/client/app.js

@@ -428,6 +428,7 @@ function doSearch(immediate = false) {
 // Track current folder for scoped search
 let currentAccountId = "";
 let currentFolderId = 0;
+let reloadDebounceTimer = null;
 searchInput?.addEventListener("input", () => {
     clearTimeout(searchTimeout);
     searchTimeout = setTimeout(() => doSearch(false), 300);
@@ -585,8 +586,19 @@ onWsEvent((event) => {
             // Incremental count update — no DOM rebuild, no jitter
             updateFolderCounts();
             updateNewMessageCount();
-            // Reload message list but keep current scroll position and selection
-            reloadCurrentFolder();
+            // Only reload message list if the synced account is the one we're viewing
+            // (or unified inbox which shows all accounts). Debounce to avoid rapid reloads
+            // during first sync which emits per-batch.
+            const syncedAccount = event.accountId;
+            const viewingThis = !currentAccountId || currentAccountId === syncedAccount;
+            if (viewingThis) {
+                if (reloadDebounceTimer)
+                    clearTimeout(reloadDebounceTimer);
+                reloadDebounceTimer = setTimeout(() => {
+                    reloadDebounceTimer = null;
+                    reloadCurrentFolder();
+                }, 500);
+            }
             // Sync finished — re-enable sync button
             const syncBtn = document.getElementById("btn-sync");
             if (syncBtn) {

package/client/components/message-list.js

@@ -94,7 +94,10 @@ export async function loadUnifiedInbox(autoSelect = true) {
         return;
     const savedScroll = !autoSelect ? body.scrollTop : 0;
     const savedUid = !autoSelect ? body.querySelector(".ml-row.selected")?.getAttribute("data-uid") : null;
-    body.innerHTML = `<div class="ml-empty">Loading...</div>`;
+    // Only show loading indicator on fresh navigation, not reloads
+    if (autoSelect) {
+        body.innerHTML = `<div class="ml-empty">Loading...</div>`;
+    }
     try {
         const result = await getUnifiedInbox(1);
         totalMessages = result.total;
@@ -103,8 +106,13 @@ export async function loadUnifiedInbox(autoSelect = true) {
             clearViewer();
             return;
         }
-        body.innerHTML = "";
-        appendMessages(body, "", result.items);
+        // Build new rows into a fragment, then swap atomically (no flash)
+        const fragment = document.createDocumentFragment();
+        const tempDiv = document.createElement("div");
+        appendMessages(tempDiv, "", result.items);
+        while (tempDiv.firstChild)
+            fragment.appendChild(tempDiv.firstChild);
+        body.replaceChildren(fragment);
         if (autoSelect) {
             const firstRow = body.querySelector(".ml-row");
             if (firstRow)
@@ -209,8 +217,13 @@ export async function loadMessages(accountId, folderId, page = 1, specialUse = "
             clearViewer();
             return;
         }
-        body.innerHTML = "";
-        appendMessages(body, accountId, result.items);
+        // Build new rows into a fragment, then swap atomically (no flash)
+        const fragment = document.createDocumentFragment();
+        const tempDiv = document.createElement("div");
+        appendMessages(tempDiv, accountId, result.items);
+        while (tempDiv.firstChild)
+            fragment.appendChild(tempDiv.firstChild);
+        body.replaceChildren(fragment);
         if (autoSelect) {
             // Explicit folder navigation — select first message
             const firstRow = body.querySelector(".ml-row");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx",
-  "version": "1.0.132",
+  "version": "1.0.133",
   "description": "Local-first email client with IMAP sync and standalone native app",
   "type": "module",
   "main": "bin/mailx.js",

package/packages/mailx-imap/index.js

@@ -866,7 +866,11 @@ export class ImapManager extends EventEmitter {
         for (let attempt = 0; attempt < 2; attempt++) {
             try {
                 const client = this.getFetchClient(accountId);
-                const msg = await client.fetchMessageByUid(folder.path, uid, { source: true });
+                // 30s timeout — prevents hanging on stale connections
+                const msg = await Promise.race([
+                    client.fetchMessageByUid(folder.path, uid, { source: true }),
+                    new Promise((_, reject) => setTimeout(() => reject(new Error("Body fetch timeout (30s)")), 30000))
+                ]);
                 if (!msg?.source)
                     return null;
                 const raw = Buffer.from(msg.source, "utf-8");
@@ -876,8 +880,16 @@ export class ImapManager extends EventEmitter {
                 return raw;
             }
             catch (e) {
-                console.error(`  Body fetch error (${accountId}/${uid}): ${e.message}`);
+                console.error(`  Body fetch error (${accountId}/${uid} attempt ${attempt + 1}): ${e.message}`);
+                // Kill stale client so retry creates a fresh connection
+                const stale = this.fetchClients.get(accountId);
                 this.fetchClients.delete(accountId);
+                if (stale) {
+                    try {
+                        await stale.logout();
+                    }
+                    catch { /* ignore */ }
+                }
                 if (attempt === 1)
                     return null;
                 // Retry with fresh client