@bobfrankston/rmfmail 1.2.204 → 1.2.206

package/bin/mailx.js

@@ -609,6 +609,20 @@ if (!isDaemon && !__isCommandInvocation && !__keepOthers) {
         clearInstanceFile();
     }
 }
+// Get the daemon's cwd OFF the npm install directory. Launchers routinely
+// start us with cwd = .../npm/node_modules/@bobfrankston/rmfmail/bin (rmfshare
+// set it explicitly; a terminal `cd` there does it too), and every child we
+// spawn — including the per-app msger launcher exe that lives for the whole
+// session — inherits it. On Windows a process's cwd holds the directory
+// locked, so `npm install -g` (the in-app self-update AND a manual update)
+// dies with EBUSY renaming .../rmfmail/bin for as long as we run
+// (Bob 2026-07-30: maroon "Update failed" banner, stuck on 1.2.202).
+// All CLI file-path arguments were consumed by the command blocks above,
+// so nothing downstream depends on the original cwd.
+try {
+    process.chdir(process.env.USERPROFILE || process.env.HOME || "/");
+}
+catch { /* */ }
 // Auto-detach: re-spawn as background process so terminal returns immediately
 // Skip for: --verbose (want console), --daemon (already detached),
 // and any command flags (setup, kill, test, etc.)
@@ -2688,7 +2702,10 @@ RFC 5322 with CRLF line endings. Bodies are quoted-printable encoded (readable i
             try {
                 const { execSync, spawn: spawnChild } = await import("child_process");
                 console.log("  [update] Installing latest version...");
-                execSync("npm install -g @bobfrankston/rmfmail", { encoding: "utf-8", timeout: 120_000, stdio: "inherit", windowsHide: true });
+                // stdio pipe (not inherit) so a failure's stderr lands in the
+                // thrown error — "Command failed: npm install" with no reason
+                // cost a three-round diagnosis of the EBUSY cwd-lock bug.
+                execSync("npm install -g @bobfrankston/rmfmail", { encoding: "utf-8", timeout: 120_000, stdio: "pipe", windowsHide: true });
                 console.log("  [update] Install complete — relaunching");
                 // Spawn the new version detached so it outlives this process
                 const child = spawnChild("rmfmail", [], { detached: true, stdio: "ignore", shell: true, windowsHide: true });
@@ -2701,7 +2718,8 @@ RFC 5322 with CRLF line endings. Bodies are quoted-printable encoded (readable i
                 // shouldn't kill the running app. Surface the failure so the
                 // banner can offer a retry instead of leaving "Updating..."
                 // pinned forever.
-                const msg = e?.message || String(e);
+                const stderrTail = (e?.stderr ? String(e.stderr) : "").trim().split(/\r?\n/).filter((l) => /error/i.test(l)).slice(0, 4).join(" | ");
+                const msg = `${e?.message || String(e)}${stderrTail ? ` — ${stderrTail}` : ""}`;
                 console.error(`  [update] Install failed: ${msg}`);
                 const offline = /ENOTFOUND|ECONNREFUSED|EAI_AGAIN|ETIMEDOUT|getaddrinfo|network|registry/i.test(msg);
                 handle.send({ _event: "updateFailed", type: "updateFailed", offline, message: msg });
@@ -2801,6 +2819,10 @@ 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" });
     });
     // 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.
@@ -2888,6 +2910,11 @@ RFC 5322 with CRLF line endings. Bodies are quoted-printable encoded (readable i
                     handle.send({ _event: "openMailto", type: "openMailto",
                         to: data.to, cc: data.cc, bcc: data.bcc,
                         subject: data.subject, body: data.body, inReplyTo: data.inReplyTo });
+                    // The user just clicked a mailto link in ANOTHER app —
+                    // bring our window to the front so the compose is visible.
+                    // _msgerWindow is a msger-native control message (consumed
+                    // by the Rust side, never forwarded to the page).
+                    handle.send({ _msgerWindow: "focus" });
                 }, 50);
             });
         }
@@ -2922,6 +2949,12 @@ RFC 5322 with CRLF line endings. Bodies are quoted-printable encoded (readable i
                     handle.send({ _event: "openShare", type: "openShare",
                         title: data.title, text: data.text, url: data.url,
                         attachments, skipped });
+                    // The share sheet was on top of another app — bring our
+                    // window to the front so the compose the share opened is
+                    // actually visible (Bob 2026-07-30: "should've brought
+                    // the new window to the front"). Native msger command —
+                    // see the mailto block above.
+                    handle.send({ _msgerWindow: "focus" });
                 }, 50);
             });
         }