@vellumai/assistant 0.10.0 → 0.10.1-staging.1

package/src/memory/migrations/143-rename-guardian-verification-values.ts

@@ -1,5 +1,4 @@
 import { type DrizzleDb, getSqliteFrom } from "../db-connection.js";
-import { withCrashRecovery } from "./validate-migration-state.js";
 
 /**
  * Reverse v23: add the "guardian_" prefix back to verification-related
@@ -48,36 +47,30 @@ export function downRenameGuardianVerificationValues(
 export function migrateRenameGuardianVerificationValues(
   database: DrizzleDb,
 ): void {
-  withCrashRecovery(
-    database,
-    "migration_rename_guardian_verification_values_v1",
-    () => {
-      const raw = getSqliteFrom(database);
+  const raw = getSqliteFrom(database);
 
-      // Rename call_mode values
-      raw.exec(
-        /*sql*/ `UPDATE call_sessions SET call_mode = 'verification' WHERE call_mode = 'guardian_verification'`,
-      );
+  // Rename call_mode values
+  raw.exec(
+    /*sql*/ `UPDATE call_sessions SET call_mode = 'verification' WHERE call_mode = 'guardian_verification'`,
+  );
 
-      // Rename event_type values
-      raw.exec(
-        /*sql*/ `UPDATE call_events SET event_type = 'voice_verification_started' WHERE event_type = 'guardian_voice_verification_started'`,
-      );
-      raw.exec(
-        /*sql*/ `UPDATE call_events SET event_type = 'voice_verification_succeeded' WHERE event_type = 'guardian_voice_verification_succeeded'`,
-      );
-      raw.exec(
-        /*sql*/ `UPDATE call_events SET event_type = 'voice_verification_failed' WHERE event_type = 'guardian_voice_verification_failed'`,
-      );
-      raw.exec(
-        /*sql*/ `UPDATE call_events SET event_type = 'outbound_voice_verification_started' WHERE event_type = 'outbound_guardian_voice_verification_started'`,
-      );
-      raw.exec(
-        /*sql*/ `UPDATE call_events SET event_type = 'outbound_voice_verification_succeeded' WHERE event_type = 'outbound_guardian_voice_verification_succeeded'`,
-      );
-      raw.exec(
-        /*sql*/ `UPDATE call_events SET event_type = 'outbound_voice_verification_failed' WHERE event_type = 'outbound_guardian_voice_verification_failed'`,
-      );
-    },
+  // Rename event_type values
+  raw.exec(
+    /*sql*/ `UPDATE call_events SET event_type = 'voice_verification_started' WHERE event_type = 'guardian_voice_verification_started'`,
+  );
+  raw.exec(
+    /*sql*/ `UPDATE call_events SET event_type = 'voice_verification_succeeded' WHERE event_type = 'guardian_voice_verification_succeeded'`,
+  );
+  raw.exec(
+    /*sql*/ `UPDATE call_events SET event_type = 'voice_verification_failed' WHERE event_type = 'guardian_voice_verification_failed'`,
+  );
+  raw.exec(
+    /*sql*/ `UPDATE call_events SET event_type = 'outbound_voice_verification_started' WHERE event_type = 'outbound_guardian_voice_verification_started'`,
+  );
+  raw.exec(
+    /*sql*/ `UPDATE call_events SET event_type = 'outbound_voice_verification_succeeded' WHERE event_type = 'outbound_guardian_voice_verification_succeeded'`,
+  );
+  raw.exec(
+    /*sql*/ `UPDATE call_events SET event_type = 'outbound_voice_verification_failed' WHERE event_type = 'outbound_guardian_voice_verification_failed'`,
   );
 }

package/src/memory/migrations/144-rename-voice-to-phone.ts

@@ -1,5 +1,4 @@
 import { type DrizzleDb, getSqliteFrom } from "../db-connection.js";
