@bobfrankston/rmfmail 1.2.102 → 1.2.105

package/client/app.js

@@ -2,8 +2,8 @@
  * mailx client entry point.
  * 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, revealMessage, selectAllVisible, exitMultiSelect, getDateBasis, setDateBasis } from "./components/message-list.js";
+import { initFolderTree, refreshFolderTree, updateFolderCounts, setFolderSynced, getFolderSynced, setOutboxTotal, highlightFolder } 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 { 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, wrapHtmlBody } from "./components/message-viewer.js";
@@ -2637,21 +2637,42 @@ let lastListReloadAt = 0;
 // BEFORE clearing search mode, then navigate to that message's folder and
 // reselect it. With no selected result (or no folderId) fall back to the old
 // reloadCurrentFolder behavior.
-function closeSearchAndReveal() {
+async function closeSearchAndReveal() {
     const focused = getCurrentFocused();
     clearSearchMode();
     const body = document.getElementById("ml-body");
     if (body)
         body.querySelectorAll(".filter-hidden").forEach(r => r.classList.remove("filter-hidden"));
     if (focused && focused.folderId) {
-        currentAccountId = focused.accountId;
-        currentFolderId = focused.folderId;
-        revealMessage(focused.accountId, focused.folderId, focused.uid);
+        await revealMessage(focused.accountId, focused.folderId, focused.uid);
     }
     else {
         reloadCurrentFolder();
     }
-    setTitle(APP_NAME);
+    // Leaving search must leave search STATE behind too. Before this, the
+    // active tab kept its "Search: …" view (re-running the dead query on any
+    // tab re-activation) and the folder tree kept the pre-search highlight —
+    // so when the reveal landed somewhere unexpected (a Drafts result, a
+    // stale folderId after a server-side rebind), every indicator still
+    // claimed search-on-All-Inboxes and the list looked broken (Bob
+    // 2026-07-04 "cleared the search but it isn't restoring the list").
+    // Sync tab, tree highlight and titles to what the list actually shows.
+    const vw = getCurrentView();
+    if (vw.unified) {
+        highlightFolder(null, -1);
+        setActiveTabView({ kind: "unified" }, "All Inboxes");
+        setTitle(`${APP_NAME} - All Inboxes`);
+        setNarrowFolderTitle("All Inboxes");
+    }
+    else {
+        currentAccountId = vw.accountId;
+        currentFolderId = vw.folderId;
+        currentFolderSpecialUse = vw.specialUse;
+        const name = highlightFolder(vw.accountId, vw.folderId) || "Mail";
+        setActiveTabView({ kind: "folder", accountId: vw.accountId, folderId: vw.folderId, specialUse: vw.specialUse }, name);
+        setTitle(`${APP_NAME} - ${name}`);
+        setNarrowFolderTitle(name);
+    }
 }
 searchInput?.addEventListener("input", () => {
     clearTimeout(searchTimeout);
@@ -3545,6 +3566,23 @@ onWsEvent((event) => {
             }).catch((e) => console.error("openComposeFromMailto failed:", e?.message || e));
             break;
         }
+        case "popoutAction": {
+            // Toolbar click inside a NATIVE popout window (msger, spawned by
+            // the daemon). That window has no IPC bridge and can't host
+            // compose, so the daemon forwards the action here; we load the
+            // message locally and reuse the overlay-popout action path —
+            // reply/replyAll/forward/editAsNew open the real editable
+            // compose, flag/delete act directly.
+            const { action, accountId, uid, folderId } = event;
+            if (!action || !accountId || !uid)
+                break;
+            (async () => {
+                const { getMessage } = await import("./lib/api-client.js");
+                const msg = await getMessage(accountId, uid, false, folderId);
+                document.dispatchEvent(new CustomEvent("mailx-popout-action", { detail: { action, msg, accountId } }));
+            })().catch((e) => console.error(`[popout-window] ${action} failed: ${e?.message || e}`));
+            break;
+        }
     }
 });
 // Store-bus consumers (post-refactor). The list's mailx-remove-stale
@@ -5868,11 +5906,11 @@ function renderDiagnosticsBadge(snapshot) {
     }).join("\n");
     host.title = `Connection issues — ${summary}\n\n${detail}`;
 }
-// Q64: pop-out a message into a floating overlay (real-OS-window pending C44).
-// Action buttons on the message pop-out window — Reply / Reply All / Forward
-// act on the popped-out message specifically (not the main viewer's), Delete
-// removes it. The pop-out lives in message-viewer.ts and can't import
-// openCompose directly, so it fires this event.
+// Q64/C44: pop-out message actions. Fired by the floating-overlay popout's
+// toolbar (message-viewer.ts can't import openCompose directly) AND by the
+// native popout window via the `popoutAction` daemon event above. Reply /
+// Reply All / Forward act on the popped-out message specifically (not the
+// main viewer's), Delete removes it.
 document.addEventListener("mailx-popout-action", ((e) => {
     const { action, msg, accountId } = e.detail || {};
     if (!msg)