imapflow 1.2.13 → 1.2.14

package/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-    ".": "1.2.13"
+    ".": "1.2.14"
 }

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## [1.2.14](https://github.com/postalsys/imapflow/compare/v1.2.13...v1.2.14) (2026-03-15)
+
+
+### Bug Fixes
+
+* correct getColorFlags treating red (index 0) as falsy and use nullish coalescing in resolveCharset ([268629c](https://github.com/postalsys/imapflow/commit/268629cf799027bf802c0231675cdc2c94a227d4))
+* use nullish coalescing in getFlagColor, match euc-jp in getDecoder, and handle BigInt(0) in uidValidity checks ([713e100](https://github.com/postalsys/imapflow/commit/713e100093edff21df020942a2c17e7b9939b42b))
+
 ## [1.2.13](https://github.com/postalsys/imapflow/compare/v1.2.12...v1.2.13) (2026-03-09)
 
 

package/lib/charsets.js

@@ -291,8 +291,5 @@ CHARACTER_SETS.forEach(entry => {
 // Returns null if the charset is not recognized.
 module.exports.resolveCharset = charset => {
     let key = charset.replace(/[_-\s]/g, '').toLowerCase();
-    if (CHARSET_MAP.has(key)) {
-        return CHARSET_MAP.get(key);
-    }
-    return null;
+    return CHARSET_MAP.get(key) ?? null;
 };

package/lib/commands/append.js

@@ -111,7 +111,7 @@ module.exports = async (connection, destination, content, flags, idate) => {
             if (responseCode.toUpperCase() === 'APPENDUID') {
                 let uidValidity = section[1] && typeof section[1].value === 'string' && !isNaN(section[1].value) ? BigInt(section[1].value) : false;
                 let uid = section[2] && typeof section[2].value === 'string' && !isNaN(section[2].value) ? Number(section[2].value) : false;
-                if (uidValidity) {
+                if (uidValidity !== false) {
                     map.uidValidity = uidValidity;
                 }
                 if (uid) {

package/lib/commands/copy.js

@@ -1,6 +1,7 @@
 'use strict';
 
-const { normalizePath, encodePath, expandRange, enhanceCommandError } = require('../tools.js');
+const { normalizePath, encodePath, enhanceCommandError } = require('../tools.js');
+const { parseCopyUid } = require('./copyuid-parser.js');
 
 /**
  * Copies messages from the current mailbox to another mailbox.
@@ -35,29 +36,7 @@ module.exports = async (connection, range, destination, options) => {
 
         // UIDPLUS (RFC 4315): the server may include a COPYUID response code in the
         // tagged OK response, providing a mapping from source UIDs to destination UIDs.
-        // Format: [COPYUID <uidValidity> <sourceUids> <destUids>]
-        let section = response.response.attributes && response.response.attributes[0] && response.response.attributes[0].section;
-        let responseCode = section && section.length && section[0] && typeof section[0].value === 'string' ? section[0].value : '';
-        switch (responseCode) {
-            case 'COPYUID':
-                {
-                    // section[1] = destination mailbox UIDVALIDITY
-                    let uidValidity = section[1] && typeof section[1].value === 'string' && !isNaN(section[1].value) ? BigInt(section[1].value) : false;
-                    if (uidValidity) {
-                        map.uidValidity = uidValidity;
-                    }
-
-                    // section[2] = source UID set, section[3] = destination UID set
-                    // Both can be ranges (e.g., "1:3") which expandRange() converts to arrays
-                    let sourceUids = section[2] && typeof section[2].value === 'string' ? expandRange(section[2].value) : false;
-                    let destinationUids = section[3] && typeof section[3].value === 'string' ? expandRange(section[3].value) : false;
-                    // Build a source->destination UID map for the caller to track where messages went
-                    if (sourceUids && destinationUids && sourceUids.length === destinationUids.length) {
-                        map.uidMap = new Map(sourceUids.map((uid, i) => [uid, destinationUids[i]]));
-                    }
-                }
-                break;
-        }
+        parseCopyUid(response.response, map);
 
         return map;
     } catch (err) {

package/lib/commands/copyuid-parser.js

@@ -0,0 +1,33 @@
+'use strict';
+
+const { expandRange } = require('../tools.js');
+
+/**
+ * Parses COPYUID response code from an IMAP response (RFC 4315).
+ * Used by both COPY and MOVE commands to extract the UID mapping
+ * from source mailbox to destination mailbox.
+ *
+ * @param {Object} response - IMAP response object with attributes
+ * @param {Object} map - Result map to populate with uidValidity and uidMap
+ */
+function parseCopyUid(response, map) {
+    let section = response.attributes && response.attributes[0] && response.attributes[0].section;
+    let responseCode = section && section.length && section[0] && typeof section[0].value === 'string' ? section[0].value : '';
+
+    if (responseCode !== 'COPYUID') {
+        return;
+    }
+
+    let uidValidity = section[1] && typeof section[1].value === 'string' && !isNaN(section[1].value) ? BigInt(section[1].value) : false;
+    if (uidValidity !== false) {
+        map.uidValidity = uidValidity;
+    }
+
+    let sourceUids = section[2] && typeof section[2].value === 'string' ? expandRange(section[2].value) : false;
+    let destinationUids = section[3] && typeof section[3].value === 'string' ? expandRange(section[3].value) : false;
+    if (sourceUids && destinationUids && sourceUids.length === destinationUids.length) {
+        map.uidMap = new Map(sourceUids.map((uid, i) => [uid, destinationUids[i]]));
+    }
+}
+
+module.exports = { parseCopyUid };

package/lib/commands/idle.js

@@ -90,9 +90,7 @@ async function runIdle(connection) {
                     }
                 }
             },
-            onSend: () => {
-                //idleSent = true;
-            }
+            onSend: () => {}
         });
 
         // Clean up: unset preCheck and resolve any remaining waiters before processing the response.
@@ -206,10 +204,10 @@ module.exports = async (connection, maxIdleTime) => {
 
                 case 'STATUS':
                     {
-                        let statusArgs = [selectCommand.arguments[0], []]; // path
-                        for (let key of ['MESSAGES', 'UIDNEXT', 'UIDVALIDITY', 'UNSEEN']) {
-                            statusArgs[1].push({ type: 'ATOM', value: key.toUpperCase() });
-                        }
+                        let statusArgs = [
+                            selectCommand.arguments[0],
+                            ['MESSAGES', 'UIDNEXT', 'UIDVALIDITY', 'UNSEEN'].map(key => ({ type: 'ATOM', value: key }))
+                        ];
                         connection.log.debug({ src: 'c', msg: `Running STATUS to detect changes in folder` });
                         response = await connection.exec('STATUS', statusArgs);
                     }

package/lib/commands/list.js

@@ -43,7 +43,7 @@ module.exports = async (connection, reference, mailbox, options) => {
 
         // Build the list of STATUS data items to request (MESSAGES, UIDNEXT, etc.)
         if (options.statusQuery) {
-            Object.keys(options.statusQuery || {}).forEach(key => {
+            Object.keys(options.statusQuery).forEach(key => {
                 if (!options.statusQuery[key]) {
                     return;
                 }

package/lib/commands/move.js

@@ -1,6 +1,7 @@
 'use strict';
 
-const { normalizePath, encodePath, expandRange, enhanceCommandError } = require('../tools.js');
+const { normalizePath, encodePath, enhanceCommandError } = require('../tools.js');
+const { parseCopyUid } = require('./copyuid-parser.js');
 
 /**
  * Moves messages from the current mailbox to another mailbox.
@@ -36,30 +37,6 @@ module.exports = async (connection, range, destination, options) => {
         return result;
     }
 
-    // Extract COPYUID response code (UIDPLUS, RFC 4315) from either an untagged
-    // OK response or the final tagged OK. MOVE uses the same COPYUID format as COPY
-    // to report the source-to-destination UID mapping.
-    let checkMoveInfo = response => {
-        let section = response.attributes && response.attributes[0] && response.attributes[0].section;
-        let responseCode = section && section.length && section[0] && typeof section[0].value === 'string' ? section[0].value : '';
-        switch (responseCode) {
-            case 'COPYUID':
-                {
-                    let uidValidity = section[1] && typeof section[1].value === 'string' && !isNaN(section[1].value) ? BigInt(section[1].value) : false;
-                    if (uidValidity) {
-                        map.uidValidity = uidValidity;
-                    }
-
-                    let sourceUids = section[2] && typeof section[2].value === 'string' ? expandRange(section[2].value) : false;
-                    let destinationUids = section[3] && typeof section[3].value === 'string' ? expandRange(section[3].value) : false;
-                    if (sourceUids && destinationUids && sourceUids.length === destinationUids.length) {
-                        map.uidMap = new Map(sourceUids.map((uid, i) => [uid, destinationUids[i]]));
-                    }
-                }
-                break;
-        }
-    };
-
     let response;
     try {
         // Some servers send COPYUID in an untagged OK before the tagged response,
@@ -67,13 +44,13 @@ module.exports = async (connection, range, destination, options) => {
         response = await connection.exec(options.uid ? 'UID MOVE' : 'MOVE', attributes, {
             untagged: {
                 OK: async untagged => {
-                    checkMoveInfo(untagged);
+                    parseCopyUid(untagged, map);
                 }
             }
         });
         response.next();
 
-        checkMoveInfo(response.response);
+        parseCopyUid(response.response, map);
         return map;
     } catch (err) {
         await enhanceCommandError(err);

package/lib/commands/store.js

@@ -28,9 +28,7 @@ module.exports = async (connection, range, flags, options) => {
     //   [+|-]FLAGS[.SILENT] or [+|-]X-GM-LABELS
     // Where: no prefix = replace all, + = add, - = remove
     // .SILENT suppresses the server from sending back updated flags (saves bandwidth).
-    let operation;
-
-    operation = 'FLAGS';
+    let operation = 'FLAGS';
 
     if (options.useLabels) {
         // Gmail labels (X-GM-EXT-1 extension): operates on labels instead of IMAP flags

package/lib/handler/imap-compiler.js

@@ -71,9 +71,7 @@ module.exports = async (response, options) => {
         // - This is the first token (resp is empty)
         // - This is a sub-array element in a consecutive-list context (no space between adjacent lists)
         if (lastType === 'LITERAL' || (!['(', '<', '['].includes(lastRespByte) && resp.length)) {
-            if (options.subArray) {
-                // ignore separator between consecutive sub-arrays in a list
-            } else {
+            if (!options.subArray) {
                 resp.push(formatRespEntry(' '));
             }
         }

package/lib/handler/imap-formal-syntax.js

@@ -39,13 +39,7 @@ function expandRange(start, end) {
  * @returns {string} The source string with excluded characters removed.
  */
 function excludeChars(source, exclude) {
-    let sourceArr = Array.prototype.slice.call(source);
-    for (let i = sourceArr.length - 1; i >= 0; i--) {
-        if (exclude.indexOf(sourceArr[i]) >= 0) {
-            sourceArr.splice(i, 1);
-        }
-    }
-    return sourceArr.join('');
+    return Array.prototype.filter.call(source, ch => exclude.indexOf(ch) < 0).join('');
 }
 
 module.exports = {

package/lib/handler/imap-parser.js

@@ -61,7 +61,7 @@ module.exports = async (command, options) => {
         // "UID MOVE", "UID SEARCH", "UID EXPUNGE", and "AUTHENTICATE PLAIN", etc.
         // For these, the first word is consumed as the command, then we read the
         // subcommand and concatenate them (e.g., "UID" + " " + "FETCH" -> "UID FETCH").
-        if (['UID', 'AUTHENTICATE'].indexOf((response.command || '').toUpperCase()) >= 0) {
+        if (['UID', 'AUTHENTICATE'].includes((response.command || '').toUpperCase())) {
             await parser.getSpace();
             response.command += ' ' + (await parser.getElement(imapFormalSyntax.command()));
         }

package/lib/handler/imap-stream.js

@@ -172,17 +172,13 @@ class ImapStream extends Transform {
                         this.literals = [];
 
                         if (payload.length) {
-                            // remove final line terminator
-                            let skipBytes = 0;
-                            if (payload.length >= 1 && payload[payload.length - 1] === LF) {
-                                skipBytes++;
-                                if (payload.length >= 2 && payload[payload.length - 2] === CR) {
-                                    skipBytes++;
+                            // remove final line terminator (\n or \r\n)
+                            if (payload[payload.length - 1] === LF) {
+                                let end = payload.length - 1;
+                                if (end > 0 && payload[end - 1] === CR) {
+                                    end--;
                                 }
-                            }
-
-                            if (skipBytes) {
-                                payload = payload.slice(0, payload.length - skipBytes);
+                                payload = payload.slice(0, end);
                             }
 
                             if (payload.length) {
@@ -278,9 +274,7 @@ class ImapStream extends Transform {
         // Queue the chunk for async processing. The 'next' callback serves as
         // backpressure: it is called only after this chunk is fully processed,
         // which signals the writable side that more data can be accepted.
-        if (chunk && chunk.length) {
-            this.inputQueue.push({ chunk, next });
-        }
+        this.inputQueue.push({ chunk, next });
 
         if (!this.processingInput) {
             this.processingInput = true;

package/lib/handler/parser-instance.js

@@ -114,7 +114,7 @@ class ParserInstance {
     async getElement(syntax) {
         let match, element, errPos;
 
-        if (this.remainder.match(/^\s/)) {
+        if (/^\s/.test(this.remainder)) {
             let error = new Error(`Unexpected whitespace at position ${this.pos} [E1]`);
             error.code = 'ParserError1';
             error.parserContext = { input: this.input, pos: this.pos };
@@ -209,7 +209,7 @@ class ParserInstance {
             throw error;
         }
 
-        if (this.remainder.match(/^\s/)) {
+        if (/^\s/.test(this.remainder)) {
             let error = new Error(`Unexpected whitespace at position ${this.pos} [E7]`);
             error.code = 'ParserError7';
             error.parserContext = { input: this.input, element: this.remainder, pos: this.pos };

package/lib/handler/token-parser.js

@@ -272,7 +272,7 @@ class TokenParser {
                         case '~': {
                             let nextChr = this.str.charAt(i + 1);
                             if (nextChr !== '{') {
-                                if (imapFormalSyntax['ATOM-CHAR']().indexOf(nextChr) >= 0) {
+                                if (imapFormalSyntax['ATOM-CHAR']().includes(nextChr)) {
                                     // '~' not followed by '{' but followed by an ATOM char: treat as ATOM
                                     this.currentNode = this.createNode(this.currentNode, this.pos + i);
                                     this.currentNode.type = 'ATOM';
@@ -369,7 +369,7 @@ class TokenParser {
                             // Allow \ as the first char for atom to support system flags
                             // Allow % to support LIST '' %
                             // Allow 8bit characters (presumably unicode)
-                            if (imapFormalSyntax['ATOM-CHAR']().indexOf(chr) < 0 && chr !== '\\' && chr !== '%' && chr.charCodeAt(0) < 0x80) {
+                            if (!imapFormalSyntax['ATOM-CHAR']().includes(chr) && chr !== '\\' && chr !== '%' && chr.charCodeAt(0) < 0x80) {
                                 let error = new Error(`Unexpected char at position ${this.pos + i} [E13: ${JSON.stringify(chr)}]`);
                                 error.code = 'ParserError13';
                                 error.parserContext = { input: this.str, pos: this.pos + i, chr };
@@ -422,7 +422,7 @@ class TokenParser {
 
                     // [ starts a section group for this element
                     // Allowed only for selected elements, otherwise falls through to regular ATOM processing
-                    if (chr === '[' && ['BODY', 'BODY.PEEK', 'BINARY', 'BINARY.PEEK'].indexOf(this.currentNode.value.toUpperCase()) >= 0) {
+                    if (chr === '[' && ['BODY', 'BODY.PEEK', 'BINARY', 'BINARY.PEEK'].includes(this.currentNode.value.toUpperCase())) {
                         this.currentNode.endPos = this.pos + i;
                         this.currentNode = this.createNode(this.currentNode.parentNode, this.pos + i);
                         this.currentNode.type = 'SECTION';
@@ -433,7 +433,7 @@ class TokenParser {
 
                     // if the char is not ATOM compatible, throw. Allow \* as an exception
                     if (
-                        imapFormalSyntax['ATOM-CHAR']().indexOf(chr) < 0 &&
+                        !imapFormalSyntax['ATOM-CHAR']().includes(chr) &&
                         chr.charCodeAt(0) < 0x80 && // allow 8bit (presumably unicode) bytes
                         chr !== ']' &&
                         !(chr === '*' && this.currentNode.value === '\\') &&
@@ -496,21 +496,21 @@ class TokenParser {
                         break;
                     }
 
-                    if (chr === '.' && (!this.currentNode.value.length || this.currentNode.value.match(/\./))) {
+                    if (chr === '.' && (this.currentNode.value === '' || this.currentNode.value.includes('.'))) {
                         let error = new Error(`Unexpected partial separator . at position ${this.pos + i} [E20]`);
                         error.code = 'ParserError20';
                         error.parserContext = { input: this.str, pos: this.pos + i, chr };
                         throw error;
                     }
 
-                    if (imapFormalSyntax.DIGIT().indexOf(chr) < 0 && chr !== '.') {
+                    if (!imapFormalSyntax.DIGIT().includes(chr) && chr !== '.') {
                         let error = new Error(`Unexpected char at position ${this.pos + i} [E21: ${JSON.stringify(chr)}]`);
                         error.code = 'ParserError21';
                         error.parserContext = { input: this.str, pos: this.pos + i, chr };
                         throw error;
                     }
 
-                    if (this.currentNode.value.match(/^0$|\.0$/) && chr !== '.') {
+                    if ((this.currentNode.value === '0' || this.currentNode.value.endsWith('.0')) && chr !== '.') {
                         let error = new Error(`Invalid partial at position ${this.pos + i} [E22: ${JSON.stringify(chr)}]`);
                         error.code = 'ParserError22';
                         error.parserContext = { input: this.str, pos: this.pos + i, chr };
@@ -606,7 +606,7 @@ class TokenParser {
                         }
                         break;
                     }
-                    if (imapFormalSyntax.DIGIT().indexOf(chr) < 0) {
+                    if (!imapFormalSyntax.DIGIT().includes(chr)) {
                         let error = new Error(`Unexpected char at position ${this.pos + i} [E25: ${JSON.stringify(chr)}]`);
                         error.code = 'ParserError25';
                         error.parserContext = { input: this.str, pos: this.pos + i, chr };
@@ -664,7 +664,7 @@ class TokenParser {
                             throw error;
                         }
                     } else if (chr === '*') {
-                        if ([',', ':'].indexOf(this.currentNode.value.at(-1)) < 0) {
+                        if (![',', ':'].includes(this.currentNode.value.at(-1))) {
                             let error = new Error(`Unexpected range wildcard at position ${this.pos + i} [E30]`);
                             error.code = 'ParserError30';
                             error.parserContext = { input: this.str, pos: this.pos + i, chr };