@bobfrankston/mailx-store 0.1.48 → 0.1.49

package/db.d.ts

@@ -15,6 +15,7 @@ export declare class MailxDB {
     readonly readOnly: boolean;
     constructor(dbDir: string, opts?: {
         readOnly?: boolean;
+        skipMigrations?: boolean;
     });
     /** Fail loud + early if expected columns are missing. Cheap (PRAGMA only
      *  runs at startup). The user-facing message names the recovery command. */

package/db.js

@@ -412,6 +412,16 @@ export class MailxDB {
         const dbPath = path.join(dbDir, "mailx.db");
         this.readOnly = !!opts.readOnly;
         this.db = new DatabaseSync(dbPath);
+        if (opts.skipMigrations && !opts.readOnly) {
+            // Write-capable connection that does NOT run schema/migrations —
+            // used by the sync worker, which opens its own writer AFTER the
+            // main thread has already built the schema. Two threads running the
+            // DDL/backfills concurrently would race on the WAL write lock.
+            this.db.exec("PRAGMA journal_mode = WAL");
+            this.db.exec("PRAGMA foreign_keys = ON");
+            this.db.exec("PRAGMA busy_timeout = 5000");
+            return;
+        }
         if (this.readOnly) {
             // Read-worker connection (Phase 0 read isolation). The main-thread
             // MailxDB has ALREADY created the schema and run every migration /
@@ -430,10 +440,19 @@ export class MailxDB {
             // so readers see committed snapshots with no journal_mode call.
             this.db.exec("PRAGMA foreign_keys = ON");
             this.db.exec("PRAGMA query_only = ON");
+            // A reader can briefly collide with a WAL checkpoint; wait, don't throw.
+            this.db.exec("PRAGMA busy_timeout = 5000");
             return;
         }
         this.db.exec("PRAGMA journal_mode = WAL");
         this.db.exec("PRAGMA foreign_keys = ON");
+        // Multi-writer foundation for the sync-worker migration: once sync runs
+        // on its own thread with its own write connection, it and the main
+        // thread are two writers on one WAL file. WAL serializes writers, so a
+        // collision returns SQLITE_BUSY *immediately* by default — busy_timeout
+        // makes the loser WAIT for the lock (up to 5s) instead of failing. Safe
+        // and inert today (single writer); required before the worker lands.
+        this.db.exec("PRAGMA busy_timeout = 5000");
         this.db.exec(SCHEMA);
         this.migrateFtsSchema();
         // Purge phantom uid=0 rows (invalid IMAP UID — empty stub letters, the

package/package.json

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