@bobfrankston/mailx 1.0.366 → 1.0.370

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx",
-  "version": "1.0.366",
+  "version": "1.0.370",
   "description": "Local-first email client with IMAP sync and standalone native app",
   "type": "module",
   "main": "bin/mailx.js",
@@ -24,7 +24,7 @@
     "@bobfrankston/iflow-node": "^0.1.7",
     "@bobfrankston/miscinfo": "^1.0.9",
     "@bobfrankston/oauthsupport": "^1.0.24",
-    "@bobfrankston/msger": "^0.1.344",
+    "@bobfrankston/msger": "^0.1.345",
     "@bobfrankston/mailx-host": "^0.1.4",
     "@capacitor/android": "^8.3.0",
     "@capacitor/cli": "^8.3.0",
@@ -88,7 +88,7 @@
       "@bobfrankston/iflow-node": "^0.1.7",
       "@bobfrankston/miscinfo": "^1.0.9",
       "@bobfrankston/oauthsupport": "^1.0.24",
-      "@bobfrankston/msger": "^0.1.344",
+      "@bobfrankston/msger": "^0.1.345",
       "@bobfrankston/mailx-host": "^0.1.4",
       "@capacitor/android": "^8.3.0",
       "@capacitor/cli": "^8.3.0",

package/packages/mailx-core/index.d.ts

@@ -58,7 +58,6 @@ export declare function getMessage(params: {
     folderName?: string;
     uid: number;
     uuid?: string;
-    pending?: boolean;
     messageId: string;
     inReplyTo: string;
     references: string[];
@@ -74,6 +73,7 @@ export declare function getMessage(params: {
     preview: string;
     bodyPath?: string;
     providerId?: string;
+    pending?: boolean;
 }>;
 export declare function updateFlags(params: {
     accountId: string;

package/packages/mailx-service/index.js

@@ -827,6 +827,14 @@ export class MailxService {
         if (!folder)
             throw new Error("Folder not found");
         this.db.deleteAllMessages(accountId, folderId);
+        // Recalc + broadcast so the folder-tree badge drops to 0 immediately.
+        // Without this, the badge kept showing the old unread count even
+        // though the list was empty (user-reported bug).
+        this.db.recalcFolderCounts(folderId);
+        try {
+            this.imapManager.emit?.("folderCountsChanged", accountId, {});
+        }
+        catch { /* non-fatal */ }
         const client = this.imapManager.createPublicClient(accountId);
         try {
             const uids = await client.getUids(folder.path);

package/packages/mailx-store/db.js

@@ -654,6 +654,18 @@ export class MailxDB {
         this.db.prepare("UPDATE messages SET flags_json = ? WHERE account_id = ? AND uid = ?").run(JSON.stringify(flags), accountId, uid);
     }
     updateMessageFolder(accountId, uid, targetFolderId) {
+        // Idempotency: if a row already exists at (account, target_folder, uid)
+        // — common with Gmail's hash-synthesized UIDs across labels, or any
+        // case where the move was already partially applied — the UPDATE
+        // would fail the (acct, folder, uid) unique constraint. Treat it as
+        // a no-op: drop the source row, the message is already where the
+        // user wants it. Previously surfaced as "Mark-as-spam failed: UNIQUE
+        // constraint failed" — bad UX for what is logically already done.
+        const existingTarget = this.db.prepare("SELECT id FROM messages WHERE account_id = ? AND folder_id = ? AND uid = ?").get(accountId, targetFolderId, uid);
+        if (existingTarget) {
+            this.db.prepare("DELETE FROM messages WHERE account_id = ? AND uid = ? AND folder_id != ?").run(accountId, uid, targetFolderId);
+            return;
+        }
         this.db.prepare("UPDATE messages SET folder_id = ? WHERE account_id = ? AND uid = ?").run(targetFolderId, accountId, uid);
     }
     updateBodyPath(accountId, uid, bodyPath) {