@bobfrankston/rmfmail 1.0.512 → 1.0.516

package/bin/lean-accounts.js

@@ -26,9 +26,9 @@
 
 import fs from "node:fs";
 import path from "node:path";
-import { fileURLToPath, pathToFileURL } from "node:url";
+import { pathToFileURL } from "node:url";
 
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const __dirname = import.meta.dirname;
 
 // ── Provider defaults: same set mailx-settings uses ────────────────────────
 const PROVIDERS = {

package/client/app.js

@@ -1905,18 +1905,16 @@ onWsEvent((event) => {
             refreshFolderTree();
             // Q53: track per-account last-sync timestamp for the status-bar hover.
             recordAccountSync(event.accountId);
-            // Reload the currently-displayed folder. folderCountsChanged
-            // already triggers a debounced reload when *new* messages arrive,
-            // but on Android the first sync of an account can produce a
-            // bunch of folderCountsChanged events that get coalesced — and
-            // if the unified inbox is the active view but no individual
-            // folder count changed *during* the debounce window, the
-            // message list never refreshes. The phone stayed on its
-            // "no messages" placeholder while Phase 1/2 sync filled the
-            // DB underneath. Forcing one more reload at sync end is cheap
-            // (a single SQL query) and makes the user-visible state catch
-            // up to the DB.
-            reloadCurrentFolder();
+            // Earlier I added reloadCurrentFolder() here to fix the
+            // "phone INBOX stays on placeholder" report. That broke desktop
+            // because syncComplete fires repeatedly on the desktop sync
+            // loop, and each reload rebuilds the message-list DOM
+            // mid-read — the focusedRow Row object becomes a stale
+            // pointer to a detached element, and the viewer's "Select a
+            // message to read" placeholder stays up while the new row
+            // shows .selected. Removed: folderCountsChanged already
+            // triggers a debounced reload when new messages actually
+            // arrive (which is what the phone case really needs anyway).
             break;
         case "folderSynced":
             // Per-folder timestamps — drives the tooltip + freshness dot.