@bobfrankston/iflow-direct 0.1.53 → 0.1.55

package/imap-native.js

@@ -553,10 +553,25 @@ export class NativeImapClient {
             return;
         if (this.selectedMailbox !== folderPath)
             await this.select(folderPath);
-        await this.fetchMessagesStream(uids.join(","), { source: true, headers: false }, (msg) => {
-            if (msg.uid && msg.source)
+        // headers default (was `headers: false`): the single-message body fetch
+        // path (fetchMessage → fetchMessagesStream, headers DEFAULT) extracts
+        // BODY[] correctly, but this batch path passed `headers: false` and
+        // returned 0 bodies 100% of the time — so prefetch never cached anything
+        // and bodies only landed when the user opened a message (Bob 2026-06-17
+        // "predownloading still broken"). The only difference between the two
+        // paths was this flag; matching the working path. Diagnostic counters
+        // confirm extraction so a regression is visible in the log.
+        let got = 0, sawResp = 0;
+        await this.fetchMessagesStream(uids.join(","), { source: true }, (msg) => {
+            sawResp++;
+            if (msg.uid && msg.source) {
+                got++;
                 onBody(msg.uid, msg.source);
+            }
         });
+        if (got === 0 && uids.length > 0) {
+            console.error(`  [imap] fetchBodiesBatch ${folderPath}: 0/${uids.length} bodies (saw ${sawResp} FETCH responses) — body literal not extracted`);
+        }
     }
     /** Fetch messages since a UID */
     async fetchSinceUid(sinceUid, options = {}, onChunk) {
@@ -1244,6 +1259,23 @@ export class NativeImapClient {
                 // decode happens once per literal, not once per chunk that
                 // arrives during the literal's transit.
                 let literal = this.utf8Decoder.decode(literalBytes);
+                // DESYNC CAPTURE (Bob 2026-06-16, overnight debug). A BODY[]
+                // literal whose content is itself an IMAP command tag means the
+                // byte-buffer parser misaligned: the declared {N} didn't match
+                // reality and we swallowed a later command's bytes. Dump the
+                // declared size + the misaligned content + the raw bytes that
+                // PRECEDED the literal, so the mechanism (how the {N} boundary
+                // slipped) is finally visible. Cheap; only fires on the desync.
+                if (this.pendingCommand.currentLiteralKey?.startsWith("BODY")
+                    && /^[A-Za-z]+\d+\s+(SELECT|FETCH|UID|LOGIN|STATUS|EXAMINE|LIST|IDLE|NOOP|STORE|SEARCH|CAPABILITY|CLOSE|LOGOUT|APPEND|COPY|MOVE|EXPUNGE|CREATE|DELETE|RENAME|SUBSCRIBE|ENABLE|STARTTLS)\b/i.test(literal.slice(0, 80).replace(/^[\s*]+/, ""))) {
+                    const litStart = this.bufferOffset - neededBytes;
+                    const ctxStart = Math.max(0, litStart - 256);
+                    const ctx = this.buffer.subarray(ctxStart, litStart);
+                    const ascii = Array.from(ctx).map(b => (b >= 32 && b < 127) ? String.fromCharCode(b) : (b === 13 ? "\\r" : b === 10 ? "\\n" : ".")).join("");
+                    console.error(`  [imap] DESYNC CAPTURE key=${this.pendingCommand.currentLiteralKey} declaredLiteral=${neededBytes}B bufferOffset=${this.bufferOffset} bufferLength=${this.bufferLength} litStart=${litStart}`);
+                    console.error(`  [imap] DESYNC literal[0..200]=${JSON.stringify(literal.slice(0, 200))}`);
+                    console.error(`  [imap] DESYNC preceding[${ctx.length}B]=${JSON.stringify(ascii)}`);
+                }
                 // For non-BODY literals (e.g. display names in ENVELOPE), wrap in quotes
                 // so tokenizeParenList treats them as a single token
                 if (!this.pendingCommand.currentLiteralKey) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/iflow-direct",
-  "version": "0.1.53",
+  "version": "0.1.55",
   "description": "Direct IMAP client — transport-agnostic, no Node.js dependencies, browser-ready",
   "main": "index.js",
   "types": "index.ts",
@@ -19,7 +19,7 @@
   "author": "Bob Frankston",
   "license": "ISC",
   "dependencies": {
-    "@bobfrankston/tcp-transport": "^0.1.6"
+    "@bobfrankston/tcp-transport": "^0.1.7"
   },
   "exports": {
     ".": {
@@ -50,7 +50,7 @@
   },
   ".transformedSnapshot": {
     "dependencies": {
-      "@bobfrankston/tcp-transport": "^0.1.6"
+      "@bobfrankston/tcp-transport": "^0.1.7"
     }
   }
 }