@withbuddi/buddi 0.1.0-pre.20 → 0.1.0-pre.22

package/node_modules/imapflow/dist/cjs/types.d.ts

@@ -207,26 +207,32 @@ export interface IdInfoObject {
     'support-url'?: string | false | undefined;
     /** Date program was released */
     date?: Date | string | false | undefined;
-    [key: string]: any;
+    /** Any other field, sent as its string form */
+    [key: string]: unknown;
+}
+/**
+ * Usage and limit of one quota resource. The `storage` values are bytes (the wire format is
+ * kilobytes), the other resources are counts
+ */
+export interface QuotaResource {
+    /** Current usage, missing when the server did not send a usable number */
+    usage?: number | undefined;
+    /** The limit, missing when the server did not send a usable number */
+    limit?: number | undefined;
+    /** Usage as a percentage of the limit, e.g. "50%", set once a limit above zero is known */
+    status?: string | undefined;
 }
 export interface QuotaResponse {
     /** Mailbox path this quota applies to */
     path: string;
-    /** Storage quota if provided by server */
-    storage?: {
-        /** Used storage in bytes */
-        used: number;
-        /** Total storage available */
-        limit: number;
-    } | undefined;
-    /** Message count quota if provided by server */
-    messages?: {
-        /** Stored messages */
-        used: number;
-        /** Maximum messages allowed */
-        limit: number;
-    } | undefined;
-    [resource: string]: any;
+    /** The quota root the server reported for the mailbox, if any */
+    quotaRoot?: string | undefined;
+    /** The STORAGE resource, if the server reports one */
+    storage?: QuotaResource | undefined;
+    /** The MESSAGE resource, if the server reports one */
+    message?: QuotaResource | undefined;
+    /** Any other resource the server reports, under its lowercased name (e.g. "mailbox") */
+    [resource: string]: QuotaResource | string | undefined;
 }
 /**
  * Status data items to request with `status()` or the `statusQuery` listing option
@@ -616,6 +622,14 @@ export interface DownloadObject {
     /** Streamed content */
     content: Readable;
 }
