@bobfrankston/rmfmail 1.1.218 → 1.1.219

package/packages/mailx-imap/index.ts

@@ -1323,13 +1323,20 @@ export class ImapManager extends EventEmitter {
                     // from wiping anything we shouldn't.
                     console.log(`  [qresync] ${accountId}/${folder.path}: UIDVALIDITY changed (was ${prevUidValidity}, now ${qr.exists}); falling back to full sync`);
                 } else if (qr.vanishedUids.length > VANISHED_ABSURD_CAP) {
-                    // Refuse the absurd VANISHED set; do NOT iterate/apply it and
-                    // do NOT return — fall through to the bounded set-diff
-                    // reconcile (same as the UIDVALIDITY-changed path). The
-                    // modseq watermark is left intact, so a healthy later QRESYNC
-                    // can still succeed.
+                    // Refuse the absurd VANISHED set; do NOT iterate/apply it.
                     const localCount = this.db.getMessageCount(accountId, folderId);
-                    console.error(`  [qresync] ${accountId}/${folder.path}: REFUSING VANISHED of ${qr.vanishedUids.length} UIDs (folder has ${localCount} locally) — almost certainly a mis-parsed range; falling back to set-diff reconcile instead of mass-deleting/wedging.`);
+                    console.error(`  [qresync] ${accountId}/${folder.path}: REFUSING VANISHED of ${qr.vanishedUids.length} UIDs (folder has ${localCount} locally) — mis-parsed range; self-healing (advance watermark) + falling back to set-diff.`);
+                    // SELF-HEAL: advance the modseq watermark to the server's
+                    // CURRENT value. Without this the next QRESYNC re-requests the
+                    // same giant `(EARLIER) 1:N` range every cycle forever (Bob saw
+                    // it recur 21:03 → 21:20) — refusing without advancing isn't
+                    // self-healing. The set-diff reconcile below discovers + applies
+                    // the ACTUAL deletions via Message-ID identity (bounded, 50%
+                    // guard), so skipping the literal VANISHED loses nothing; it
+                    // just breaks the stale-modseq loop so future QRESYNCs are clean.
+                    if (qr.newHighestModSeq !== undefined) {
+                        this.db.updateFolderSync(folderId, prevUidValidity, String(qr.newHighestModSeq));
+                    }
                 } else {
                     // Apply VANISHED — server says these UIDs are gone. No
                     // tombstone, no diff, just delete the local rows.

package/packages/mailx-imap/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.83",
+  "version": "0.1.84",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@bobfrankston/mailx-imap",
-      "version": "0.1.83",
+      "version": "0.1.84",
       "license": "ISC",
       "dependencies": {
         "@bobfrankston/iflow-direct": "^0.1.27",

package/packages/mailx-imap/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.83",
+  "version": "0.1.84",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",