@bobfrankston/mailx 1.0.348 → 1.0.349

package/client/components/message-list.js

@@ -266,8 +266,13 @@ export async function loadSearchResults(query, scope = "all", accountId = "", fo
 export async function loadMessages(accountId, folderId, page = 1, specialUse = "", autoSelect = true) {
     searchMode = false;
     unifiedMode = false;
-    showToInsteadOfFrom = ["sent", "drafts", "outbox"].includes(specialUse) ||
-        ["Sent", "Drafts", "Outbox", "Sent Items"].includes(specialUse);
+    // specialUse is either the DB tag ("sent"/"drafts"/"outbox") or the
+    // folder path lowercased (folder-tree fallback when tag is missing — common
+    // on Dovecot which doesn't advertise \Sent). Match both cases.
+    const su = (specialUse || "").toLowerCase();
+    showToInsteadOfFrom = su === "sent" || su === "drafts" || su === "outbox"
+        || su.endsWith("sent") || su.endsWith("drafts") || su.endsWith("outbox")
+        || su === "sent items" || su === "sent mail" || su.endsWith("/sent items") || su.endsWith(".sent items");
     currentAccountId = accountId;
     currentFolderId = folderId;
     currentPage = 1;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx",
-  "version": "1.0.348",
+  "version": "1.0.349",
   "description": "Local-first email client with IMAP sync and standalone native app",
   "type": "module",
   "main": "bin/mailx.js",
@@ -20,7 +20,7 @@
     "postinstall": "node bin/postinstall.js"
   },
   "dependencies": {
-    "@bobfrankston/iflow-direct": "^0.1.23",
+    "@bobfrankston/iflow-direct": "^0.1.24",
     "@bobfrankston/iflow-node": "^0.1.7",
     "@bobfrankston/miscinfo": "^1.0.9",
     "@bobfrankston/oauthsupport": "^1.0.24",
@@ -84,7 +84,7 @@
   },
   ".transformedSnapshot": {
     "dependencies": {
-      "@bobfrankston/iflow-direct": "^0.1.23",
+      "@bobfrankston/iflow-direct": "^0.1.24",
       "@bobfrankston/iflow-node": "^0.1.7",
       "@bobfrankston/miscinfo": "^1.0.9",
       "@bobfrankston/oauthsupport": "^1.0.24",

package/packages/mailx-service/index.js

@@ -694,22 +694,19 @@ export class MailxService {
     /** Move messages to the account's configured spam folder (accounts.jsonc "spam" path).
      *  Throws if the account has no spam folder configured or the folder doesn't exist locally. */
     async markAsSpamMessages(accountId, uids) {
-        // Cached accounts — same reason as send/saveDraft: a stalled GDrive
-        // mount could turn `Mark as spam` into a 120s IPC timeout.
-        let account = this.getCachedAccounts().find(a => a.id === accountId);
-        if (!account) {
-            this._accountsCache = null;
-            account = this.getCachedAccounts().find(a => a.id === accountId);
-        }
-        if (!account)
-            throw new Error(`Account ${accountId} not found`);
-        const spamPath = account.spam;
-        if (!spamPath)
-            throw new Error(`Account ${accountId} has no "spam" folder configured`);
+        // The spam folder is whatever the provider's getSpecialFolders() said
+        // it is. iflow-direct's compat client fills this in from RFC 6154
+        // \Junk / \Spam flags (with sensible defaults if the server doesn't
+        // advertise them); Gmail has SPAM built in. mailx stores the result
+        // as `specialUse: "junk"` on the matching folder row.
+        //
+        // Earlier versions required an explicit `spam:` field in accounts.jsonc
+        // and the button erroring out when that was absent. That's obsolete —
+        // the provider knows where spam goes. Just look up the flagged folder.
         const folders = this.db.getFolders(accountId);
-        const target = folders.find(f => f.path.toLowerCase() === spamPath.toLowerCase());
+        const target = folders.find(f => f.specialUse === "junk");
         if (!target)
-            throw new Error(`Spam folder "${spamPath}" not found in ${accountId}`);
+            throw new Error(`No \\Junk/\\Spam folder found for ${accountId}`);
         await this.moveMessages(accountId, uids, target.id);
         return { targetFolderId: target.id, moved: uids.length };
     }