@bobfrankston/rmfmail 1.2.1 → 1.2.3

package/packages/mailx-store/db.ts

@@ -1232,6 +1232,7 @@ export class MailxDB {
             accountId: r.account_id,
             folderId: r.folder_id,
             uid: r.uid,
+            uuid: r.uuid || "",   // stable identity for row data-uuid / selection restore
             messageId: r.message_id || "",
             inReplyTo: r.in_reply_to || "",
             references: JSON.parse(r.refs || "[]"),
@@ -2082,6 +2083,13 @@ export class MailxDB {
             // Bcc). The unified-inbox UI shows a small ⇆ badge on these
             // rows so the user knows "this is a copy of the same message".
             dupeCount: (r.dupeCount as number) | 0,
+            // uuid is the stable local identity each message-list row carries as
+            // data-uuid. Without it, restoreSelection() / rememberPosition() /
+            // the set-diff scroll anchor all match on empty string and fail, so
+            // the selected-row highlight vanishes on every reload (Bob
+            // 2026-06-13: "no highlighting in the summary"). getMessages already
+            // maps uuid; this unified-inbox map dropped it. SELECT m.* exposes it.
+            uuid: r.uuid || "",
         } as any));
 
         return { items, total, page, pageSize };
@@ -3403,6 +3411,7 @@ export class MailxDB {
                 folderId: r.folder_id,
                 folderName: r.folder_name || "",
                 uid: r.uid,
+                uuid: r.uuid || "",   // stable identity for row data-uuid / selection restore
                 messageId: r.message_id || "",
                 inReplyTo: r.in_reply_to || "",
                 references: JSON.parse(r.refs || "[]"),

package/packages/mailx-store/package.json

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

package/test/phase0-deepcompare.mjs

@@ -0,0 +1,51 @@
+/** Deep-compare worker vs in-process envelopes to catch any field the
+ *  structured-clone boundary drops/alters (suspected highlight regression). */
+import fs from "node:fs";
+import path from "node:path";
+import os from "node:os";
+import { MailxDB, Store, FileMessageStore } from "../packages/mailx-store/index.js";
+import { spawnDbWorker } from "../packages/mailx-service/db-worker-client.js";
+
+const liveDir = process.env.RMFMAIL_DIR || path.join(os.homedir(), ".rmfmail");
+const tmp = fs.mkdtempSync(path.join(os.tmpdir(), "rmf-deep-"));
+for (const s of ["", "-wal", "-shm"]) {
+    const src = path.join(liveDir, "mailx.db" + s);
+    if (fs.existsSync(src)) try { fs.copyFileSync(src, path.join(tmp, "mailx.db" + s)); } catch {}
+}
+const db = new MailxDB(tmp);
+const store = new Store(db, new FileMessageStore(path.join(tmp, "store")));
+const worker = await spawnDbWorker({ dbPath: tmp, storePath: path.join(tmp, "store") });
+
+const ip = store.getUnifiedInbox(1, 10, false);
+const wk = await worker.bus.request("db:getUnifiedInbox", { page: 1, pageSize: 10, flaggedOnly: false });
+
+function describe(v) {
+    if (v === null) return "null";
+    if (v instanceof Date) return `Date(${v.getTime()})`;
+    if (Array.isArray(v)) return `Array[${v.length}]`;
+    return typeof v;
+}
+let mismatches = 0;
+for (let i = 0; i < Math.min(ip.items.length, wk.items.length); i++) {
+    const a = ip.items[i], b = wk.items[i];
+    const keys = new Set([...Object.keys(a), ...Object.keys(b)]);
+    for (const k of keys) {
+        const sa = JSON.stringify(a[k]), sb = JSON.stringify(b[k]);
+        if (sa !== sb) {
+            mismatches++;
+            console.log(`  item[${i}].${k}: in-proc=${describe(a[k])} ${sa}  ||  worker=${describe(b[k])} ${sb}`);
+        }
+    }
+}
+console.log(mismatches === 0
+    ? `\n✓ ALL fields identical across ${ip.items.length} envelopes — highlighting is NOT a worker data issue`
+    : `\n✗ ${mismatches} field mismatch(es) — likely the highlight cause`);
+
+// Show which fields exist + the flags shape (unread highlight driver).
+console.log("\nfields on envelope[0]:", Object.keys(ip.items[0] || {}).join(", "));
+console.log("flags[0]:", JSON.stringify(ip.items[0]?.flags), "| unread driver candidates:",
+    ["unread", "seen", "isRead", "flags"].filter(k => k in (ip.items[0] || {})).join(", "));
+
+await worker.close();
+try { fs.rmSync(tmp, { recursive: true, force: true }); } catch {}
+process.exit(0);