@bobfrankston/mailx 1.0.351 → 1.0.353

package/client/app.js

@@ -5,7 +5,7 @@
 import { initFolderTree, refreshFolderTree, updateFolderCounts, setFolderSynced, getFolderSynced } from "./components/folder-tree.js";
 import { initMessageList, loadMessages, loadUnifiedInbox, loadSearchResults, reloadCurrentFolder, getSelectedMessages, markBodiesCached } from "./components/message-list.js";
 import { showMessage, getCurrentMessage, initViewer } from "./components/message-viewer.js";
-import { connectWebSocket, onWsEvent, triggerSync, syncAccount, reauthenticate, getAccounts, getFolders, deleteMessages, undeleteMessage, restartServer, getSyncPending, getVersion, getSettings, saveSettings, getAutocompleteSettings, saveAutocompleteSettings, repairAccounts, updateFlags, markAsSpamMessages } from "./lib/api-client.js";
+import { connectWebSocket, onWsEvent, triggerSync, syncAccount, reauthenticate, getAccounts, getFolders, deleteMessages, undeleteMessage, restartServer, getSyncPending, getVersion, getSettings, saveSettings, getAutocompleteSettings, saveAutocompleteSettings, repairAccounts, updateFlags, markAsSpamMessages, logClientEvent } from "./lib/api-client.js";
 import * as messageState from "./lib/message-state.js";
 // ── New message badge (favicon + title) ──
 let baseTitle = "mailx";
