@bobfrankston/mailx-store 0.1.57 → 0.1.58

package/db.d.ts

@@ -301,6 +301,13 @@ export declare class MailxDB {
          *  message_folders. Omitted/false for Gmail (label = folder, many per
          *  message). Set true by the IMAP storeMessages path. */
         exclusive?: boolean;
+        /** UIDs the server currently lists for THIS folder, when the caller
+         *  just enumerated them (the set-diff backfill path). Lets move-detect
+         *  distinguish a real move (old uid gone) from a DUPLICATE DELIVERY
+         *  (same Message-ID at two live uids in one folder — e.g. a message
+         *  addressed to two aliases of the same mailbox). Without this the
+         *  single row ping-pongs between the two uids every sync cycle. */
+        liveServerUids?: Set<number>;
     }): number;
     /** Backfill the FTS5 `body_text` column for a message after its body
      *  has been parsed. Capped at ~64 KB of text per row — FTS5 stores the

package/db.js

@@ -1828,7 +1828,19 @@ export class MailxDB {
         // surface transient move-dups. Keeping the collapse for now.
         if (msg.messageId) {
             const moved = this.db.prepare("SELECT id, folder_id, uid FROM messages WHERE account_id = ? AND message_id = ? LIMIT 1").get(msg.accountId, msg.messageId);
-            if (moved) {
+            // Duplicate delivery, NOT a move: same folder, different uid, and
+            // the caller's server UID list confirms the OLD uid is still in the
+            // mailbox. Rebinding here would ping-pong the row between the two
+            // uids on every sync (each cycle sees the unbound uid as
+            // "server-only", refetches it, and rebinds). Fall through to a
+            // fresh insert instead — both copies get their own row, matching
+            // what the server (and every other client) actually shows, and the
+            // set-diff converges.
+            if (moved && moved.folder_id === msg.folderId && moved.uid !== msg.uid
+                && msg.liveServerUids?.has(moved.uid)) {
+                console.log(`  [move-detect] ${msg.accountId} ${msg.messageId}: uid ${moved.uid} still on server — duplicate copy at uid ${msg.uid}, inserting second row`);
+            }
+            else if (moved) {
                 console.log(`  [move-detect] ${msg.accountId} ${msg.messageId}: rebinding row ${moved.id} (folder ${moved.folder_id}/uid ${moved.uid} → folder ${msg.folderId}/uid ${msg.uid})`);
                 this.db.prepare("UPDATE messages SET folder_id = ?, uid = ?, cached_at = ? WHERE id = ?").run(msg.folderId, msg.uid, Date.now(), moved.id);
                 this.upsertMessageFolder(moved.id, msg.folderId, msg.uid, msg.exclusive);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-store",
-  "version": "0.1.57",
+  "version": "0.1.58",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",