@bobfrankston/mailx-store 0.1.29 → 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

@@ -1501,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.29",
+  "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.15",
-    "@bobfrankston/mailx-settings": "^0.1.18",
+    "@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.15",
-      "@bobfrankston/mailx-settings": "^0.1.18",
+      "@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 { /* */ }
         }