imapflow 1.4.6 → 1.4.7

package/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-    ".": "1.4.6"
+    ".": "1.4.7"
 }

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [1.4.7](https://github.com/postalsys/imapflow/compare/v1.4.6...v1.4.7) (2026-07-10)
+
+
+### Bug Fixes
+
+* treat \NonExistent mailboxes as \Noselect in LIST responses ([af21245](https://github.com/postalsys/imapflow/commit/af21245dab4e1320afc4c4458afe33d1508180c1))
+
 ## [1.4.6](https://github.com/postalsys/imapflow/compare/v1.4.5...v1.4.6) (2026-07-05)
 
 

package/lib/commands/list.js

@@ -87,6 +87,15 @@ module.exports = async (connection, reference, mailbox, options) => {
             specialUseMatches[type].push({ entry, source });
         };
 
+        // RFC 5258: the \NonExistent attribute implies \Noselect. Some servers only
+        // return \NonExistent for phantom folders, so add \Noselect as well to keep
+        // the flags consistent for consumers that only check \Noselect.
+        let normalizeFlags = entry => {
+            if (entry.flags.has('\\NonExistent')) {
+                entry.flags.add('\\Noselect');
+            }
+        };
+
         // User-provided hints map mailbox paths to special-use types (e.g., {sent: "Sent Items"}).
         // These override server-reported flags and name-based guesses. Converted to a
         // path-keyed lookup: { "Sent Items" => "\\Sent" }
@@ -132,6 +141,8 @@ module.exports = async (connection, reference, mailbox, options) => {
                             listed: true
                         };
 
+                        normalizeFlags(entry);
+
                         // Check user-provided hints first (highest priority)
                         if (specialUseHints[entry.path]) {
                             addSpecialUseMatch(entry, specialUseHints[entry.path], 'user');
@@ -300,6 +311,7 @@ module.exports = async (connection, reference, mailbox, options) => {
                             existing.subscribed = true;
                             // Merge any additional flags from LSUB into the LIST entry
                             entry.flags.forEach(flag => existing.flags.add(flag));
+                            normalizeFlags(existing);
                         }
                         // Non-listed subscribed folders are intentionally ignored
                     }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "imapflow",
-    "version": "1.4.6",
+    "version": "1.4.7",
     "description": "IMAP Client for Node",
     "main": "lib/imap-flow.js",
     "types": "lib/imap-flow.d.ts",

package/test/commands-integration-test.js

@@ -3550,6 +3550,94 @@ module.exports['Commands: list skips Noselect folders for status'] = async test
     test.done();
 };
 
+module.exports['Commands: list adds Noselect to NonExistent mailboxes'] = async test => {
+    const connection = createMockConnection({
+        state: 3,
+        exec: async (cmd, attrs, opts) => {
+            if (cmd === 'LIST' && opts && opts.untagged && opts.untagged.LIST) {
+                await opts.untagged.LIST({
+                    attributes: [[{ value: '\\HasNoChildren' }], { value: '/' }, { value: 'INBOX' }]
+                });
+                await opts.untagged.LIST({
+                    attributes: [[{ value: '\\NonExistent' }], { value: '/' }, { value: 'Phantom' }]
+                });
+            }
+            return { next: () => {} };
+        }
+    });
+
+    const result = await listCommand(connection, '', '*');
+    const phantom = result.find(e => e.path === 'Phantom');
+    test.ok(phantom);
+    // RFC 5258: \\NonExistent implies \\Noselect
+    test.equal(phantom.flags.has('\\Noselect'), true);
+    // The original flag is preserved, not replaced
+    test.equal(phantom.flags.has('\\NonExistent'), true);
+    const inbox = result.find(e => e.path === 'INBOX');
+    test.ok(inbox);
+    test.equal(inbox.flags.has('\\Noselect'), false);
+    test.done();
+};
+
+module.exports['Commands: list LSUB merge adds Noselect to NonExistent'] = async test => {
+    const connection = createMockConnection({
+        state: 3,
+        exec: async (cmd, attrs, opts) => {
+            if (cmd === 'LIST' && opts && opts.untagged && opts.untagged.LIST) {
+                await opts.untagged.LIST({
+                    attributes: [[{ value: '\\HasNoChildren' }], { value: '/' }, { value: 'Folder1' }]
+                });
+            }
+            if (cmd === 'LSUB' && opts && opts.untagged && opts.untagged.LSUB) {
+                // Some servers only report \\NonExistent in LSUB responses
+                await opts.untagged.LSUB({
+                    attributes: [[{ value: '\\NonExistent' }], { value: '/' }, { value: 'Folder1' }]
+                });
+            }
+            return { next: () => {} };
+        }
+    });
+
+    const result = await listCommand(connection, '', '*');
+    const folder = result.find(e => e.path === 'Folder1');
+    test.ok(folder);
+    test.equal(folder.subscribed, true);
+    test.equal(folder.flags.has('\\NonExistent'), true);
+    // RFC 5258: \\NonExistent merged from LSUB implies \\Noselect
+    test.equal(folder.flags.has('\\Noselect'), true);
+    test.done();
+};
+
+module.exports['Commands: list does not STATUS NonExistent mailboxes'] = async test => {
+    let statusPaths = [];
+    const connection = createMockConnection({
+        state: 3,
+        capabilities: new Map(),
+        run: async (cmd, path) => {
+            if (cmd === 'STATUS') {
+                statusPaths.push(path);
+                return { messages: 10 };
+            }
+        },
+        exec: async (cmd, attrs, opts) => {
+            if (cmd === 'LIST' && opts && opts.untagged && opts.untagged.LIST) {
+                await opts.untagged.LIST({
+                    attributes: [[{ value: '\\HasNoChildren' }], { value: '/' }, { value: 'INBOX' }]
+                });
+                await opts.untagged.LIST({
+                    attributes: [[{ value: '\\NonExistent' }], { value: '/' }, { value: 'Phantom' }]
+                });
+            }
+            return { next: () => {} };
+        }
+    });
+
+    await listCommand(connection, '', '*', { statusQuery: { messages: true } });
+    // STATUS runs for the selectable mailbox only
+    test.deepEqual(statusPaths, ['INBOX']);
+    test.done();
+};
+
 module.exports['Commands: list XLIST removes Inbox flag from non-INBOX'] = async test => {
     const connection = createMockConnection({
         state: 3,