@vellumai/assistant 0.10.0 → 0.10.1-dev.202606240317.ea25efe

package/src/memory/migrations/150-oauth-apps-client-secret-path.ts

@@ -1,6 +1,5 @@
 import type { DrizzleDb } from "../db-connection.js";
 import { getSqliteFrom } from "../db-connection.js";
-import { withCrashRecovery } from "./validate-migration-state.js";
 
 /**
  * Add client_secret_credential_path column to oauth_apps.
@@ -15,87 +14,81 @@ import { withCrashRecovery } from "./validate-migration-state.js";
  * Idempotent — skips if the column already exists with NOT NULL.
  */
 export function migrateOAuthAppsClientSecretPath(database: DrizzleDb): void {
-  withCrashRecovery(
-    database,
-    "migration_oauth_apps_client_secret_path_v1",
-    () => {
-      const raw = getSqliteFrom(database);
-
-      // Guard: table must exist
-      const tableExists = raw
-        .query(
-          `SELECT 1 FROM sqlite_master WHERE type = 'table' AND name = 'oauth_apps'`,
-        )
-        .get();
-      if (!tableExists) return;
-
-      // Guard: check if column already exists with NOT NULL
-      const colInfo = raw
-        .query(
-          `SELECT "notnull" FROM pragma_table_info('oauth_apps') WHERE name = 'client_secret_credential_path'`,
-        )
-        .get() as { notnull: number } | null;
-      if (colInfo && colInfo.notnull === 1) return;
-
-      // Step 1: Add the column (nullable) — wrapped in try/catch for idempotency
-      if (!colInfo) {
-        try {
-          raw.exec(
-            /*sql*/ `ALTER TABLE oauth_apps ADD COLUMN client_secret_credential_path TEXT`,
-          );
-        } catch {
-          // Column may already exist from a previous partial run
-        }
-      }
-
-      // Step 2: Backfill existing rows
+  const raw = getSqliteFrom(database);
+
+  // Guard: table must exist
+  const tableExists = raw
+    .query(
+      `SELECT 1 FROM sqlite_master WHERE type = 'table' AND name = 'oauth_apps'`,
+    )
+    .get();
+  if (!tableExists) return;
+
+  // Guard: check if column already exists with NOT NULL
+  const colInfo = raw
+    .query(
+      `SELECT "notnull" FROM pragma_table_info('oauth_apps') WHERE name = 'client_secret_credential_path'`,
+    )
+    .get() as { notnull: number } | null;
+  if (colInfo && colInfo.notnull === 1) return;
+
+  // Step 1: Add the column (nullable) — wrapped in try/catch for idempotency
+  if (!colInfo) {
+    try {
       raw.exec(
-        /*sql*/ `UPDATE oauth_apps SET client_secret_credential_path = 'oauth_app/' || id || '/client_secret' WHERE client_secret_credential_path IS NULL`,
+        /*sql*/ `ALTER TABLE oauth_apps ADD COLUMN client_secret_credential_path TEXT`,
       );
+    } catch {
+      // Column may already exist from a previous partial run
+    }
+  }
 
-      // Step 3: Rebuild the table to enforce NOT NULL
-      raw.exec("PRAGMA foreign_keys = OFF");
-      try {
-        raw.exec("BEGIN");
-
-        raw.exec(/*sql*/ `
-          CREATE TABLE oauth_apps_new (
-            id TEXT PRIMARY KEY,
-            provider_key TEXT NOT NULL REFERENCES oauth_providers(provider_key),
-            client_id TEXT NOT NULL,
-            client_secret_credential_path TEXT NOT NULL DEFAULT '',
-            created_at INTEGER NOT NULL,
-            updated_at INTEGER NOT NULL
-          )
-        `);
-
-        raw.exec(/*sql*/ `
-          INSERT INTO oauth_apps_new
-          SELECT id, provider_key, client_id, client_secret_credential_path, created_at, updated_at
-          FROM oauth_apps
-        `);
-
-        raw.exec(/*sql*/ `DROP TABLE oauth_apps`);
-        raw.exec(/*sql*/ `ALTER TABLE oauth_apps_new RENAME TO oauth_apps`);
-
-        // Recreate the unique index
-        raw.exec(
-          /*sql*/ `CREATE UNIQUE INDEX IF NOT EXISTS idx_oauth_apps_provider_client ON oauth_apps(provider_key, client_id)`,
-        );
-
-        raw.exec("COMMIT");
-      } catch (e) {
-        try {
-          raw.exec("ROLLBACK");
-        } catch {
-          /* no active transaction */
-        }
-        throw e;
-      } finally {
-        raw.exec("PRAGMA foreign_keys = ON");
-      }
-    },
+  // Step 2: Backfill existing rows
+  raw.exec(
+    /*sql*/ `UPDATE oauth_apps SET client_secret_credential_path = 'oauth_app/' || id || '/client_secret' WHERE client_secret_credential_path IS NULL`,
   );
+
+  // Step 3: Rebuild the table to enforce NOT NULL
+  raw.exec("PRAGMA foreign_keys = OFF");
+  try {
+    raw.exec("BEGIN");
+
+    raw.exec(/*sql*/ `
+      CREATE TABLE oauth_apps_new (
+        id TEXT PRIMARY KEY,
+        provider_key TEXT NOT NULL REFERENCES oauth_providers(provider_key),
+        client_id TEXT NOT NULL,
+        client_secret_credential_path TEXT NOT NULL DEFAULT '',
+        created_at INTEGER NOT NULL,
+        updated_at INTEGER NOT NULL
+      )
+    `);
+
+    raw.exec(/*sql*/ `
+      INSERT INTO oauth_apps_new
+      SELECT id, provider_key, client_id, client_secret_credential_path, created_at, updated_at
+      FROM oauth_apps
+    `);
+
+    raw.exec(/*sql*/ `DROP TABLE oauth_apps`);
+    raw.exec(/*sql*/ `ALTER TABLE oauth_apps_new RENAME TO oauth_apps`);
+
+    // Recreate the unique index
+    raw.exec(
+      /*sql*/ `CREATE UNIQUE INDEX IF NOT EXISTS idx_oauth_apps_provider_client ON oauth_apps(provider_key, client_id)`,
+    );
+
+    raw.exec("COMMIT");
+  } catch (e) {
+    try {
+      raw.exec("ROLLBACK");
+    } catch {
+      /* no active transaction */
+    }
+    throw e;
+  } finally {
+    raw.exec("PRAGMA foreign_keys = ON");
+  }
 }
 
 /**

package/src/memory/migrations/157-invite-contact-id.ts

@@ -1,5 +1,4 @@
 import { type DrizzleDb, getSqliteFrom } from "../db-connection.js";
-import { withCrashRecovery } from "./validate-migration-state.js";
 
 /**
  * Add a required contact_id column to assistant_ingress_invites.
@@ -13,92 +12,90 @@ import { withCrashRecovery } from "./validate-migration-state.js";
  *  3. Rebuild the table to enforce NOT NULL (SQLite cannot ALTER COLUMN).
  */
 export function migrateInviteContactId(database: DrizzleDb): void {
-  withCrashRecovery(database, "migration_invite_contact_id_v1", () => {
-    const raw = getSqliteFrom(database);
+  const raw = getSqliteFrom(database);
 
-    const cols = (
-      raw.query(`PRAGMA table_info(assistant_ingress_invites)`).all() as Array<{
-        name: string;
-        notnull: number;
-      }>
-    ).map((c) => ({ name: c.name, notnull: c.notnull }));
+  const cols = (
+    raw.query(`PRAGMA table_info(assistant_ingress_invites)`).all() as Array<{
+      name: string;
+      notnull: number;
+    }>
+  ).map((c) => ({ name: c.name, notnull: c.notnull }));
 
-    const col = cols.find((c) => c.name === "contact_id");
+  const col = cols.find((c) => c.name === "contact_id");
 
-    // Step 1: Add the column as nullable
-    if (!col) {
-      raw.exec(
-        /*sql*/ `ALTER TABLE assistant_ingress_invites ADD COLUMN contact_id TEXT`,
-      );
-    }
-
-    // Step 2: Delete legacy rows with no contact binding
+  // Step 1: Add the column as nullable
+  if (!col) {
     raw.exec(
-      /*sql*/ `DELETE FROM assistant_ingress_invites WHERE contact_id IS NULL OR contact_id = ''`,
+      /*sql*/ `ALTER TABLE assistant_ingress_invites ADD COLUMN contact_id TEXT`,
     );
+  }
 
-    // Step 3: Rebuild the table with NOT NULL on contact_id
-    if (col && col.notnull === 1) return; // already NOT NULL
+  // Step 2: Delete legacy rows with no contact binding
+  raw.exec(
+    /*sql*/ `DELETE FROM assistant_ingress_invites WHERE contact_id IS NULL OR contact_id = ''`,
+  );
 
-    raw.exec("PRAGMA foreign_keys = OFF");
-    try {
-      raw.exec(/*sql*/ `
-          BEGIN;
+  // Step 3: Rebuild the table with NOT NULL on contact_id
+  if (col && col.notnull === 1) return; // already NOT NULL
 
-          CREATE TABLE assistant_ingress_invites_new (
-            id TEXT PRIMARY KEY,
-            source_channel TEXT NOT NULL,
-            token_hash TEXT NOT NULL,
-            created_by_session_id TEXT,
-            note TEXT,
-            max_uses INTEGER NOT NULL DEFAULT 1 CHECK (max_uses > 0),
-            use_count INTEGER NOT NULL DEFAULT 0 CHECK (use_count >= 0),
-            expires_at INTEGER NOT NULL,
-            status TEXT NOT NULL DEFAULT 'active',
-            redeemed_by_external_user_id TEXT,
-            redeemed_by_external_chat_id TEXT,
-            redeemed_at INTEGER,
-            expected_external_user_id TEXT,
-            voice_code_hash TEXT,
-            voice_code_digits INTEGER,
-            invite_code_hash TEXT,
-            friend_name TEXT,
-            guardian_name TEXT,
-            contact_id TEXT NOT NULL,
-            created_at INTEGER NOT NULL,
-            updated_at INTEGER NOT NULL
-          );
+  raw.exec("PRAGMA foreign_keys = OFF");
+  try {
+    raw.exec(/*sql*/ `
+        BEGIN;
 
-          INSERT INTO assistant_ingress_invites_new
-          SELECT id, source_channel, token_hash, created_by_session_id, note,
-                 max_uses, use_count, expires_at, status,
-                 redeemed_by_external_user_id, redeemed_by_external_chat_id,
-                 redeemed_at, expected_external_user_id, voice_code_hash,
-                 voice_code_digits, invite_code_hash, friend_name, guardian_name,
-                 contact_id, created_at, updated_at
-          FROM assistant_ingress_invites;
+        CREATE TABLE assistant_ingress_invites_new (
+          id TEXT PRIMARY KEY,
+          source_channel TEXT NOT NULL,
+          token_hash TEXT NOT NULL,
+          created_by_session_id TEXT,
+          note TEXT,
+          max_uses INTEGER NOT NULL DEFAULT 1 CHECK (max_uses > 0),
+          use_count INTEGER NOT NULL DEFAULT 0 CHECK (use_count >= 0),
+          expires_at INTEGER NOT NULL,
+          status TEXT NOT NULL DEFAULT 'active',
+          redeemed_by_external_user_id TEXT,
+          redeemed_by_external_chat_id TEXT,
+          redeemed_at INTEGER,
+          expected_external_user_id TEXT,
+          voice_code_hash TEXT,
+          voice_code_digits INTEGER,
+          invite_code_hash TEXT,
+          friend_name TEXT,
+          guardian_name TEXT,
+          contact_id TEXT NOT NULL,
+          created_at INTEGER NOT NULL,
+          updated_at INTEGER NOT NULL
+        );
 
-          DROP TABLE assistant_ingress_invites;
-          ALTER TABLE assistant_ingress_invites_new RENAME TO assistant_ingress_invites;
+        INSERT INTO assistant_ingress_invites_new
+        SELECT id, source_channel, token_hash, created_by_session_id, note,
+               max_uses, use_count, expires_at, status,
+               redeemed_by_external_user_id, redeemed_by_external_chat_id,
+               redeemed_at, expected_external_user_id, voice_code_hash,
+               voice_code_digits, invite_code_hash, friend_name, guardian_name,
+               contact_id, created_at, updated_at
+        FROM assistant_ingress_invites;
 
-          CREATE UNIQUE INDEX IF NOT EXISTS idx_ingress_invites_token_hash
-            ON assistant_ingress_invites(token_hash);
-          CREATE INDEX IF NOT EXISTS idx_ingress_invites_channel_status
-            ON assistant_ingress_invites(source_channel, status, expires_at);
-          CREATE INDEX IF NOT EXISTS idx_ingress_invites_channel_created
-            ON assistant_ingress_invites(source_channel, created_at);
+        DROP TABLE assistant_ingress_invites;
+        ALTER TABLE assistant_ingress_invites_new RENAME TO assistant_ingress_invites;
 
-          COMMIT;
-        `);
-    } catch (e) {
-      try {
-        raw.exec("ROLLBACK");
-      } catch {
-        /* no active transaction */
-      }
-      throw e;
-    } finally {
-      raw.exec("PRAGMA foreign_keys = ON");
+        CREATE UNIQUE INDEX IF NOT EXISTS idx_ingress_invites_token_hash
+          ON assistant_ingress_invites(token_hash);
+        CREATE INDEX IF NOT EXISTS idx_ingress_invites_channel_status
+          ON assistant_ingress_invites(source_channel, status, expires_at);
+        CREATE INDEX IF NOT EXISTS idx_ingress_invites_channel_created
+          ON assistant_ingress_invites(source_channel, created_at);
+
+        COMMIT;
+      `);
+  } catch (e) {
+    try {
+      raw.exec("ROLLBACK");
+    } catch {
+      /* no active transaction */
     }
-  });
+    throw e;
+  } finally {
+    raw.exec("PRAGMA foreign_keys = ON");
+  }
 }

package/src/memory/migrations/162-guardian-timestamps-epoch-ms.ts

@@ -1,6 +1,5 @@
 import { getLogger } from "../../util/logger.js";
 import { type DrizzleDb, getSqliteFrom } from "../db-connection.js";
-import { withCrashRecovery } from "./validate-migration-state.js";
 
 const log = getLogger("migration-162");
 
@@ -18,71 +17,58 @@ const log = getLogger("migration-162");
  */
 export function migrateGuardianTimestampsEpochMs(database: DrizzleDb): void {
   // Step 1: Convert existing text timestamps to integer epoch ms values.
-  withCrashRecovery(
-    database,
-    "migration_guardian_timestamps_epoch_ms_v1",
-    () => {
-      const raw = getSqliteFrom(database);
-
-      // Convert canonical_guardian_requests timestamp columns
-      raw.exec(/*sql*/ `
-      UPDATE canonical_guardian_requests
-      SET created_at = CAST(strftime('%s', created_at) AS INTEGER) * 1000 + CAST(substr(created_at, 21, 3) AS INTEGER),
-          updated_at = CAST(strftime('%s', updated_at) AS INTEGER) * 1000 + CAST(substr(updated_at, 21, 3) AS INTEGER),
-          expires_at = CASE
-            WHEN expires_at IS NOT NULL
-            THEN CAST(strftime('%s', expires_at) AS INTEGER) * 1000 + CAST(substr(expires_at, 21, 3) AS INTEGER)
-            ELSE NULL
-          END
-      WHERE typeof(created_at) = 'text'
-    `);
-
-      // Convert canonical_guardian_deliveries timestamp columns
-      raw.exec(/*sql*/ `
-      UPDATE canonical_guardian_deliveries
-      SET created_at = CAST(strftime('%s', created_at) AS INTEGER) * 1000 + CAST(substr(created_at, 21, 3) AS INTEGER),
-          updated_at = CAST(strftime('%s', updated_at) AS INTEGER) * 1000 + CAST(substr(updated_at, 21, 3) AS INTEGER)
-      WHERE typeof(created_at) = 'text'
-    `);
+  const raw = getSqliteFrom(database);
 
-      // Convert scoped_approval_grants timestamp columns
-      raw.exec(/*sql*/ `
-      UPDATE scoped_approval_grants
-      SET expires_at = CAST(strftime('%s', expires_at) AS INTEGER) * 1000 + CAST(substr(expires_at, 21, 3) AS INTEGER),
-          created_at = CAST(strftime('%s', created_at) AS INTEGER) * 1000 + CAST(substr(created_at, 21, 3) AS INTEGER),
-          updated_at = CAST(strftime('%s', updated_at) AS INTEGER) * 1000 + CAST(substr(updated_at, 21, 3) AS INTEGER),
-          consumed_at = CASE
-            WHEN consumed_at IS NOT NULL
-            THEN CAST(strftime('%s', consumed_at) AS INTEGER) * 1000 + CAST(substr(consumed_at, 21, 3) AS INTEGER)
-            ELSE NULL
-          END
-      WHERE typeof(created_at) = 'text'
-    `);
+  // Convert canonical_guardian_requests timestamp columns
+  raw.exec(/*sql*/ `
+  UPDATE canonical_guardian_requests
+  SET created_at = CAST(strftime('%s', created_at) AS INTEGER) * 1000 + CAST(substr(created_at, 21, 3) AS INTEGER),
+      updated_at = CAST(strftime('%s', updated_at) AS INTEGER) * 1000 + CAST(substr(updated_at, 21, 3) AS INTEGER),
+      expires_at = CASE
+        WHEN expires_at IS NOT NULL
+        THEN CAST(strftime('%s', expires_at) AS INTEGER) * 1000 + CAST(substr(expires_at, 21, 3) AS INTEGER)
+        ELSE NULL
+      END
+  WHERE typeof(created_at) = 'text'
+`);
+
+  // Convert canonical_guardian_deliveries timestamp columns
+  raw.exec(/*sql*/ `
+  UPDATE canonical_guardian_deliveries
+  SET created_at = CAST(strftime('%s', created_at) AS INTEGER) * 1000 + CAST(substr(created_at, 21, 3) AS INTEGER),
+      updated_at = CAST(strftime('%s', updated_at) AS INTEGER) * 1000 + CAST(substr(updated_at, 21, 3) AS INTEGER)
+  WHERE typeof(created_at) = 'text'
+`);
 
-      log.info(
-        "Converted guardian table timestamps from ISO 8601 text to epoch ms",
-      );
-    },
+  // Convert scoped_approval_grants timestamp columns
+  raw.exec(/*sql*/ `
+  UPDATE scoped_approval_grants
+  SET expires_at = CAST(strftime('%s', expires_at) AS INTEGER) * 1000 + CAST(substr(expires_at, 21, 3) AS INTEGER),
+      created_at = CAST(strftime('%s', created_at) AS INTEGER) * 1000 + CAST(substr(created_at, 21, 3) AS INTEGER),
+      updated_at = CAST(strftime('%s', updated_at) AS INTEGER) * 1000 + CAST(substr(updated_at, 21, 3) AS INTEGER),
+      consumed_at = CASE
+        WHEN consumed_at IS NOT NULL
+        THEN CAST(strftime('%s', consumed_at) AS INTEGER) * 1000 + CAST(substr(consumed_at, 21, 3) AS INTEGER)
+        ELSE NULL
+      END
+  WHERE typeof(created_at) = 'text'
+`);
+
+  log.info(
+    "Converted guardian table timestamps from ISO 8601 text to epoch ms",
   );
 
   // Step 2: Rebuild tables so timestamp columns have INTEGER affinity.
   // Databases created before the CREATE TABLE migrations were updated still
   // have TEXT affinity on these columns, which coerces integer values back
   // to text strings on read.
-  withCrashRecovery(
-    database,
-    "migration_guardian_timestamps_rebuild_v1",
-    () => {
-      const raw = getSqliteFrom(database);
-
-      rebuildCanonicalGuardianRequests(raw);
-      rebuildCanonicalGuardianDeliveries(raw);
-      rebuildScopedApprovalGrants(raw);
-
-      log.info(
-        "Rebuilt guardian tables with INTEGER affinity on timestamp columns",
-      );
-    },
+
+  rebuildCanonicalGuardianRequests(raw);
+  rebuildCanonicalGuardianDeliveries(raw);
+  rebuildScopedApprovalGrants(raw);
+
+  log.info(
+    "Rebuilt guardian tables with INTEGER affinity on timestamp columns",
   );
 }
 

package/src/memory/migrations/169-rename-gmail-provider-key-to-google.ts

@@ -1,6 +1,5 @@
 import type { DrizzleDb } from "../db-connection.js";
 import { getSqliteFrom } from "../db-connection.js";
-import { withCrashRecovery } from "./validate-migration-state.js";
 
 /**
  * One-shot migration: rename the `integration:gmail` provider key to
@@ -18,50 +17,44 @@ import { withCrashRecovery } from "./validate-migration-state.js";
 export function migrateRenameGmailProviderKeyToGoogle(
   database: DrizzleDb,
 ): void {
-  withCrashRecovery(
-    database,
-    "migration_rename_gmail_provider_key_to_google_v1",
-    () => {
-      const raw = getSqliteFrom(database);
+  const raw = getSqliteFrom(database);
 
-      raw.exec("PRAGMA foreign_keys = OFF");
-      try {
-        // If `integration:google` already exists (runtime seeded it after the
-        // code rename), the old `integration:gmail` rows are orphaned — just
-        // delete them instead of renaming.
-        const googleExists = raw
-          .prepare(
-            /*sql*/ `SELECT 1 FROM oauth_providers WHERE provider_key = 'integration:google'`,
-          )
-          .get();
+  raw.exec("PRAGMA foreign_keys = OFF");
+  try {
+    // If `integration:google` already exists (runtime seeded it after the
+    // code rename), the old `integration:gmail` rows are orphaned — just
+    // delete them instead of renaming.
+    const googleExists = raw
+      .prepare(
+        /*sql*/ `SELECT 1 FROM oauth_providers WHERE provider_key = 'integration:google'`,
+      )
+      .get();
 
-        if (googleExists) {
-          raw.exec(
-            /*sql*/ `DELETE FROM oauth_connections WHERE provider_key = 'integration:gmail'`,
-          );
-          raw.exec(
-            /*sql*/ `DELETE FROM oauth_apps WHERE provider_key = 'integration:gmail'`,
-          );
-          raw.exec(
-            /*sql*/ `DELETE FROM oauth_providers WHERE provider_key = 'integration:gmail'`,
-          );
-        } else {
-          // Update child tables first, then the parent.
-          raw.exec(
-            /*sql*/ `UPDATE oauth_connections SET provider_key = 'integration:google' WHERE provider_key = 'integration:gmail'`,
-          );
-          raw.exec(
-            /*sql*/ `UPDATE oauth_apps SET provider_key = 'integration:google' WHERE provider_key = 'integration:gmail'`,
-          );
-          raw.exec(
-            /*sql*/ `UPDATE oauth_providers SET provider_key = 'integration:google' WHERE provider_key = 'integration:gmail'`,
-          );
-        }
-      } finally {
-        raw.exec("PRAGMA foreign_keys = ON");
-      }
-    },
-  );
+    if (googleExists) {
+      raw.exec(
+        /*sql*/ `DELETE FROM oauth_connections WHERE provider_key = 'integration:gmail'`,
+      );
+      raw.exec(
+        /*sql*/ `DELETE FROM oauth_apps WHERE provider_key = 'integration:gmail'`,
+      );
+      raw.exec(
+        /*sql*/ `DELETE FROM oauth_providers WHERE provider_key = 'integration:gmail'`,
+      );
+    } else {
+      // Update child tables first, then the parent.
+      raw.exec(
+        /*sql*/ `UPDATE oauth_connections SET provider_key = 'integration:google' WHERE provider_key = 'integration:gmail'`,
+      );
+      raw.exec(
+        /*sql*/ `UPDATE oauth_apps SET provider_key = 'integration:google' WHERE provider_key = 'integration:gmail'`,
+      );
+      raw.exec(
+        /*sql*/ `UPDATE oauth_providers SET provider_key = 'integration:google' WHERE provider_key = 'integration:gmail'`,
+      );
+    }
+  } finally {
+    raw.exec("PRAGMA foreign_keys = ON");
+  }
 }
 
 /**

package/src/memory/migrations/174-rename-thread-starters-table.ts

@@ -1,6 +1,5 @@
 import type { DrizzleDb } from "../db-connection.js";
 import { getSqliteFrom } from "../db-connection.js";
-import { withCrashRecovery } from "./validate-migration-state.js";
 
 /**
  * Rename `thread_starters` table to `conversation_starters` and recreate
@@ -8,48 +7,42 @@ import { withCrashRecovery } from "./validate-migration-state.js";
  * terminology unification.
  */
 export function migrateRenameThreadStartersTable(database: DrizzleDb): void {
-  withCrashRecovery(
-    database,
-    "migration_rename_thread_starters_table_v1",
-    () => {
-      const raw = getSqliteFrom(database);
+  const raw = getSqliteFrom(database);
 
-      // Check the old table exists before attempting anything
-      const oldTableExists = raw
-        .query(
-          `SELECT 1 FROM sqlite_master WHERE type = 'table' AND name = 'thread_starters'`,
-        )
-        .get();
-      if (!oldTableExists) return;
+  // Check the old table exists before attempting anything
+  const oldTableExists = raw
+    .query(
+      `SELECT 1 FROM sqlite_master WHERE type = 'table' AND name = 'thread_starters'`,
+    )
+    .get();
+  if (!oldTableExists) return;
 
-      // If the new table already exists (crash recovery), skip the rename
-      const newTableExists = raw
-        .query(
-          `SELECT 1 FROM sqlite_master WHERE type = 'table' AND name = 'conversation_starters'`,
-        )
-        .get();
-      if (newTableExists) return;
+  // If the new table already exists (crash recovery), skip the rename
+  const newTableExists = raw
+    .query(
+      `SELECT 1 FROM sqlite_master WHERE type = 'table' AND name = 'conversation_starters'`,
+    )
+    .get();
+  if (newTableExists) return;
 
-      // Rename the physical table
-      raw.exec(
-        /*sql*/ `ALTER TABLE thread_starters RENAME TO conversation_starters`,
-      );
+  // Rename the physical table
+  raw.exec(
+    /*sql*/ `ALTER TABLE thread_starters RENAME TO conversation_starters`,
+  );
 
-      // Drop old indexes and recreate with new names.
-      // SQLite automatically updates index table references on RENAME, but the
-      // index names still reference the old naming convention — drop and recreate
-      // with consistent names pointing at the new table.
+  // Drop old indexes and recreate with new names.
+  // SQLite automatically updates index table references on RENAME, but the
+  // index names still reference the old naming convention — drop and recreate
+  // with consistent names pointing at the new table.
 
-      raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_thread_starters_batch`);
-      raw.exec(
-        /*sql*/ `CREATE INDEX IF NOT EXISTS idx_conversation_starters_batch ON conversation_starters(generation_batch, created_at)`,
-      );
+  raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_thread_starters_batch`);
+  raw.exec(
+    /*sql*/ `CREATE INDEX IF NOT EXISTS idx_conversation_starters_batch ON conversation_starters(generation_batch, created_at)`,
+  );
 
-      raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_thread_starters_card_type`);
-      raw.exec(
-        /*sql*/ `CREATE INDEX IF NOT EXISTS idx_conversation_starters_card_type ON conversation_starters(card_type, scope_id)`,
-      );
-    },
+  raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_thread_starters_card_type`);
+  raw.exec(
+    /*sql*/ `CREATE INDEX IF NOT EXISTS idx_conversation_starters_card_type ON conversation_starters(card_type, scope_id)`,
   );
 }