@bobfrankston/rmfmail 1.2.136 → 1.2.139

package/bin/mailx.ts

@@ -1768,10 +1768,28 @@ async function main(): Promise<void> {
             const d = new Date();
             return `${pad2(d.getHours())}:${pad2(d.getMinutes())}:${pad2(d.getSeconds())}.${pad3(d.getMilliseconds())}`;
         };
-        const logPath = path.join(logDir, `rmfmail-${localDate()}.log`);
-        const logStream = fs.createWriteStream(logPath, { flags: "a" });
-        console.log = (...a: any[]) => { logStream.write(`${ts()} ${a.join(" ")}\n`); };
-        console.error = (...a: any[]) => { logStream.write(`${ts()} ERROR ${a.join(" ")}\n`); };
+        // Day-aware stream: recompute the filename on every write and swap
+        // the stream at midnight. The old code resolved the path ONCE at
+        // boot, so a daemon that ran past midnight kept appending to the
+        // boot-day file forever — while the sync worker's lines (written
+        // through its own channel) landed in the new day's file. Result: the
+        // morning of 2026-07-14 had ALL main-thread diagnostics ([ipc],
+        // [client], [popout]) hiding in rmfmail-2026-07-13.log and the
+        // current file showing only [w] worker lines — which made the
+        // popout-send data-loss incident look untraceable (S68).
+        let logDate = localDate();
+        let logStream = fs.createWriteStream(path.join(logDir, `rmfmail-${logDate}.log`), { flags: "a" });
+        const logWrite = (line: string): void => {
+            const d = localDate();
+            if (d !== logDate) {
+                try { logStream.end(); } catch { /* old handle already dead */ }
+                logDate = d;
+                logStream = fs.createWriteStream(path.join(logDir, `rmfmail-${d}.log`), { flags: "a" });
+            }
+            logStream.write(line);
+        };
+        console.log = (...a: any[]) => { logWrite(`${ts()} ${a.join(" ")}\n`); };
+        console.error = (...a: any[]) => { logWrite(`${ts()} ERROR ${a.join(" ")}\n`); };
         // Redirect daemon's process.stderr to the log too. msger forwards its
         // Rust child's stderr to our process.stderr; in daemon mode that's
         // stdio:"ignore" → /dev/null, which buries diagnostics like
