@bobfrankston/rmfmail 1.2.335 → 1.2.337

package/client/app.js

@@ -3,7 +3,7 @@
  * Wires together all UI components and WebSocket connection.
  */
 import { initFolderTree, refreshFolderTree, updateFolderCounts, setFolderSynced, getFolderSynced, setOutboxTotal, highlightFolder, setFolderFilter } from "./components/folder-tree.js";
-import { initMessageList, loadMessages, loadUnifiedInbox, loadSearchResults, reloadCurrentFolder, clearSearchMode, getSelectedMessages, markBodiesCached, getCurrentFocused, releaseFocus, removeMessagesAndReconcile, setRowFlagged, scrollFocusedIntoView, refreshPriorityIndex, revealMessage, getCurrentView, selectAllVisible, exitMultiSelect, getDateBasis, setDateBasis } from "./components/message-list.js";
+import { initMessageList, loadMessages, loadUnifiedInbox, loadSearchResults, reloadCurrentFolder, clearSearchMode, getSelectedMessages, markBodiesCached, getCurrentFocused, releaseFocus, removeMessagesAndReconcile, setRowFlagged, scrollFocusedIntoView, refreshPriorityIndex, revealMessage, getCurrentView, selectAllMessages, exitMultiSelect, getDateBasis, setDateBasis } 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, popOutToWindow, printCurrentMessage, toggleFullscreenPreview, showPreviewBodyMenu, setSearchHighlightTerms, parseHighlightTerms } from "./components/message-viewer.js";
@@ -2955,7 +2955,11 @@ function doSearch(immediate = false) {
     if (serverOn) {
         serverSearchTimer = setTimeout(() => {
             serverSearchTimer = null;
-            loadSearchResults(query, "server", currentAccountId, currentFolderId, includeTrash);
+            // 2026-09-16: the scope dropdown applies to the server pass too —
+            // "This folder" sweeps only the current folder; "All folders"
+            // sends folderId 0, which the service reads as every folder.
+            const serverFolderId = localScopeEff === "current" ? currentFolderId : 0;
+            loadSearchResults(query, "server", currentAccountId, serverFolderId, includeTrash);
         }, SERVER_SEARCH_DALLY_MS);
     }
 }
@@ -4166,10 +4170,12 @@ document.addEventListener("keydown", (e) => {
         const mlBody = document.getElementById("ml-body");
         if (mlBody) {
             e.preventDefault();
-            // 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();
+            // selectAllMessages() loads the remaining pages first, then sets
+            // `multi-select-on` + selects — so a following ★ click clears the
+            // WHOLE selection, not just the clicked row (Bob 2026-06-26), and
+            // the selection covers every message in the view, not the 50-row
+            // pages loaded so far (Bob 2026-09-16).
+            void selectAllMessages();
         }
     }
     // Ctrl+D or Delete = Delete selected messages.