@bobfrankston/mailx-imap 0.1.59 → 0.1.61

package/index.js

@@ -2597,6 +2597,14 @@ export class ImapManager extends EventEmitter {
      *  the user clicked, they're waiting, this jumps ahead of any background
      *  prefetch sitting in the slow lane. */
     async fetchMessageBody(accountId, folderId, uid) {
+        // Belt-and-braces against `UID FETCH 0`. The IMAP server rejects it as
+        // BAD "Invalid uidset" and the connection slot is consumed for the
+        // round-trip. The enqueue path now guards too — this catches direct
+        // callers that bypass the queue (Bob 2026-05-25).
+        if (!uid || !Number.isFinite(uid) || uid <= 0) {
+            console.error(`[imap] fetchMessageBody rejected invalid uid=${uid} for ${accountId}/folder${folderId}`);
+            return null;
+        }
         const envelope = this.db.getMessageByUid(accountId, uid, folderId);
         let storedPath = envelope?.bodyPath || "";
         if (!storedPath)
@@ -4441,7 +4449,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)
@@ -4485,7 +4505,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.59",
+  "version": "0.1.61",
   "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.35",
+    "@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.35",
+      "@bobfrankston/mailx-store": "^0.1.37",
       "@bobfrankston/iflow-direct": "^0.1.50",
       "@bobfrankston/tcp-transport": "^0.1.6",
       "@bobfrankston/smtp-direct": "^0.1.8",