@bobfrankston/mailx 1.0.232 → 1.0.233

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx",
-  "version": "1.0.232",
+  "version": "1.0.233",
   "description": "Local-first email client with IMAP sync and standalone native app",
   "type": "module",
   "main": "bin/mailx.js",
@@ -24,7 +24,7 @@
     "@bobfrankston/iflow-node": "^0.1.2",
     "@bobfrankston/miscinfo": "^1.0.8",
     "@bobfrankston/oauthsupport": "^1.0.22",
-    "@bobfrankston/msger": "^0.1.294",
+    "@bobfrankston/msger": "^0.1.295",
     "@capacitor/android": "^8.3.0",
     "@capacitor/cli": "^8.3.0",
     "@capacitor/core": "^8.3.0",
@@ -78,7 +78,7 @@
       "@bobfrankston/iflow-node": "^0.1.2",
       "@bobfrankston/miscinfo": "^1.0.8",
       "@bobfrankston/oauthsupport": "^1.0.22",
-      "@bobfrankston/msger": "^0.1.294",
+      "@bobfrankston/msger": "^0.1.295",
       "@capacitor/android": "^8.3.0",
       "@capacitor/cli": "^8.3.0",
       "@capacitor/core": "^8.3.0",

package/packages/mailx-store/db.js

@@ -127,8 +127,10 @@ export class MailxDB {
         this.db.exec("PRAGMA foreign_keys = ON");
         this.db.exec(SCHEMA);
         // Idempotent migrations for older databases that predate new columns.
-        // SQLite doesn't support "ADD COLUMN IF NOT EXISTS", so we probe the
-        // table schema and add missing columns one by one.
+        // SQLite doesn't support "ADD COLUMN IF NOT EXISTS", so we just try the
+        // ALTER and catch the "duplicate column" error. Simpler and more robust
+        // than probing via PRAGMA table_info (which can behave differently
+        // across sqlite drivers).
         this.addColumnIfMissing("messages", "thread_id", "TEXT");
         try {
             this.db.exec("CREATE INDEX IF NOT EXISTS idx_messages_thread_id ON messages(account_id, thread_id)");
@@ -137,10 +139,17 @@ export class MailxDB {
     }
     /** Idempotently add a column to a table if it's missing. */
     addColumnIfMissing(table, column, sqlType) {
-        const cols = this.db.prepare(`PRAGMA table_info(${table})`).all();
-        if (cols.some(c => c.name === column))
-            return;
-        this.db.exec(`ALTER TABLE ${table} ADD COLUMN ${column} ${sqlType}`);
+        try {
+            this.db.exec(`ALTER TABLE ${table} ADD COLUMN ${column} ${sqlType}`);
+            console.log(`  [db] added column ${table}.${column}`);
+        }
+        catch (e) {
+            const msg = String(e?.message || e);
+            // "duplicate column name" is the expected case when column already exists
+            if (!/duplicate column/i.test(msg)) {
+                console.error(`  [db] migration ${table}.${column} failed: ${msg}`);
+            }
+        }
     }
     /** Compute a thread id for an incoming message. Strategy:
      *  1. If any ancestor (in_reply_to or references) is already present in