@bobfrankston/rmfmail 1.1.199 → 1.1.201

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 } from "./components/message-list.js";
+import { initMessageList, loadMessages, loadUnifiedInbox, loadSearchResults, reloadCurrentFolder, clearSearchMode, setLiveFilter, getSelectedMessages, markBodiesCached, getCurrentFocused, releaseFocus, removeMessagesAndReconcile, setRowFlagged, scrollFocusedIntoView, refreshPriorityIndex } 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";
@@ -860,7 +860,7 @@ async function openCompose(mode, overrideMsg, overrideAccountId) {
     // anything missing. Don't show "still loading" alerts — the message IS
     // loaded (it's in the list), body is a separate fetch that isn't needed
     // for Reply's headers. Missing fields become empty strings.
-    if ((mode === "reply" || mode === "replyAll" || mode === "forward") && !current) {
+    if ((mode === "reply" || mode === "replyAll" || mode === "forward" || mode === "editAsNew") && !current) {
         // Only true blocker: no message selected at all.
         console.warn(`[compose] ${mode} — no message selected`);
         return;
@@ -881,7 +881,8 @@ async function openCompose(mode, overrideMsg, overrideAccountId) {
     const titlePrefix = mode === "reply" ? "Reply" :
         mode === "replyAll" ? "Reply All" :
             mode === "forward" ? "Forward" :
-                "Compose";
+                mode === "editAsNew" ? "Edit as new" :
+                    "Compose";
     const titleSubject = mode === "new" ? "" : (msg?.subject || "");
     const frame = showComposeOverlay(titleSubject ? `${titlePrefix}: ${titleSubject}` : titlePrefix);
     // Now finish initialisation off the critical path — editor bootstrap
@@ -1032,6 +1033,18 @@ async function openCompose(mode, overrideMsg, overrideAccountId) {
         init.bodyHtml = forwardBody(msg);
         init.fromAddress = detectReplyFrom();
     }
+    else if (msg && mode === "editAsNew") {
+        // Edit-as-new clones an existing message into a fresh compose as if
+        // the user authored it: original To/Cc/Subject/body, all editable.
+        // No `Re:`/`Fwd:` prefix (subject verbatim), no quote/forward wrapper,
+        // no In-Reply-To / References (this is NOT a reply — it gets a brand-
+        // new Message-ID at send time). Replaces the move-to-Drafts kludge for
+        // "tweak this and send it again" (Q152, Bob 2026-05-31).
+        init.to = Array.isArray(msg.to) ? msg.to : [];
+        init.cc = Array.isArray(msg.cc) ? msg.cc : [];
+        init.subject = msg.subject || "";
+        init.bodyHtml = editAsNewBody(msg);
+    }
     // Store init data for compose window to pick up. Two parallel paths
     // because sessionStorage propagation to a freshly-loading iframe has
     // intermittently failed under WebView2's custom-protocol host (Bob
@@ -1393,6 +1406,13 @@ function quoteBody(msg) {
     // inside it.
     return `<p></p><br><br><div class="reply"><p>On ${date}, ${from} wrote:</p><blockquote>${body}</blockquote></div>`;
 }
+function editAsNewBody(msg) {
+    // The message body verbatim (same plain-text/HTML cleaning a quoted reply
+    // gets), with NO attribution line and NO quote/forward wrapper — the user
+    // is re-authoring this content, not quoting it. A leading empty paragraph
+    // gives the caret a block to land in above the cloned content.
+    return `<p></p>${sanitizeQuotedBody(msg)}`;
+}
 function forwardBody(msg) {
     const date = new Date(msg.date).toLocaleString();
     const from = msg.from.name ? `${msg.from.name} &lt;${msg.from.address}&gt;` : msg.from.address;
@@ -2220,14 +2240,12 @@ function doSearch(immediate = false) {
     // Only when the server checkbox is OFF — with it on we want the real
     // local+server search, not a row filter.
     if (localScope === "current" && !serverOn && !immediate) {
-        const body = document.getElementById("ml-body");
-        if (body) {
-            const lower = query.toLowerCase();
-            for (const row of body.querySelectorAll(".ml-row")) {
-                const text = row.textContent?.toLowerCase() || "";
-                row.classList.toggle("filter-hidden", !text.includes(lower));
-            }
-        }
+        // Register as a live list filter (not a raw DOM toggle) so it survives
+        // background sync reloads and blocks reloadCurrentFolder — otherwise a
+        // sync mid-read wipes the filter, deselects the open message, and pops
+        // back to the full list while the search box still shows the query
+        // (Bob 2026-05-31). setLiveFilter applies the hide to current rows.
+        setLiveFilter(query);
         return;
     }
     // ── Phase 1: LOCAL search — always, immediately ──