-import { withCrashRecovery } from "./validate-migration-state.js";
 
 /**
  * Reverse v24: rename "phone" channel values back to "voice" across all
@@ -145,139 +144,137 @@ export function downRenameVoiceToPhone(database: DrizzleDb): void {
  * as the canonical channel ID for phone/voice calls.
  */
 export function migrateRenameVoiceToPhone(database: DrizzleDb): void {
-  withCrashRecovery(database, "migration_rename_voice_to_phone_v1", () => {
-    const raw = getSqliteFrom(database);
-
-    // contact_channels.type
-    raw.exec(
-      /*sql*/ `UPDATE contact_channels SET type = 'phone' WHERE type = 'voice'`,
-    );
-
-    // conversations.origin_channel
-    raw.exec(
-      /*sql*/ `UPDATE conversations SET origin_channel = 'phone' WHERE origin_channel = 'voice'`,
-    );
-
-    // conversations.origin_interface
-    raw.exec(
-      /*sql*/ `UPDATE conversations SET origin_interface = 'phone' WHERE origin_interface = 'voice'`,
-    );
-
-    // messages.metadata — JSON blobs may contain "voice" as a channel/interface value
-    // (e.g. userMessageChannel, provenanceSourceChannel). Replace the quoted JSON
-    // string so that messageMetadataSchema (which uses z.enum(CHANNEL_IDS)) can
-    // still parse historical rows.
-    raw.exec(
-      /*sql*/ `UPDATE messages SET metadata = REPLACE(metadata, '"voice"', '"phone"') WHERE metadata LIKE '%"voice"%'`,
-    );
-
-    // assistant_ingress_invites.source_channel
-    raw.exec(
-      /*sql*/ `UPDATE assistant_ingress_invites SET source_channel = 'phone' WHERE source_channel = 'voice'`,
-    );
-
-    // assistant_inbox_thread_state.source_channel
-    raw.exec(
-      /*sql*/ `UPDATE assistant_inbox_thread_state SET source_channel = 'phone' WHERE source_channel = 'voice'`,
-    );
-
-    // guardian_action_requests.source_channel
-    raw.exec(
-      /*sql*/ `UPDATE guardian_action_requests SET source_channel = 'phone' WHERE source_channel = 'voice'`,
-    );
-
-    // guardian_action_requests.answered_by_channel
-    raw.exec(
-      /*sql*/ `UPDATE guardian_action_requests SET answered_by_channel = 'phone' WHERE answered_by_channel = 'voice'`,
-    );
-
-    // channel_verification_sessions.channel
-    raw.exec(
-      /*sql*/ `UPDATE channel_verification_sessions SET channel = 'phone' WHERE channel = 'voice'`,
-    );
-
-    // channel_guardian_approval_requests.channel
-    raw.exec(
-      /*sql*/ `UPDATE channel_guardian_approval_requests SET channel = 'phone' WHERE channel = 'voice'`,
-    );
-
-    // channel_guardian_rate_limits.channel
-    // Dedup: remove voice rows that would collide with existing phone rows
-    // on the UNIQUE index (channel, actor_external_user_id, actor_chat_id).
-    raw.exec(/*sql*/ `DELETE FROM channel_guardian_rate_limits WHERE channel = 'voice' AND EXISTS (
-        SELECT 1 FROM channel_guardian_rate_limits AS t2
-        WHERE t2.channel = 'phone'
-          AND t2.actor_external_user_id = channel_guardian_rate_limits.actor_external_user_id
-          AND t2.actor_chat_id = channel_guardian_rate_limits.actor_chat_id
-      )`);
-    raw.exec(
-      /*sql*/ `UPDATE channel_guardian_rate_limits SET channel = 'phone' WHERE channel = 'voice'`,
-    );
-
-    // notification_events.source_channel
-    raw.exec(
-      /*sql*/ `UPDATE notification_events SET source_channel = 'phone' WHERE source_channel = 'voice'`,
-    );
-
-    // notification_deliveries.channel
-    raw.exec(
-      /*sql*/ `UPDATE notification_deliveries SET channel = 'phone' WHERE channel = 'voice'`,
-    );
-
-    // external_conversation_bindings.source_channel
-    raw.exec(
-      /*sql*/ `UPDATE external_conversation_bindings SET source_channel = 'phone' WHERE source_channel = 'voice'`,
-    );
-
-    // channel_inbound_events.source_channel
-    raw.exec(
-      /*sql*/ `UPDATE channel_inbound_events SET source_channel = 'phone' WHERE source_channel = 'voice'`,
-    );
-
-    // conversation_attention_events.source_channel
-    raw.exec(
-      /*sql*/ `UPDATE conversation_attention_events SET source_channel = 'phone' WHERE source_channel = 'voice'`,
-    );
-
-    // conversation_assistant_attention_state.last_seen_source_channel
-    raw.exec(
-      /*sql*/ `UPDATE conversation_assistant_attention_state SET last_seen_source_channel = 'phone' WHERE last_seen_source_channel = 'voice'`,
-    );
-
-    // canonical_guardian_requests.source_channel
-    raw.exec(
-      /*sql*/ `UPDATE canonical_guardian_requests SET source_channel = 'phone' WHERE source_channel = 'voice'`,
-    );
-
-    // canonical_guardian_deliveries.destination_channel
-    raw.exec(
-      /*sql*/ `UPDATE canonical_guardian_deliveries SET destination_channel = 'phone' WHERE destination_channel = 'voice'`,
-    );
-
-    // guardian_action_deliveries.destination_channel
-    raw.exec(
-      /*sql*/ `UPDATE guardian_action_deliveries SET destination_channel = 'phone' WHERE destination_channel = 'voice'`,
-    );
-
-    // scoped_approval_grants: request_channel, decision_channel, execution_channel
-    raw.exec(
-      /*sql*/ `UPDATE scoped_approval_grants SET request_channel = 'phone' WHERE request_channel = 'voice'`,
-    );
-    raw.exec(
-      /*sql*/ `UPDATE scoped_approval_grants SET decision_channel = 'phone' WHERE decision_channel = 'voice'`,
-    );
-    raw.exec(
-      /*sql*/ `UPDATE scoped_approval_grants SET execution_channel = 'phone' WHERE execution_channel = 'voice'`,
-    );
-
-    // sequences.channel
-    raw.exec(
-      /*sql*/ `UPDATE sequences SET channel = 'phone' WHERE channel = 'voice'`,
-    );
-
-    // followups.channel
-    raw.exec(
-      /*sql*/ `UPDATE followups SET channel = 'phone' WHERE channel = 'voice'`,
-    );
-  });
+  const raw = getSqliteFrom(database);
+
+  // contact_channels.type
+  raw.exec(
+    /*sql*/ `UPDATE contact_channels SET type = 'phone' WHERE type = 'voice'`,
+  );
+
+  // conversations.origin_channel
+  raw.exec(
+    /*sql*/ `UPDATE conversations SET origin_channel = 'phone' WHERE origin_channel = 'voice'`,
+  );
+
+  // conversations.origin_interface
+  raw.exec(
+    /*sql*/ `UPDATE conversations SET origin_interface = 'phone' WHERE origin_interface = 'voice'`,
+  );
+
+  // messages.metadata — JSON blobs may contain "voice" as a channel/interface value
+  // (e.g. userMessageChannel, provenanceSourceChannel). Replace the quoted JSON
+  // string so that messageMetadataSchema (which uses z.enum(CHANNEL_IDS)) can
+  // still parse historical rows.
+  raw.exec(
+    /*sql*/ `UPDATE messages SET metadata = REPLACE(metadata, '"voice"', '"phone"') WHERE metadata LIKE '%"voice"%'`,
+  );
+
+  // assistant_ingress_invites.source_channel
+  raw.exec(
+    /*sql*/ `UPDATE assistant_ingress_invites SET source_channel = 'phone' WHERE source_channel = 'voice'`,
+  );
+
+  // assistant_inbox_thread_state.source_channel
+  raw.exec(
+    /*sql*/ `UPDATE assistant_inbox_thread_state SET source_channel = 'phone' WHERE source_channel = 'voice'`,
+  );
+
+  // guardian_action_requests.source_channel
+  raw.exec(
+    /*sql*/ `UPDATE guardian_action_requests SET source_channel = 'phone' WHERE source_channel = 'voice'`,
+  );
+
+  // guardian_action_requests.answered_by_channel
+  raw.exec(
+    /*sql*/ `UPDATE guardian_action_requests SET answered_by_channel = 'phone' WHERE answered_by_channel = 'voice'`,
+  );
+
+  // channel_verification_sessions.channel
+  raw.exec(
+    /*sql*/ `UPDATE channel_verification_sessions SET channel = 'phone' WHERE channel = 'voice'`,
+  );
+
+  // channel_guardian_approval_requests.channel
+  raw.exec(
+    /*sql*/ `UPDATE channel_guardian_approval_requests SET channel = 'phone' WHERE channel = 'voice'`,
+  );
+
+  // channel_guardian_rate_limits.channel
+  // Dedup: remove voice rows that would collide with existing phone rows
+  // on the UNIQUE index (channel, actor_external_user_id, actor_chat_id).
+  raw.exec(/*sql*/ `DELETE FROM channel_guardian_rate_limits WHERE channel = 'voice' AND EXISTS (
+      SELECT 1 FROM channel_guardian_rate_limits AS t2
+      WHERE t2.channel = 'phone'
+        AND t2.actor_external_user_id = channel_guardian_rate_limits.actor_external_user_id
+        AND t2.actor_chat_id = channel_guardian_rate_limits.actor_chat_id
+    )`);
+  raw.exec(
+    /*sql*/ `UPDATE channel_guardian_rate_limits SET channel = 'phone' WHERE channel = 'voice'`,
+  );
+
+  // notification_events.source_channel
+  raw.exec(
+    /*sql*/ `UPDATE notification_events SET source_channel = 'phone' WHERE source_channel = 'voice'`,
+  );
+
+  // notification_deliveries.channel
+  raw.exec(
+    /*sql*/ `UPDATE notification_deliveries SET channel = 'phone' WHERE channel = 'voice'`,
+  );
+
+  // external_conversation_bindings.source_channel
+  raw.exec(
+    /*sql*/ `UPDATE external_conversation_bindings SET source_channel = 'phone' WHERE source_channel = 'voice'`,
+  );
+
+  // channel_inbound_events.source_channel
+  raw.exec(
+    /*sql*/ `UPDATE channel_inbound_events SET source_channel = 'phone' WHERE source_channel = 'voice'`,
+  );
+
+  // conversation_attention_events.source_channel
+  raw.exec(
+    /*sql*/ `UPDATE conversation_attention_events SET source_channel = 'phone' WHERE source_channel = 'voice'`,
+  );
+
+  // conversation_assistant_attention_state.last_seen_source_channel
+  raw.exec(
+    /*sql*/ `UPDATE conversation_assistant_attention_state SET last_seen_source_channel = 'phone' WHERE last_seen_source_channel = 'voice'`,
+  );
+
+  // canonical_guardian_requests.source_channel
+  raw.exec(
+    /*sql*/ `UPDATE canonical_guardian_requests SET source_channel = 'phone' WHERE source_channel = 'voice'`,
+  );
+
+  // canonical_guardian_deliveries.destination_channel
+  raw.exec(
+    /*sql*/ `UPDATE canonical_guardian_deliveries SET destination_channel = 'phone' WHERE destination_channel = 'voice'`,
+  );
+
+  // guardian_action_deliveries.destination_channel
+  raw.exec(
+    /*sql*/ `UPDATE guardian_action_deliveries SET destination_channel = 'phone' WHERE destination_channel = 'voice'`,
+  );
+
+  // scoped_approval_grants: request_channel, decision_channel, execution_channel
+  raw.exec(
+    /*sql*/ `UPDATE scoped_approval_grants SET request_channel = 'phone' WHERE request_channel = 'voice'`,
+  );
+  raw.exec(
+    /*sql*/ `UPDATE scoped_approval_grants SET decision_channel = 'phone' WHERE decision_channel = 'voice'`,
+  );
+  raw.exec(
+    /*sql*/ `UPDATE scoped_approval_grants SET execution_channel = 'phone' WHERE execution_channel = 'voice'`,
+  );
+
+  // sequences.channel
+  raw.exec(
+    /*sql*/ `UPDATE sequences SET channel = 'phone' WHERE channel = 'voice'`,
+  );
+
+  // followups.channel
+  raw.exec(
+    /*sql*/ `UPDATE followups SET channel = 'phone' WHERE channel = 'voice'`,
+  );
 }

