@bobfrankston/mailx-store 0.1.28 → 0.1.30

package/db.d.ts

@@ -259,9 +259,15 @@ export declare class MailxDB {
     /** 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
      *  raw text and indexes tokens; we don't need every byte of a 1 MB
-     *  marketing email to find a word in the first paragraph. Called from
-     *  LocalStore.getMessage right after simpleParser succeeds. */
-    updateFtsBodyByUid(accountId: string, folderId: number, uid: number, bodyText: string): void;
+     *  marketing email to find a word in the first paragraph.
+     *
+     *  Keyed on the FTS rowid (= messages.id) directly. The earlier
+     *  (account, folder_id, uid) lookup keyed on the LEGACY messages.folder_id
+     *  column — a message can live in several folders via message_folders,
+     *  so that lookup could miss the row entirely and silently leave the
+     *  body un-indexed (Bob 2026-05-18: "mcdade" in an opened message's
+     *  body, search found nothing). */
+    updateFtsBody(rowId: number, bodyText: string): void;
     /** List view: messages currently in (account, folder).
      *  Joins through `message_folders` so the UID + folder location come
      *  from membership rows, not from the legacy messages.folder_id /

package/db.js

@@ -1355,13 +1355,18 @@ export class MailxDB {
         // quoted-strings (RFC-strictly forbidden, common in the wild). Run
         // libmime over Subject and every address display name once; safe
         // to call on already-decoded text (no `=?...?=` markers → no-op).
+        // The `address` is decoded too: a legit address never carries an
+        // encoded-word, so it's a no-op there — but spam sometimes packs the
+        // whole `name <addr>` into the mailbox local-part as one encoded
+        // word (`=?utf-8?q?...=3C...=40...=3E?=@host`). Decoding it at least
+        // renders readable text instead of raw `=3C`/`=C3=A9` gibberish.
         msg.subject = decodeHeaderWords(msg.subject);
         if (msg.from)
-            msg.from = { name: decodeHeaderWords(msg.from.name || ""), address: msg.from.address || "" };
+            msg.from = { name: decodeHeaderWords(msg.from.name || ""), address: decodeHeaderWords(msg.from.address || "") };
         if (msg.to)
-            msg.to = msg.to.map(a => ({ name: decodeHeaderWords(a.name || ""), address: a.address || "" }));
+            msg.to = msg.to.map(a => ({ name: decodeHeaderWords(a.name || ""), address: decodeHeaderWords(a.address || "") }));
         if (msg.cc)
-            msg.cc = msg.cc.map(a => ({ name: decodeHeaderWords(a.name || ""), address: a.address || "" }));
+            msg.cc = msg.cc.map(a => ({ name: decodeHeaderWords(a.name || ""), address: decodeHeaderWords(a.address || "") }));
         const existing = this.db.prepare("SELECT id, provider_id FROM messages WHERE account_id = ? AND folder_id = ? AND uid = ?").get(msg.accountId, msg.folderId, msg.uid);
         if (existing) {
             // Backfill provider_id on existing rows that predate this column —
@@ -1496,15 +1501,20 @@ export class MailxDB {
     /** 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
      *  raw text and indexes tokens; we don't need every byte of a 1 MB
-     *  marketing email to find a word in the first paragraph. Called from
-     *  LocalStore.getMessage right after simpleParser succeeds. */
-    updateFtsBodyByUid(accountId, folderId, uid, bodyText) {
+     *  marketing email to find a word in the first paragraph.
+     *
+     *  Keyed on the FTS rowid (= messages.id) directly. The earlier
+     *  (account, folder_id, uid) lookup keyed on the LEGACY messages.folder_id
+     *  column — a message can live in several folders via message_folders,
+     *  so that lookup could miss the row entirely and silently leave the
+     *  body un-indexed (Bob 2026-05-18: "mcdade" in an opened message's
+     *  body, search found nothing). */
+    updateFtsBody(rowId, bodyText) {
+        if (!rowId)
+            return;
         try {
-            const row = this.db.prepare("SELECT id FROM messages WHERE account_id = ? AND folder_id = ? AND uid = ?").get(accountId, folderId, uid);
-            if (!row)
-                return;
             const capped = bodyText.length > 64_000 ? bodyText.slice(0, 64_000) : bodyText;
-            this.db.prepare("UPDATE messages_fts SET body_text = ? WHERE rowid = ?").run(capped, row.id);
+            this.db.prepare("UPDATE messages_fts SET body_text = ? WHERE rowid = ?").run(capped, rowId);
         }
         catch { /* FTS update is best-effort */ }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-store",
-  "version": "0.1.28",
+  "version": "0.1.30",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",
@@ -9,8 +9,8 @@
   },
   "license": "ISC",
   "dependencies": {
-    "@bobfrankston/mailx-types": "^0.1.14",
-    "@bobfrankston/mailx-settings": "^0.1.17",
+    "@bobfrankston/mailx-types": "^0.1.17",
+    "@bobfrankston/mailx-settings": "^0.1.20",
     "@bobfrankston/mailx-bus": "^0.1.2",
     "mailparser": "^3.7.2"
   },
@@ -29,8 +29,8 @@
   },
   ".transformedSnapshot": {
     "dependencies": {
-      "@bobfrankston/mailx-types": "^0.1.14",
-      "@bobfrankston/mailx-settings": "^0.1.17",
+      "@bobfrankston/mailx-types": "^0.1.17",
+      "@bobfrankston/mailx-settings": "^0.1.20",
       "@bobfrankston/mailx-bus": "^0.1.2",
       "mailparser": "^3.7.2"
     }

package/store.js

@@ -305,9 +305,9 @@ export class Store {
         // backfill, searches miss any word that only appears deeper in the
         // body. Fire-and-forget — failures are non-fatal, never block the
         // user's preview render.
-        if (bodyText) {
+        if (bodyText && envelope.id) {
             try {
-                this.db.updateFtsBodyByUid(accountId, envelope.folderId, uid, bodyText);
+                this.db.updateFtsBody(envelope.id, bodyText);
             }
             catch { /* */ }
         }