@bobfrankston/iflow-direct 0.1.8 → 0.1.9

package/imap-compat.js

@@ -73,44 +73,51 @@ export class CompatImapClient {
     }
     /** Detect special folders from folder list */
     getSpecialFolders(folders) {
-        // Match IMAP special-use flags (RFC 6154) first, then fall back to exact
-        // LEAF NAME match. Do NOT use substring matches on the full path —
-        // "Drafts.Older" used to get tagged as the Drafts folder because of a
-        // lower.includes("draft") test, which made it impossible to rename or
-        // delete the subfolder.
+        // Two-pass detection so RFC 6154 \Drafts / \Sent / \Trash / \Junk /
+        // \Archive flags always win over name-based guesses. Previous single-pass
+        // `else if (flag || name)` could tag a stray folder named "DRAFT" as the
+        // drafts folder on Gmail, beating out [Imap]/Drafts which has the real
+        // \Drafts flag — Gmail then rejects APPEND to "DRAFT" as NONEXISTENT.
         const result = {};
         const leafName = (f) => {
             const delim = f.delimiter || ".";
             return (f.path.split(delim).pop() || f.path).toLowerCase();
         };
+        const flagsLower = (f) => f.flags.map(fl => fl.toLowerCase());
+        // Pass 1: RFC 6154 flags only — authoritative if present
         for (const f of folders) {
-            const name = leafName(f);
-            const flags = f.flags.map(fl => fl.toLowerCase());
-            if (flags.includes("\\inbox") || name === "inbox") {
-                if (!result.inbox)
-                    result.inbox = f.path;
-            }
-            else if (flags.includes("\\sent") || name === "sent" || name === "sent items" || name === "sent mail") {
-                if (!result.sent)
-                    result.sent = f.path;
-            }
-            else if (flags.includes("\\trash") || name === "trash" || name === "deleted items" || name === "deleted") {
-                if (!result.trash)
-                    result.trash = f.path;
-            }
-            else if (flags.includes("\\drafts") || name === "drafts" || name === "draft") {
-                if (!result.drafts)
-                    result.drafts = f.path;
-            }
-            else if (flags.includes("\\junk") || flags.includes("\\spam") || name === "spam" || name === "junk" || name === "junk email" || name === "junk e-mail") {
-                if (!result.junk)
-                    result.junk = f.path;
+            const flags = flagsLower(f);
+            if (!result.inbox && (flags.includes("\\inbox") || leafName(f) === "inbox"))
+                result.inbox = f.path;
+            if (!result.sent && flags.includes("\\sent"))
+                result.sent = f.path;
+            if (!result.trash && flags.includes("\\trash"))
+                result.trash = f.path;
+            if (!result.drafts && flags.includes("\\drafts"))
+                result.drafts = f.path;
+            if (!result.archive && flags.includes("\\archive"))
+                result.archive = f.path;
+            if (!result.junk && (flags.includes("\\junk") || flags.includes("\\spam"))) {
+                result.junk = f.path;
                 if (!result.spam)
                     result.spam = f.path;
             }
-            else if (flags.includes("\\archive") || name === "archive" || name === "archives") {
-                if (!result.archive)
-                    result.archive = f.path;
+        }
+        // Pass 2: exact leaf-name fallback for servers that don't advertise flags
+        for (const f of folders) {
+            const name = leafName(f);
+            if (!result.sent && (name === "sent" || name === "sent items" || name === "sent mail"))
+                result.sent = f.path;
+            if (!result.trash && (name === "trash" || name === "deleted items" || name === "deleted"))
+                result.trash = f.path;
+            if (!result.drafts && (name === "drafts" || name === "draft"))
+                result.drafts = f.path;
+            if (!result.archive && (name === "archive" || name === "archives"))
+                result.archive = f.path;
+            if (!result.junk && (name === "spam" || name === "junk" || name === "junk email" || name === "junk e-mail")) {
+                result.junk = f.path;
+                if (!result.spam)
+                    result.spam = f.path;
             }
         }
         return result;

package/package.json

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