@bobfrankston/iflow-direct 0.1.50 → 0.1.52

package/imap-native.js

@@ -132,8 +132,29 @@ export class NativeImapClient {
             }
         });
         this.transport.onError((err) => {
+            // Diagnose transport-level errors fully. Node emits socket errors
+            // with a populated `.message` in normal cases (ECONNRESET, ETIMEDOUT,
+            // etc.) — but TLS-layer errors, abrupt FINs surfaced as errors,
+            // and some wrapper paths produce an Error with empty `.message`
+            // and the useful info on `.code` / `.errno` / `.syscall` instead.
+            // Logging only `.message` left "Transport error: " in the log
+            // with no diagnostic content, which masked why bobma INBOX prefetch
+            // kept dying (2026-05-27). Build a richer description.
+            const errAny = err;
+            const parts = [];
+            if (errAny?.message)
+                parts.push(String(errAny.message));
+            if (errAny?.code)
+                parts.push(`code=${errAny.code}`);
+            if (errAny?.errno !== undefined)
+                parts.push(`errno=${errAny.errno}`);
+            if (errAny?.syscall)
+                parts.push(`syscall=${errAny.syscall}`);
+            const desc = parts.length > 0
+                ? parts.join(" ")
+                : `<no message> ${typeof err} ${err?.constructor?.name || ""}`.trim();
             if (this.verbose)
-                console.error(`  [imap] Transport error: ${err.message}`);
+                console.error(`  [imap] Transport error: ${desc}`);
             // Transport errors (DNS failure, ECONNRESET, TLS failure, etc.) mean
             // the connection is dead — clear the flag so ensureConnected() will
             // reconnect on the next call, and drop the stale IDLE state so we
@@ -149,7 +170,17 @@ export class NativeImapClient {
             if (this.pendingCommand) {
                 const { reject } = this.pendingCommand;
                 this.pendingCommand = null;
-                reject(err);
+                // Make sure the error that propagates to upper-layer catches
+                // (prefetch, sync, etc.) has a non-empty message — wrap if
+                // the underlying err has a blank message.
+                if (err instanceof Error && !err.message) {
+                    const wrapped = new Error(desc);
+                    wrapped.cause = err;
+                    reject(wrapped);
+                }
+                else {
+                    reject(err);
+                }
             }
         });
         await this.transport.connect(this.config.server, this.config.port, useTls, this.config.server);
@@ -325,8 +356,15 @@ export class NativeImapClient {
                 // is the QRESYNC case; without it, the message just vanished
                 // mid-session (post-EXPUNGE on another client).
                 const text = r.text.replace(/^\(EARLIER\)\s*/i, "").trim();
-                for (const uid of proto.parseUidSet(text))
+                // Cap expansion (see parseUidSet): a stale-modseq VANISHED range
+                // can be millions of UIDs. Stop at 100k — the caller's bounded
+                // reconcile takes over for an over-cap set rather than trusting
+                // it (Bob 2026-06-04). 100k > any real single-cycle vanish.
+                const VANISHED_EXPAND_CAP = 100_000;
+                for (const uid of proto.parseUidSet(text, VANISHED_EXPAND_CAP))
                     vanishedUids.push(uid);
+                if (vanishedUids.length >= VANISHED_EXPAND_CAP)
+                    break;
             }
         }
         this.selectedMailbox = mailbox;

package/imap-protocol.d.ts

@@ -90,7 +90,7 @@ export declare function enableCommand(tag: string, extensions: string[]): string
  *  when it has one; here `*` is treated as a sentinel and skipped so the
  *  call site can decide what to do (typically: ignore — VANISHED never
  *  emits `*` because that would be open-ended). */
-export declare function parseUidSet(set: string): number[];
+export declare function parseUidSet(set: string, maxExpand?: number): number[];
 /** Build EXAMINE command (read-only SELECT) */
 export declare function examineCommand(tag: string, mailbox: string): string;
 /** Build STATUS command */

package/imap-protocol.js

@@ -56,9 +56,17 @@ export function enableCommand(tag, extensions) {
  *  when it has one; here `*` is treated as a sentinel and skipped so the
  *  call site can decide what to do (typically: ignore — VANISHED never
  *  emits `*` because that would be open-ended). */
-export function parseUidSet(set) {
+export function parseUidSet(set, maxExpand = Infinity) {
+    // maxExpand caps the expanded output. A VANISHED *range* like
+    // `(EARLIER) 1:4828883` (sent when the client's modseq is very stale)
+    // would otherwise expand to MILLIONS of integers — burning CPU/memory
+    // here and wedging the consumer downstream. The caller passes a cap and
+    // treats an at-cap result as "too stale, reconcile some other way"
+    // (Bob 2026-06-04). Default Infinity keeps every existing caller unchanged.
     const out = [];
     for (const part of set.split(",")) {
+        if (out.length >= maxExpand)
+            break;
         const trimmed = part.trim();
         if (!trimmed)
             continue;
@@ -73,7 +81,7 @@ export function parseUidSet(set) {
             const hi = range[1] === "*" ? lo : parseInt(range[1], 10);
             if (!Number.isFinite(lo) || !Number.isFinite(hi) || hi < lo)
                 continue;
-            for (let u = lo; u <= hi; u++)
+            for (let u = lo; u <= hi && out.length < maxExpand; u++)
                 out.push(u);
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/iflow-direct",
-  "version": "0.1.50",
+  "version": "0.1.52",
   "description": "Direct IMAP client — transport-agnostic, no Node.js dependencies, browser-ready",
   "main": "index.js",
   "types": "index.ts",