@bobfrankston/rmfmail 1.1.190 → 1.1.192

package/client/app.js

@@ -893,6 +893,20 @@ async function openCompose(mode, overrideMsg, overrideAccountId) {
     const identityDomains = foldDomains(explicitDomains.length > 0
         ? explicitDomains
         : (accountDomain ? [accountDomain] : []));
+    // Extract the bare email address from a header value that may be a full
+    // `"Display Name" <addr@host>` form. detectReplyFrom must return a BARE
+    // address because the compose side wraps it as `${accountName} <${addr}>`
+    // — passing a full name+addr there produced the nested garbage
+    // `Bob Frankston <"mailing list" <multicians@groups.io>>` (Bob 2026-05-28).
+    // Delivered-To on mailing-list mail commonly carries a display name.
+    function bareAddress(raw) {
+        if (!raw)
+            return raw;
+        const angle = raw.match(/<([^>]+)>/);
+        if (angle)
+            return angle[1].trim();
+        return raw.trim().replace(/^"+|"+$/g, "");
+    }
     function detectReplyFrom() {
         if (!msg)
             return undefined;
@@ -902,15 +916,16 @@ async function openCompose(mode, overrideMsg, overrideAccountId) {
         // when their domain matches the account's identityDomains, since To/Cc
         // can be set by the sender and aren't authoritative.
         if (msg.deliveredTo) {
-            console.log(`[compose] reply From → ${msg.deliveredTo} (Delivered-To)`);
-            return msg.deliveredTo;
+            const addr = bareAddress(msg.deliveredTo);
+            console.log(`[compose] reply From → ${addr} (Delivered-To, bare of "${msg.deliveredTo}")`);
+            return addr;
         }
         if (identityDomains.length === 0)
             return undefined;
         const candidates = [
             ...((msg.to || []).map((a) => a.address)),
             ...((msg.cc || []).map((a) => a.address)),
-        ].filter(Boolean);
+        ].filter(Boolean).map(bareAddress);
         for (const addr of candidates) {
             const domain = addr.split("@")[1]?.toLowerCase();
             if (domain && identityDomains.some(d => domain === d || domain.endsWith(`.${d}`))) {