@bobfrankston/mailx-imap 0.1.60 → 0.1.62

package/index.js

@@ -744,6 +744,20 @@ export class ImapManager extends EventEmitter {
                     tokenDirectory: tokenDir,
                     credentialsKey: "installed",
                     loginHint: account.imap.user,
+                    // Route info/error to the daemon log. oauthsupport's
+                    // default logFn drops info silently — that masked "full
+                    // reauth keeps firing" with no log trail (Bob 2026-05-25
+                    // "the log should be showing the auth request"). With
+                    // this, "Forcing consent prompt", "No valid token found,
+                    // starting OAuth authentication", "Token refresh failed"
+                    // all land in rmfmail-*.log under [oauth] prefix.
+                    logFn: (type, message) => {
+                        const prefix = `  [oauth ${account.id}]`;
+                        if (type === "error")
+                            console.error(`${prefix} ${message}`);
+                        else
+                            console.log(`${prefix} ${message}`);
+                    },
                 });
                 const timeoutPromise = new Promise((_, reject) => setTimeout(() => reject(new Error(`OAuth token fetch timeout (${TOKEN_FETCH_TIMEOUT_MS / 1000}s)`)), TOKEN_FETCH_TIMEOUT_MS));
                 const result = await Promise.race([authPromise, timeoutPromise]);
@@ -4449,7 +4463,19 @@ export class ImapManager extends EventEmitter {
             return;
         let reconciled = 0;
         let appended = 0;
-        await this.withConnection(accountId, async (client) => {
+        // Use a DEDICATED IMAP client for sent-sweep — do NOT share the
+        // slow-lane ops client. The priority-INBOX sync grabs the slow-lane
+        // client via getOpsClient() (no lane queueing) and runs SEARCH→FETCH
+        // as a logical transaction; if sent-sweep's searchByHeader (SELECT
+        // Sent → SEARCH → CLOSE) interleaves on the same wire between the
+        // sync's SEARCH and FETCH, the FETCH lands with no mailbox selected
+        // and Dovecot returns "BAD No mailbox selected" — new mail never
+        // lands locally (Bob 2026-05-25: "not seeing recent mail").
+        // iflow-direct serializes per-command, not per-task, so reusing the
+        // shared client is unsafe across logical transactions. A dedicated
+        // client is the smallest surgical fix.
+        const client = await this.createClientWithLimit(accountId);
+        try {
             for (const row of rows) {
                 const msgId = row.message_id;
                 if (!msgId)
@@ -4493,7 +4519,17 @@ export class ImapManager extends EventEmitter {
                     }
                 }
             }
-        }, { slow: true, timeoutMs: 120_000 });
+        }
+        finally {
+            try {
+                await (client._realLogout || client.logout)();
+            }
+            catch { /* */ }
+            try {
+                client.destroy?.();
+            }
+            catch { /* */ }
+        }
         if (reconciled + appended > 0) {
             this.emit("folderCountsChanged", accountId, {});
             console.log(`  [sent-sweep] ${accountId}: ${reconciled} rebound, ${appended} re-appended`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.60",
+  "version": "0.1.62",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",
@@ -11,7 +11,7 @@
   "dependencies": {
     "@bobfrankston/mailx-types": "^0.1.18",
     "@bobfrankston/mailx-settings": "^0.1.22",
-    "@bobfrankston/mailx-store": "^0.1.36",
+    "@bobfrankston/mailx-store": "^0.1.37",
     "@bobfrankston/iflow-direct": "^0.1.50",
     "@bobfrankston/tcp-transport": "^0.1.6",
     "@bobfrankston/smtp-direct": "^0.1.8",
@@ -39,7 +39,7 @@
     "dependencies": {
       "@bobfrankston/mailx-types": "^0.1.18",
       "@bobfrankston/mailx-settings": "^0.1.22",
-      "@bobfrankston/mailx-store": "^0.1.36",
+      "@bobfrankston/mailx-store": "^0.1.37",
       "@bobfrankston/iflow-direct": "^0.1.50",
       "@bobfrankston/tcp-transport": "^0.1.6",
       "@bobfrankston/smtp-direct": "^0.1.8",