@bobfrankston/mailx-imap 0.1.97 → 0.1.99

package/index.js

@@ -1197,6 +1197,7 @@ export class ImapManager extends EventEmitter {
                         to: toEmailAddresses(msg.to || []),
                         cc: toEmailAddresses(msg.cc || []),
                         flags, size: msg.size || 0, hasAttachments, preview, bodyPath,
+                        exclusive: true, // IMAP path → one folder per message
                     },
                     ftsBody,
                 });
@@ -1360,6 +1361,7 @@ export class ImapManager extends EventEmitter {
         // where a full UID SEARCH is cheap and reveals server-side deletes
         // of older messages).
         let statusMessageCount = null;
+        let serverUidNext = null;
         if (highestUid > 0) {
             try {
                 console.log(`  [sync-status] ${accountId}/${folder.path}: calling STATUS...`);
@@ -1370,6 +1372,7 @@ export class ImapManager extends EventEmitter {
                 if (status && typeof status.messages === "number")
                     statusMessageCount = status.messages;
                 if (status && typeof status.uidNext === "number") {
+                    serverUidNext = status.uidNext;
                     const serverHighest = status.uidNext - 1;
                     const noNewUids = serverHighest <= highestUid;
                     const countsMatch = typeof status.messages === "number" && status.messages === localCount;
@@ -1755,6 +1758,7 @@ export class ImapManager extends EventEmitter {
                         hasAttachments: parsed.hasAttachments,
                         preview: parsed.preview,
                         bodyPath,
+                        exclusive: true, // IMAP path → one folder per message
                     },
                     ftsBody: parsed.bodyText || "",
                 });
@@ -1811,6 +1815,36 @@ export class ImapManager extends EventEmitter {
         //      data (the "ubiquiti letter disappeared after reply" case had no trace).
         let deletedCount = 0;
         if (!firstSync) {
+            // PHANTOM SWEEP (Bob 2026-06-17). A local row whose uid >= the server's
+            // UIDNEXT is PROVABLY impossible: UIDNEXT is the next uid the server
+            // will ever assign in this folder, so any uid >= it was never assigned
+            // here. These are cross-wired rows ("UID is not identity" corruption —
+            // CEBog had 106/120 rows with uid >= UIDNEXT=94, real messages stamped
+            // with the wrong folder_id). They break prefetch: it asks the server
+            // for uids the folder doesn't have → 0 FETCH responses → no body ever
+            // caches. The normal set-diff reconcile WON'T remove them because the
+            // 50%-deletion safety guard refuses (88% of CEBog would delete). But
+            // these uids can't be a real message, a move target, or a transient
+            // server hiccup — so delete them unconditionally (the real copy lives
+            // in its true folder and re-syncs from there; the local store is a
+            // cache). Only runs when STATUS gave a trustworthy UIDNEXT.
+            if (serverUidNext != null && serverUidNext > 0) {
+                try {
+                    const phantoms = this.db.getUidsForFolder(accountId, folderId).filter(u => u >= serverUidNext);
+                    if (phantoms.length > 0) {
+                        for (const uid of phantoms) {
+                            this.unlinkBodyFile(accountId, uid, folderId).catch(() => { });
+                            this.db.deleteMessage(accountId, folderId, uid, `phantom: uid>=server UIDNEXT ${serverUidNext}`, `mailx-imap phantom sweep (${folder.path})`);
+                        }
+                        deletedCount += phantoms.length;
+                        this.db.recalcFolderCounts(folderId);
+                        console.log(`  [phantom-sweep] ${accountId}/${folder.path}: removed ${phantoms.length} cross-wired row(s) with uid >= UIDNEXT ${serverUidNext}`);
+                    }
+                }
+                catch (e) {
+                    console.error(`  [phantom-sweep] ${accountId}/${folder.path}: ${e?.message || e}`);
+                }
+            }
             try {
                 // Reuse the server UID list set-diff already fetched.
                 // Without this we made TWO `UID SEARCH` calls per folder

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.97",
+  "version": "0.1.99",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",
@@ -11,8 +11,8 @@
   "dependencies": {
     "@bobfrankston/mailx-types": "^0.1.19",
     "@bobfrankston/mailx-settings": "^0.1.28",
-    "@bobfrankston/mailx-store": "^0.1.51",
-    "@bobfrankston/iflow-direct": "^0.1.54",
+    "@bobfrankston/mailx-store": "^0.1.53",
+    "@bobfrankston/iflow-direct": "^0.1.55",
     "@bobfrankston/tcp-transport": "^0.1.7",
     "@bobfrankston/smtp-direct": "^0.1.9",
     "@bobfrankston/mailx-sync": "^0.1.22",
@@ -39,8 +39,8 @@
     "dependencies": {
       "@bobfrankston/mailx-types": "^0.1.19",
       "@bobfrankston/mailx-settings": "^0.1.28",
-      "@bobfrankston/mailx-store": "^0.1.51",
-      "@bobfrankston/iflow-direct": "^0.1.54",
+      "@bobfrankston/mailx-store": "^0.1.53",
+      "@bobfrankston/iflow-direct": "^0.1.55",
       "@bobfrankston/tcp-transport": "^0.1.7",
       "@bobfrankston/smtp-direct": "^0.1.9",
       "@bobfrankston/mailx-sync": "^0.1.22",