@bobfrankston/rmfmail 1.2.220 → 1.2.222

package/bin/mailx.js

@@ -178,6 +178,34 @@ const isDaemon = hasFlag("daemon"); // internal: re-spawned detached process
         process.exit(0);
     }
 }
+/** App entries appended to the NATIVE right-click menu inside compose. msger
+ *  keeps the WebView2 default menu (that is the only place spellcheck
+ *  suggestions live) and gates these to editable targets, so they never show
+ *  on the read-only list or message preview.
+ *
+ *  Clipboard entries are ours rather than the browser's on purpose: Chromium
+ *  OMITS Cut/Copy from the default menu when the caret is collapsed, and a
+ *  right-click on a word is a collapsed caret — so the menu the user actually
+ *  sees while editing has no Cut or Copy (Bob 2026-08-07). Ours expand to the
+ *  word under the caret first, exactly like the formatting entries below
+ *  already do, so "right-click a word → Copy" means what it looks like.
+ *
+ *  Ids are dispatched to compose.ts via window.__msgerContextCommand — one
+ *  list, both windows (in-window compose overlay + popout compose), because
+ *  drift between the two is what produced "right mouse isn't showing any
+ *  formatting options" in the first place (Bob 2026-07-18). */
+const COMPOSE_CONTEXT_MENU_ITEMS = [
+    { id: "cut", label: "Cut" },
+    { id: "copy", label: "Copy" },
+    { id: "paste", label: "Paste" },
+    { id: "bold", label: "Bold" },
+    { id: "italic", label: "Italic" },
+    { id: "underline", label: "Underline" },
+    { id: "strike", label: "Strikethrough" },
+    { id: "link", label: "Link…" },
+    { id: "clear", label: "Clear formatting" },
+    { id: "source", label: "View HTML source…" },
+];
 /** Windows mailto: registration — registry keys via reg.exe. Shared by the
  *  `-register-mailto` CLI flag and the Settings → "Make default email app"
  *  menu action (injected into MailxService; the registry work stays HERE in
@@ -2509,15 +2537,7 @@ RFC 5322 with CRLF line endings. Bodies are quoted-printable encoded (readable i
                     // App entries on the NATIVE right-click menu (native stays
                     // so spellcheck suggestions survive). compose.ts maps ids
                     // to editor commands via window.__msgerContextCommand.
-                    contextMenuItems: [
-                        { id: "bold", label: "Bold" },
-                        { id: "italic", label: "Italic" },
-                        { id: "underline", label: "Underline" },
-                        { id: "strike", label: "Strikethrough" },
-                        { id: "link", label: "Link…" },
-                        { id: "clear", label: "Clear formatting" },
-                        { id: "source", label: "View HTML source…" },
-                    ],
+                    contextMenuItems: COMPOSE_CONTEXT_MENU_ITEMS,
                 });
                 popoutWindows.set(key, h);
                 if (typeof h.pid === "number")
@@ -2653,15 +2673,7 @@ RFC 5322 with CRLF line endings. Bodies are quoted-printable encoded (readable i
         // "right mouse isn't showing any formatting options let alone source").
         // msger gates these to editable targets, so they DON'T appear on
         // right-clicks in the read-only list / message preview.
-        contextMenuItems: [
-            { id: "bold", label: "Bold" },
-            { id: "italic", label: "Italic" },
-            { id: "underline", label: "Underline" },
-            { id: "strike", label: "Strikethrough" },
-            { id: "link", label: "Link…" },
-            { id: "clear", label: "Clear formatting" },
-            { id: "source", label: "View HTML source…" },
-        ],
+        contextMenuItems: COMPOSE_CONTEXT_MENU_ITEMS,
     });
     // Late-bind the popout-action forwarder now that the main window's
     // service channel exists.
@@ -2905,10 +2917,22 @@ RFC 5322 with CRLF line endings. Bodies are quoted-printable encoded (readable i
     // and emits this when Word writes; compose.ts listens and reloads Quill.
     imapManager.on("wordEditUpdated", (payload) => {
         handle.send({ _event: "wordEditUpdated", type: "wordEditUpdated", ...payload });
-        // Bring compose back to the front after a Word save — the service's
-        // old mailx-host focusMainWindow call was a silent no-op (the helper
-        // never existed); this native msger command is the real mechanism.
-        handle.send({ _msgerWindow: "focus" });
+        // Deliberately NO { _msgerWindow: "focus" } here. The autosave sidecar
+        // saves every ~2s while the user types, and each save lands in this
+        // handler — so this line pulled the foreground off Word mid-sentence,
+        // every couple of seconds, for the whole editing session (Bob
+        // 2026-08-06: "keeps flipping back to rmfmail on any change"). The
+        // focus now happens once, on wordEditClosed, below.
+    });
+    // Word (or the document) closed — the external edit is over. THIS is the
+    // moment to bring rmfmail forward, replacing the old per-save focus grab.
+    imapManager.on("wordEditClosed", (payload) => {
+        handle.send({ _event: "wordEditClosed", type: "wordEditClosed", ...payload });
+        // Only raise the MAIN window when the compose lives IN it. A popout
+        // compose is a separate window with no service channel, so focusing
+        // main would just cover it up.
+        if (!payload.fromPopout)
+            handle.send({ _msgerWindow: "focus" });
     });
     // Cloud-write/read failures from mailx-settings → push to UI as a banner so
     // silent fall-back-to-local can no longer swallow Drive errors.