@bobfrankston/mailx-store 0.1.9 → 0.1.10

package/db.js

@@ -1188,7 +1188,13 @@ export class MailxDB {
     }
     /** Get messages without cached bodies (for background prefetch) */
     getMessagesWithoutBody(accountId, limit = 50) {
-        return this.db.prepare("SELECT uid, folder_id as folderId FROM messages WHERE account_id = ? AND (body_path IS NULL OR body_path = '') ORDER BY date DESC LIMIT ?").all(accountId, limit);
+        // Prefetch order: smallest first, NULLs last, recent within tiebreak.
+        // Reasoning: on slow / metered Android networks, the user feels the
+        // cache "fill up" much faster when the queue is dominated by short
+        // notification mails (a handful of KB each) instead of a single
+        // multi-megabyte attachment that monopolizes bandwidth for minutes.
+        // Size 0 / NULL fall to the end so they don't masquerade as small.
+        return this.db.prepare("SELECT uid, folder_id as folderId FROM messages WHERE account_id = ? AND (body_path IS NULL OR body_path = '') ORDER BY (size IS NULL OR size = 0), size ASC, date DESC LIMIT ?").all(accountId, limit);
     }
     getHighestUid(accountId, folderId) {
         const r = this.db.prepare("SELECT MAX(uid) as maxUid FROM messages WHERE account_id = ? AND folder_id = ?").get(accountId, folderId);
@@ -1613,6 +1619,23 @@ export class MailxDB {
         const score = (r) => (r.match_rank || 0) * 10_000
             + (r.use_count || 0) * Math.pow(0.5, Math.max(0, now - (r.last_used || 0)) / HALF_LIFE_MS);
         rows.sort((a, b) => score(b) - score(a));
+        // Dedup by lowercased email — same address often appears as both
+        // source='google' (synced from Google Contacts) and source='discovered'
+        // (auto-collected from sent mail). The user only wants to see the
+        // best entry; we keep the higher-ranked source (which the sort above
+        // has already put first), and silently drop the duplicate. Without
+        // this, the autocomplete dropdown showed two identical Kevin Healy
+        // rows just labeled GOOGLE and DISCOVERED.
+        const seenEmails = new Set();
+        rows = rows.filter(r => {
+            const k = (r.email || "").toLowerCase();
+            if (!k)
+                return true;
+            if (seenEmails.has(k))
+                return false;
+            seenEmails.add(k);
+            return true;
+        });
         rows = rows.slice(0, limit);
         return rows.map(r => ({ name: r.name, email: r.email, source: r.source, useCount: r.use_count }));
     }

package/file-store.d.ts

@@ -28,6 +28,12 @@ export declare class FileMessageStore implements MessageStore {
      *  it in `body_path`. The (folderId, uid) args are kept for interface
      *  compatibility; they do NOT affect the filename. */
     putMessage(accountId: string, _folderId: number, _uid: number, raw: Buffer): Promise<string>;
+    /** Resolve a stored body_path (relative or absolute) to an absolute
+     *  filesystem path. Returns "" if the input doesn't resolve to a file
+     *  inside the store. UI / "Source" actions need the absolute path so
+     *  they can open / pass it to OS file pickers without re-introducing
+     *  the basePath context every time. */
+    absolutePath(stored: string): string;
     /** Read by stored path (relative or absolute). */
     readByPath(stored: string): Promise<Buffer>;
     hasByPath(stored: string): Promise<boolean>;

package/file-store.js

@@ -51,6 +51,14 @@ export class FileMessageStore {
         fs.writeFileSync(abs, raw);
         return rel;
     }
+    /** Resolve a stored body_path (relative or absolute) to an absolute
+     *  filesystem path. Returns "" if the input doesn't resolve to a file
+     *  inside the store. UI / "Source" actions need the absolute path so
+     *  they can open / pass it to OS file pickers without re-introducing
+     *  the basePath context every time. */
+    absolutePath(stored) {
+        return this.resolveStored(stored);
+    }
     /** Read by stored path (relative or absolute). */
     async readByPath(stored) {
         const abs = this.resolveStored(stored);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-store",
-  "version": "0.1.9",
+  "version": "0.1.10",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",
@@ -10,7 +10,7 @@
   "license": "ISC",
   "dependencies": {
     "@bobfrankston/mailx-types": "^0.1.10",
-    "@bobfrankston/mailx-settings": "^0.1.11"
+    "@bobfrankston/mailx-settings": "^0.1.12"
   },
   "repository": {
     "type": "git",
@@ -26,7 +26,7 @@
   ".transformedSnapshot": {
     "dependencies": {
       "@bobfrankston/mailx-types": "^0.1.10",
-      "@bobfrankston/mailx-settings": "^0.1.11"
+      "@bobfrankston/mailx-settings": "^0.1.12"
     }
   }
 }