@bobfrankston/mailx 1.0.180 → 1.0.182

package/bin/mailx.js

@@ -28,6 +28,20 @@ function hasFlag(name) { return args.includes(`-${name}`) || args.includes(`--${
 const serverMode = hasFlag("server");
 const noBrowser = hasFlag("no-browser");
 const verbose = hasFlag("verbose");
+const isDaemon = hasFlag("daemon"); // internal: re-spawned detached process
+// Auto-detach: re-spawn as background process so terminal returns immediately
+// Skip for: --verbose (want console), --server (needs terminal), --daemon (already detached),
+// and any command flags (setup, kill, test, etc.)
+if (!verbose && !serverMode && !isDaemon && !process.argv.slice(2).some(a => /^-/.test(a) && !["--no-browser"].includes(a))) {
+    const { spawn } = await import("node:child_process");
+    const child = spawn(process.execPath, [...process.argv.slice(1), "--daemon"], {
+        detached: true,
+        stdio: "ignore",
+        windowsHide: true,
+    });
+    child.unref();
+    process.exit(0);
+}
 const setupMode = hasFlag("setup");
 const addMode = hasFlag("add");
 const testMode = hasFlag("test");
@@ -35,7 +49,7 @@ const rebuildMode = hasFlag("rebuild");
 const repairMode = hasFlag("repair");
 const importMode = hasFlag("import");
 // Validate arguments
-const knownFlags = ["server", "no-browser", "verbose", "external", "kill", "v", "version", "setup", "add", "test", "rebuild", "repair", "native-imap", "log", "import", "email", "mail"];
+const knownFlags = ["server", "no-browser", "verbose", "external", "kill", "v", "version", "setup", "add", "test", "rebuild", "repair", "native-imap", "log", "import", "email", "mail", "daemon"];
 for (const arg of args) {
     const flag = arg.replace(/^--?/, "");
     if (arg.startsWith("-") && !knownFlags.includes(flag)) {
@@ -665,6 +679,10 @@ async function main() {
     // Pass server version to dispatch so getVersion returns it
     const rootPkg = JSON.parse(fs.readFileSync(path.join(import.meta.dirname, "..", "package.json"), "utf-8"));
     handle.onRequest(async (req) => {
+        if (!req._action) {
+            console.log(`[ipc] ← ignored (no _action): ${JSON.stringify(req).substring(0, 100)}`);
+            return;
+        }
         console.log(`[ipc] ← ${req._action} (${req._cbid})`);
         try {
             const response = await dispatch(svc, req);

package/client/.msger-window.json

@@ -1 +1 @@
-{"height":1047,"width":1844,"x":531,"y":264}
+{"height":1344,"width":1438,"x":216,"y":107}

package/client/app.js

@@ -265,12 +265,31 @@ async function openCompose(mode) {
         references: [],
         accounts: accounts.map((a) => ({ id: a.id, name: a.name, email: a.email })),
     };
+    // Auto-detect reply From: if the message was delivered to an identity domain,
+    // reply from that address instead of the default account address.
+    // Identity domains configured per-account in accounts.jsonc (identityDomains array).
+    // Default identity domains for bob.ma account:
+    const identityDomains = ["bob.ma", "frankston.com"];
+    function detectReplyFrom() {
+        if (!msg)
+            return undefined;
+        // Check deliveredTo first (most reliable), then To addresses
+        const candidates = [msg.deliveredTo, ...(msg.to || []).map((a) => a.address)].filter(Boolean);
+        for (const addr of candidates) {
+            const domain = addr.split("@")[1]?.toLowerCase();
+            if (domain && identityDomains.some(d => domain === d || domain.endsWith(`.${d}`))) {
+                return addr;
+            }
+        }
+        return undefined;
+    }
     if (msg && mode === "reply") {
         init.to = [msg.from];
         init.subject = `Re: ${cleanSubject}`;
         init.bodyHtml = quoteBody(msg);
         init.inReplyTo = msg.messageId;
         init.references = [...(msg.references || []), msg.messageId];
+        init.fromAddress = detectReplyFrom();
     }
     else if (msg && mode === "replyAll") {
         init.to = [msg.from, ...msg.to.filter((a) => a.address !== msg.from.address)];
@@ -279,6 +298,7 @@ async function openCompose(mode) {
         init.bodyHtml = quoteBody(msg);
         init.inReplyTo = msg.messageId;
         init.references = [...(msg.references || []), msg.messageId];
+        init.fromAddress = detectReplyFrom();
     }
     else if (msg && mode === "forward") {
         init.subject = `Fwd: ${cleanSubject}`;
@@ -565,22 +585,10 @@ onWsEvent((event) => {
             break;
         }
         case "folderCountsChanged": {
-            // Incremental count update — no DOM rebuild, no jitter
+            // Update folder badges only — never reload the message list or touch the viewer.
+            // The list refreshes when the user clicks a folder or presses Sync.
             updateFolderCounts();
             updateNewMessageCount();
-            // Only reload message list if the synced account is the one we're viewing
-            // (or unified inbox which shows all accounts). Debounce to avoid rapid reloads
-            // during first sync which emits per-batch.
-            const syncedAccount = event.accountId;
-            const viewingThis = !currentAccountId || currentAccountId === syncedAccount;
-            if (viewingThis) {
-                if (reloadDebounceTimer)
-                    clearTimeout(reloadDebounceTimer);
-                reloadDebounceTimer = setTimeout(() => {
-                    reloadDebounceTimer = null;
-                    reloadCurrentFolder();
-                }, 500);
-            }
             // Sync finished — re-enable sync button
             const syncBtn = document.getElementById("btn-sync");
             if (syncBtn) {

package/client/components/message-viewer.js

@@ -17,16 +17,23 @@ export function getCurrentMessage() {
 /** Initialize viewer — subscribe to state changes */
 export function initViewer() {
     state.subscribe((change) => {
-        if (change === "removed" || change === "messages") {
+        if (change === "removed") {
+            // Message was deleted/moved — show auto-selected next, or clear
             const sel = state.getSelected();
             if (!sel) {
                 clearViewer();
             }
             else if (sel.uid !== currentMessage?.uid || sel.accountId !== currentAccountId) {
-                // State auto-selected a new message after removal — show it
                 showMessage(sel.accountId, sel.uid, sel.folderId);
             }
         }
+        else if (change === "selected") {
+            // Explicit deselect (folder switch, clearViewer)
+            if (!state.getSelected()) {
+                clearViewer();
+            }
+        }
+        // "messages" change (sync reload) — don't touch the viewer
     });
 }
 function clearViewer() {

package/client/compose/compose.js

@@ -260,6 +260,13 @@ function parseAddrs(s) {
 function applyInit(init) {
     // Populate From dropdown
     populateFromSelect(init.accounts, init.accountId);
+    // Auto-detect reply From: if fromAddress is set (identity domain match),
+    // use it as the From address via the "Other..." custom field
+    if (init.fromAddress) {
+        fromSelect.value = "__custom__";
+        fromCustom.hidden = false;
+        fromCustom.value = init.fromAddress;
+    }
     toInput.value = formatAddrs(init.to);
     ccInput.value = formatAddrs(init.cc);
     subjectInput.value = init.subject;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx",
-  "version": "1.0.180",
+  "version": "1.0.182",
   "description": "Local-first email client with IMAP sync and standalone native app",
   "type": "module",
   "main": "bin/mailx.js",
@@ -24,7 +24,7 @@
     "@bobfrankston/iflow-node": "^0.1.2",
     "@bobfrankston/miscinfo": "^1.0.8",
     "@bobfrankston/oauthsupport": "^1.0.21",
-    "@bobfrankston/msger": "^0.1.230",
+    "@bobfrankston/msger": "^0.1.232",
     "@capacitor/android": "^8.3.0",
     "@capacitor/cli": "^8.3.0",
     "@capacitor/core": "^8.3.0",

package/packages/mailx-service/index.js

@@ -106,7 +106,7 @@ export class MailxService {
         for (const cfg of settings.accounts) {
             const a = dbAccounts.find(d => d.id === cfg.id);
             if (a)
-                ordered.push({ ...a, label: cfg.label, defaultSend: cfg.defaultSend || false });
+                ordered.push({ ...a, label: cfg.label, defaultSend: cfg.defaultSend || false, identityDomains: cfg.identityDomains || [] });
         }
         // Append any DB accounts not in settings
         for (const a of dbAccounts) {
@@ -339,7 +339,9 @@ export class MailxService {
         const account = settings.accounts.find(a => a.id === msg.from);
         if (!account)
             throw new Error(`Unknown account: ${msg.from}`);
-        const fromHeader = msg.fromAddress || `${account.name} <${account.email}>`;
+        // Use custom From address if set (identity domain reply), but always wrap with account display name
+        const fromAddr = msg.fromAddress || account.email;
+        const fromHeader = `${account.name} <${fromAddr}>`;
         const to = msg.to.map((a) => a.name ? `${a.name} <${a.address}>` : a.address).join(", ");
         const cc = msg.cc?.map((a) => a.name ? `${a.name} <${a.address}>` : a.address).join(", ");
         const bcc = msg.bcc?.map((a) => a.name ? `${a.name} <${a.address}>` : a.address).join(", ");