@bobfrankston/rmfmail 1.1.91 → 1.1.93

package/.commitmsg

@@ -1,11 +1,20 @@
-Fix: typing after a URL no longer swallows text into the link
+Search: index full message bodies, not just the preview snippet
 
-In the TinyMCE editor, typing or pasting a URL auto-linked it, but the
-caret stayed inside the <a>, so every following character was appended
-into the link text — a whole sentence turned into one link.
+Body-text search only ever covered messages the user had opened. At sync
+time the FTS body_text was seeded with the short `preview` snippet; the
+full text was backfilled only when LocalStore.getMessage parsed a body
+on view — and even that backfill keyed on the legacy messages.folder_id
+column via a (account,folder,uid) lookup that could miss the row, so a
+word buried in an opened message's body still wasn't found (Bob:
+"mcdade" in a message body, search found nothing).
 
-rmf-tiny's adapter now has a keypress handler: when a printable key is
-pressed with the caret collapsed at the trailing edge of an <a>, the
-caret hops to just after the <a> first, so the new text lands outside
-the link. Covers autolink, pasted URLs, and hand-made links.
-(client/lib/rmf-tiny.js rebuilt from the adapter source.)
+Two fixes:
+- updateFtsBody(rowId, text) keys on the FTS rowid (= messages.id)
+  directly — no fragile folder lookup. store.ts passes envelope.id.
+- The IMAP sync path now indexes the full body into FTS right after
+  upsertMessage. extractPreview already parsed `source`, so the body
+  text is free — every synced message is body-searchable without the
+  user opening it first.
+
+Existing already-synced messages stay preview-only until re-synced
+(mailx -repair re-indexes them).

package/client/app.bundle.js

@@ -1095,7 +1095,8 @@ async function showMessage(accountId, uid, folderId, specialUse, isRetry = false
   const headerEl = document.getElementById("mv-header");
   const bodyEl = document.getElementById("mv-body");
   const attEl = document.getElementById("mv-attachments");
-  if (!isRetry && currentMessage && currentAccountId === accountId && currentMessage.uid === uid && (folderId === void 0 || currentMessage.folderId === folderId) && bodyEl.querySelector("iframe")) {
+  const sameMessage = !!currentMessage && currentAccountId === accountId && (envelope?.uuid && currentMessage.uuid ? envelope.uuid === currentMessage.uuid : currentMessage.uid === uid && (folderId === void 0 || currentMessage.folderId === folderId));
+  if (!isRetry && sameMessage && bodyEl.querySelector("iframe")) {
     if (envelope) {
       lastEnvelope = envelope;
       try {