@bobfrankston/mailx-imap 0.1.23 → 0.1.24

package/index.js

@@ -1871,19 +1871,70 @@ export class ImapManager extends EventEmitter {
                 // is parked in IDLE, it's unusable for any other command, so
                 // it can't share the ops queue. Counts against the per-host
                 // semaphore (one slot for the IDLE socket).
-                const watchClient = await this.createClient(accountId, "idle");
-                const stop = await watchClient.watchMailbox("INBOX", (newCount) => {
-                    console.log(`  [idle] ${accountId}: ${newCount} new message(s)`);
-                    // Fetch only the new UIDs — the heavyweight gap/reconcile
-                    // path runs on the 5-minute STATUS poll, so EXISTS lands
-                    // in the UI in roughly one round-trip.
-                    this.syncInboxNewOnly(accountId).catch(e => console.error(`  [idle] sync error: ${e.message}`));
-                });
+                //
+                // We watch INBOX (incoming mail) and Sent + Drafts (outgoing
+                // changes from another client — Thunderbird, phone, web).
+                // Without IDLE on Sent, a message just sent from another
+                // device would only show in mailx after the next periodic
+                // full sync.
+                const stops = [];
+                const clients = [];
+                const watchOne = async (mailboxLabel, path) => {
+                    const client = await this.createClient(accountId, "idle");
+                    clients.push(client);
+                    const stop = await client.watchMailbox(path, (newCount) => {
+                        console.log(`  [idle] ${accountId} ${path}: ${newCount} new message(s)`);
+                        if (mailboxLabel === "inbox") {
+                            // Fast path: incremental fetch of NEW UIDs only.
+                            // Heavy reconcile runs on the 5-minute STATUS poll.
+                            this.syncInboxNewOnly(accountId).catch(e => console.error(`  [idle] inbox sync error: ${e.message}`));
+                        }
+                        else {
+                            // Sent / Drafts changed elsewhere. Use the
+                            // standard folder sync — picks up the new UID,
+                            // rebinds any optimistic local row by Message-ID.
+                            const folder = this.findFolder(accountId, mailboxLabel);
+                            if (folder) {
+                                this.syncFolder(accountId, folder.id).catch(e => console.error(`  [idle] ${path} sync error: ${e.message}`));
+                            }
+                        }
+                    });
+                    stops.push(stop);
+                };
+                await watchOne("inbox", "INBOX");
+                const sent = this.findFolder(accountId, "sent");
+                if (sent) {
+                    try {
+                        await watchOne("sent", sent.path);
+                    }
+                    catch (e) {
+                        console.error(`  [idle] Failed to watch ${sent.path}: ${e.message}`);
+                    }
+                }
+                const drafts = this.findFolder(accountId, "drafts");
+                if (drafts) {
+                    try {
+                        await watchOne("drafts", drafts.path);
+                    }
+                    catch (e) {
+                        console.error(`  [idle] Failed to watch ${drafts.path}: ${e.message}`);
+                    }
+                }
                 this.watchers.set(accountId, async () => {
-                    await stop();
-                    await watchClient.logout();
+                    for (const stop of stops) {
+                        try {
+                            await stop();
+                        }
+                        catch { /* ignore */ }
+                    }
+                    for (const c of clients) {
+                        try {
+                            await c.logout();
+                        }
+                        catch { /* ignore */ }
+                    }
                 });
-                console.log(`  [idle] Watching INBOX for ${accountId}`);
+                console.log(`  [idle] Watching INBOX${sent ? "+Sent" : ""}${drafts ? "+Drafts" : ""} for ${accountId}`);
             }
             catch (e) {
                 console.error(`  [idle] Failed to watch ${accountId}: ${e.message}`);
@@ -2612,9 +2663,12 @@ export class ImapManager extends EventEmitter {
                 bodyPath,
             });
             // Folder-tree badge refresh + message-list reload if the user
-            // is currently on Sent — same event the sync path emits.
+            // is currently on Sent — same event shape the sync path emits.
+            // (Was sending {accountId,folderId} as a single arg, which the
+            // IPC forwarder + UI handler decoded as garbage — the optimistic
+            // row landed in the DB but never appeared in the list.)
             this.db.recalcFolderCounts(sent.id);
-            this.emit("folderCountsChanged", { accountId, folderId: sent.id });
+            this.emit("folderCountsChanged", accountId, {});
             console.log(`  [sent-local] wrote optimistic row in Sent (uid=${synthUid}) for ${accountId}: ${envelope.subject}`);
         }
         catch (e) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.23",
+  "version": "0.1.24",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",
@@ -10,8 +10,8 @@
   "license": "ISC",
   "dependencies": {
     "@bobfrankston/mailx-types": "^0.1.10",
-    "@bobfrankston/mailx-settings": "^0.1.11",
-    "@bobfrankston/mailx-store": "^0.1.9",
+    "@bobfrankston/mailx-settings": "^0.1.13",
+    "@bobfrankston/mailx-store": "^0.1.10",
     "@bobfrankston/iflow-direct": "^0.1.30",
     "@bobfrankston/tcp-transport": "^0.1.5",
     "@bobfrankston/smtp-direct": "^0.1.5",
@@ -38,8 +38,8 @@
   ".transformedSnapshot": {
     "dependencies": {
       "@bobfrankston/mailx-types": "^0.1.10",
-      "@bobfrankston/mailx-settings": "^0.1.11",
-      "@bobfrankston/mailx-store": "^0.1.9",
+      "@bobfrankston/mailx-settings": "^0.1.13",
+      "@bobfrankston/mailx-store": "^0.1.10",
       "@bobfrankston/iflow-direct": "^0.1.30",
       "@bobfrankston/tcp-transport": "^0.1.5",
       "@bobfrankston/smtp-direct": "^0.1.5",