@bobfrankston/mailx 1.0.289 → 1.0.291

package/bin/mailx.js

@@ -939,6 +939,9 @@ async function main() {
     imapManager.on("syncComplete", (accountId) => {
         handle.send({ _event: "syncComplete", type: "syncComplete", accountId });
     });
+    imapManager.on("syncActionFailed", (accountId, action, uid, error) => {
+        handle.send({ _event: "syncActionFailed", type: "syncActionFailed", accountId, action, uid, error });
+    });
     // 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.
     const { onCloudError } = await import("@bobfrankston/mailx-settings");

package/client/app.js

@@ -608,12 +608,12 @@ async function deleteSelectedMessages() {
         if (selected.length === 1) {
             lastDeleted = { ...selected[0], subject: "" };
             if (statusSync)
-                statusSync.textContent = `Deleted 1 message — Ctrl+Z to undo`;
+                statusSync.textContent = `Trashed 1 message (syncing) — Ctrl+Z to undo`;
         }
         else {
             lastDeleted = null;
             if (statusSync)
-                statusSync.textContent = `Deleted ${selected.length} messages`;
+                statusSync.textContent = `Trashed ${selected.length} messages (syncing)`;
         }
         if (undoTimeout)
             clearTimeout(undoTimeout);
@@ -711,7 +711,7 @@ async function spamSelectedMessages() {
             await markAsSpamMessages(accountId, uids);
         }
         if (statusSync)
-            statusSync.textContent = `Marked ${selected.length} as spam`;
+            statusSync.textContent = `Spam: ${selected.length} queued — pending server sync`;
         messageState.removeMessages(selected);
     }
     catch (e) {
@@ -1019,7 +1019,11 @@ onWsEvent((event) => {
             }
             break;
         case "folderCountsChanged": {
-            // Update folder badges + silently refresh message list (preserves selection and viewer)
+            // Update folder badges + full tree refresh (not just counts) so
+            // newly-synced folders appear. Without this, the first sync after
+            // setup creates DB folders but the tree never picks them up because
+            // syncComplete fires during page reload and gets lost.
+            refreshFolderTree();
             updateFolderCounts();
             updateNewMessageCount();
             // Debounced silent reload — preserves scroll position, selection, and viewer
@@ -1040,6 +1044,14 @@ onWsEvent((event) => {
                 statusSync.textContent = `Synced ${new Date().toLocaleTimeString(undefined, { hour: "2-digit", minute: "2-digit", hour12: false })}`;
             break;
         }
+        case "syncActionFailed": {
+            // Surface sync failures (move/delete/flag not applied on server)
+            // so the user knows local-first actions haven't propagated yet.
+            const action = event.action === "move" ? "Move" : event.action === "delete" ? "Delete" : event.action;
+            if (statusSync)
+                statusSync.textContent = `Sync failed: ${action} — ${event.error}`;
+            break;
+        }
         case "reload":
             location.reload();
             break;

package/client/components/folder-tree.js

@@ -482,22 +482,34 @@ async function loadFolderTree(container) {
                     "aol.com": "Use an app password: AOL Settings → Account Security → Generate app password",
                     "icloud.com": "Use an app-specific password: appleid.apple.com → Sign-In and Security → App-Specific Passwords",
                 };
+                let oauthAutoFired = false;
                 emailInput?.addEventListener("input", () => {
                     const email = emailInput.value.trim();
                     const domain = email.split("@")[1]?.toLowerCase() || "";
                     const hasAt = email.includes("@") && domain.length > 0;
                     const isOAuth = ["gmail.com", "googlemail.com", "outlook.com", "hotmail.com", "live.com"].includes(domain);
-                    // Progressive reveal: email first, then name + submit,
-                    // password only for non-OAuth providers.
+                    const isGmailLike = ["gmail.com", "googlemail.com"].includes(domain);
+                    // OAuth providers: auto-fire setup immediately once domain
+                    // is recognized — don't show name/password (name is auto-
+                    // detected from Google profile, no password needed). This
+                    // eliminates the "form flash" where fields briefly appear
+                    // before the page reloads.
+                    if (hasAt && isOAuth && !oauthAutoFired && !setupTriggered) {
+                        oauthAutoFired = true;
+                        statusEl.textContent = `Connecting to ${isGmailLike ? "Gmail" : "Outlook"}...`;
+                        trySetup();
+                        return;
+                    }
+                    // Non-OAuth: progressive reveal of name + password + submit.
                     const nameRow = document.getElementById("setup-name-row");
                     const pwRow = document.getElementById("setup-password-row");
                     const submitBtn = document.getElementById("setup-submit");
                     if (nameRow)
-                        nameRow.style.display = hasAt ? "block" : "none";
+                        nameRow.style.display = hasAt && !isOAuth ? "block" : "none";
                     if (pwRow)
                         pwRow.style.display = hasAt && !isOAuth ? "block" : "none";
                     if (submitBtn)
-                        submitBtn.style.display = hasAt ? "block" : "none";
+                        submitBtn.style.display = hasAt && !isOAuth ? "block" : "none";
                     const helpEl = document.getElementById("setup-app-password-help");
                     if (helpEl) {
                         const help = APP_PASSWORD_HELP[domain];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx",
-  "version": "1.0.289",
+  "version": "1.0.291",
   "description": "Local-first email client with IMAP sync and standalone native app",
   "type": "module",
   "main": "bin/mailx.js",

package/packages/mailx-imap/index.d.ts

@@ -22,6 +22,7 @@ export interface ImapManagerEvents {
     /** Fired after a message body has been written to the local store — lets
      *  the UI flip a row's "not-downloaded" indicator without re-rendering. */
     bodyCached: (accountId: string, uid: number) => void;
+    syncActionFailed: (accountId: string, action: string, uid: number, error: string) => void;
 }
 export declare class ImapManager extends EventEmitter {
     private configs;

package/packages/mailx-imap/index.js

@@ -2078,9 +2078,11 @@ export class ImapManager extends EventEmitter {
                 catch (e) {
                     console.error(`  [sync] Failed action ${action.action} UID ${action.uid}: ${e.message}`);
                     this.db.failSyncAction(action.id, e.message);
+                    this.emit("syncActionFailed", accountId, action.action, action.uid, e.message);
                     if (action.attempts >= 5) {
                         console.error(`  [sync] Giving up on action ${action.id} after 5 attempts`);
                         this.db.completeSyncAction(action.id);
+                        this.emit("syncActionFailed", accountId, action.action, action.uid, `Gave up after 5 attempts: ${e.message}`);
                     }
                 }
             }