@bobfrankston/rmfmail 1.1.134 → 1.1.135

package/.commitmsg

@@ -1,27 +1,20 @@
-fix: -repair handles FTS5 schema drift + uses correct folder columns
+feat: elapsed-time wait indicator + autocomplete + invalid-addr UX
 
-Symptom: "rmfmail -repair" crashed with "no such column: T.to_text"
-on Bob's bobma DB. Root cause is a long-standing FTS5 schema drift —
-the messages_fts table is contentless (content=messages, content_rowid=id)
-and declares to_text/cc_text, but the live messages schema has
-to_json/cc_json. DELETE FROM messages_fts tries to materialize each
-row's content via the broken linkage and SQL-error'd before any work
-could land.
-
-Fix:
-- Use the FTS5 "delete-all" idiom (INSERT … VALUES('delete-all'))
-  to clear the index without touching the broken content lookup. Falls
-  back to DROP + CREATE on whatever exotic failure could survive that.
-- folders.total / .unread → .total_count / .unread_count (the live
-  column names; the old names never existed on this schema).
-- Reset highest_modseq to NULL so QRESYNC re-bootstraps; otherwise
-  the post-repair sync trusts a stale ModSeq and skips folders that
-  are actually empty locally.
-
-This unblocks the stale-folder_id repair path: bobma's Added2.issues
-has 552 ghost local rows tagged to a small folder but carrying
-INBOX-sized UIDs. Server FETCH on those UIDs in the wrong folder
-returns zero, the prefetch loop logs "0/N bodies" every tick, and
-the preview pane shows "Fetching body from server..." forever. Repair
-wipes the metadata, re-fetches envelopes, and folder_id gets rebuilt
-from the server's actual UID lists.
+- Body-fetch placeholder now renders an elapsed-time counter that ticks
+  every second ("Fetching body from server… (12s)"). After 30 s the
+  counter flips red+bold so a real wedge is visually distinct from a
+  normal slow Dovecot read. Cleared automatically when bodyAvailable
+  fires, generation advances, or the placeholder leaves the DOM.
+- Compose address autocomplete: clicking a "Frankston, Bob" suggestion
+  against a `bob` token used to produce `bob, "Frankston, Bob" <addr>`
+  (the original token preserved instead of replaced) in some edge cases.
+  Fix is in replaceCaretToken — strip the trailing comma/whitespace from
+  the pre-token segment, re-add a clean ", " separator. tokenSpanAtCaret
+  also now skips leading whitespace inside the span so the separator
+  belongs to "before", not the token itself.
+- "Invalid address" send-block now uses high-contrast white-on-red
+  styling (was muted red text on light grey, easy to miss). Both
+  classList wiring and CSS updated: the .compose-status-error class
+  gets a saturated red background, white bold text, padding/radius.
+  Message text bumped to "Invalid address: '...' — Send blocked" so
+  the user knows it's the gating condition, not a transient note.

package/client/app.bundle.js

@@ -1179,8 +1179,25 @@ async function showMessage(accountId, uid, folderId, specialUse, isRetry = false
       } catch {
       }
       const previewText = (msg.preview || cached?.preview || "").trim();
-      bodyEl.innerHTML = previewText ? `<div class="mv-preview-placeholder">${escapeHtml(previewText)}</div>` : `<div class="mv-empty">Fetching body from server\u2026</div>`;
+      const waitStart = Date.now();
+      const indicatorHtml = `<span class="mv-wait-elapsed" data-start="${waitStart}">(0s)</span>`;
+      bodyEl.innerHTML = previewText ? `<div class="mv-preview-placeholder">${escapeHtml(previewText)}<div class="mv-wait-line">Fetching body from server\u2026 ${indicatorHtml}</div></div>` : `<div class="mv-empty">Fetching body from server\u2026 ${indicatorHtml}</div>`;
       const captureGen = gen;
+      const tick = setInterval(() => {
+        if (captureGen !== showMessageGeneration) {
+          clearInterval(tick);
+          return;
+        }
+        const span = bodyEl.querySelector(".mv-wait-elapsed");
+        if (!span) {
+          clearInterval(tick);
+          return;
+        }
+        const secs = Math.round((Date.now() - waitStart) / 1e3);
+        span.textContent = secs >= 60 ? `(${Math.floor(secs / 60)}m ${secs % 60}s)` : `(${secs}s)`;
+        if (secs === 30)
+          span.classList.add("mv-wait-slow");
+      }, 1e3);
       const off = subscribeStore("*", (ev) => {
         if (ev.kind !== "bodyAvailable")
           return;
@@ -1188,9 +1205,11 @@ async function showMessage(accountId, uid, folderId, specialUse, isRetry = false
           return;
         if (captureGen !== showMessageGeneration) {
           off();
+          clearInterval(tick);
           return;
         }
         off();
+        clearInterval(tick);
         showMessage(accountId, uid, folderId, specialUse, true, envelope || cached || void 0).catch(() => {
         });
       });