@bobfrankston/rmfmail 1.2.67 → 1.2.69

package/client/app.js

@@ -3,7 +3,7 @@
  * Wires together all UI components and WebSocket connection.
  */
 import { initFolderTree, refreshFolderTree, updateFolderCounts, setFolderSynced, getFolderSynced, setOutboxTotal } from "./components/folder-tree.js";
-import { initMessageList, loadMessages, loadUnifiedInbox, loadSearchResults, reloadCurrentFolder, clearSearchMode, getSelectedMessages, markBodiesCached, getCurrentFocused, releaseFocus, removeMessagesAndReconcile, setRowFlagged, scrollFocusedIntoView, refreshPriorityIndex, showSelectionMenu, revealMessage } from "./components/message-list.js";
+import { initMessageList, loadMessages, loadUnifiedInbox, loadSearchResults, reloadCurrentFolder, clearSearchMode, getSelectedMessages, markBodiesCached, getCurrentFocused, releaseFocus, removeMessagesAndReconcile, setRowFlagged, scrollFocusedIntoView, refreshPriorityIndex, showSelectionMenu, revealMessage, selectAllVisible } from "./components/message-list.js";
 import { seenOf, flaggedOf, draftOf, setSeen, setFlagged } from "@bobfrankston/mailx-types";
 import { initTabs, setActiveView as setActiveTabView, openTab } from "./components/tabs.js";
 import { getCurrentMessage, initViewer, popOutCurrentMessage, printCurrentMessage, toggleFullscreenPreview, showPreviewBodyMenu, wrapHtmlBody } from "./components/message-viewer.js";
@@ -1682,7 +1682,7 @@ async function performUndo() {
                 const flagsAfter = entry.prevFlagged
                     ? [flag] // simplest restore — full set is unknown here
                     : [];
-                await updateFlags(entry.accountId, entry.uid, flagsAfter);
+                await updateFlags(entry.accountId, entry.uid, flagsAfter, entry.folderId);
                 setRowFlagged(entry.accountId, entry.uid, entry.prevFlagged);
                 messageState.updateMessageFlags(entry.accountId, entry.uid, flagsAfter);
             }
@@ -1913,7 +1913,7 @@ async function bulkFlagSelected() {
     const anyUnflagged = rows.some(r => !flaggedOf(r));
     const targetFlagged = anyUnflagged; // mixed or all-unflagged → flag; all-flagged → unflag
     const undoPayload = rows.map(r => ({
-        accountId: r.accountId, uid: r.uid, prevFlagged: flaggedOf(r),
+        accountId: r.accountId, uid: r.uid, prevFlagged: flaggedOf(r), folderId: r.folderId,
     }));
     const { updateFlags } = await import("./lib/api-client.js");
     for (const r of rows) {
@@ -1923,7 +1923,7 @@ async function bulkFlagSelected() {
         setRowFlagged(r.accountId, r.uid, targetFlagged);
         messageState.updateMessageFlags(r.accountId, r.uid, r.flags);
         try {
-            await updateFlags(r.accountId, r.uid, r.flags);
+            await updateFlags(r.accountId, r.uid, r.flags, r.folderId);
         }
         catch (e) {
             console.error(`Bulk flag failed for ${r.accountId}/${r.uid}: ${e?.message || e}`);
@@ -1945,7 +1945,7 @@ document.getElementById("btn-flag")?.addEventListener("click", async () => {
     setFlagged(sel, !wasFlagged);
     updateFlagButton();
     try {
-        await updateFlags(sel.accountId, sel.uid, sel.flags);
+        await updateFlags(sel.accountId, sel.uid, sel.flags, sel.folderId);
         messageState.updateMessageFlags(sel.accountId, sel.uid, sel.flags);
         // Row owns its own DOM \u2014 go through the row object so class + star
         // update atomically and the list/preview stay in sync.
@@ -2075,7 +2075,7 @@ document.getElementById("btn-mark-unread")?.addEventListener("click", () => {
         return;
     const wasSeen = seenOf(sel);
     setSeen(sel, !wasSeen);
-    updateFlags(sel.accountId, sel.uid, sel.flags).then(() => {
+    updateFlags(sel.accountId, sel.uid, sel.flags, sel.folderId).then(() => {
         messageState.updateMessageFlags(sel.accountId, sel.uid, sel.flags);
         const row = document.querySelector(`.ml-row[data-uid="${sel.uid}"][data-account-id="${sel.accountId}"]`);
         if (row)
@@ -3682,7 +3682,10 @@ document.addEventListener("keydown", (e) => {
         const mlBody = document.getElementById("ml-body");
         if (mlBody) {
             e.preventDefault();
-            mlBody.querySelectorAll(".ml-row").forEach(r => r.classList.add("selected"));
+            // selectAllVisible() also sets `multi-select-on` so a following ★
+            // click clears the WHOLE selection, not just the clicked row
+            // (Bob 2026-06-26). Plain add-.selected was the bug.
+            selectAllVisible();
         }
     }
     // Ctrl+D or Delete = Delete selected messages.
@@ -3739,7 +3742,7 @@ document.addEventListener("keydown", (e) => {
         e.preventDefault();
         const applySeen = (m, seen) => {
             setSeen(m, seen);
-            updateFlags(m.accountId, m.uid, m.flags).then(() => {
+            updateFlags(m.accountId, m.uid, m.flags, m.folderId).then(() => {
                 messageState.updateMessageFlags(m.accountId, m.uid, m.flags);
                 const row = document.querySelector(`.ml-row[data-uid="${m.uid}"][data-account-id="${CSS.escape(m.accountId)}"]`);
                 if (row)
@@ -5954,13 +5957,18 @@ function saveBootSnapshot() {
         // (e.g. user clicked a folder, list is empty for a tick) would
         // otherwise clobber the previous good snapshot.
         const ftHtml = folderTree?.innerHTML?.trim() || "";
-        const mlHtml = messageList?.innerHTML?.trim() || "";
+        // Search is ephemeral: never snapshot the list while a search is
+        // showing, or a cold start would repaint stale search results before
+        // the real folder/inbox loads (Bob 2026-06-26). Empty string = the
+        // loader skips this region and the normal boot load fills it.
+        const searching = (searchInput?.value.trim() || "") !== "";
+        const mlHtml = searching ? "" : (messageList?.innerHTML?.trim() || "");
         if (!ftHtml && !mlHtml)
             return;
         const snap = {
             folderTree: ftHtml,
             messageList: mlHtml,
-            folderTitle: folderTitle?.innerHTML?.trim() || "",
+            folderTitle: searching ? "" : (folderTitle?.innerHTML?.trim() || ""),
             savedAt: Date.now(),
         };
         localStorage.setItem("mailx-boot-snapshot", JSON.stringify(snap));