+/**
+ * What `download()` resolves with when there is nothing to download: no mailbox is selected, or
+ * the message or part was not found. Check `content` before using the result.
+ */
+export interface DownloadNotFound {
+    meta?: undefined;
+    content?: undefined;
+}
 export interface DownloadOptions {
     /** If true then uses UID number instead of sequence number for `range` */
     uid?: boolean | undefined;

package/node_modules/imapflow/dist/esm/commands/append.js

@@ -1,4 +1,4 @@
-import { formatFlag, canUseFlag, formatDateTime, normalizePath, encodePath, comparePaths, enhanceCommandError, parseBigIntValue, parseUintValue, MAX_UINT32_DIGITS } from '../tools.js';
+import { formatFlag, canUseFlag, formatDateTime, normalizePath, encodePath, comparePaths, parseBigIntValue, parseUintValue, MAX_UINT32_DIGITS, getSelectedMailbox, emitSafe, isAuthenticatedState, reportCommandError } from '../tools.js';
 /**
  * Appends a message to a mailbox.
  *
@@ -11,7 +11,7 @@ import { formatFlag, canUseFlag, formatDateTime, normalizePath, encodePath, comp
  * @throws {Error} If the APPEND command fails or message exceeds APPENDLIMIT
  */
 export default async function append(connection, destination, content, flags, idate) {
-    if (![connection.states.AUTHENTICATED, connection.states.SELECTED].includes(connection.state) || !destination) {
+    if (!isAuthenticatedState(connection) || !destination) {
         // nothing to do here
         return;
     }
@@ -31,7 +31,9 @@ export default async function append(connection, destination, content, flags, id
     destination = normalizePath(connection, destination);
     // If appending to the currently selected mailbox, we can listen for the
     // untagged EXISTS response to capture the new message's sequence number.
-    let expectExists = comparePaths(connection, connection.mailbox.path, destination);
+    let selected = getSelectedMailbox(connection);
+    // The selected mailbox when appending to it, false otherwise
+    const targetMailbox = selected && comparePaths(connection, selected.path, destination) ? selected : false;
     // Validate and format flags. Only flags allowed by the mailbox's permanentFlags are included.
     flags = (Array.isArray(flags) ? flags : [].concat(flags || []))
         .map(flag => flag && formatFlag(flag.toString()))
@@ -74,13 +76,12 @@ export default async function append(connection, destination, content, flags, id
         map.seq = seq;
         // Update the connection's mailbox state and emit 'exists' event if the
         // count changed (notifies listeners about the new message).
-        if (expectExists) {
-            let mailbox = connection.mailbox;
-            let prevCount = mailbox.exists;
+        if (targetMailbox) {
+            let prevCount = targetMailbox.exists;
             if (map.seq !== prevCount) {
-                mailbox.exists = map.seq;
-                connection.emit('exists', {
-                    path: mailbox.path,
+                targetMailbox.exists = map.seq;
+                emitSafe(connection, 'exists', {
+                    path: targetMailbox.path,
                     count: map.seq,
                     prevCount
                 });
@@ -91,7 +92,7 @@ export default async function append(connection, destination, content, flags, id
     try {
         response = await connection.exec('APPEND', attributes, {
             // Only listen for EXISTS if we're appending to the currently selected mailbox
-            untagged: expectExists ? { EXISTS: handleExistsUpdate } : false
+            untagged: targetMailbox ? { EXISTS: handleExistsUpdate } : false
         });
         // UIDPLUS (RFC 4315): the server may include APPENDUID response code in
         // the tagged OK. Format: [APPENDUID <uidValidity> <uid>]
@@ -116,7 +117,7 @@ export default async function append(connection, destination, content, flags, id
         response.next();
         // If we didn't get an EXISTS during APPEND (some servers don't send it
         // until the next command), issue a NOOP to flush pending notifications.
-        if (expectExists && !map.seq) {
+        if (targetMailbox && !map.seq) {
             try {
                 response = await connection.exec('NOOP', false, {
                     untagged: { EXISTS: handleExistsUpdate },
@@ -139,8 +140,7 @@ export default async function append(connection, destination, content, flags, id
         return map;
     }
     catch (err) {
-        await enhanceCommandError(err);
-        connection.log.warn({ err, cid: connection.id });
+        await reportCommandError(connection, err);
         throw err;
     }
 }

package/node_modules/imapflow/dist/esm/commands/authenticate.d.ts

@@ -1,16 +1,11 @@
 import type { ImapFlow } from '../imap-flow.js';
+import type { AuthOptions } from '../types.js';
 /**
- * Credentials for the AUTHENTICATE command
+ * Credentials for the AUTHENTICATE command: the auth options, with the password passed as `password`
  */
-export interface AuthenticateCredentials {
-    /** OAuth2 access token for OAUTHBEARER/XOAUTH2 authentication */
-    accessToken?: string | undefined;
+export interface AuthenticateCredentials extends Omit<AuthOptions, 'user' | 'pass'> {
     /** Password for PLAIN or LOGIN authentication */
     password?: string | undefined;
-    /** Force a specific login method (e.g., 'AUTH=PLAIN', 'AUTH=LOGIN') */
-    loginMethod?: string | undefined;
-    /** Authorization identity for PLAIN authentication */
-    authzid?: string | undefined;
 }
 /**
  * Authenticates user using the best available method.

package/node_modules/imapflow/dist/esm/commands/close.js

@@ -1,3 +1,4 @@
+import { emitSafe } from '../tools.js';
 /**
  * Closes the currently selected mailbox.
  *
@@ -23,7 +24,7 @@ export default async function close(connection) {
         connection.currentSelectCommand = false;
         connection.state = connection.states.AUTHENTICATED;
         if (currentMailbox) {
-            connection.emit('mailboxClose', currentMailbox);
+            emitSafe(connection, 'mailboxClose', currentMailbox);
         }
         return true;
     }

package/node_modules/imapflow/dist/esm/commands/copy.js

@@ -1,4 +1,4 @@
-import { normalizePath, encodePath, enhanceCommandError } from '../tools.js';
+import { normalizePath, encodePath, reportCommandError, getSelectedMailbox } from '../tools.js';
 import { parseCopyUid } from './copyuid-parser.js';
 /**
  * Copies messages from the current mailbox to another mailbox.
@@ -11,7 +11,8 @@ import { parseCopyUid } from './copyuid-parser.js';
  * @returns Copy result with UID mapping if available, false on failure, or undefined if preconditions not met
  */
 export default async function copy(connection, range, destination, options) {
-    if (connection.state !== connection.states.SELECTED || !range || !destination) {
+    let mailbox = getSelectedMailbox(connection);
+    if (!mailbox || !range || !destination) {
         // nothing to do here
         return;
     }
@@ -25,15 +26,14 @@ export default async function copy(connection, range, destination, options) {
     try {
         response = await connection.exec(options.uid ? 'UID COPY' : 'COPY', attributes);
         response.next();
-        let map = { path: connection.mailbox.path, destination };
+        let map = { path: mailbox.path, destination };
         // 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.
         parseCopyUid(response.response, map);
         return map;
     }
     catch (err) {
-        await enhanceCommandError(err);
-        connection.log.warn({ err, cid: connection.id });
+        await reportCommandError(connection, err);
         return false;
     }
 }

package/node_modules/imapflow/dist/esm/commands/create.js

@@ -1,4 +1,4 @@
-import { encodePath, normalizePath, getStatusCode, enhanceCommandError } from '../tools.js';
+import { encodePath, normalizePath, getStatusCode, isAuthenticatedState, reportCommandError } from '../tools.js';
 /**
  * Creates a new mailbox and subscribes to it.
  *
@@ -8,7 +8,7 @@ import { encodePath, normalizePath, getStatusCode, enhanceCommandError } from '.
  * @throws If the CREATE command fails (except when mailbox already exists)
  */
 export default async function create(connection, path) {
-    if (![connection.states.AUTHENTICATED, connection.states.SELECTED].includes(connection.state)) {
+    if (!isAuthenticatedState(connection)) {
         // nothing to do here
         return;
     }
@@ -68,8 +68,7 @@ export default async function create(connection, path) {
                 created: false
             };
         }
-        await enhanceCommandError(err);
-        connection.log.warn({ err, cid: connection.id });
+        await reportCommandError(connection, err);
         throw err;
     }
 }

package/node_modules/imapflow/dist/esm/commands/delete.js

@@ -1,4 +1,4 @@
-import { encodePath, normalizePath, enhanceCommandError } from '../tools.js';
+import { encodePath, normalizePath, isAuthenticatedState, reportCommandError, getSelectedMailbox } from '../tools.js';
 /**
  * Deletes an existing mailbox.
  *
@@ -8,14 +8,15 @@ import { encodePath, normalizePath, enhanceCommandError } from '../tools.js';
  * @throws If the DELETE command fails
  */
 export default async function deleteMailbox(connection, path) {
-    if (![connection.states.AUTHENTICATED, connection.states.SELECTED].includes(connection.state)) {
+    if (!isAuthenticatedState(connection)) {
         // nothing to do here
         return;
     }
     path = normalizePath(connection, path);
     // If the mailbox to delete is currently selected, we must close/deselect it first.
     // IMAP servers reject DELETE on the currently selected mailbox (RFC 3501 6.3.4).
-    if (connection.state === connection.states.SELECTED && connection.mailbox.path === path) {
+    let selected = getSelectedMailbox(connection);
+    if (selected && selected.path === path) {
         await connection.run('CLOSE');
     }
     let response;
@@ -28,8 +29,7 @@ export default async function deleteMailbox(connection, path) {
         return map;
     }
     catch (err) {
-        await enhanceCommandError(err);
-        connection.log.warn({ err, cid: connection.id });
+        await reportCommandError(connection, err);
         throw err;
     }
 }

package/node_modules/imapflow/dist/esm/commands/expunge.js

@@ -1,4 +1,4 @@
-import { enhanceCommandError, hasCapability, parseBigIntValue } from '../tools.js';
+import { hasCapability, parseBigIntValue, reportCommandError, getSelectedMailbox } from '../tools.js';
 /**
  * Deletes specified messages by flagging them as Deleted and expunging.
  *
@@ -9,14 +9,19 @@ import { enhanceCommandError, hasCapability, parseBigIntValue } from '../tools.j
  * @returns True on success, false on failure, or undefined if preconditions not met
  */
 export default async function expunge(connection, range, options) {
-    if (connection.state !== connection.states.SELECTED || !range) {
+    let mailbox = getSelectedMailbox(connection);
+    if (!mailbox || !range) {
         // nothing to do here
         return;
     }
     options = options || {};
     // Two-step deletion process per IMAP protocol:
     // Step 1: Mark the target messages with the \Deleted flag.
-    await connection.messageFlagsAdd(range, ['\\Deleted'], options);
+    // If that failed, EXPUNGE would not remove the target messages (and without
+    // UIDPLUS it would remove unrelated \Deleted messages), so report the failure instead.
+    if (!(await connection.messageFlagsAdd(range, ['\\Deleted'], options))) {
+        return false;
+    }
     // Step 2: Issue EXPUNGE to permanently remove \Deleted messages.
     // With UIDPLUS (RFC 4315): "UID EXPUNGE <uids>" removes only the specified UIDs,
     // leaving other \Deleted messages untouched, important for concurrent access.
@@ -35,7 +40,6 @@ export default async function expunge(connection, range, options) {
         if (responseCode.toUpperCase() === 'HIGHESTMODSEQ') {
             // A response code always comes with its section, see responseCode above
             let codeSection = section;
-            let mailbox = connection.mailbox;
             // Bounded digit runs only: isNaN() also passes '1e5', which BigInt() rejects with
             // a throw that the catch below would swallow, making messageDelete() report false
             // even though the server expunged the messages.
@@ -48,8 +52,7 @@ export default async function expunge(connection, range, options) {
         return true;
     }
     catch (err) {
-        await enhanceCommandError(err);
-        connection.log.warn({ err, cid: connection.id });
+        await reportCommandError(connection, err);
         return false;
     }
 }

package/node_modules/imapflow/dist/esm/commands/fetch.js

@@ -1,4 +1,4 @@
-import { formatMessageResponse, isRev2Active } from '../tools.js';
+import { formatMessageResponse, isRev2Active, getSelectedMailbox } from '../tools.js';
 /**
  * Fetches emails from the server.
  *
@@ -9,23 +9,23 @@ import { formatMessageResponse, isRev2Active } from '../tools.js';
  * @returns Object with message count and list, or undefined if not in SELECTED state
  */
 export default async function fetch(connection, range, query, options) {
-    if (connection.state !== connection.states.SELECTED || !range) {
+    let mailbox = getSelectedMailbox(connection);
+    if (!mailbox || !range) {
         // nothing to do here
         return;
     }
     options = options || {};
-    let mailbox = connection.mailbox;
     // Use BINARY extension for fetching if supported and requested, otherwise fall back to BODY.
     // RFC 9051 folds the FETCH side of the BINARY extension into base IMAP4rev2, so an active
     // rev2 session can use it even without the BINARY capability token (the APPEND side is NOT
     // folded in and stays gated on the token in append.ts)
     const canUseBinary = connection.capabilities.has('BINARY') || isRev2Active(connection);
     const commandKey = canUseBinary && options.binary && !connection.disableBinary ? 'BINARY' : 'BODY';
-    // Retry logic for ETHROTTLE errors (server rate limiting) with exponential backoff
-    let retryCount = 0;
+    // Retry logic for ETHROTTLE errors (server rate limiting) with exponential backoff.
+    // Every pass returns or throws: the last throttled attempt throws instead of retrying.
     const maxRetries = 4;
     const baseDelay = 1000; // Start with 1 second delay
-    while (retryCount < maxRetries) {
+    for (let retryCount = 0;; retryCount++) {
         let messages = {
             count: 0,
             list: []
@@ -202,15 +202,18 @@ export default async function fetch(connection, range, query, options) {
             return messages;
         }
         catch (err) {
-            if (err.code === 'ETHROTTLE') {
+            // The last throttled attempt falls through and throws, so running out of retries
+            // is never mistaken for an empty result
+            if (err.code === 'ETHROTTLE' && retryCount < maxRetries - 1) {
                 // Server returned a throttle error (rate limiting). Retry with exponential backoff.
-                // Delay doubles each retry: 1s, 2s, 4s, 8s (capped at 30s).
+                // Delay doubles each retry: 1s, 2s, 4s.
                 // If server provides a throttleReset hint, use that if longer.
                 const backoffDelay = Math.min(baseDelay * Math.pow(2, retryCount), 30000); // Cap at 30 seconds
                 // Use throttle reset time if provided and longer than backoff. The hint is
                 // server-controlled, so the wait goes through connection.throttleWait(), which caps
-                // it and keeps the timer tracked and abortable.
-                const delay = err.throttleReset && err.throttleReset > backoffDelay ? err.throttleReset : backoffDelay;
+                // it and keeps the timer tracked and abortable. The connection already waited
+                // part of the back-off before rejecting (throttleWaited), so only the rest is left.
+                const delay = Math.max(err.throttleReset || 0, backoffDelay) - (err.throttleWaited || 0);
                 connection.log.warn({
                     msg: 'Retrying throttled request with exponential backoff',
                     cid: connection.id,
@@ -226,7 +229,6 @@ export default async function fetch(connection, range, query, options) {
                 if (aborted) {
                     throw connection.createNoConnectionError(connection.byeReason, { rejectedFrom: 'throttleAbort', command: 'FETCH' });
                 }
-                retryCount++;
                 continue;
             }
             connection.log.warn({ err, cid: connection.id });

package/node_modules/imapflow/dist/esm/commands/idle.js

@@ -1,4 +1,4 @@
-import { guardedPromise, hasCapability, logConnectionError, restampConnectionError, unrefTimer, clearTimer } from '../tools.js';
+import { guardedPromise, hasCapability, logConnectionError, restampConnectionError, unrefTimer, clearTimer, getSelectedMailbox } from '../tools.js';
 const NOOP_INTERVAL = 2 * 60 * 1000;
 /**
  * Marks the connection as idling on behalf of one session and returns a release function.
@@ -121,7 +121,11 @@ async function runIdle(connection) {
     }
     catch (err) {
         logConnectionError(connection, 'IDLE session failed', err);
-        if (preCheckWaitQueue.length) {
+        // A tagged NO or BAD only means the server refused IDLE; the connection is still usable,
+        // so the waiters are released by the finally block below and their own commands run.
+        // Anything else (close, lost socket, parser failure) fails the waiters too.
+        let refusedByServer = ['NO', 'BAD'].includes(err.responseStatus);
+        if (preCheckWaitQueue.length && !refusedByServer) {
             // One error for the whole queue: every waiter failed at the same site, for the same
             // reason. Built inside the guard so a teardown with nothing queued - the common case -
             // does not pay for an Error and its stack capture.
@@ -237,7 +241,7 @@ async function runPollingFallback(connection, maxIdleTime) {
                     return;
                 }
                 // The transport or the mailbox may be gone by the time the timer fires
-                if (!connection.socket || connection.socket.destroyed || connection.state !== connection.states.SELECTED || !connection.mailbox) {
+                if (!connection.socket || connection.socket.destroyed || !getSelectedMailbox(connection)) {
                     return cancel();
                 }
                 pollOnce(connection, session)

package/node_modules/imapflow/dist/esm/commands/list.js

@@ -148,10 +148,12 @@ export default async function list(connection, reference, mailbox, options) {
                         if (!untagged.attributes || !untagged.attributes.length) {
                             return;
                         }
+                        // A name sent as a literal arrives as a Buffer, so convert it once for both fields
+                        let rawPath = ((untagged.attributes[2] && untagged.attributes[2].value) || '').toString();
                         let entry = {
                             // Decode from modified UTF-7 wire format and normalize the path
-                            path: normalizePath(connection, decodePath(connection, ((untagged.attributes[2] && untagged.attributes[2].value) || ''))),
-                            pathAsListed: ((untagged.attributes[2] && untagged.attributes[2].value) || ''),
+                            path: normalizePath(connection, decodePath(connection, rawPath)),
+                            pathAsListed: rawPath,
                             flags: new Set(getStringList(untagged.attributes[0])),
                             delimiter: (untagged.attributes[1] && untagged.attributes[1].value),
                             listed: true
@@ -181,7 +183,7 @@ export default async function list(connection, reference, mailbox, options) {
                             entry.path = entry.path.slice(1);
                         }
                         // Build parent path hierarchy for tree construction and sorting
-                        entry.parentPath = entry.delimiter && entry.path ? entry.path.substr(0, entry.path.lastIndexOf(entry.delimiter)) : '';
+                        entry.parentPath = entry.delimiter && entry.path ? entry.path.substring(0, entry.path.lastIndexOf(entry.delimiter)) : '';
                         entry.parent = entry.delimiter ? entry.path.split(entry.delimiter) : [entry.path];
                         entry.name = entry.parent.pop();
                         // Try to detect special-use from server flags or well-known names
@@ -379,9 +381,11 @@ export default async function list(connection, reference, mailbox, options) {
                         if (!untagged.attributes || !untagged.attributes.length) {
                             return;
                         }
+                        // A name sent as a literal arrives as a Buffer, so convert it once for both fields
+                        let rawPath = ((untagged.attributes[2] && untagged.attributes[2].value) || '').toString();
                         let entry = {
-                            path: normalizePath(connection, decodePath(connection, ((untagged.attributes[2] && untagged.attributes[2].value) || ''))),
-                            pathAsListed: ((untagged.attributes[2] && untagged.attributes[2].value) || ''),
+                            path: normalizePath(connection, decodePath(connection, rawPath)),
+                            pathAsListed: rawPath,
                             flags: new Set(getStringList(untagged.attributes[0])),
                             delimiter: (untagged.attributes[1] && untagged.attributes[1].value),
                             subscribed: true
@@ -392,7 +396,7 @@ export default async function list(connection, reference, mailbox, options) {
                         if (entry.delimiter && entry.path.charAt(0) === entry.delimiter) {
                             entry.path = entry.path.slice(1);
                         }
-                        entry.parentPath = entry.delimiter && entry.path ? entry.path.substr(0, entry.path.lastIndexOf(entry.delimiter)) : '';
+                        entry.parentPath = entry.delimiter && entry.path ? entry.path.substring(0, entry.path.lastIndexOf(entry.delimiter)) : '';
                         entry.parent = entry.delimiter ? entry.path.split(entry.delimiter) : [entry.path];
                         entry.name = entry.parent.pop();
                         // Merge LSUB data into existing LIST entry if found
@@ -444,23 +448,35 @@ export default async function list(connection, reference, mailbox, options) {
                 connection.log.warn({ msg: 'Failed to request subscription info', err, cid: connection.id });
             }
         }
-        // Resolve special-use conflicts: for each type, pick the best candidate
-        // based on source priority (user > extension > name), then alphabetically.
-        // Only the winning entry gets the specialUse property set.
-        for (let type of Object.keys(specialUseMatches)) {
-            let sortedEntries = specialUseMatches[type].sort((a, b) => {
-                let aSource = SOURCE_SORT_ORDER.indexOf(a.source);
-                let bSource = SOURCE_SORT_ORDER.indexOf(b.source);
-                if (aSource === bSource) {
-                    return a.entry.path.localeCompare(b.entry.path);
-                }
+        // Resolve special-use conflicts. Each type goes to one mailbox and each mailbox gets
+        // at most one type. Candidates are taken in priority order across all types (user >
+        // extension > name, then alphabetically), so a mailbox claimed by a stronger match
+        // leaves its other type to that type's next candidate instead of to nobody. Within a
+        // source a shallower mailbox wins before the alphabetical order is consulted, so that
+        // INBOX.Sent is preferred over INBOX.Archive.Sent.
+        let candidates = Object.entries(specialUseMatches).flatMap(([type, matches]) => matches.map(match => ({ type, ...match })));
+        candidates.sort((a, b) => {
+            let aSource = SOURCE_SORT_ORDER.indexOf(a.source);
+            let bSource = SOURCE_SORT_ORDER.indexOf(b.source);
+            if (aSource !== bSource) {
                 return aSource - bSource;
-            });
-            if (!sortedEntries[0].entry.specialUse) {
-                let source = sortedEntries[0].source;
-                sortedEntries[0].entry.specialUse = type;
-                sortedEntries[0].entry.specialUseSource = PUBLIC_SOURCE[source] || source;
             }
+            // parent is set on every listed entry before the candidates are ranked
+            let aDepth = a.entry.parent.length;
+            let bDepth = b.entry.parent.length;
+            if (aDepth !== bDepth) {
+                return aDepth - bDepth;
+            }
+            return a.entry.path.localeCompare(b.entry.path);
+        });
+        let assignedTypes = new Set();
+        for (let { type, entry, source } of candidates) {
+            if (assignedTypes.has(type) || entry.specialUse) {
+                continue;
+            }
+            entry.specialUse = type;
+            entry.specialUseSource = PUBLIC_SOURCE[source] || source;
+            assignedTypes.add(type);
         }
         // No source answered, so "not subscribed" was never actually reported for any of
         // these folders - the state is unknown, not false. Reporting the whole listing as

package/node_modules/imapflow/dist/esm/commands/move.js

@@ -1,4 +1,4 @@
-import { normalizePath, encodePath, enhanceCommandError, hasCapability } from '../tools.js';
+import { normalizePath, encodePath, hasCapability, reportCommandError, getSelectedMailbox } from '../tools.js';
 import { parseCopyUid } from './copyuid-parser.js';
 /**
  * Moves messages from the current mailbox to another mailbox.
@@ -11,7 +11,8 @@ import { parseCopyUid } from './copyuid-parser.js';
  * @returns Move result with UID mapping if available, false on failure, or undefined if preconditions not met
  */
 export default async function move(connection, range, destination, options) {
-    if (connection.state !== connection.states.SELECTED || !range || !destination) {
+    let mailbox = getSelectedMailbox(connection);
+    if (!mailbox || !range || !destination) {
         // nothing to do here
         return;
     }
@@ -21,13 +22,18 @@ export default async function move(connection, range, destination, options) {
         { type: 'SEQUENCE', value: range },
         { type: 'ATOM', value: encodePath(connection, destination) }
     ];
-    let map = { path: connection.mailbox.path, destination };
+    let map = { path: mailbox.path, destination };
     // Fallback for servers without the MOVE extension (RFC 6851):
     // emulate MOVE using COPY + flag as \Deleted + EXPUNGE.
     if (!hasCapability(connection, 'MOVE')) {
         let result = await connection.messageCopy(range, destination, options);
-        await connection.messageDelete(range, Object.assign({ silent: true }, options));
-        return result;
+        if (!result) {
+            // The source must stay untouched when the copy failed, otherwise the messages are lost
+            return result;
+        }
+        let deleted = await connection.messageDelete(range, Object.assign({ silent: true }, options));
+        // Messages that were copied but not removed from the source mean the move did not complete
+        return deleted ? result : false;
     }
     let response;
     try {
@@ -45,8 +51,7 @@ export default async function move(connection, range, destination, options) {
         return map;
     }
     catch (err) {
-        await enhanceCommandError(err);
-        connection.log.warn({ err, cid: connection.id });
+        await reportCommandError(connection, err);
         return false;
     }
 }

package/node_modules/imapflow/dist/esm/commands/namespace.js

@@ -1,4 +1,4 @@
-import { hasCapability, getStringList } from '../tools.js';
+import { hasCapability, getStringList, isAuthenticatedState } from '../tools.js';
 /**
  * Requests NAMESPACE info from the server.
  *
@@ -6,7 +6,7 @@ import { hasCapability, getStringList } from '../tools.js';
  * @returns The primary personal namespace, or an error object on failure
  */
 export default async function namespace(connection) {
-    if (![connection.states.AUTHENTICATED, connection.states.SELECTED].includes(connection.state)) {
+    if (!isAuthenticatedState(connection)) {
         // nothing to do here
         return;
     }

package/node_modules/imapflow/dist/esm/commands/quota.js

@@ -1,4 +1,4 @@
-import { encodePath, normalizePath, enhanceCommandError, parseUintValue, isUnsafeKey } from '../tools.js';
+import { encodePath, normalizePath, parseUintValue, isUnsafeKey, isAuthenticatedState, reportCommandError } from '../tools.js';
 /**
  * Requests quota information for a mailbox.
  *
@@ -7,7 +7,7 @@ import { encodePath, normalizePath, enhanceCommandError, parseUintValue, isUnsaf
  * @returns Quota information object, false if QUOTA not supported or on failure, or undefined if preconditions not met
  */
 export default async function quota(connection, path) {
-    if (![connection.states.AUTHENTICATED, connection.states.SELECTED].includes(connection.state) || !path) {
+    if (!isAuthenticatedState(connection) || !path) {
         // nothing to do here
         return;
     }
@@ -47,19 +47,21 @@ export default async function quota(connection, path) {
             if (isUnsafeKey(key) || key === 'path' || key === 'quotaroot') {
                 return;
             }
-            if (!map[key]) {
-                map[key] = {};
+            let resource = map[key];
+            if (typeof resource !== 'object') {
+                resource = {};
+                map[key] = resource;
             }
             // Storage quota is reported in KB by IMAP; convert to bytes for consistency
             const multiplier = key === 'storage' ? 1024 : 1;
             if (position === 1) {
-                map[key].usage = value * multiplier;
+                resource.usage = value * multiplier;
             }
             else if (position === 2) {
-                map[key].limit = value * multiplier;
+                resource.limit = value * multiplier;
                 // Calculate usage percentage for convenient display
-                if (map[key].limit) {
-                    map[key].status = Math.round(((map[key].usage || 0) / map[key].limit) * 100) + '%';
+                if (resource.limit) {
+                    resource.status = Math.round(((resource.usage || 0) / resource.limit) * 100) + '%';
                 }
             }
         });
@@ -107,8 +109,7 @@ export default async function quota(connection, path) {
         return map;
     }
     catch (err) {
-        await enhanceCommandError(err);
-        connection.log.warn({ err, cid: connection.id });
+        await reportCommandError(connection, err);
         return false;
     }
 }