@bobfrankston/rmfmail 1.2.16 → 1.2.18

package/packages/mailx-store/db.ts

@@ -488,6 +488,18 @@ export class MailxDB {
         try {
             this.db.exec("CREATE INDEX IF NOT EXISTS idx_messages_thread_id ON messages(account_id, thread_id)");
         } catch { /* already exists */ }
+        // (account_id, message_id) composite — THE hot-path lookup. upsertMessage's
+        // move-detect and computeThreadId both query `WHERE account_id=? AND
+        // message_id=?`. The old idx_messages_message_id is on message_id ALONE, so
+        // with an account_id predicate the planner instead picks an account_id-only
+        // index and SCANS every row for the account (Bob's bobma INBOX = 136k rows).
+        // One new-message upsert ran several such scans (one per reference); a
+        // 50-message batch = the 62-SECOND write txn the slow-txn logger caught
+        // (2026-06-16) — which held the write lock and starved every other writer.
+        // A leading composite turns each into an O(log n) seek.
+        try {
+            this.db.exec("CREATE INDEX IF NOT EXISTS idx_messages_acct_msgid ON messages(account_id, message_id)");
+        } catch { /* already exists */ }
         // is_replied: set when ANY other message in this account has in_reply_to
         // pointing at this row's message_id. Primary source of truth for the ↩
         // marker — \Answered is plan B (some servers strip it, Gmail labels
@@ -2900,7 +2912,13 @@ export class MailxDB {
         // straddling an `await` — the wrapper is nesting-safe (savepoints)
         // so it can't collide with the sync backfill's own transactions
         // when the two interleave across the yield below.
-        const WRITE_CHUNK = 500;
+        // 100, not 500: each chunk is one held write transaction, and the
+        // slow-txn logger caught these running 2+ SECONDS apiece under WAL
+        // pressure — long enough to lock out the sync worker's storeMessages
+        // ("database is locked"). Smaller chunks = shorter lock holds + a
+        // yield (below) after each, giving the worker write windows (Bob
+        // 2026-06-16, live debug).
+        const WRITE_CHUNK = 100;
         const entries = [...agg.entries()];
         for (let i = 0; i < entries.length; i += WRITE_CHUNK) {
             const slice = entries.slice(i, i + WRITE_CHUNK);

package/packages/mailx-store/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-store",
-  "version": "0.1.51",
+  "version": "0.1.52",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",