@vellumai/assistant 0.4.55 → 0.4.57

package/src/memory/migrations/111-media-assets.ts

@@ -1,8 +1,7 @@
 import type { DrizzleDb } from "../db-connection.js";
 
 /**
- * Media assets, processing stages, keyframes, vision outputs,
- * timelines, events, tracking profiles, and event feedback tables with indexes.
+ * Media assets, processing stages, and keyframes tables with indexes.
  */
 export function createMediaAssetsTables(database: DrizzleDb): void {
   database.run(/*sql*/ `
@@ -64,108 +63,4 @@ export function createMediaAssetsTables(database: DrizzleDb): void {
   database.run(
     /*sql*/ `CREATE INDEX IF NOT EXISTS idx_media_keyframes_asset_timestamp ON media_keyframes(asset_id, timestamp)`,
   );
-
-  database.run(/*sql*/ `
-    CREATE TABLE IF NOT EXISTS media_vision_outputs (
-      id TEXT PRIMARY KEY,
-      asset_id TEXT NOT NULL REFERENCES media_assets(id) ON DELETE CASCADE,
-      keyframe_id TEXT NOT NULL REFERENCES media_keyframes(id) ON DELETE CASCADE,
-      analysis_type TEXT NOT NULL,
-      output TEXT NOT NULL,
-      confidence REAL,
-      created_at INTEGER NOT NULL
-    )
-  `);
-
-  database.run(
-    /*sql*/ `CREATE INDEX IF NOT EXISTS idx_media_vision_outputs_asset_id ON media_vision_outputs(asset_id)`,
-  );
-  database.run(
-    /*sql*/ `CREATE INDEX IF NOT EXISTS idx_media_vision_outputs_keyframe_id ON media_vision_outputs(keyframe_id)`,
-  );
-  database.run(
-    /*sql*/ `CREATE INDEX IF NOT EXISTS idx_media_vision_outputs_asset_type ON media_vision_outputs(asset_id, analysis_type)`,
-  );
-
-  database.run(/*sql*/ `
-    CREATE TABLE IF NOT EXISTS media_timelines (
-      id TEXT PRIMARY KEY,
-      asset_id TEXT NOT NULL REFERENCES media_assets(id) ON DELETE CASCADE,
-      start_time REAL NOT NULL,
-      end_time REAL NOT NULL,
-      segment_type TEXT NOT NULL,
-      attributes TEXT,
-      confidence REAL,
-      created_at INTEGER NOT NULL
-    )
-  `);
-
-  database.run(
-    /*sql*/ `CREATE INDEX IF NOT EXISTS idx_media_timelines_asset_id ON media_timelines(asset_id)`,
-  );
-  database.run(
-    /*sql*/ `CREATE INDEX IF NOT EXISTS idx_media_timelines_asset_time ON media_timelines(asset_id, start_time)`,
-  );
-
-  database.run(/*sql*/ `
-    CREATE TABLE IF NOT EXISTS media_events (
-      id TEXT PRIMARY KEY,
-      asset_id TEXT NOT NULL REFERENCES media_assets(id) ON DELETE CASCADE,
-      event_type TEXT NOT NULL,
-      start_time REAL NOT NULL,
-      end_time REAL NOT NULL,
-      confidence REAL NOT NULL,
-      reasons TEXT NOT NULL,
-      metadata TEXT,
-      created_at INTEGER NOT NULL
-    )
-  `);
-
-  database.run(
-    /*sql*/ `CREATE INDEX IF NOT EXISTS idx_media_events_asset_id ON media_events(asset_id)`,
-  );
-  database.run(
-    /*sql*/ `CREATE INDEX IF NOT EXISTS idx_media_events_asset_type ON media_events(asset_id, event_type)`,
-  );
-  database.run(
-    /*sql*/ `CREATE INDEX IF NOT EXISTS idx_media_events_confidence ON media_events(confidence DESC)`,
-  );
-
-  database.run(/*sql*/ `
-    CREATE TABLE IF NOT EXISTS media_tracking_profiles (
-      id TEXT PRIMARY KEY,
-      asset_id TEXT NOT NULL REFERENCES media_assets(id) ON DELETE CASCADE,
-      capabilities TEXT NOT NULL,
-      created_at INTEGER NOT NULL
-    )
-  `);
-
-  database.run(
-    /*sql*/ `CREATE INDEX IF NOT EXISTS idx_media_tracking_profiles_asset_id ON media_tracking_profiles(asset_id)`,
-  );
-
-  database.run(/*sql*/ `
-    CREATE TABLE IF NOT EXISTS media_event_feedback (
-      id TEXT PRIMARY KEY,
-      asset_id TEXT NOT NULL REFERENCES media_assets(id) ON DELETE CASCADE,
-      event_id TEXT NOT NULL REFERENCES media_events(id) ON DELETE CASCADE,
-      feedback_type TEXT NOT NULL,
-      original_start_time REAL,
-      original_end_time REAL,
-      corrected_start_time REAL,
-      corrected_end_time REAL,
-      notes TEXT,
-      created_at INTEGER NOT NULL
-    )
-  `);
-
-  database.run(
-    /*sql*/ `CREATE INDEX IF NOT EXISTS idx_media_event_feedback_asset_id ON media_event_feedback(asset_id)`,
-  );
-  database.run(
-    /*sql*/ `CREATE INDEX IF NOT EXISTS idx_media_event_feedback_event_id ON media_event_feedback(event_id)`,
-  );
-  database.run(
-    /*sql*/ `CREATE INDEX IF NOT EXISTS idx_media_event_feedback_type ON media_event_feedback(asset_id, feedback_type)`,
-  );
 }

package/src/memory/migrations/121-canonical-guardian-requests.ts

@@ -27,9 +27,9 @@ export function createCanonicalGuardianTables(database: DrizzleDb): void {
       answer_text TEXT,
       decided_by_external_user_id TEXT,
       followup_state TEXT,
-      expires_at TEXT,
-      created_at TEXT NOT NULL,
-      updated_at TEXT NOT NULL
+      expires_at INTEGER,
+      created_at INTEGER NOT NULL,
+      updated_at INTEGER NOT NULL
     )
   `);
 
@@ -61,8 +61,8 @@ export function createCanonicalGuardianTables(database: DrizzleDb): void {
       destination_chat_id TEXT,
       destination_message_id TEXT,
       status TEXT NOT NULL DEFAULT 'pending',
-      created_at TEXT NOT NULL,
-      updated_at TEXT NOT NULL
+      created_at INTEGER NOT NULL,
+      updated_at INTEGER NOT NULL
     )
   `);
 

package/src/memory/migrations/159-drop-contact-interaction-columns.ts

@@ -8,6 +8,11 @@ export function migrateDropContactInteractionColumns(
   } catch {
     /* already dropped or doesn't exist */
   }
+
+  // Drop the index on last_interaction before dropping the column — SQLite
+  // rejects ALTER TABLE DROP COLUMN when a dependent index exists.
+  database.run(/*sql*/ `DROP INDEX IF EXISTS idx_contacts_last_interaction`);
+
   try {
     database.run(/*sql*/ `ALTER TABLE contacts DROP COLUMN last_interaction`);
   } catch {

package/src/memory/migrations/161-drop-orphaned-media-tables.ts

@@ -0,0 +1,18 @@
+import type { DrizzleDb } from "../db-connection.js";
+import { getSqliteFrom } from "../db-connection.js";
+
+/**
+ * Drop orphaned media tables that are no longer used.
+ *
+ * These tables were previously created by 111-media-assets.ts but the CREATE
+ * TABLE statements were removed in PR #16739. This migration cleans up
+ * existing databases that still have them.
+ */
+export function migrateDropOrphanedMediaTables(database: DrizzleDb): void {
+  const raw = getSqliteFrom(database);
+  raw.exec(/*sql*/ `DROP TABLE IF EXISTS media_vision_outputs`);
+  raw.exec(/*sql*/ `DROP TABLE IF EXISTS media_timelines`);
+  raw.exec(/*sql*/ `DROP TABLE IF EXISTS media_events`);
+  raw.exec(/*sql*/ `DROP TABLE IF EXISTS media_tracking_profiles`);
+  raw.exec(/*sql*/ `DROP TABLE IF EXISTS media_event_feedback`);
+}

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

@@ -0,0 +1,283 @@
+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");
+
+/**
+ * Convert guardian table timestamps from ISO 8601 text to epoch ms integers,
+ * then rebuild the tables so that timestamp columns have INTEGER affinity.
+ *
+ * The `canonical_guardian_requests`, `canonical_guardian_deliveries`, and
+ * `scoped_approval_grants` tables were originally created with TEXT columns
+ * for timestamps. Migration step 1 converts existing data values in-place,
+ * and step 2 rebuilds the tables so the column declarations use INTEGER.
+ * Without the rebuild, SQLite's TEXT affinity coerces the integer values
+ * back to text strings on read, causing downstream code (e.g.
+ * `new Date(req.expiresAt).getTime()`) to produce NaN.
+ */
+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'
+    `);
+
+      // 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",
+      );
+    },
+  );
+}
+
+// ---------------------------------------------------------------------------
+// Table rebuild helpers
+// ---------------------------------------------------------------------------
+
+type RawDb = ReturnType<typeof getSqliteFrom>;
+
+function hasIntegerAffinity(
+  raw: RawDb,
+  table: string,
+  column: string,
+): boolean {
+  const row = raw
+    .query(
+      `SELECT type FROM pragma_table_info('${table}') WHERE name = '${column}'`,
+    )
+    .get() as { type: string } | null;
+  if (!row) return true; // column doesn't exist — nothing to fix
+  return row.type.toUpperCase() === "INTEGER";
+}
+
+function rebuildCanonicalGuardianRequests(raw: RawDb): void {
+  if (hasIntegerAffinity(raw, "canonical_guardian_requests", "created_at"))
+    return;
+
+  raw.exec("PRAGMA foreign_keys = OFF");
+  try {
+    raw.exec(/*sql*/ `
+      BEGIN;
+
+      CREATE TABLE canonical_guardian_requests_new (
+        id TEXT PRIMARY KEY,
+        kind TEXT NOT NULL,
+        source_type TEXT NOT NULL,
+        source_channel TEXT,
+        conversation_id TEXT,
+        requester_external_user_id TEXT,
+        requester_chat_id TEXT,
+        guardian_external_user_id TEXT,
+        guardian_principal_id TEXT,
+        call_session_id TEXT,
+        pending_question_id TEXT,
+        question_text TEXT,
+        request_code TEXT,
+        tool_name TEXT,
+        input_digest TEXT,
+        status TEXT NOT NULL DEFAULT 'pending',
+        answer_text TEXT,
+        decided_by_external_user_id TEXT,
+        decided_by_principal_id TEXT,
+        followup_state TEXT,
+        expires_at INTEGER,
+        created_at INTEGER NOT NULL,
+        updated_at INTEGER NOT NULL
+      );
+
+      INSERT INTO canonical_guardian_requests_new
+      SELECT id, kind, source_type, source_channel, conversation_id,
+             requester_external_user_id, requester_chat_id,
+             guardian_external_user_id, guardian_principal_id,
+             call_session_id, pending_question_id, question_text,
+             request_code, tool_name, input_digest, status, answer_text,
+             decided_by_external_user_id, decided_by_principal_id,
+             followup_state, expires_at, created_at, updated_at
+      FROM canonical_guardian_requests;
+
+      DROP TABLE canonical_guardian_requests;
+      ALTER TABLE canonical_guardian_requests_new RENAME TO canonical_guardian_requests;
+
+      CREATE INDEX IF NOT EXISTS idx_canonical_guardian_requests_status ON canonical_guardian_requests(status);
+      CREATE INDEX IF NOT EXISTS idx_canonical_guardian_requests_guardian ON canonical_guardian_requests(guardian_external_user_id, status);
+      CREATE INDEX IF NOT EXISTS idx_canonical_guardian_requests_conversation ON canonical_guardian_requests(conversation_id, status);
+      CREATE INDEX IF NOT EXISTS idx_canonical_guardian_requests_source ON canonical_guardian_requests(source_type, status);
+      CREATE INDEX IF NOT EXISTS idx_canonical_guardian_requests_kind ON canonical_guardian_requests(kind, status);
+      CREATE INDEX IF NOT EXISTS idx_canonical_guardian_requests_request_code ON canonical_guardian_requests(request_code);
+
+      COMMIT;
+    `);
+  } catch (e) {
+    try {
+      raw.exec("ROLLBACK");
+    } catch {
+      /* no active transaction */
+    }
+    throw e;
+  } finally {
+    raw.exec("PRAGMA foreign_keys = ON");
+  }
+}
+
+function rebuildCanonicalGuardianDeliveries(raw: RawDb): void {
+  if (hasIntegerAffinity(raw, "canonical_guardian_deliveries", "created_at"))
+    return;
+
+  raw.exec("PRAGMA foreign_keys = OFF");
+  try {
+    raw.exec(/*sql*/ `
+      BEGIN;
+
+      CREATE TABLE canonical_guardian_deliveries_new (
+        id TEXT PRIMARY KEY,
+        request_id TEXT NOT NULL REFERENCES canonical_guardian_requests(id) ON DELETE CASCADE,
+        destination_channel TEXT NOT NULL,
+        destination_conversation_id TEXT,
+        destination_chat_id TEXT,
+        destination_message_id TEXT,
+        status TEXT NOT NULL DEFAULT 'pending',
+        created_at INTEGER NOT NULL,
+        updated_at INTEGER NOT NULL
+      );
+
+      INSERT INTO canonical_guardian_deliveries_new
+      SELECT id, request_id, destination_channel, destination_conversation_id,
+             destination_chat_id, destination_message_id, status,
+             created_at, updated_at
+      FROM canonical_guardian_deliveries;
+
+      DROP TABLE canonical_guardian_deliveries;
+      ALTER TABLE canonical_guardian_deliveries_new RENAME TO canonical_guardian_deliveries;
+
+      CREATE INDEX IF NOT EXISTS idx_canonical_guardian_deliveries_request_id ON canonical_guardian_deliveries(request_id);
+      CREATE INDEX IF NOT EXISTS idx_canonical_guardian_deliveries_status ON canonical_guardian_deliveries(status);
+      CREATE INDEX IF NOT EXISTS idx_canonical_guardian_deliveries_destination ON canonical_guardian_deliveries(destination_channel, destination_chat_id);
+
+      COMMIT;
+    `);
+  } catch (e) {
+    try {
+      raw.exec("ROLLBACK");
+    } catch {
+      /* no active transaction */
+    }
+    throw e;
+  } finally {
+    raw.exec("PRAGMA foreign_keys = ON");
+  }
+}
+
+function rebuildScopedApprovalGrants(raw: RawDb): void {
+  if (hasIntegerAffinity(raw, "scoped_approval_grants", "created_at")) return;
+
+  raw.exec("PRAGMA foreign_keys = OFF");
+  try {
+    raw.exec(/*sql*/ `
+      BEGIN;
+
+      CREATE TABLE scoped_approval_grants_new (
+        id TEXT PRIMARY KEY,
+        scope_mode TEXT NOT NULL,
+        request_id TEXT,
+        tool_name TEXT,
+        input_digest TEXT,
+        request_channel TEXT NOT NULL,
+        decision_channel TEXT NOT NULL,
+        execution_channel TEXT,
+        conversation_id TEXT,
+        call_session_id TEXT,
+        requester_external_user_id TEXT,
+        guardian_external_user_id TEXT,
+        status TEXT NOT NULL,
+        expires_at INTEGER NOT NULL,
+        consumed_at INTEGER,
+        consumed_by_request_id TEXT,
+        created_at INTEGER NOT NULL,
+        updated_at INTEGER NOT NULL
+      );
+
+      INSERT INTO scoped_approval_grants_new
+      SELECT id, scope_mode, request_id, tool_name, input_digest,
+             request_channel, decision_channel, execution_channel,
+             conversation_id, call_session_id,
+             requester_external_user_id, guardian_external_user_id,
+             status, expires_at, consumed_at, consumed_by_request_id,
+             created_at, updated_at
+      FROM scoped_approval_grants;
+
+      DROP TABLE scoped_approval_grants;
+      ALTER TABLE scoped_approval_grants_new RENAME TO scoped_approval_grants;
+
+      CREATE INDEX IF NOT EXISTS idx_scoped_grants_request_id ON scoped_approval_grants(request_id) WHERE request_id IS NOT NULL;
+      CREATE INDEX IF NOT EXISTS idx_scoped_grants_tool_sig ON scoped_approval_grants(tool_name, input_digest) WHERE tool_name IS NOT NULL;
+      CREATE INDEX IF NOT EXISTS idx_scoped_grants_status_expires ON scoped_approval_grants(status, expires_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/163-rename-notification-thread-columns.ts

@@ -0,0 +1,35 @@
+import { type DrizzleDb } from "../db-connection.js";
+
+/**
+ * Rename notification_deliveries columns from thread-based names to
+ * conversation-based names, aligning with the broader thread → conversation
+ * terminology unification.
+ *
+ * Each rename is wrapped in try/catch so re-running the migration is
+ * idempotent (SQLite throws if the source column does not exist).
+ */
+export function migrateRenameNotificationThreadColumns(
+  database: DrizzleDb,
+): void {
+  try {
+    database.run(
+      `ALTER TABLE notification_deliveries RENAME COLUMN thread_action TO conversation_action`,
+    );
+  } catch {
+    /* already renamed */
+  }
+  try {
+    database.run(
+      `ALTER TABLE notification_deliveries RENAME COLUMN thread_target_conversation_id TO conversation_target_id`,
+    );
+  } catch {
+    /* already renamed */
+  }
+  try {
+    database.run(
+      `ALTER TABLE notification_deliveries RENAME COLUMN thread_decision_fallback_used TO conversation_fallback_used`,
+    );
+  } catch {
+    /* already renamed */
+  }
+}

package/src/memory/migrations/164-rename-conversation-type-column.ts

@@ -0,0 +1,19 @@
+import type { DrizzleDb } from "../db-connection.js";
+
+/**
+ * Rename the `thread_type` column to `conversation_type` in the conversations
+ * table and recreate the index with the new column name.
+ */
+export function migrateRenameConversationTypeColumn(database: DrizzleDb): void {
+  try {
+    database.run(
+      /*sql*/ `ALTER TABLE conversations RENAME COLUMN thread_type TO conversation_type`,
+    );
+  } catch {
+    /* already renamed */
+  }
+  database.run(/*sql*/ `DROP INDEX IF EXISTS idx_conversations_thread_type`);
+  database.run(
+    /*sql*/ `CREATE INDEX IF NOT EXISTS idx_conversations_conversation_type ON conversations(conversation_type)`,
+  );
+}

package/src/memory/migrations/165-rename-inbox-thread-state-table.ts

@@ -0,0 +1,15 @@
+import type { DrizzleDb } from "../db-connection.js";
+
+/**
+ * Rename assistant_inbox_thread_state -> assistant_inbox_conversation_state
+ * as part of the thread -> conversation terminology unification.
+ */
+export function migrateRenameInboxThreadStateTable(database: DrizzleDb): void {
+  try {
+    database.run(
+      /*sql*/ `ALTER TABLE assistant_inbox_thread_state RENAME TO assistant_inbox_conversation_state`,
+    );
+  } catch {
+    /* already renamed */
+  }
+}

package/src/memory/migrations/166-rename-followups-thread-id.ts

@@ -0,0 +1,20 @@
+import type { DrizzleDb } from "../db-connection.js";
+
+/**
+ * Rename followups.thread_id → conversation_id, aligning with the broader
+ * thread → conversation terminology unification.
+ *
+ * Wrapped in try/catch for idempotency (SQLite throws if the source column
+ * does not exist, meaning it was already renamed).
+ */
+export function migrateRenameFollowupsThreadIdColumn(
+  database: DrizzleDb,
+): void {
+  try {
+    database.run(
+      `ALTER TABLE followups RENAME COLUMN thread_id TO conversation_id`,
+    );
+  } catch {
+    /* already renamed */
+  }
+}

package/src/memory/migrations/167-rename-sequence-enrollments-thread-id.ts

@@ -0,0 +1,20 @@
+import type { DrizzleDb } from "../db-connection.js";
+
+/**
+ * Rename sequence_enrollments.thread_id → conversation_id, aligning with the
+ * broader thread → conversation terminology unification.
+ *
+ * Wrapped in try/catch for idempotency (SQLite throws if the source column
+ * does not exist, meaning it was already renamed).
+ */
+export function migrateRenameSequenceEnrollmentsThreadIdColumn(
+  database: DrizzleDb,
+): void {
+  try {
+    database.run(
+      `ALTER TABLE sequence_enrollments RENAME COLUMN thread_id TO conversation_id`,
+    );
+  } catch {
+    /* already renamed */
+  }
+}

package/src/memory/migrations/168-rename-sequence-steps-reply-key.ts

@@ -0,0 +1,17 @@
+import { type DrizzleDb, getSqliteFrom } from "../db-connection.js";
+
+/**
+ * Backfill existing sequence steps JSON blobs to rename the
+ * `replyToThread` key to `replyInSameConversation`, aligning with
+ * the broader thread → conversation terminology unification.
+ *
+ * The UPDATE is naturally idempotent — REPLACE is a no-op when
+ * the old key does not appear in the text, so no checkpoint guard
+ * is needed.
+ */
+export function migrateRenameSequenceStepsReplyKey(database: DrizzleDb): void {
+  const raw = getSqliteFrom(database);
+  raw.exec(
+    /*sql*/ `UPDATE sequences SET steps = REPLACE(steps, '"replyToThread":', '"replyInSameConversation":') WHERE steps LIKE '%"replyToThread":%'`,
+  );
+}

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

@@ -0,0 +1,64 @@
+import { type DrizzleDb, getSqliteFrom } from "../db-connection.js";
+import { withCrashRecovery } from "./validate-migration-state.js";
+
+/**
+ * One-shot migration: rename the `integration:gmail` provider key to
+ * `integration:google` across all three OAuth tables.
+ *
+ * PR #16355 renamed the provider key in code but did not include a data
+ * migration. Without this, existing users who connected Gmail before the
+ * rename have their connections orphaned — runtime lookups for
+ * `integration:google` never find the old `integration:gmail` rows.
+ *
+ * FK constraints require us to update child tables (oauth_apps,
+ * oauth_connections) before the parent (oauth_providers), or disable FKs.
+ * We disable FKs for safety and update all three tables atomically.
+ */
+export function migrateRenameGmailProviderKeyToGoogle(
+  database: DrizzleDb,
+): void {
+  withCrashRecovery(
+    database,
+    "migration_rename_gmail_provider_key_to_google_v1",
+    () => {
+      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();
+
+        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");
+      }
+    },
+  );
+}