@bobfrankston/rmfmail 1.0.679 → 1.0.681

package/client/app.ts

@@ -171,9 +171,15 @@ function updateBadge(count: number): void {
 async function updateNewMessageCount(): Promise<void> {
     try {
         const accounts = await getAccounts();
+        // Fan out folder queries in parallel — earlier code awaited each
+        // account's `getFolders` in series, so an N-account setup paid N
+        // back-to-back IPC round-trips on every count refresh (folderCountsChanged,
+        // sync events, IDLE updates).
+        const folderLists = await Promise.all(
+            accounts.map((acct: any) => getFolders(acct.id).catch(() => [] as any[])),
+        );
         let totalUnread = 0;
-        for (const acct of accounts) {
-            const folders = await getFolders(acct.id);
+        for (const folders of folderLists) {
             const inbox = folders.find((f: any) => f.specialUse === "inbox");
             if (inbox) totalUnread += inbox.unreadCount || 0;
         }
@@ -242,9 +248,12 @@ window.addEventListener("focus", stopTitleFlash);
 /** Call when user actively views messages — resets the badge */
 function markAsSeen(): void {
     getAccounts().then(async (accounts: any[]) => {
+        // Parallel folder fetch — see updateNewMessageCount for rationale.
+        const folderLists = await Promise.all(
+            accounts.map((acct: any) => getFolders(acct.id).catch(() => [] as any[])),
+        );
         let total = 0;
-        for (const acct of accounts) {
-            const folders = await getFolders(acct.id);
+        for (const folders of folderLists) {
             const inbox = folders.find((f: any) => f.specialUse === "inbox");
             if (inbox) total += inbox.unreadCount || 0;
         }
@@ -718,9 +727,30 @@ async function openCompose(mode: ComposeMode): Promise<void> {
         console.warn(`[compose] ${mode} — no message selected`);
         return;
     }
-    const accounts = await getAccounts();
-    const accountId = current?.accountId || accounts[0]?.id || "";
+    // Parallel-load: kick off getAccounts AND open the iframe in the same
+    // tick. The iframe doesn't need the account list until after its editor
+    // bootstraps (200-500 ms for TinyMCE, less for Quill); by then the IPC
+    // round-trip has resolved. Earlier code awaited getAccounts FIRST,
+    // adding the IPC latency to the perceived Ctrl+N → editor-visible time.
+    // We post `compose-init-ready` to the iframe once init is in
+    // sessionStorage so compose.ts's IIFE can read synchronously without
+    // polling.
+    const accountsP = getAccounts();
     const msg = current?.message;
+    // Title bar text needs the subject from msg (no IPC dependency) — build
+    // it now so the iframe can be opened with the final title and avoid a
+    // flash of placeholder text.
+    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);
+    // Now finish initialisation off the critical path — editor bootstrap
+    // inside the iframe runs concurrently with this await.
+    const accounts = await accountsP;
+    const accountId = current?.accountId || accounts[0]?.id || "";
     const rePrefix = /^(re|fwd?):\s*/i;
     const cleanSubject = msg ? msg.subject.replace(rePrefix, "") : "";
 
@@ -733,7 +763,7 @@ async function openCompose(mode: ComposeMode): Promise<void> {
         bodyHtml: "",
         inReplyTo: "",
         references: [],
-        accounts: accounts.map((a: any) => ({ id: a.id, name: a.name, email: a.email, signature: a.signature })),
+        accounts: accounts.map((a: any) => ({ id: a.id, name: a.name, email: a.email, signature: a.signature, sig: a.sig })),
     };
 
     // Auto-detect reply From: if the message was delivered to an identity address
@@ -806,23 +836,16 @@ async function openCompose(mode: ComposeMode): Promise<void> {
     }
 
 
-    // Store init data for compose window to pick up
+    // Store init data for compose window to pick up. sessionStorage is the
+    // canonical handoff path — same origin between parent and iframe; the
+    // compose IIFE reads it after the editor finishes booting. We also
+    // postMessage the iframe so it can short-circuit the listen-for-message
+    // wait if it's already past editor init.
     sessionStorage.setItem("composeInit", JSON.stringify(init));
-    // Inline compose: load compose.html in an overlay iframe (same origin, same IPC)
-    // Popup windows don't work in IPC mode (custom protocol doesn't propagate to child windows)
-    // Title reflects mode + subject so the user can see what they're replying to
-    // ("Re: Stars of STEM 2026" instead of just "Compose"). Forward shows the
-    // forward target subject; new compose stays generic.
-    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"): void {
+function showComposeOverlay(title = "Compose"): HTMLIFrameElement {
     const wrapper = document.createElement("div");
     wrapper.className = "compose-overlay";
     // Full-screen on small/short screens, floating on larger
@@ -972,6 +995,7 @@ function showComposeOverlay(title = "Compose"): void {
     wrapper.appendChild(frame);
     if (!isSmall) addComposeResizeHandles(wrapper, frame);
     document.body.appendChild(wrapper);
+    return frame;
 }
 
 /** Drop a transparent full-viewport shield in front of every other element
@@ -1071,6 +1095,12 @@ function sanitizeQuotedBody(msg: any): string {
     // future provider/path that didn't go through that pipeline.
     const isPlainText = !msg.bodyHtml;
     if (isPlainText) {
+        // Full HTML escape. Leaving `>` unescaped was tempting for source
+        // readability but breaks HTML in edge cases — TinyMCE's normalize-
+        // on-paste re-interprets the input, and stray `>` near sequences
+        // like `<!--` / `-->` / `<!` in plain-text bodies can be misread
+        // by the parser. Per Bob 2026-05-12: "not just ugly, it breaks
+        // the HTML." Trivial source-clutter is the lesser evil.
         const escaped = String(msg.bodyText || "")
             .replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;");
         return `<div style="white-space:pre-wrap;font-family:inherit;margin:0">${escaped}</div>`;
@@ -2519,7 +2549,12 @@ async function openComposeFromMailto(m: {
     to: string[]; cc: string[]; bcc: string[];
     subject: string; body: string; inReplyTo: string;
 }): Promise<void> {
-    const accounts = await getAccounts();
+    // Open the iframe immediately and load accounts in parallel — same
+    // pattern as openCompose. The mailto handler should never feel like
+    // "waiting for the system" to a user who clicked a link.
+    const accountsP = getAccounts();
+    const frame = showComposeOverlay(m.subject ? `Compose: ${m.subject}` : "Compose");
+    const accounts = await accountsP;
     const accountId = accounts[0]?.id || "";
     const escape = (s: string) => s.replace(/[&<>]/g, c =>
         ({ "&": "&amp;", "<": "&lt;", ">": "&gt;" }[c]!));
@@ -2536,10 +2571,10 @@ async function openComposeFromMailto(m: {
         bodyHtml,
         inReplyTo: m.inReplyTo,
         references: m.inReplyTo ? [m.inReplyTo] : [],
-        accounts: accounts.map((a: any) => ({ id: a.id, name: a.name, email: a.email, signature: a.signature })),
+        accounts: accounts.map((a: any) => ({ 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 { /* */ }
 }
 
 // ── Keyboard shortcuts ──
@@ -4090,16 +4125,20 @@ function renderOutboxStatus(s: any): void {
 setInterval(async () => {
     try {
         const { getOutboxStatus, getDiagnostics } = await import("./lib/api-client.js");
-        renderOutboxStatus(await getOutboxStatus());
-        renderDiagnosticsBadge(await getDiagnostics());
+        // Run in parallel — neither call depends on the other and each is a
+        // separate IPC round-trip. Earlier code awaited them serially.
+        const [outbox, diag] = await Promise.all([getOutboxStatus(), getDiagnostics()]);
+        renderOutboxStatus(outbox);
+        renderDiagnosticsBadge(diag);
     } catch { /* service unreachable */ }
 }, 15000);
 // First read on startup so the bar isn't blank.
 (async () => {
     try {
         const { getOutboxStatus, getDiagnostics } = await import("./lib/api-client.js");
-        renderOutboxStatus(await getOutboxStatus());
-        renderDiagnosticsBadge(await getDiagnostics());
+        const [outbox, diag] = await Promise.all([getOutboxStatus(), getDiagnostics()]);
+        renderOutboxStatus(outbox);
+        renderDiagnosticsBadge(diag);
     } catch { /* */ }
 })();
 

package/client/compose/compose.bundle.js

@@ -596,6 +596,27 @@ async function createTinyMceEditor(container2, opts = {}) {
       // everything that the schema allows.
       paste_word_valid_elements: "@[style|class],-strong/b,-em/i,-u,-s,-sub,-sup,-strike,-p,-ol,-ul,-li,-h1,-h2,-h3,-h4,-h5,-h6,-blockquote,-table[border|cellpadding|cellspacing|width|height|class|style],-tr,-td[colspan|rowspan|width|height|class|style|valign|align|background|bgcolor],-th,-thead,-tbody,-tfoot,-pre,-br,-a[href|target|title],-img[src|alt|width|height|style|class]",
       paste_retain_style_properties: "color background background-color font-family font-size font-weight font-style text-decoration text-align padding padding-top padding-bottom padding-left padding-right margin margin-top margin-bottom margin-left margin-right border border-top border-bottom border-left border-right",
+      // Auto-link bare URLs in pasted content. TinyMCE's `autolink`
+      // plugin only fires on TYPED space/enter; URLs that arrive via
+      // clipboard (browser address bar, terminal copy) come in as
+      // plain text and stay un-linked. paste_preprocess runs on the
+      // HTML the paste plugin produced.
+      //
+      // CRITICAL: skip auto-link when the content already contains
+      // anchors. Naive regex over the whole content would wrap
+      // `<a href="X">X</a>` in ANOTHER anchor (nested anchors are
+      // invalid HTML and browsers split them, producing visible
+      // junk). For HTML pastes that already have linked URLs (the
+      // common case from a browser address bar or another mail
+      // client), TinyMCE preserves them — no auto-link pass needed.
+      // For plain-text pastes (no anchors present), wrap any bare
+      // http(s)://… runs. Trailing sentence punctuation is excluded
+      // from the URL.
+      paste_preprocess: (_plugin, args) => {
+        if (/<a[\s>]/i.test(args.content))
+          return;
+        args.content = args.content.replace(/(^|[\s(\[])((?:https?|ftp):\/\/[^\s<>"']+[^\s<>"'.,;:!?)\]])/gi, (_m, lead, url) => `${lead}<a href="${url}">${url}</a>`);
+      },
       content_style: "body { font-family: system-ui, sans-serif; font-size: 14px; }",
       init_instance_callback: (ed) => resolve(ed),
       setup: (ed) => {
@@ -2941,6 +2962,15 @@ document.addEventListener("contextmenu", () => {
 
 // client/compose/compose.ts
 logClientEvent("compose-module-loaded", { href: location.href, version: window.mailxVersion || "?" });
+var _composeT0 = performance.now();
+function _ctick(label) {
+  const ms = (performance.now() - _composeT0).toFixed(0).padStart(5);
+  try {
+    logClientEvent(`compose-tick ${ms}ms ${label}`);
+  } catch {
+  }
+}
+_ctick("module body executing");
 function closeCompose() {
   logClientEvent("compose-close");
   try {
@@ -2979,8 +3009,8 @@ async function loadEditorAssets(type) {
       loadScript(`${cdn}/@tiptap/extension-placeholder@2/dist/index.umd.js`)
     ]);
   } else {
-    loadCSS("https://cdn.jsdelivr.net/npm/quill@2/dist/quill.snow.css");
-    await loadScript("https://cdn.jsdelivr.net/npm/quill@2/dist/quill.js");
+    loadCSS("../lib/quill/quill.snow.css");
+    await loadScript("../lib/quill/quill.js");
   }
 }
 var editorType = "quill";
@@ -3062,7 +3092,9 @@ async function tryEditor(type) {
   }
 }
 var activeEditorType = editorType;
+_ctick(`editor load start (${editorType})`);
 editor = await tryEditor(editorType);
+_ctick(`editor load end (${editorType}, ok=${!!editor})`);
 if (!editor) {
   const fallbackType = editorType === "quill" ? "tiptap" : "quill";
   logClientEvent("compose-editor-fallback-other", { from: editorType, to: fallbackType });
@@ -3647,13 +3679,41 @@ function scheduleDraftSave() {
     });
   }, DRAFT_INPUT_DEBOUNCE_MS);
 }
+var _parentInitReady = !!sessionStorage.getItem("composeInit");
+var _parentInitListeners = [];
+window.addEventListener("message", (e) => {
+  if (e.data?.type !== "compose-init-ready") return;
+  _parentInitReady = true;
+  for (const fn of _parentInitListeners.splice(0)) fn();
+});
+function waitForParentInit(maxMs) {
+  if (_parentInitReady) return Promise.resolve();
+  return new Promise((resolve) => {
+    const timer = setTimeout(() => {
+      _parentInitReady = true;
+      resolve();
+    }, maxMs);
+    _parentInitListeners.push(() => {
+      clearTimeout(timer);
+      resolve();
+    });
+  });
+}
 (async () => {
+  _ctick("init IIFE start");
+  if (!sessionStorage.getItem("composeInit")) {
+    _ctick("waiting for parent init");
+    await waitForParentInit(1500);
+    _ctick("parent init received");
+  }
   const stored = sessionStorage.getItem("composeInit");
   if (stored) {
     sessionStorage.removeItem("composeInit");
     const init = JSON.parse(stored);
+    _ctick(`init parsed (mode=${init.mode}, bodyHtml=${init.bodyHtml?.length || 0} bytes)`);
     if (init.accounts && init.accounts.length > 0) {
       applyInit(init);
+      _ctick("applyInit done \u2014 compose visible");
       getAccounts().then((fresh) => {
         if (Array.isArray(fresh) && fresh.length > 0) {
           init.accounts = fresh;