@bobfrankston/iflow-direct 0.1.47 → 0.1.49

package/imap-native.js

@@ -467,19 +467,27 @@ export class NativeImapClient {
             items.push("BODY.PEEK[]");
         const tag = proto.nextTag();
         const streamed = [];
-        const cb = onMessage
-            ? (resp) => {
-                if (resp.tag !== "*" || resp.type !== "FETCH")
-                    return;
-                const parsed = this.parseFetchResponses([resp]);
-                for (const msg of parsed) {
-                    streamed.push(msg);
+        // ALWAYS parse incrementally in the per-response callback — even
+        // when the caller passed no `onMessage`. The callback runs from
+        // processBuffer's 10 ms time-budgeted loop, so envelope parsing of a
+        // large FETCH (a sync-backfill chunk of 100+ messages) is sliced
+        // across event-loop turns. The old `onMessage ? stream : batch`
+        // path parsed every envelope in ONE synchronous block after the
+        // tagged OK — profiled 2026-05-15 as a ~30 s daemon wedge
+        // (tokenizeParenList ~40 % of CPU), starving every getMessage /
+        // move / delete IPC for the duration.
+        const cb = (resp) => {
+            if (resp.tag !== "*" || resp.type !== "FETCH")
+                return;
+            const parsed = this.parseFetchResponses([resp]);
+            for (const msg of parsed) {
+                streamed.push(msg);
+                if (onMessage)
                     onMessage(msg);
-                }
             }
-            : undefined;
-        const responses = await this.sendCommand(tag, proto.fetchCommand(tag, range, items), cb);
-        return onMessage ? streamed : this.parseFetchResponses(responses);
+        };
+        await this.sendCommand(tag, proto.fetchCommand(tag, range, items), cb);
+        return streamed;
     }
     /**
      * Folder-scoped batch body fetch. Selects `folderPath`, issues a single

package/imap-protocol.js

@@ -422,6 +422,16 @@ function tokenizeParenList(s) {
             let j = i;
             while (j < end && str[j] !== " " && str[j] !== ")" && str[j] !== "(")
                 j++;
+            if (j === i) {
+                // str[i] is a stray ')' (or '(') that none of the branches
+                // above consumed — an unbalanced/malformed parenthesised
+                // list (Dovecot mis-parses some Outlook `From:` headers and
+                // emits exactly this). Without advancing, `i` is stuck and
+                // the while-loop spins FOREVER, pinning a CPU core and
+                // wedging the whole daemon event loop. Skip the stray char.
+                i++;
+                continue;
+            }
             tokens.push(str.substring(i, j));
             i = j;
         }

package/package.json

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