@bobfrankston/mailx-imap 0.1.96 → 0.1.97

package/index.js

@@ -3430,27 +3430,19 @@ export class ImapManager extends EventEmitter {
                                             }
                                         }
                                         catch (e) {
-                                            const emsg = String(e?.message || e);
-                                            // "IMAP command leaked into response stream" /
-                                            // "corrupt body" means the byte-buffer parser lost
-                                            // sync — a partial/flaky read shifted a literal
-                                            // boundary so this FETCH body captured a later
-                                            // command's bytes. The MESSAGE is probably fine; the
-                                            // CONNECTION is poisoned (every subsequent read on
-                                            // it is now garbage). Re-throw so withConnection
-                                            // discards this client and the next chunk reconnects
-                                            // clean — instead of marking a good message as poison
-                                            // and limping on a dead socket until the 300s timeout,
-                                            // which left bobma INBOX stuck and "no new mail"
-                                            // (Bob 2026-06-15). Do NOT back the UID off; it
-                                            // retries next tick on a clean connection.
-                                            if (/command leaked|corrupt body|desync/i.test(emsg)) {
-                                                console.error(`  [prefetch] ${accountId}/${uid}: connection desync — discarding socket to resync: ${emsg.slice(0, 90)}`);
-                                                throw new Error(`imap-desync: ${emsg.slice(0, 80)}`);
-                                            }
-                                            console.error(`  [prefetch] ${accountId}/${uid}: store write failed: ${emsg}`);
-                                            // Genuine poison body (not a desync): back off
-                                            // 5m→30m→2h→12h so it isn't re-fetched on a loop.
+                                            console.error(`  [prefetch] ${accountId}/${uid}: store write failed: ${e?.message || e}`);
+                                            // "IMAP command leaked into response stream" is NOT
+                                            // a live connection desync — the DESYNC CAPTURE
+                                            // (2026-06-16) proved the parser is perfectly aligned
+                                            // (server announces BODY[] {N}, we read exactly N).
+                                            // It's a message whose body is PERMANENTLY corrupt on
+                                            // the server (mailx APPENDed it with a leaked command
+                                            // prepended in some past session). Discarding the
+                                            // connection (my 1.2.12 mistake) just re-fetches the
+                                            // same bad message and thrashes the socket — which
+                                            // disrupted the SHARED slow-lane connection that sync
+                                            // uses. Correct handling: back this one message off
+                                            // (5m→30m→2h→12h) and leave the connection alone.
                                             this.markPrefetchEmpty(accountId, folderId, uid);
                                         }
                                     })());
@@ -3496,9 +3488,9 @@ export class ImapManager extends EventEmitter {
                             // (markPrefetchEmpty is per-message poison only) —
                             // they retry next tick. Genuine per-body errors are
                             // handled per-UID above and never reach here.
-                            const dead = /not connected|ETIMEDOUT|ECONNRESET|ECONNREFUSED|EPIPE|socket|connection (closed|reset|lost)|disconnect|imap-desync/i.test(msg);
+                            const dead = /not connected|ETIMEDOUT|ECONNRESET|ECONNREFUSED|EPIPE|socket|connection (closed|reset|lost)|disconnect/i.test(msg);
                             if (dead) {
-                                console.log(`  [prefetch] ${accountId}/${folder.path}: connection down/desync — deferring rest to next tick (fresh socket)`);
+                                console.log(`  [prefetch] ${accountId}/${folder.path}: connection down — deferring rest to next tick (fresh socket)`);
                                 break;
                             }
                             if (counters.errors >= ERROR_BUDGET)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.96",
+  "version": "0.1.97",
   "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.50",
-    "@bobfrankston/iflow-direct": "^0.1.53",
+    "@bobfrankston/mailx-store": "^0.1.51",
+    "@bobfrankston/iflow-direct": "^0.1.54",
     "@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.50",
-      "@bobfrankston/iflow-direct": "^0.1.53",
+      "@bobfrankston/mailx-store": "^0.1.51",
+      "@bobfrankston/iflow-direct": "^0.1.54",
       "@bobfrankston/tcp-transport": "^0.1.7",
       "@bobfrankston/smtp-direct": "^0.1.9",
       "@bobfrankston/mailx-sync": "^0.1.22",