@bobfrankston/rmfmail 1.0.680 → 1.0.686

package/bin/build-quill.js

@@ -0,0 +1,35 @@
+#!/usr/bin/env node
+// Copy node_modules/quill/dist/{quill.js,quill.snow.css} → client/lib/quill/
+// so compose can load Quill from disk instead of jsdelivr CDN. The CDN
+// fetch was a 100-500 ms (network-dependent) tax on every compose open,
+// since iframes ran a fresh `await loadScript(jsdelivr…)` chain even on
+// warm starts. Local copy = single msger-protocol fetch = ~30 ms.
+//
+// Same shape as build-tinymce.js (sibling script). Build runs after
+// `npm install`, before bundling, so the assets are present when the
+// browser first reaches them.
+import fs from "node:fs";
+import path from "node:path";
+
+const root = path.resolve(import.meta.dirname, "..");
+const srcDir = path.join(root, "node_modules", "quill", "dist");
+const dstDir = path.join(root, "client", "lib", "quill");
+
+if (!fs.existsSync(srcDir)) {
+    console.error(`build-quill: source not found at ${srcDir} — run npm install first`);
+    process.exit(1);
+}
+
+// Only the two files the editor actually loads — Quill's other dist
+// artifacts (bubble theme, core-only build, sourcemaps) aren't on the
+// runtime path and would just bloat the client tree.
+const files = ["quill.js", "quill.snow.css"];
+fs.mkdirSync(dstDir, { recursive: true });
+let totalBytes = 0;
+for (const name of files) {
+    const s = path.join(srcDir, name);
+    const d = path.join(dstDir, name);
+    fs.copyFileSync(s, d);
+    totalBytes += fs.statSync(d).size;
+}
+console.log(`build-quill: copied ${files.length} files (${(totalBytes / 1024).toFixed(1)} KB) → client/lib/quill/`);

package/bin/lean-accounts.js

