@bobfrankston/rmfmail 1.2.208 → 1.2.210

package/bin/mailx.js

@@ -2266,24 +2266,46 @@ RFC 5322 with CRLF line endings. Bodies are quoted-printable encoded (readable i
         // crash) the main window's browser process — see S67 note on the
         // popout windows below. Service-supplied opts may override.
         const { popupKey, ...boxOpts } = opts || {};
+        // ONE window per occurrence, however many app windows are open.
+        // Every window runs its own alarm poller (alarms.ts is client-side),
+        // so a second main window meant the same reminder spawned two
+        // identical popups — and the second registration clobbered the
+        // first's closer, leaving a popup that cross-machine retraction
+        // could no longer close (Bob 2026-07-31: "two instances running,
+        // two popups"). The late caller joins the SAME result, so both
+        // pollers persist the same dismiss/snooze.
+        if (popupKey) {
+            const open = reminderPopups.get(popupKey);
+            if (open) {
+                console.log(`  [reminder] popup ${popupKey} already open — joining it instead of opening a second`);
+                return await open.result;
+            }
+        }
         const h = showMessageBoxEx({ profile: "popout", ...boxOpts });
         if (typeof h.pid === "number")
             addChildPid(h.pid);
-        if (popupKey)
-            reminderPopups.set(popupKey, { close: () => { try {
-                    h.close();
-                }
-                catch { /* already gone */ } } });
-        try {
+        const result = (async () => {
             const r = await h.result;
             return { button: r.button, closed: r.closed, dismissed: r.dismissed, form: r.form };
-        }
-        finally {
+        })();
+        // Cleanup rides on the shared promise, not on this caller — joiners
+        // must not tear down registration out from under the real window.
+        void result.catch(() => { }).finally(() => {
             if (typeof h.pid === "number")
                 removeChildPid(h.pid);
             if (popupKey)
                 reminderPopups.delete(popupKey);
+        });
+        if (popupKey) {
+            reminderPopups.set(popupKey, {
+                close: () => { try {
+                    h.close();
+                }
+                catch { /* already gone */ } },
+                result,
+            });
         }
+        return await result;
     });
     svc.setPopupCloserFn((key) => {
         const p = reminderPopups.get(key);