@bobfrankston/rmfmail 1.0.703 → 1.0.704

package/client/app.js

@@ -801,9 +801,18 @@ async function openCompose(mode) {
     const account = accounts.find((a) => a.id === accountId);
     const explicitDomains = (account?.identityDomains || []).map((d) => d.toLowerCase());
     const accountDomain = (account?.email || "").split("@")[1]?.toLowerCase();
-    const identityDomains = explicitDomains.length > 0
+    // Fold subsumed subdomains. If the list has both `frankston.com` and
+    // `bobf.frankston.com`, drop the longer entry — the matcher's
+    // `endsWith(".frankston.com")` test already catches it, so listing
+    // both is wasted clutter. Same-pair dedup + parent-shadow filter in
+    // one pass.
+    function foldDomains(list) {
+        const unique = Array.from(new Set(list));
+        return unique.filter(d => !unique.some(p => p !== d && d.endsWith(`.${p}`)));
+    }
+    const identityDomains = foldDomains(explicitDomains.length > 0
         ? explicitDomains
-        : (accountDomain ? [accountDomain] : []);
+        : (accountDomain ? [accountDomain] : []));
     function detectReplyFrom() {
         if (!msg)
             return undefined;
@@ -832,6 +841,41 @@ async function openCompose(mode) {
         console.log(`[compose] no identity match`);
         return undefined;
     }
+    // Trace what we're feeding the reply — `[reply-init]` on the daemon log
+    // will show the exact `from`/`to`/`cc` values present on the source
+    // message at construction time, so a "To field empty" report can be
+    // diagnosed without re-deriving state. Bob 2026-05-13: had empty To on
+    // a clear-From mailing list message; need to see whether msg.from was
+    // populated or arrived as null.
+    if (msg) {
+        try {
+            const dump = {
+                mode,
+                accountId,
+                hasMsg: true,
+                from: msg.from || null,
+                toLen: Array.isArray(msg.to) ? msg.to.length : -1,
+                ccLen: Array.isArray(msg.cc) ? msg.cc.length : -1,
+                deliveredTo: msg.deliveredTo || "",
+                identityDomains,
+                subject: msg.subject || "",
+            };
+            const apiClient = window.mailxapi;
+            if (apiClient?.logClientEvent) {
+                apiClient.logClientEvent("reply-init", dump);
+            }
+            else {
+                console.log("[reply-init]", dump);
+            }
+        }
+        catch { /* tracing must never break compose */ }
+    }
+    else {
+        try {
+            window.mailxapi?.logClientEvent?.("reply-init", { mode, accountId, hasMsg: false });
+        }
+        catch { /* */ }
+    }
     // Defensive: msg.from / msg.to may be missing on rows that arrived before
     // headers finished loading. Don't push undefined into init.to — that
     // bubbles to the compose form as literal "undefined". Empty-out gracefully.