@@ -524,6 +524,7 @@ document.getElementById("btn-factory-reset")?.addEventListener("click", async ()
     }
 });
 async function openCompose(mode) {
+    logClientEvent("openCompose-entry", { mode });
     const current = getCurrentMessage();
     // Local-first: if the row is selected we already have its headers in the
     // local DB. Populate the compose form unconditionally; the user can edit

package/client/compose/compose.js

@@ -4,9 +4,13 @@
  * Receives init data via window.opener.postMessage or URL params.
  */
 import { createEditor } from "./editor.js";
-import { getSettings, getAccounts, searchContacts, sendMessage, saveDraft as apiSaveDraft, deleteDraft } from "../lib/api-client.js";
+import { getSettings, getAccounts, searchContacts, sendMessage, saveDraft as apiSaveDraft, deleteDraft, logClientEvent } from "../lib/api-client.js";
+// Very first line the iframe runs — if this doesn't reach Node, the iframe
+// itself isn't loading or the bridge is completely broken.
+logClientEvent("compose-module-loaded", { href: location.href, version: window.mailxVersion || "?" });
 /** Close compose window */
 function closeCompose() {
+    logClientEvent("compose-close");
     window.close();
 }
 // ── Load editor scripts dynamically ──
@@ -576,6 +580,11 @@ window.addEventListener("blur", () => {
     catch { /* */ }
 });
 document.getElementById("btn-send")?.addEventListener("click", () => {
+    // Loud tracing through the whole send pipeline. Every step ships a
+    // `[client] compose-send-*` event to the Node log so a "vanished message"
+    // report can be traced end-to-end without devtools. If the log stops
+    // at any step, that's where the pipeline broke.
+    logClientEvent("compose-send-click");
     const body = {
         from: getFromAccountId(),
         fromAddress: getFromAddress(),
@@ -587,10 +596,12 @@ document.getElementById("btn-send")?.addEventListener("click", () => {
         bodyText: editor.getText(),
         attachments: attachments.map(a => ({ filename: a.filename, mimeType: a.mimeType, dataBase64: a.dataBase64 })),
     };
+    logClientEvent("compose-send-body-built", { from: body.from, toCount: body.to.length, subjectLen: (body.subject || "").length, bodyHtmlLen: (body.bodyHtml || "").length, atts: body.attachments.length });
     // Local validity (one missing-To check) — must run before close so the
     // user gets an inline error instead of silent loss. Anything else (real
     // address validation, MIME assembly, disk write) happens server-side.
     if (!body.to.length) {
+        logClientEvent("compose-send-rejected-no-to");
         alert("Please add at least one To recipient.");
         return;
     }
@@ -611,11 +622,14 @@ document.getElementById("btn-send")?.addEventListener("click", () => {
         statusEl.textContent = "";
     const sendStart = Date.now();
     let ipcPromise;
+    logClientEvent("compose-send-pre-ipc");
     try {
         ipcPromise = sendMessage(body);
+        logClientEvent("compose-send-ipc-invoked", { promiseType: typeof ipcPromise, isThenable: !!(ipcPromise && typeof ipcPromise.then === "function") });
     }
     catch (e) {
         const msg = e?.message || String(e);
+        logClientEvent("compose-send-sync-throw", { error: msg });
         console.error(`[compose] Send threw synchronously: ${msg}`);
         if (sendBtn) {
             sendBtn.disabled = false;
@@ -629,6 +643,7 @@ document.getElementById("btn-send")?.addEventListener("click", () => {
     }
     Promise.resolve(ipcPromise)
         .then(() => {
+        logClientEvent("compose-send-ipc-resolved", { ms: Date.now() - sendStart });
         console.log(`[compose] Send IPC returned OK in ${Date.now() - sendStart}ms`);
         // Stop autosave only after ACK — if send threw we want the draft
         // autosave to keep the message safe.
@@ -643,6 +658,7 @@ document.getElementById("btn-send")?.addEventListener("click", () => {
     })
         .catch((e) => {
         const msg = e?.message || String(e);
+        logClientEvent("compose-send-ipc-rejected", { error: msg, ms: Date.now() - sendStart });
         console.error(`[compose] Send IPC failed after ${Date.now() - sendStart}ms: ${msg}`);
         if (sendBtn) {
             sendBtn.disabled = false;

package/client/lib/api-client.js

@@ -129,6 +129,20 @@ export function deleteFolder(accountId, folderId) {
 export function emptyFolder(accountId, folderId) {
     return ipc().emptyFolder?.(accountId, folderId);
 }
+/** Ship a named event to the Node log as `[client] <tag> <data>`. Fire and
+ *  forget — never awaits, never throws, never blocks the caller. If the IPC
+ *  bridge is unavailable, the call is a no-op so tracing calls scattered
+ *  through the UI don't become failure points themselves. */
+export function logClientEvent(tag, data) {
+    try {
+        const bridge = typeof globalThis.mailxapi !== "undefined" && globalThis.mailxapi?.isApp ? globalThis.mailxapi
+            : window.opener?.mailxapi?.isApp ? window.opener.mailxapi
+                : window.parent?.mailxapi?.isApp ? window.parent.mailxapi
+                    : null;
+        bridge?.logClientEvent?.(tag, data);
+    }
+    catch { /* never throw from tracing */ }
+}
 export function sendMessage(body) {
     return ipc().sendMessage?.(body);
 }

package/client/lib/mailxapi.js

@@ -86,6 +86,12 @@
             return callNode("undeleteMessage", { accountId: accountId, uid: uid, folderId: folderId });
         },
 
+        // Diagnostic tracing — callers (compose iframe, app, components) can
+        // ship arbitrary named events to the Node log. Surfaces as
+        // `[client] <tag> <data>` so a failing pipeline can be traced end
+        // to end in a single log file.
+        logClientEvent: function(tag, data) { return callNode("logClientEvent", { tag: tag, data: data }); },
+
         // Compose
         sendMessage: function(msg) { return callNode("sendMessage", msg); },
         saveDraft: function(params) { return callNode("saveDraft", params); },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx",
-  "version": "1.0.351",
+  "version": "1.0.353",
   "description": "Local-first email client with IMAP sync and standalone native app",
   "type": "module",
   "main": "bin/mailx.js",
@@ -20,7 +20,7 @@
     "postinstall": "node bin/postinstall.js"
   },
   "dependencies": {
-    "@bobfrankston/iflow-direct": "^0.1.25",
+    "@bobfrankston/iflow-direct": "^0.1.26",
     "@bobfrankston/iflow-node": "^0.1.7",
     "@bobfrankston/miscinfo": "^1.0.9",
     "@bobfrankston/oauthsupport": "^1.0.24",
@@ -84,7 +84,7 @@
   },
   ".transformedSnapshot": {
     "dependencies": {
-      "@bobfrankston/iflow-direct": "^0.1.25",
+      "@bobfrankston/iflow-direct": "^0.1.26",
       "@bobfrankston/iflow-node": "^0.1.7",
       "@bobfrankston/miscinfo": "^1.0.9",
       "@bobfrankston/oauthsupport": "^1.0.24",

package/packages/mailx-service/jsonrpc.js

@@ -128,6 +128,14 @@ async function dispatchAction(svc, action, p) {
             return { content: await svc.readConfigHelp(p.name) };
         case "unsubscribeOneClick":
             return await svc.unsubscribeOneClick(p.url);
+        // Client-side tracing — lets webview / iframe code ship events to the
+        // Node log so a "compose→send→vanished" report can be diagnosed without
+        // opening devtools. Every call shows up as `[client] <tag> <data>` in
+        // the main log. Keep it on the top of the switch so it's cheap + first
+        // to dispatch.
+        case "logClientEvent":
+            console.log(`  [client] ${p.tag || "?"}${p.data ? " " + JSON.stringify(p.data).slice(0, 400) : ""}`);
+            return { ok: true };
         // Settings
         case "getSettings":
             return svc.getSettings();