@@ -98,7 +98,6 @@ function denormalizeAccount(acct, globalName) {
     if (acct.defaultSend) out.defaultSend = true;
     if (acct.enabled === false) out.enabled = false;
     if (acct.relayDomains?.length > 0) out.relayDomains = acct.relayDomains;
-    if (acct.deliveredToPrefix?.length > 0) out.deliveredToPrefix = acct.deliveredToPrefix;
     if (acct.identityDomains?.length > 0) out.identityDomains = acct.identityDomains;
 
     const syncContactsDefault = provider?.imap.auth === "oauth2";

package/client/app.bundle.js

@@ -62,6 +62,7 @@ __export(api_client_exports, {
   logClientEvent: () => logClientEvent,
   markAsSpamMessages: () => markAsSpamMessages,
   markFolderRead: () => markFolderRead,
+  moveFolderToTrash: () => moveFolderToTrash,
   moveMessage: () => moveMessage,
   moveMessages: () => moveMessages,
   onEvent: () => onEvent,
@@ -334,6 +335,9 @@ function renameFolder(accountId, folderId, newName) {
 function deleteFolder(accountId, folderId) {
   return ipc().deleteFolder?.(accountId, folderId);
 }
+function moveFolderToTrash(accountId, folderId) {
+  return ipc().moveFolderToTrash?.(accountId, folderId);
+}
 function emptyFolder(accountId, folderId) {
   return ipc().emptyFolder?.(accountId, folderId);
 }
@@ -489,6 +493,8 @@ function showContextMenu(x, y, items) {
     const el = document.createElement("div");
     el.className = "ctx-item" + (item.disabled ? " ctx-disabled" : "");
     el.textContent = item.label;
+    if (item.tooltip)
+      el.title = item.tooltip;
     if (!item.disabled) {
       el.addEventListener("click", () => {
         closeContextMenu();
@@ -2164,6 +2170,57 @@ function cacheKey(mode, a, f, flagged, q) {
     return `folder:${a}:${f}:${flagged ? "flag" : ""}`;
   return `search:${q}`;
 }
+function persistPositions() {
+  try {
+    const obj = {};
+    for (const [k, v] of positionMemory)
+      obj[k] = v;
+    sessionStorage.setItem(POSITION_STORAGE_KEY, JSON.stringify(obj));
+  } catch {
+  }
+}
+function currentViewKey() {
+  if (searchMode)
+    return cacheKey("search", void 0, void 0, void 0, currentSearchQuery);
+  if (unifiedMode)
+    return CACHE_KEY_UNIFIED;
+  if (!currentAccountId2 || currentFolderId == null)
+    return null;
+  const flaggedOnly = document.getElementById("ml-body")?.classList.contains("flagged-only") || false;
+  return cacheKey("folder", currentAccountId2, currentFolderId, flaggedOnly);
+}
+function rememberPosition() {
+  const key = currentViewKey();
+  if (!key)
+    return;
+  const body = document.getElementById("ml-body");
+  if (!body)
+    return;
+  const sel = body.querySelector(".ml-row.selected");
+  if (!sel)
+    return;
+  const uid = Number(sel.dataset.uid);
+  if (!Number.isFinite(uid))
+    return;
+  positionMemory.set(key, { uid, scroll: body.scrollTop });
+  persistPositions();
+}
+function pickRestoreUid(items, saved) {
+  if (!items.length)
+    return null;
+  if (items.some((m) => m.uid === saved))
+    return saved;
+  let best = -1;
+  for (const m of items) {
+    if (typeof m.uid !== "number")
+      continue;
+    if (m.uid < saved && m.uid > best)
+      best = m.uid;
+  }
+  if (best >= 0)
+    return best;
+  return typeof items[0]?.uid === "number" ? items[0].uid : null;
+}
 function listResultsEqual(a, b) {
   if (!a || a.length !== b.length)
     return false;
@@ -2210,6 +2267,7 @@ function focusRow(row) {
   showMessage(row.accountId, row.msg.uid, row.msg.folderId, void 0, false, row.msg);
   onMessageSelect(row.accountId, row.msg.uid, row.msg.folderId);
   document.dispatchEvent(new CustomEvent("mailx-focus-changed", { detail: row.msg }));
+  rememberPosition();
 }
 function getCurrentFocused() {
   return focusedRow ? focusedRow.msg : null;
@@ -2467,19 +2525,19 @@ async function loadUnifiedInbox(autoSelect = true) {
   const fromHeader = document.querySelector(".ml-col-from");
   if (fromHeader)
     fromHeader.textContent = "From";
-  const savedScroll = !autoSelect ? body.scrollTop : 0;
-  const currentSelectedUid = () => !autoSelect ? body.querySelector(".ml-row.selected")?.getAttribute("data-uid") || null : null;
+  const remembered = positionMemory.get(CACHE_KEY_UNIFIED);
+  const savedScroll = remembered?.scroll ?? (!autoSelect ? body.scrollTop : 0);
   const cached = listCache.get(CACHE_KEY_UNIFIED);
   if (cached) {
-    const preCacheUid = currentSelectedUid();
     totalMessages = cached.total;
     setMessages(cached.items);
     renderMessages(body, "", cached.items);
-    if (autoSelect)
-      selectFirst(body);
-    else {
+    const targetUid = remembered ? pickRestoreUid(cached.items, remembered.uid) : null;
+    if (targetUid != null) {
       body.scrollTop = savedScroll;
-      restoreSelection(body, preCacheUid);
+      restoreSelection(body, String(targetUid));
+    } else if (autoSelect) {
+      selectFirst(body);
     }
   } else if (autoSelect) {
     body.innerHTML = `<div class="ml-empty">Loading...</div>`;
@@ -2497,14 +2555,14 @@ async function loadUnifiedInbox(autoSelect = true) {
       body.innerHTML = `<div class="ml-empty">${result.total > 0 ? `${result.total} messages syncing...` : "Syncing \u2014 messages will appear shortly"}</div>`;
       return;
     }
-    const preRenderUid = currentSelectedUid();
     setMessages(result.items);
     renderMessages(body, "", result.items);
-    if (autoSelect) {
-      selectFirst(body);
-    } else {
+    const targetUid = remembered ? pickRestoreUid(result.items, remembered.uid) : null;
+    if (targetUid != null) {
       body.scrollTop = savedScroll;
-      restoreSelection(body, preRenderUid);
+      restoreSelection(body, String(targetUid));
+    } else if (autoSelect) {
+      selectFirst(body);
     }
   } catch (e) {
     if (e.name === "AbortError")
@@ -2604,23 +2662,23 @@ async function loadMessages(accountId, folderId, page = 1, specialUse = "", auto
   const fromHeader = document.querySelector(".ml-col-from");
   if (fromHeader)
     fromHeader.textContent = showToInsteadOfFrom ? "To" : "From";
-  const savedScroll = !autoSelect ? body.scrollTop : 0;
-  const currentSelectedUid = () => !autoSelect ? body.querySelector(".ml-row.selected")?.getAttribute("data-uid") || null : null;
   const flaggedOnly = document.getElementById("ml-body")?.classList.contains("flagged-only") || false;
   const cKey = cacheKey("folder", accountId, folderId, flaggedOnly);
+  const remembered = positionMemory.get(cKey);
+  const savedScroll = remembered?.scroll ?? (!autoSelect ? body.scrollTop : 0);
   const cached = listCache.get(cKey);
   if (cached) {
-    const preCacheUid = currentSelectedUid();
     totalMessages = cached.total;
     setMessages(cached.items);
     renderMessages(body, accountId, cached.items);
-    if (autoSelect)
-      selectFirst(body);
-    else {
+    const targetUid = remembered ? pickRestoreUid(cached.items, remembered.uid) : null;
+    if (targetUid != null) {
       requestAnimationFrame(() => {
         body.scrollTop = savedScroll;
-        restoreSelection(body, preCacheUid);
+        restoreSelection(body, String(targetUid));
       });
+    } else if (autoSelect) {
+      selectFirst(body);
     }
   } else if (autoSelect) {
     body.innerHTML = `<div class="ml-empty">Loading...</div>`;
@@ -2639,18 +2697,18 @@ async function loadMessages(accountId, folderId, page = 1, specialUse = "", auto
       body.innerHTML = `<div class="ml-empty">${flaggedOnly ? "No flagged messages" : "No messages"}</div>`;
       return;
     }
-    const preRenderUid = currentSelectedUid();
     setMessages(result.items);
     renderMessages(body, accountId, result.items);
-    if (autoSelect) {
-      selectFirst(body);
-    } else {
+    const targetUid = remembered ? pickRestoreUid(result.items, remembered.uid) : null;
+    if (targetUid != null) {
       requestAnimationFrame(() => {
         if (myGen !== loadGen)
           return;
         body.scrollTop = savedScroll;
-        restoreSelection(body, preRenderUid);
+        restoreSelection(body, String(targetUid));
       });
+    } else if (autoSelect) {
+      selectFirst(body);
     }
   } catch (e) {
     if (e.name === "AbortError")
@@ -2812,7 +2870,7 @@ function escapeHtml2(s) {
   div.textContent = s;
   return div.innerHTML;
 }
-var onMessageSelect, currentAccountId2, currentFolderId, currentSpecialUse, lastClickedRow, currentPage, totalMessages, loading, unifiedMode, searchMode, currentSearchQuery, wasUnifiedBeforeSearch, showToInsteadOfFrom, touchWasScroll, currentSort, currentSortDir, loadGen, listCache, CACHE_KEY_UNIFIED, focusedRow, rowByKey, prioritySenders, priorityDomains, timeFmt, dateFmt, dateFmtSameYear, MessageRow;
+var onMessageSelect, currentAccountId2, currentFolderId, currentSpecialUse, lastClickedRow, currentPage, totalMessages, loading, unifiedMode, searchMode, currentSearchQuery, wasUnifiedBeforeSearch, showToInsteadOfFrom, touchWasScroll, currentSort, currentSortDir, loadGen, listCache, CACHE_KEY_UNIFIED, positionMemory, POSITION_STORAGE_KEY, focusedRow, rowByKey, prioritySenders, priorityDomains, timeFmt, dateFmt, dateFmtSameYear, MessageRow;
 var init_message_list = __esm({
   "client/components/message-list.js"() {
     "use strict";
@@ -2835,6 +2893,19 @@ var init_message_list = __esm({
     loadGen = 0;
     listCache = /* @__PURE__ */ new Map();
     CACHE_KEY_UNIFIED = "unified";
+    positionMemory = /* @__PURE__ */ new Map();
+    POSITION_STORAGE_KEY = "mailx-list-positions";
+    try {
+      const raw = sessionStorage.getItem(POSITION_STORAGE_KEY);
+      if (raw) {
+        const parsed = JSON.parse(raw);
+        for (const [k, v] of Object.entries(parsed || {})) {
+          if (typeof v?.uid === "number")
+            positionMemory.set(k, v);
+        }
+      }
+    } catch {
+    }
     focusedRow = null;
     rowByKey = /* @__PURE__ */ new Map();
     prioritySenders = /* @__PURE__ */ new Set();
@@ -4918,18 +4989,47 @@ function renderNode(node, container, depth) {
           alert(`Failed: ${err.message}`);
         }
       }, disabled: !!node.specialUse },
-      { label: "Delete folder", action: async () => {
-        if (!confirm(`Delete folder "${node.name}"? Messages will be moved to Trash.`))
-          return;
-        try {
-          await deleteFolder(node.accountId, node.id);
-          const treeContainer = document.getElementById("folder-tree");
-          if (treeContainer)
-            loadFolderTree(treeContainer);
-        } catch (err) {
-          alert(`Failed: ${err.message}`);
-        }
-      }, disabled: !!node.specialUse },
+      // Two delete entries. Move-to-Trash is the default; permanent
+      // delete is a second, separated item with a tooltip so it's
+      // visible from the menu rather than requiring a discovery
+      // shortcut. The IMAP RENAME under Trash brings messages +
+      // subfolders along; the server-side fallback (when Trash is
+      // \Noinferiors) moves messages to Trash root and then deletes
+      // the empty folder. Permanent skips Trash entirely.
+      {
+        label: "Move folder to Trash",
+        action: async () => {
+          if (!confirm(`Move folder "${node.name}" to Trash? It can be restored by dragging back out of Trash.`))
+            return;
+          try {
+            await moveFolderToTrash(node.accountId, node.id);
+            const treeContainer = document.getElementById("folder-tree");
+            if (treeContainer)
+              loadFolderTree(treeContainer);
+          } catch (err) {
+            alert(`Failed: ${err.message}`);
+          }
+        },
+        disabled: !!node.specialUse,
+        tooltip: "Renames the folder into Trash (date-suffixed if needed); use Delete permanently below to skip Trash."
+      },
+      {
+        label: "Delete folder permanently",
+        action: async () => {
+          if (!confirm(`Permanently delete folder "${node.name}" and ALL its messages? This cannot be undone.`))
+            return;
+          try {
+            await deleteFolder(node.accountId, node.id);
+            const treeContainer = document.getElementById("folder-tree");
+            if (treeContainer)
+              loadFolderTree(treeContainer);
+          } catch (err) {
+            alert(`Failed: ${err.message}`);
+          }
+        },
+        disabled: !!node.specialUse,
+        tooltip: "Skips Trash. Same as Shift+Delete on a regular file. No undo."
+      },
       { label: "", action: () => {
       }, separator: true },
       // Q57: copy IMAP path so user can paste into accounts.jsonc as
@@ -5758,9 +5858,11 @@ function updateBadge(count) {
 async function updateNewMessageCount() {
   try {
     const accounts = await getAccounts();
+    const folderLists = await Promise.all(
+      accounts.map((acct) => getFolders(acct.id).catch(() => []))
+    );
     let totalUnread = 0;
-    for (const acct of accounts) {
-      const folders = await getFolders(acct.id);
+    for (const folders of folderLists) {
       const inbox = folders.find((f) => f.specialUse === "inbox");
       if (inbox) totalUnread += inbox.unreadCount || 0;
     }
@@ -5822,9 +5924,11 @@ document.addEventListener("visibilitychange", () => {
 window.addEventListener("focus", stopTitleFlash);
 function markAsSeen() {
   getAccounts().then(async (accounts) => {
+    const folderLists = await Promise.all(
+      accounts.map((acct) => getFolders(acct.id).catch(() => []))
+    );
     let total = 0;
-    for (const acct of accounts) {
-      const folders = await getFolders(acct.id);
+    for (const folders of folderLists) {
       const inbox = folders.find((f) => f.specialUse === "inbox");
       if (inbox) total += inbox.unreadCount || 0;
     }
@@ -6166,9 +6270,13 @@ async function openCompose(mode) {
     console.warn(`[compose] ${mode} \u2014 no message selected`);
     return;
   }
-  const accounts = await getAccounts();
-  const accountId = current?.accountId || accounts[0]?.id || "";
+  const accountsP = getAccounts();
   const msg = current?.message;
+  const titlePrefix = mode === "reply" ? "Reply" : mode === "replyAll" ? "Reply All" : mode === "forward" ? "Forward" : "Compose";
+  const titleSubject = mode === "new" ? "" : msg?.subject || "";
+  const frame = showComposeOverlay(titleSubject ? `${titlePrefix}: ${titleSubject}` : titlePrefix);
+  const accounts = await accountsP;
+  const accountId = current?.accountId || accounts[0]?.id || "";
   const rePrefix = /^(re|fwd?):\s*/i;
   const cleanSubject = msg ? msg.subject.replace(rePrefix, "") : "";
   const init = {
@@ -6180,7 +6288,7 @@ async function openCompose(mode) {
     bodyHtml: "",
     inReplyTo: "",
     references: [],
-    accounts: accounts.map((a) => ({ id: a.id, name: a.name, email: a.email, signature: a.signature }))
+    accounts: accounts.map((a) => ({ id: a.id, name: a.name, email: a.email, signature: a.signature, sig: a.sig }))
   };
   const account = accounts.find((a) => a.id === accountId);
   const explicitDomains = (account?.identityDomains || []).map((d) => d.toLowerCase());
@@ -6234,9 +6342,10 @@ async function openCompose(mode) {
     init.fromAddress = detectReplyFrom();
   }
   sessionStorage.setItem("composeInit", JSON.stringify(init));
-  const titlePrefix = mode === "reply" ? "Reply" : mode === "replyAll" ? "Reply All" : mode === "forward" ? "Forward" : "Compose";
-  const titleSubject = mode === "new" ? "" : msg?.subject || init.subject || "";
-  showComposeOverlay(titleSubject ? `${titlePrefix}: ${titleSubject}` : titlePrefix);
+  try {
+    frame?.contentWindow?.postMessage({ type: "compose-init-ready" }, "*");
+  } catch {
+  }
 }
 function showComposeOverlay(title = "Compose") {
   const wrapper = document.createElement("div");
@@ -6357,6 +6466,7 @@ function showComposeOverlay(title = "Compose") {
   wrapper.appendChild(frame);
   if (!isSmall) addComposeResizeHandles(wrapper, frame);
   document.body.appendChild(wrapper);
+  return frame;
 }
 function installDragShield(cursor) {
   const shield = document.createElement("div");
@@ -6425,7 +6535,7 @@ function addComposeResizeHandles(wrapper, frame) {
 function sanitizeQuotedBody(msg) {
   const isPlainText = !msg.bodyHtml;
   if (isPlainText) {
-    const escaped = String(msg.bodyText || "").replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;");
+    const escaped = String(msg.bodyText || "").replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/\r\n?/g, "\n").replace(/\n/g, "<br>");
     return `<div style="white-space:pre-wrap;font-family:inherit;margin:0">${escaped}</div>`;
   }
   let body = msg.bodyHtml;
@@ -7574,7 +7684,9 @@ onWsEvent((event) => {
   }
 });
 async function openComposeFromMailto(m) {
-  const accounts = await getAccounts();
+  const accountsP = getAccounts();
+  const frame = showComposeOverlay(m.subject ? `Compose: ${m.subject}` : "Compose");
+  const accounts = await accountsP;
   const accountId = accounts[0]?.id || "";
   const escape = (s) => s.replace(/[&<>]/g, (c) => ({ "&": "&amp;", "<": "&lt;", ">": "&gt;" })[c]);
   const bodyHtml = m.body ? "<p>" + escape(m.body).replace(/\r?\n/g, "</p><p>") + "</p>" : "";
@@ -7588,10 +7700,13 @@ async function openComposeFromMailto(m) {
     bodyHtml,
     inReplyTo: m.inReplyTo,
     references: m.inReplyTo ? [m.inReplyTo] : [],
-    accounts: accounts.map((a) => ({ id: a.id, name: a.name, email: a.email, signature: a.signature }))
+    accounts: accounts.map((a) => ({ id: a.id, name: a.name, email: a.email, signature: a.signature, sig: a.sig }))
   };
   sessionStorage.setItem("composeInit", JSON.stringify(init));
-  showComposeOverlay(m.subject ? `Compose: ${m.subject}` : "Compose");
+  try {
+    frame?.contentWindow?.postMessage({ type: "compose-init-ready" }, "*");
+  } catch {
+  }
 }
 window.addEventListener("keydown", (e) => {
   if (!e.ctrlKey || e.altKey || e.metaKey) return;
@@ -8955,16 +9070,18 @@ function renderOutboxStatus(s) {
 setInterval(async () => {
   try {
     const { getOutboxStatus: getOutboxStatus2, getDiagnostics: getDiagnostics2 } = await Promise.resolve().then(() => (init_api_client(), api_client_exports));
-    renderOutboxStatus(await getOutboxStatus2());
-    renderDiagnosticsBadge(await getDiagnostics2());
+    const [outbox, diag] = await Promise.all([getOutboxStatus2(), getDiagnostics2()]);
+    renderOutboxStatus(outbox);
+    renderDiagnosticsBadge(diag);
   } catch {
   }
 }, 15e3);
 (async () => {
   try {
     const { getOutboxStatus: getOutboxStatus2, getDiagnostics: getDiagnostics2 } = await Promise.resolve().then(() => (init_api_client(), api_client_exports));
-    renderOutboxStatus(await getOutboxStatus2());
-    renderDiagnosticsBadge(await getDiagnostics2());
+    const [outbox, diag] = await Promise.all([getOutboxStatus2(), getDiagnostics2()]);
+    renderOutboxStatus(outbox);
+    renderDiagnosticsBadge(diag);
   } catch {
   }
 })();