@bobfrankston/iflow 1.0.55 → 1.0.57

package/imaplib/imap-native.js

@@ -293,7 +293,8 @@ export class NativeImapClient {
         const uids = await this.search(`UID ${sinceUid + 1}:*`);
         if (uids.length === 0)
             return [];
-        console.log(`  [fetch] ${uids.length} UIDs since ${sinceUid}`);
+        uids.reverse(); // Newest first
+        console.log(`  [fetch] ${uids.length} UIDs since ${sinceUid} (newest first)`);
         if (uids.length <= NativeImapClient.INITIAL_CHUNK_SIZE) {
             const msgs = await this.fetchMessages(uids.join(","), options);
             if (onChunk)
@@ -323,7 +324,9 @@ export class NativeImapClient {
         const uids = await this.search(criteria);
         if (uids.length === 0)
             return [];
-        console.log(`  [fetch] ${uids.length} UIDs to fetch`);
+        // Reverse so newest messages (highest UIDs) come first
+        uids.reverse();
+        console.log(`  [fetch] ${uids.length} UIDs to fetch (newest first)`);
         const allMessages = [];
         let chunkSize = NativeImapClient.INITIAL_CHUNK_SIZE;
         for (let i = 0; i < uids.length; i += chunkSize) {
@@ -521,11 +524,18 @@ export class NativeImapClient {
     }
     processBuffer() {
         while (true) {
-            // Check for literal {size}\r\n — reading exact byte count of literal data
+            // Check for literal {size}\r\n — reading exact BYTE count of literal data
+            // CRITICAL: literalBytes is in octets (from IMAP {N}), but this.buffer is a
+            // JavaScript string where multi-byte UTF-8 characters count as 1 character.
+            // We must use Buffer.byteLength to find the correct character boundary.
             if (this.pendingCommand?.literalBytes != null) {
-                if (this.buffer.length >= this.pendingCommand.literalBytes) {
-                    let literal = this.buffer.substring(0, this.pendingCommand.literalBytes);
-                    this.buffer = this.buffer.substring(this.pendingCommand.literalBytes);
+                const neededBytes = this.pendingCommand.literalBytes;
+                const bufferBytes = Buffer.byteLength(this.buffer, "utf-8");
+                if (bufferBytes >= neededBytes) {
+                    // Find the character position that corresponds to the byte boundary
+                    const buf = Buffer.from(this.buffer, "utf-8");
+                    let literal = buf.subarray(0, neededBytes).toString("utf-8");
+                    this.buffer = buf.subarray(neededBytes).toString("utf-8");
                     // 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) {
@@ -549,7 +559,7 @@ export class NativeImapClient {
                     continue;
                 }
                 if (this.verbose && this.pendingCommand.literalBytes > 0) {
-                    console.log(`  [imap] waiting for literal: need ${this.pendingCommand.literalBytes}, have ${this.buffer.length}`);
+                    console.log(`  [imap] waiting for literal: need ${neededBytes} bytes, have ${bufferBytes} bytes`);
                 }
                 break; // Wait for more data
             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/iflow",
-  "version": "1.0.55",
+  "version": "1.0.57",
   "description": "IMAP client wrapper library",
   "main": "index.js",
   "types": "index.ts",