package/src/memory/migrations/145-drop-accounts-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";
 
 /**
  * Drop the unused legacy accounts table and its indexes.
@@ -9,13 +8,11 @@ import { withCrashRecovery } from "./validate-migration-state.js";
  * account-store path, so retaining the table only leaves dead state around.
  */
 export function migrateDropAccountsTable(database: DrizzleDb): void {
-  withCrashRecovery(database, "migration_drop_accounts_table_v1", () => {
-    const raw = getSqliteFrom(database);
+  const raw = getSqliteFrom(database);
 
-    raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_accounts_service`);
-    raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_accounts_status`);
-    raw.exec(/*sql*/ `DROP TABLE IF EXISTS accounts`);
-  });
+  raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_accounts_service`);
+  raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_accounts_status`);
+  raw.exec(/*sql*/ `DROP TABLE IF EXISTS accounts`);
 }
 
 /**

package/src/memory/migrations/147-migrate-reminders-to-schedules.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";
 
 /**
  * Migrate all existing reminders into the cron_jobs (schedules) table as
@@ -31,98 +30,96 @@ import { withCrashRecovery } from "./validate-migration-state.js";
  * preserved for the subsequent cleanup PR.
  */
 export function migrateRemindersToSchedules(database: DrizzleDb): void {
-  withCrashRecovery(database, "migration_reminders_to_schedules_v1", () => {
-    const raw = getSqliteFrom(database);
+  const raw = getSqliteFrom(database);
 
-    // Guard: if the reminders table doesn't exist, nothing to migrate.
-    const hasReminders = raw
-      .query(
-        "SELECT name FROM sqlite_master WHERE type='table' AND name='reminders'",
-      )
-      .get();
-    if (!hasReminders) return;
+  // Guard: if the reminders table doesn't exist, nothing to migrate.
+  const hasReminders = raw
+    .query(
+      "SELECT name FROM sqlite_master WHERE type='table' AND name='reminders'",
+    )
+    .get();
+  if (!hasReminders) return;
 
-    // Read all reminders into memory. We use the reminder's original ID as
-    // the cron_jobs primary key so that INSERT OR IGNORE deduplicates
-    // correctly if the migration re-runs after a crash.
-    const reminders = raw.query("SELECT * FROM reminders").all() as Array<{
-      id: string;
-      label: string;
-      message: string;
-      fire_at: number;
-      mode: string;
-      status: string;
-      fired_at: number | null;
-      routing_intent: string;
-      routing_hints_json: string;
-      created_at: number;
-      updated_at: number;
-    }>;
+  // Read all reminders into memory. We use the reminder's original ID as
+  // the cron_jobs primary key so that INSERT OR IGNORE deduplicates
+  // correctly if the migration re-runs after a crash.
+  const reminders = raw.query("SELECT * FROM reminders").all() as Array<{
+    id: string;
+    label: string;
+    message: string;
+    fire_at: number;
+    mode: string;
+    status: string;
+    fired_at: number | null;
+    routing_intent: string;
+    routing_hints_json: string;
+    created_at: number;
+    updated_at: number;
+  }>;
 
-    if (reminders.length === 0) return;
+  if (reminders.length === 0) return;
 
-    const insert = raw.query(/*sql*/ `
-        INSERT OR IGNORE INTO cron_jobs (
-          id, name, enabled, cron_expression, schedule_syntax, timezone,
-          message, next_run_at, last_run_at, last_status, retry_count,
-          created_by, mode, routing_intent, routing_hints_json, status,
-          created_at, updated_at
-        ) VALUES (
-          ?, ?, ?, NULL, 'cron', NULL,
-          ?, ?, ?, ?, 0,
-          'agent', ?, ?, ?, ?,
-          ?, ?
-        )
-      `);
+  const insert = raw.query(/*sql*/ `
+      INSERT OR IGNORE INTO cron_jobs (
+        id, name, enabled, cron_expression, schedule_syntax, timezone,
+        message, next_run_at, last_run_at, last_status, retry_count,
+        created_by, mode, routing_intent, routing_hints_json, status,
+        created_at, updated_at
+      ) VALUES (
+        ?, ?, ?, NULL, 'cron', NULL,
+        ?, ?, ?, ?, 0,
+        'agent', ?, ?, ?, ?,
+        ?, ?
+      )
+    `);
 
-    // Mark migrated pending reminders as fired so the legacy
-    // claimDueReminders path doesn't fire them a second time.
-    const markFired = raw.query(/*sql*/ `
-        UPDATE reminders SET status = 'fired', fired_at = ?
-        WHERE id = ? AND status = 'pending'
-      `);
+  // Mark migrated pending reminders as fired so the legacy
+  // claimDueReminders path doesn't fire them a second time.
+  const markFired = raw.query(/*sql*/ `
+      UPDATE reminders SET status = 'fired', fired_at = ?
+      WHERE id = ? AND status = 'pending'
+    `);
 
-    try {
-      raw.exec("BEGIN");
+  try {
+    raw.exec("BEGIN");
 
-      for (const r of reminders) {
-        const statusMap: Record<string, string> = {
-          pending: "active",
-          firing: "firing",
-          fired: "fired",
-          cancelled: "cancelled",
-        };
+    for (const r of reminders) {
+      const statusMap: Record<string, string> = {
+        pending: "active",
+        firing: "firing",
+        fired: "fired",
+        cancelled: "cancelled",
+      };
 
-        insert.run(
-          r.id,
-          r.label,
-          r.status === "pending" ? 1 : 0,
-          // message, next_run_at, last_run_at, last_status
-          r.message,
-          r.fire_at,
-          r.fired_at,
-          r.status === "fired" ? "ok" : null,
-          // mode, routing_intent, routing_hints_json, status
-          r.mode,
-          r.routing_intent,
-          r.routing_hints_json,
-          statusMap[r.status] ?? "active",
-          // created_at, updated_at
-          r.created_at,
-          r.updated_at,
-        );
+      insert.run(
+        r.id,
+        r.label,
+        r.status === "pending" ? 1 : 0,
+        // message, next_run_at, last_run_at, last_status
+        r.message,
+        r.fire_at,
+        r.fired_at,
+        r.status === "fired" ? "ok" : null,
+        // mode, routing_intent, routing_hints_json, status
+        r.mode,
+        r.routing_intent,
+        r.routing_hints_json,
+        statusMap[r.status] ?? "active",
+        // created_at, updated_at
+        r.created_at,
+        r.updated_at,
+      );
 
-        if (r.status === "pending") {
-          markFired.run(Date.now(), r.id);
-        }
+      if (r.status === "pending") {
+        markFired.run(Date.now(), r.id);
       }
-
-      raw.exec("COMMIT");
-    } catch (err) {
-      raw.exec("ROLLBACK");
-      throw err;
     }
-  });
+
+    raw.exec("COMMIT");
+  } catch (err) {
+    raw.exec("ROLLBACK");
+    throw err;
+  }
 }
 
 /**

package/src/memory/migrations/148-drop-reminders-table.ts

@@ -1,17 +1,14 @@
 import type { DrizzleDb } from "../db-connection.js";
 import { getSqliteFrom } from "../db-connection.js";
-import { withCrashRecovery } from "./validate-migration-state.js";
 
 /**
  * Drop the legacy reminders table and its index now that all data has been
  * migrated into cron_jobs as one-shot schedules (migration 147).
  */
 export function migrateDropRemindersTable(database: DrizzleDb): void {
-  withCrashRecovery(database, "migration_drop_reminders_table_v1", () => {
-    const raw = getSqliteFrom(database);
-    raw.run("DROP INDEX IF EXISTS idx_reminders_status_fire_at");
-    raw.run("DROP TABLE IF EXISTS reminders");
-  });
+  const raw = getSqliteFrom(database);
+  raw.run("DROP INDEX IF EXISTS idx_reminders_status_fire_at");
+  raw.run("DROP TABLE IF EXISTS reminders");
 }
 
 /**