@bobfrankston/iflow-direct 0.1.23 → 0.1.24

package/imap-compat.d.ts

@@ -36,7 +36,7 @@ export declare class CompatImapClient {
     logout(): Promise<void>;
     /** Get folder list */
     getFolderList(): Promise<NativeFolder[]>;
-    /** Detect special folders from folder list */
+    /** Extract special folders from folder list */
     getSpecialFolders(folders: NativeFolder[]): SpecialFolders;
     /** Fetch messages since a UID in a mailbox */
     fetchMessagesSinceUid(mailbox: string, sinceUid: number, options?: {

package/imap-compat.js

@@ -5,6 +5,16 @@
  */
 import { NativeImapClient } from "./imap-native.js";
 import { FetchedMessage } from "./fetched-message.js";
+// We may need some special cases in the future
+const defaultFolders = {
+    inbox: "INBOX", // Uppercase -- we'll need a reality check
+    sent: "Sent",
+    trash: "Trash",
+    drafts: "Drafts",
+    spam: "_Spam",
+    junk: "_Junk",
+    archive: "Archive"
+};
 /**
  * Compatibility IMAP client — wraps NativeImapClient with the old ImapClient API.
  * Each method creates its own select/operation/close cycle.
@@ -39,55 +49,63 @@ export class CompatImapClient {
         await this.ensureConnected();
         return this.native.listFolders();
     }
-    /** Detect special folders from folder list */
+    /** Extract special folders from folder list */
     getSpecialFolders(folders) {
         // 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 = {};
+        let result = {};
         const leafName = (f) => {
             const delim = f.delimiter || ".";
-            return (f.path.split(delim).pop() || f.path).toLowerCase();
+            return (f.path.split(delim).at(-1) ?? f.path).toLowerCase();
         };
-        const flagsLower = (f) => f.flags.map(fl => fl.toLowerCase());
-        // Pass 1: RFC 6154 flags only — authoritative if present
+        const flagSet = (f) => new Set(f.flags.map(fl => fl.toLowerCase()));
+        // const df = 'y:\\temp\\folders.csv';
+        // fs.writeFileSync(df, "path,flags\n" + folders.map(f => `${f.path},"${f.flags.join(";")}"`).join("\n"));
+        // console.log(`Wrote folder list to ${df}`);
+        // RFC 6154 flags only — authoritative if present
+        // Note that Dovecot does not have them but we check INBOX for the case
         for (const f of folders) {
-            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"))) {
+            const flags = flagSet(f);
+            // Debubing 
+            const dbgignore = ['\\haschildren', '\\hasnochildren', '\\noselect', '\\unmarked'];
+            const dbgfgall = Array.from(flags);
+            const dbgfg = dbgfgall.filter(fl => !dbgignore.includes(fl));
+            if (flags.has("\\inbox") || leafName(f) === "inbox")
+                result.inbox ??= f.path;
+            if (flags.has("\\sent"))
+                result.sent ??= f.path;
+            if (flags.has("\\trash"))
+                result.trash ??= f.path;
+            if (flags.has("\\drafts"))
+                result.drafts ??= f.path;
+            if (flags.has("\\archive"))
+                result.archive ??= f.path;
+            if (flags.has("\\junk"))
                 result.junk = f.path;
-                if (!result.spam)
-                    result.spam = 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;
-            }
+            if (flags.has("\\spam"))
+                result.spam ??= f.path;
         }
+        result = { ...defaultFolders, ...result }; // Fill in any missing defaults
+        // // Pass 2: exact leaf-name fallback for servers that don't advertise flags
+        // const sentNames = new Set(["sent", "sent items", "sent mail"]);
+        // const trashNames = new Set(["trash", "deleted items", "deleted"]);
+        // const draftsNames = new Set(["drafts", "draft"]);
+        // const archiveNames = new Set(["archive", "archives"]);
+        // const junkNames = new Set(["spam", "junk", "junk email", "junk e-mail"]);
+        // for (const f of folders) {
+        //     const name = leafName(f);
+        //     if (sentNames.has(name)) result.sent ??= f.path;
+        //     if (trashNames.has(name)) result.trash ??= f.path;
+        //     if (draftsNames.has(name)) result.drafts ??= f.path;
+        //     if (archiveNames.has(name)) result.archive ??= f.path;
+        //     if (junkNames.has(name)) {
+        //         result.junk ??= f.path;
+        //         result.spam ??= f.path;
+        //     }
+        // }
         return result;
     }
     /** Fetch messages since a UID in a mailbox */

package/imap-native.js

@@ -386,7 +386,7 @@ export class NativeImapClient {
             const chunk = uids.slice(i, i + chunkSize);
             const msgs = await this.fetchMessages(chunk.join(","), options);
             allMessages.push(...msgs);
-            console.log(`  [fetch] ${allMessages.length}/${uids.length} (chunk of ${chunk.length})`);
+            // console.log(`  [fetch] ${allMessages.length}/${uids.length} (chunk of ${chunk.length})`);
             if (onChunk)
                 onChunk(msgs);
             if (chunkSize < this.fetchChunkSizeMax)
@@ -405,7 +405,7 @@ export class NativeImapClient {
             return [];
         // Reverse so newest messages (highest UIDs) come first
         uids.reverse();
-        console.log(`  [fetch] ${uids.length} UIDs to fetch (newest first)`);
+        // console.log(`  [fetch] ${uids.length} UIDs to fetch (newest first)`);
         const allMessages = [];
         let chunkSize = this.fetchChunkSize;
         for (let i = 0; i < uids.length; i += chunkSize) {
@@ -788,9 +788,9 @@ export class NativeImapClient {
                     const rest_bytes = encoded.subarray(neededBytes);
                     let literal = new TextDecoder().decode(literal_bytes);
                     this.buffer = new TextDecoder().decode(rest_bytes);
-                    if (neededBytes !== literal.length) {
-                        console.log(`  [imap] literal: ${neededBytes} bytes → ${literal.length} chars (multi-byte corrected)`);
-                    }
+                    // if (neededBytes !== literal.length) {
+                    //     console.log(`iflow-direct:  [imap] literal: ${neededBytes} bytes → ${literal.length} chars (multi-byte corrected)`);
+                    // }
                     // 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) {

package/package.json

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

package/tasks.json

@@ -0,0 +1,20 @@
+{
+    "version": "2.0.0",
+    "tasks": [
+        {
+            "label": "tsc: watch",
+            "type": "shell",
+            "command": "tsc",
+            "args": ["--watch"],
+            "runOptions": {
+                "runOn": "folderOpen"
+            },
+            "problemMatcher": "$tsc-watch",
+            "isBackground": true,
+            "group": {
+                "kind": "build",
+                "isDefault": true
+            }
+        }
+    ]
+}