@bobfrankston/mailx 1.0.339 → 1.0.348

package/bin/mailx.js

@@ -130,6 +130,8 @@ if (!isDaemon && !__isCommandInvocation) {
 if (!verbose && !isDaemon && !process.argv.slice(2).some(a => /^-/.test(a))) {
     const { spawn } = await import("node:child_process");
     const child = spawn(process.execPath, [...process.argv.slice(1), "--daemon"], {
+        // windowsHide on the spawn options below — prevents the brief
+        // console-window flash when the daemon launches.
         detached: true,
         stdio: "ignore",
         windowsHide: true,
@@ -909,6 +911,53 @@ async function main() {
         }
     }
     const db = new MailxDB(getConfigDir());
+    // Auto-create the sending/ recovery README on every startup. Stays in
+    // sync with the running version of mailx; user can ignore once the
+    // disk-staging fallback is no longer needed.
+    try {
+        const sendingDir = path.join(getConfigDir(), "sending");
+        fs.mkdirSync(sendingDir, { recursive: true });
+        const readmePath = path.join(sendingDir, "README.md");
+        const readmeBody = `# \`~/.mailx/sending/\` and \`~/.mailx/outbox/\` — outgoing-mail staging
+
+Auto-generated by mailx on startup. Manual recovery reference for when mailx is broken or you need to feed an outgoing message into another mail program.
+
+## Layout
+
+\`\`\`
+~/.mailx/
+├── outbox/<account>/
+│   └── *.ltr                  ← THE QUEUE. Worker scans every 10s, sends, deletes on success.
+└── sending/<account>/
+    ├── editing/               ← Last 3 draft autosaves while composing.
+    ├── queued/                ← Manual drop-in / crash-recovery copies.
+    └── sent/                  ← Audit trail of successfully sent messages.
+\`\`\`
+
+In-flight files are atomically renamed to \`<file>.sending-<host>-<pid>\` while the worker is processing them — same-machine claim so two mailx instances don't double-send. Stale claims (dead PIDs on this host) are recovered on the next tick.
+
+## Manual fallback
+
+- **mailx is dead, need to send a draft** — most recent file in \`sending/<account>/editing/\` is a complete RFC 822 message; copy the body into another mail client and resend.
+- **Feed a raw .eml to mailx** — drop into \`sending/<account>/queued/\`. Picked up within 10s.
+- **mailx says queued but server doesn't have it** — look in \`outbox/<account>/\`. \`.ltr\` still there → worker hasn't sent yet (check \`~/.mailx/logs/\`). \`.sending-<host>-<pid>\` → in flight. Gone → success.
+
+## Format
+
+RFC 5322 with CRLF line endings. Bodies are quoted-printable encoded (readable in a text editor). Every message carries \`Message-ID:\` for cross-device dedup; \`X-Mailx-Retry\` marks retry attempts.
+`;
+        // Only rewrite if content drifted (avoids gratuitous mtime updates).
+        let existing = "";
+        try {
+            existing = fs.readFileSync(readmePath, "utf-8");
+        }
+        catch { /* missing */ }
+        if (existing !== readmeBody)
+            fs.writeFileSync(readmePath, readmeBody);
+    }
+    catch (e) {
+        console.error(`  [readme] Could not write sending README: ${e?.message || e}`);
+    }
     const { NodeTcpTransport } = await import("@bobfrankston/node-tcp-transport");
     const imapManager = new ImapManager(db, () => new NodeTcpTransport());
     // Native client is the only option (iflow-direct)
@@ -1023,7 +1072,7 @@ async function main() {
             try {
                 clearInstanceFile();
                 const { spawn: spawnChild } = await import("child_process");
-                const child = spawnChild("mailx", [], { detached: true, stdio: "ignore", shell: true });
+                const child = spawnChild("mailx", [], { detached: true, stdio: "ignore", shell: true, windowsHide: true });
                 child.unref();
                 console.log("  [restart] Spawned fresh daemon; shutting down current");
                 // Give the spawn a moment to take hold before we start
@@ -1043,10 +1092,10 @@ async function main() {
             try {
                 const { execSync, spawn: spawnChild } = await import("child_process");
                 console.log("  [update] Installing latest version...");
-                execSync("npm install -g @bobfrankston/mailx", { encoding: "utf-8", timeout: 120_000, stdio: "inherit" });
+                execSync("npm install -g @bobfrankston/mailx", { encoding: "utf-8", timeout: 120_000, stdio: "inherit", windowsHide: true });
                 console.log("  [update] Install complete — relaunching");
                 // Spawn the new version detached so it outlives this process
-                const child = spawnChild("mailx", [], { detached: true, stdio: "ignore", shell: true });
+                const child = spawnChild("mailx", [], { detached: true, stdio: "ignore", shell: true, windowsHide: true });
                 child.unref();
             }
             catch (e) {
@@ -1055,13 +1104,18 @@ async function main() {
             gracefulShutdown("Update applied");
             return;
         }
+        // Per-action wall-clock timing so a "took N seconds" report tells us
+        // where between Rust→stdin→dispatch→service the time actually went.
+        const ipcT0 = Date.now();
         try {
             const response = await dispatch(svc, req);
-            console.log(`[ipc] → ${req._action} (${req._cbid}) ok`);
+            const elapsed = Date.now() - ipcT0;
+            console.log(`[ipc] → ${req._action} (${req._cbid}) ok in ${elapsed}ms`);
             handle.send(response);
         }
         catch (e) {
-            console.error(`[ipc] → ${req._action} (${req._cbid}) error: ${e.message}`);
+            const elapsed = Date.now() - ipcT0;
+            console.error(`[ipc] → ${req._action} (${req._cbid}) error in ${elapsed}ms: ${e.message}`);
             handle.send({ _cbid: req._cbid, error: e.message });
         }
     });
@@ -1119,6 +1173,9 @@ async function main() {
     imapManager.on("configChanged", (filename) => {
         handle.send({ _event: "configChanged", type: "configChanged", filename });
     });
+    imapManager.on("outboxStatus", (status) => {
+        handle.send({ _event: "outboxStatus", type: "outboxStatus", ...status });
+    });
     // syncComplete drives the folder-tree refresh that picks up newly-discovered
     // folders on first run (Gmail accounts have no folders in the DB until the
     // first sync fetches the labels). Without this forward, the UI shows the
@@ -1237,8 +1294,31 @@ async function main() {
     const UPDATE_CHECK_MS = 30 * 60_000; // 30 minutes
     async function checkForUpdate() {
         try {
-            const { execSync } = await import("child_process");
-            const latest = execSync("npm view @bobfrankston/mailx version", { encoding: "utf-8", timeout: 15_000 }).trim();
+            // spawn with windowsHide:true — execSync briefly flashes a cmd
+            // window on Windows every time the periodic check fires.
+            const { spawn } = await import("child_process");
+            const latest = await new Promise((resolve, reject) => {
+                const child = spawn("npm", ["view", "@bobfrankston/mailx", "version"], {
+                    windowsHide: true,
+                    shell: true,
+                });
+                let out = "";
+                let err = "";
+                child.stdout.on("data", (d) => { out += d.toString(); });
+                child.stderr.on("data", (d) => { err += d.toString(); });
+                const killer = setTimeout(() => { try {
+                    child.kill();
+                }
+                catch { /* */ } reject(new Error("npm view timed out")); }, 15_000);
+                child.on("error", (e) => { clearTimeout(killer); reject(e); });
+                child.on("exit", (code) => {
+                    clearTimeout(killer);
+                    if (code === 0)
+                        resolve(out.trim());
+                    else
+                        reject(new Error(err.trim() || `npm view exit ${code}`));
+                });
+            });
             const current = rootPkgVersion;
             if (latest && latest !== current) {
                 console.log(`  [update] New version available: ${current} → ${latest}`);