@@ -1781,7 +1799,7 @@ async function main(): Promise<void> {
         process.stderr.write = ((chunk: any, ...rest: any[]): boolean => {
             try {
                 const s = typeof chunk === "string" ? chunk : chunk.toString();
-                logStream.write(`${ts()} STDERR ${s}${s.endsWith("\n") ? "" : "\n"}`);
+                logWrite(`${ts()} STDERR ${s}${s.endsWith("\n") ? "" : "\n"}`);
             } catch { /* ignore — best-effort logging */ }
             return origStderrWrite(chunk, ...rest);
         }) as typeof process.stderr.write;
@@ -1800,7 +1818,7 @@ async function main(): Promise<void> {
         { MailxDB, prewarmParseWorker, storeBus, Store },
         { ImapManager },
         { MailxService, spawnSyncWorker },
-        { dispatch },
+        { dispatch, setDebugEvalSink },
         { loadSettings, loadAccountsAsync, loadAllowlistAsync, getConfigDir, getStorageInfo, getStorePath },
         { NodeTcpTransport },
         { FileMessageStore },
@@ -2026,7 +2044,10 @@ RFC 5322 with CRLF line endings. Bodies are quoted-printable encoded (readable i
     // see memory: feedback_cross_platform_strong_rule). Remove on
     // resolution so the file doesn't accumulate stale PIDs.
     svc.setPopupFn(async (opts: any) => {
-        const h = showMessageBoxEx(opts);
+        // Own WebView2 profile so a reminder popup can never share (and
+        // crash) the main window's browser process — see S67 note on the
+        // popout windows below. Service-supplied opts may override.
+        const h = showMessageBoxEx({ profile: "popout", ...opts });
         if (typeof h.pid === "number") addChildPid(h.pid);
         try {
             const r = await h.result;
@@ -2051,6 +2072,40 @@ RFC 5322 with CRLF line endings. Bodies are quoted-printable encoded (readable i
             const { ports } = await import("@bobfrankston/miscinfo");
             const dbgApp = express();
             dbgApp.use(express.json({ limit: "25mb" }));
+            // Live-page eval: POST /api/eval {code} → daemon pushes a
+            // `debugEval` event to the main window → app.ts runs the code and
+            // answers via the `debugEvalResult` action → we reply here. The
+            // expression may be async (it's wrapped in `(async () => (…))()`),
+            // must evaluate as an EXPRESSION, and should return JSON-able
+            // data. Lets an agent inspect the real DOM (elementFromPoint,
+            // computed styles, scroll geometry) without devtools. Registered
+            // BEFORE the generic /api router so it isn't shadowed.
+            const pendingEvals = new Map<string, { finish: (v: any) => void; timer: NodeJS.Timeout }>();
+            setDebugEvalSink((id, result, error) => {
+                const p = pendingEvals.get(id);
+                if (!p) return;
+                pendingEvals.delete(id);
+                clearTimeout(p.timer);
+                p.finish(error ? { error } : { result });
+            });
+            dbgApp.post("/api/eval", (req, res) => {
+                const code = req.body?.code;
+                if (typeof code !== "string" || !code.trim()) {
+                    res.status(400).json({ error: "body must be { code: string } — a JS expression" });
+                    return;
+                }
+                if (!sendToClient) {
+                    res.status(503).json({ error: "main window not connected yet" });
+                    return;
+                }
+                const id = `ev-${Date.now()}-${Math.random().toString(36).slice(2, 8)}`;
+                const timer = setTimeout(() => {
+                    pendingEvals.delete(id);
+                    res.status(504).json({ error: "eval timeout (10s) — window busy, or client bundle predates debugEval" });
+                }, 10_000);
+                pendingEvals.set(id, { finish: (v) => res.json(v), timer });
+                sendToClient({ _event: "debugEval", type: "debugEval", id, code });
+            });
             dbgApp.use("/api", createApiRouter(store, imapManager));
             // Port: `--debug-server=<n>` / `--debug-server <n>` if given,
             // else the reserved `ports.rmfmaildbg` from miscinfo (the named
@@ -2141,6 +2196,13 @@ RFC 5322 with CRLF line endings. Bodies are quoted-printable encoded (readable i
                 // window shows unactivated BEHIND the focused main window
                 // and reads as "nothing opened" (Bob 2026-07-09).
                 focusOnCreate: true,
+                // Own WebView2 profile: a window sharing the main window's
+                // user-data dir shares its browser process, and a crash
+                // there blacks out the MAIN window permanently (S67,
+                // 2026-07-14 + 2026-07-15: BROWSER_PROCESS_EXITED during
+                // popout spawn). One shared "popout" profile for all
+                // secondary windows — decoupled from main, bounded disk.
+                profile: "popout",
             });
             popoutWindows.set(key, h);
             if (typeof h.pid === "number") addChildPid(h.pid);
@@ -2173,6 +2235,8 @@ RFC 5322 with CRLF line endings. Bodies are quoted-printable encoded (readable i
                     // Same as the message popout: the daemon is background,
                     // so an unactivated window lands behind the main window.
                     focusOnCreate: true,
+                    // Own profile — same S67 rationale as the message popout.
+                    profile: "popout",
                     // App entries on the NATIVE right-click menu (native stays
                     // so spellcheck suggestions survive). compose.ts maps ids
                     // to editor commands via window.__msgerContextCommand.
@@ -2221,6 +2285,10 @@ RFC 5322 with CRLF line endings. Bodies are quoted-printable encoded (readable i
                 aumid: "com.frankston.rmfmail",
                 escapeCloses: false,
                 focusOnCreate: true,
+                // Own profile — same S67 rationale as the message popout.
+                // Note: localStorage view prefs won't be shared with the
+                // main window; preferences.jsonc (the real store) is.
+                profile: "popout",
             });
             popoutWindows.set(key, h);
             if (typeof h.pid === "number") addChildPid(h.pid);

package/client/app.bundle.js

@@ -37,6 +37,7 @@ __export(api_client_exports, {
   createCalendarEvent: () => createCalendarEvent,
   createFolder: () => createFolder,
   createTask: () => createTask,
+  debugEvalResult: () => debugEvalResult,
   deleteCalendarEvent: () => deleteCalendarEvent,
   deleteContact: () => deleteContact,
   deleteDraft: () => deleteDraft,
@@ -496,6 +497,9 @@ function logClientEvent(tag, data) {
 function sendMessage(body) {
   return ipc().sendMessage?.(body);
 }
+function debugEvalResult(id, result, error) {
+  return ipc().debugEvalResult?.(id, result, error);
+}
 function saveDraft(body) {
   return ipc().saveDraft?.(body);
 }
@@ -3849,11 +3853,16 @@ function clearSearchMode() {
 }
 async function loadUnifiedInbox(autoSelect = true) {
   const myGen = ++loadGen;
+  const wasUnified = unifiedMode;
   unifiedMode = true;
   searchMode = false;
   currentSpecialUse = "";
   showToInsteadOfFrom = false;
-  currentPage = 1;
+  const loadedPages = !autoSelect && wasUnified ? Math.min(Math.max(1, currentPage), MAX_REFRESH_PAGES) : 1;
+  if (!autoSelect && wasUnified && currentPage > MAX_REFRESH_PAGES) {
+    console.log(`  [ml-refresh] depth capped: ${currentPage} loaded pages, refreshing only ${MAX_REFRESH_PAGES}`);
+  }
+  currentPage = loadedPages;
   totalMessages = 0;
   const body = document.getElementById("ml-body");
   if (!body)
@@ -3886,7 +3895,7 @@ async function loadUnifiedInbox(autoSelect = true) {
     body.innerHTML = `<div class="ml-empty">Loading...</div>`;
   }
   try {
-    const result = await getUnifiedInbox(1, 50, flaggedOnly, dateBasis);
+    const result = await getUnifiedInbox(1, loadedPages * 50, flaggedOnly, dateBasis);
     if (myGen !== loadGen)
       return;
     totalMessages = result.total;
@@ -4016,9 +4025,14 @@ async function loadMessages(accountId, folderId, page = 1, specialUse = "", auto
     currentSpecialUse = specialUse;
   const su = currentSpecialUse.toLowerCase();
   showToInsteadOfFrom = su === "sent" || su === "drafts" || su === "outbox" || su.endsWith("sent") || su.endsWith("drafts") || su.endsWith("outbox") || su === "sent items" || su === "sent mail" || su.endsWith("/sent items") || su.endsWith(".sent items");
+  const sameView = accountId === currentAccountId2 && folderId === currentFolderId;
   currentAccountId2 = accountId;
   currentFolderId = folderId;
-  currentPage = 1;
+  const loadedPages = !autoSelect && sameView ? Math.min(Math.max(1, currentPage), MAX_REFRESH_PAGES) : 1;
+  if (!autoSelect && sameView && currentPage > MAX_REFRESH_PAGES) {
+    console.log(`  [ml-refresh] depth capped: ${currentPage} loaded pages, refreshing only ${MAX_REFRESH_PAGES}`);
+  }
+  currentPage = loadedPages;
   totalMessages = 0;
   const body = document.getElementById("ml-body");
   if (!body)
@@ -4058,7 +4072,7 @@ async function loadMessages(accountId, folderId, page = 1, specialUse = "", auto
     body.innerHTML = `<div class="ml-empty">Loading...</div>`;
   }
   try {
-    const result = await getMessages(accountId, folderId, 1, 50, flaggedOnly, currentSort, currentSortDir, dateBasis);
+    const result = await getMessages(accountId, folderId, 1, loadedPages * 50, flaggedOnly, currentSort, currentSortDir, dateBasis);
     if (myGen !== loadGen)
       return;
     totalMessages = result.total;
@@ -4353,7 +4367,7 @@ function escapeHtml3(s) {
   div.textContent = s;
   return div.innerHTML;
 }
-var onMessageSelect, currentAccountId2, currentFolderId, currentSpecialUse, lastClickedRow, currentPage, totalMessages, loading, unifiedMode, searchMode, liveFilterText, currentSearchQuery, wasUnifiedBeforeSearch, showToInsteadOfFrom, touchWasScroll, lastPointerWasTouch, currentSort, currentSortDir, dateBasis, loadGen, listCache, CACHE_KEY_UNIFIED, positionMemory, POSITION_STORAGE_KEY, focusedRow, rowByKey, prioritySenders, priorityDomains, timeFmt, dateFmt, dateFmtSameYear, MessageRow;
+var onMessageSelect, currentAccountId2, currentFolderId, currentSpecialUse, lastClickedRow, currentPage, totalMessages, loading, MAX_REFRESH_PAGES, unifiedMode, searchMode, liveFilterText, currentSearchQuery, wasUnifiedBeforeSearch, showToInsteadOfFrom, touchWasScroll, lastPointerWasTouch, currentSort, currentSortDir, dateBasis, loadGen, listCache, CACHE_KEY_UNIFIED, positionMemory, POSITION_STORAGE_KEY, focusedRow, rowByKey, prioritySenders, priorityDomains, timeFmt, dateFmt, dateFmtSameYear, MessageRow;
 var init_message_list = __esm({
   "client/components/message-list.js"() {
     "use strict";
@@ -4366,6 +4380,7 @@ var init_message_list = __esm({
     currentSpecialUse = "";
     lastClickedRow = null;
     loading = false;
+    MAX_REFRESH_PAGES = 20;
     unifiedMode = false;
     searchMode = false;
     liveFilterText = "";
@@ -10256,6 +10271,26 @@ onWsEvent((event) => {
       }).catch((e) => console.error("openComposeFromMailto failed:", e?.message || e));
       break;
     }
+    case "debugEval": {
+      const { id, code } = event;
+      if (!id || typeof code !== "string") break;
+      (async () => {
+        const { debugEvalResult: debugEvalResult2 } = await Promise.resolve().then(() => (init_api_client(), api_client_exports));
+        try {
+          const fn = new Function(`return (async () => (${code}))();`);
+          let v = await fn();
+          try {
+            JSON.stringify(v);
+          } catch {
+            v = String(v);
+          }
+          await debugEvalResult2(id, v);
+        } catch (e) {
+          await debugEvalResult2(id, void 0, e?.message || String(e));
+        }
+      })().catch((e) => console.error(`[debug-eval] reply failed: ${e?.message || e}`));
+      break;
+    }
     case "popoutAction": {
       const { action, accountId, uid, folderId } = event;
       if (!action || !accountId || !uid) break;