@vellumai/assistant 0.4.33 → 0.4.34

package/src/memory/migrations/134-contacts-notes-column.ts

@@ -1,68 +1,87 @@
 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-134");
 
 export function migrateContactsNotesColumn(database: DrizzleDb): void {
-  const raw = getSqliteFrom(database);
+  withCrashRecovery(database, "migration_contacts_notes_column_v1", () => {
+    const raw = getSqliteFrom(database);
 
-  try {
-    raw.exec(/*sql*/ `ALTER TABLE contacts ADD COLUMN notes TEXT`);
-  } catch {
-    /* already exists */
-  }
+    try {
+      raw.exec(/*sql*/ `ALTER TABLE contacts ADD COLUMN notes TEXT`);
+    } catch {
+      /* already exists */
+    }
+
+    // Check which legacy columns still exist — handles partial completion
+    // if a previous run crashed after dropping some columns but not all.
+    const cols = new Set(
+      (
+        raw.query(`PRAGMA table_info(contacts)`).all() as Array<{
+          name: string;
+        }>
+      ).map((c) => c.name),
+    );
 
-  // Check if legacy columns still exist before attempting backfill + drop
-  const cols = new Set(
-    (
-      raw.query(`PRAGMA table_info(contacts)`).all() as Array<{ name: string }>
-    ).map((c) => c.name),
-  );
+    const legacyCols = [
+      "relationship",
+      "importance",
+      "response_expectation",
+      "preferred_tone",
+    ] as const;
+    const remaining = legacyCols.filter((c) => cols.has(c));
 
-  if (cols.has("relationship")) {
-    const rows = raw
-      .query(
-        `SELECT id, relationship, importance, response_expectation, preferred_tone
+    // Backfill notes from legacy columns if any are still present and notes
+    // haven't been populated yet (only run once, before any columns are dropped).
+    if (remaining.length === legacyCols.length) {
+      const rows = raw
+        .query(
+          `SELECT id, relationship, importance, response_expectation, preferred_tone
        FROM contacts
        WHERE relationship IS NOT NULL
           OR importance != 0.5
           OR response_expectation IS NOT NULL
           OR preferred_tone IS NOT NULL`,
-      )
-      .all() as Array<{
-      id: string;
-      relationship: string | null;
-      importance: number;
-      response_expectation: string | null;
-      preferred_tone: string | null;
-    }>;
+        )
+        .all() as Array<{
+        id: string;
+        relationship: string | null;
+        importance: number;
+        response_expectation: string | null;
+        preferred_tone: string | null;
+      }>;
 
-    const update = raw.prepare(`UPDATE contacts SET notes = ? WHERE id = ?`);
+      const update = raw.prepare(`UPDATE contacts SET notes = ? WHERE id = ?`);
 
-    for (const row of rows) {
-      const parts: string[] = [];
-      if (row.relationship) parts.push(`Relationship: ${row.relationship}`);
-      if (row.importance !== 0.5) parts.push(`Importance: ${row.importance}`);
-      if (row.response_expectation)
-        parts.push(`Response expectation: ${row.response_expectation}`);
-      if (row.preferred_tone)
-        parts.push(`Preferred tone: ${row.preferred_tone}`);
-      if (parts.length > 0) {
-        update.run(parts.join("\n"), row.id);
+      for (const row of rows) {
+        const parts: string[] = [];
+        if (row.relationship) parts.push(`Relationship: ${row.relationship}`);
+        if (row.importance !== 0.5) parts.push(`Importance: ${row.importance}`);
+        if (row.response_expectation)
+          parts.push(`Response expectation: ${row.response_expectation}`);
+        if (row.preferred_tone)
+          parts.push(`Preferred tone: ${row.preferred_tone}`);
+        if (parts.length > 0) {
+          update.run(parts.join("\n"), row.id);
+        }
       }
-    }
 
-    const migrated = rows.length;
-    if (migrated > 0) {
-      log.info({ migrated }, "Migrated contact metadata to notes field");
+      const migrated = rows.length;
+      if (migrated > 0) {
+        log.info({ migrated }, "Migrated contact metadata to notes field");
+      }
     }
 
-    // Drop indexes that reference columns we're about to remove
+    // Drop indexes that reference columns we're about to remove.
+    // Must happen before the column drops — SQLite rejects dropping a column
+    // that is still referenced by an index.
     raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_contacts_importance`);
 
-    raw.exec(/*sql*/ `ALTER TABLE contacts DROP COLUMN relationship`);
-    raw.exec(/*sql*/ `ALTER TABLE contacts DROP COLUMN importance`);
-    raw.exec(/*sql*/ `ALTER TABLE contacts DROP COLUMN response_expectation`);
-    raw.exec(/*sql*/ `ALTER TABLE contacts DROP COLUMN preferred_tone`);
-  }
+    // Drop each legacy column individually so partial completion is safe:
+    // on crash recovery the loop picks up only the columns that remain.
+    for (const col of remaining) {
+      raw.exec(/*sql*/ `ALTER TABLE contacts DROP COLUMN ${col}`);
+    }
+  });
 }

package/src/memory/migrations/136-drop-assistant-id-columns.ts

@@ -0,0 +1,263 @@
+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-136");
+
+/**
+ * Drop `assistant_id` columns from all 16 daemon tables that carried the
+ * per-assistant scoping column. After wave-1 PRs normalised every value to
+ * 'self' (the implicit single-tenant identity), the column is dead weight.
+ *
+ * Steps:
+ *  1. Safety assertion: verify all rows are 'self' or NULL.
+ *  2. Drop composite indexes that include `assistant_id`.
+ *  3. `ALTER TABLE ... DROP COLUMN assistant_id` for each table.
+ *  4. Recreate indexes without the `assistant_id` column.
+ */
+export function migrateDropAssistantIdColumns(database: DrizzleDb): void {
+  withCrashRecovery(database, "migration_drop_assistant_id_columns_v1", () => {
+    const raw = getSqliteFrom(database);
+
+    // The 16 tables that carry assistant_id.
+    const tables = [
+      "contacts",
+      "assistant_ingress_invites",
+      "assistant_inbox_thread_state",
+      "call_sessions",
+      "channel_guardian_verification_challenges",
+      "channel_guardian_approval_requests",
+      "channel_guardian_rate_limits",
+      "guardian_action_requests",
+      "scoped_approval_grants",
+      "notification_events",
+      "notification_preferences",
+      "notification_deliveries",
+      "conversation_attention_events",
+      "conversation_assistant_attention_state",
+      "actor_token_records",
+      "actor_refresh_token_records",
+    ];
+
+    // --- Safety assertion ---
+    // Verify all existing assistant_id values are 'self' or NULL before dropping.
+    for (const table of tables) {
+      const cols = new Set(
+        (
+          raw.query(`PRAGMA table_info(${table})`).all() as Array<{
+            name: string;
+          }>
+        ).map((c) => c.name),
+      );
+
+      if (!cols.has("assistant_id")) {
+        log.info(
+          { table },
+          "Table does not have assistant_id column — skipping",
+        );
+        continue;
+      }
+
+      const unexpected = raw
+        .query(
+          `SELECT DISTINCT assistant_id FROM ${table} WHERE assistant_id IS NOT NULL AND assistant_id != 'self'`,
+        )
+        .all() as Array<{ assistant_id: string }>;
+
+      if (unexpected.length > 0) {
+        log.warn(
+          { table, values: unexpected.map((r) => r.assistant_id) },
+          "Unexpected assistant_id values found — skipping table",
+        );
+        continue;
+      }
+    }
+
+    // --- Drop ALL indexes that include assistant_id ---
+    // Every index below references the assistant_id column. SQLite will error
+    // on ALTER TABLE ... DROP COLUMN if any index still references the column.
+
+    // channel_guardian_verification_challenges indexes (migrations 110, 026, 027a)
+    raw.exec(
+      /*sql*/ `DROP INDEX IF EXISTS idx_channel_guardian_challenges_lookup`,
+    );
+    raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_guardian_sessions_active`);
+    raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_guardian_sessions_identity`);
+    raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_guardian_sessions_destination`);
+    raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_guardian_sessions_bootstrap`);
+
+    // channel_guardian_rate_limits indexes (migration 110)
+    raw.exec(
+      /*sql*/ `DROP INDEX IF EXISTS idx_channel_guardian_rate_limits_actor`,
+    );
+
+    // assistant_ingress_invites indexes (migration 112)
+    raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_ingress_invites_channel_status`);
+    raw.exec(
+      /*sql*/ `DROP INDEX IF EXISTS idx_ingress_invites_channel_created`,
+    );
+
+    // assistant_inbox_thread_state indexes (migration 112)
+    raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_inbox_thread_state_channel`);
+    raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_inbox_thread_state_last_msg`);
+    raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_inbox_thread_state_escalation`);
+
+    // notification_preferences indexes (migration 114)
+    raw.exec(
+      /*sql*/ `DROP INDEX IF EXISTS idx_notification_preferences_assistant_id`,
+    );
+    raw.exec(
+      /*sql*/ `DROP INDEX IF EXISTS idx_notification_preferences_assistant_priority`,
+    );
+
+    // notification_events indexes (migration 114)
+    raw.exec(
+      /*sql*/ `DROP INDEX IF EXISTS idx_notification_events_assistant_event_created`,
+    );
+    raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_notification_events_dedupe`);
+
+    // notification_deliveries indexes (migration 114)
+    raw.exec(
+      /*sql*/ `DROP INDEX IF EXISTS idx_notification_deliveries_assistant_status`,
+    );
+
+    // conversation_attention_events indexes (migration 117)
+    raw.exec(
+      /*sql*/ `DROP INDEX IF EXISTS idx_conv_attn_events_assistant_observed`,
+    );
+
+    // conversation_assistant_attention_state indexes (migration 117)
+    raw.exec(
+      /*sql*/ `DROP INDEX IF EXISTS idx_conv_attn_state_assistant_latest_msg`,
+    );
+    raw.exec(
+      /*sql*/ `DROP INDEX IF EXISTS idx_conv_attn_state_assistant_last_seen`,
+    );
+
+    // actor_token_records indexes (migration 038)
+    raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_actor_tokens_active_device`);
+
+    // actor_refresh_token_records indexes (migration 039)
+    raw.exec(/*sql*/ `DROP INDEX IF EXISTS idx_refresh_tokens_active_device`);
+
+    // --- Drop assistant_id column from each table ---
+    for (const table of tables) {
+      const cols = new Set(
+        (
+          raw.query(`PRAGMA table_info(${table})`).all() as Array<{
+            name: string;
+          }>
+        ).map((c) => c.name),
+      );
+
+      if (!cols.has("assistant_id")) continue;
+
+      // Re-verify safety before each drop
+      const unexpected = raw
+        .query(
+          `SELECT DISTINCT assistant_id FROM ${table} WHERE assistant_id IS NOT NULL AND assistant_id != 'self'`,
+        )
+        .all() as Array<{ assistant_id: string }>;
+
+      if (unexpected.length > 0) {
+        log.warn(
+          { table, values: unexpected.map((r) => r.assistant_id) },
+          "Unexpected assistant_id values — skipping column drop",
+        );
+        continue;
+      }
+
+      raw.exec(/*sql*/ `ALTER TABLE ${table} DROP COLUMN assistant_id`);
+      log.info({ table }, "Dropped assistant_id column");
+    }
+
+    // --- Recreate indexes without assistant_id ---
+    // Each index below is the equivalent of the dropped index but with the
+    // assistant_id column removed. Index names are updated to avoid
+    // collisions with the old names.
+
+    // channel_guardian_verification_challenges
+    raw.exec(
+      /*sql*/ `CREATE INDEX IF NOT EXISTS idx_channel_guardian_challenges_lookup ON channel_guardian_verification_challenges(channel, challenge_hash, status)`,
+    );
+    raw.exec(
+      /*sql*/ `CREATE INDEX IF NOT EXISTS idx_guardian_sessions_active ON channel_guardian_verification_challenges(channel, status)`,
+    );
+    raw.exec(
+      /*sql*/ `CREATE INDEX IF NOT EXISTS idx_guardian_sessions_identity ON channel_guardian_verification_challenges(channel, expected_external_user_id, expected_chat_id, status)`,
+    );
+    raw.exec(
+      /*sql*/ `CREATE INDEX IF NOT EXISTS idx_guardian_sessions_destination ON channel_guardian_verification_challenges(channel, destination_address)`,
+    );
+    raw.exec(
+      /*sql*/ `CREATE INDEX IF NOT EXISTS idx_guardian_sessions_bootstrap ON channel_guardian_verification_challenges(channel, bootstrap_token_hash, status)`,
+    );
+
+    // channel_guardian_rate_limits
+    raw.exec(
+      /*sql*/ `CREATE UNIQUE INDEX IF NOT EXISTS idx_channel_guardian_rate_limits_actor ON channel_guardian_rate_limits(channel, actor_external_user_id, actor_chat_id)`,
+    );
+
+    // assistant_ingress_invites
+    raw.exec(
+      /*sql*/ `CREATE INDEX IF NOT EXISTS idx_ingress_invites_channel_status ON assistant_ingress_invites(source_channel, status, expires_at)`,
+    );
+    raw.exec(
+      /*sql*/ `CREATE INDEX IF NOT EXISTS idx_ingress_invites_channel_created ON assistant_ingress_invites(source_channel, created_at)`,
+    );
+
+    // assistant_inbox_thread_state
+    raw.exec(
+      /*sql*/ `CREATE UNIQUE INDEX IF NOT EXISTS idx_inbox_thread_state_channel ON assistant_inbox_thread_state(source_channel, external_chat_id)`,
+    );
+    raw.exec(
+      /*sql*/ `CREATE INDEX IF NOT EXISTS idx_inbox_thread_state_last_msg ON assistant_inbox_thread_state(last_message_at)`,
+    );
+    raw.exec(
+      /*sql*/ `CREATE INDEX IF NOT EXISTS idx_inbox_thread_state_escalation ON assistant_inbox_thread_state(has_pending_escalation, last_message_at)`,
+    );
+
+    // notification_preferences
+    raw.exec(
+      /*sql*/ `CREATE INDEX IF NOT EXISTS idx_notification_preferences_priority ON notification_preferences(priority DESC)`,
+    );
+
+    // notification_events
+    raw.exec(
+      /*sql*/ `CREATE INDEX IF NOT EXISTS idx_notification_events_event_created ON notification_events(source_event_name, created_at)`,
+    );
+    raw.exec(
+      /*sql*/ `CREATE UNIQUE INDEX IF NOT EXISTS idx_notification_events_dedupe ON notification_events(dedupe_key) WHERE dedupe_key IS NOT NULL`,
+    );
+
+    // notification_deliveries
+    raw.exec(
+      /*sql*/ `CREATE INDEX IF NOT EXISTS idx_notification_deliveries_status ON notification_deliveries(status)`,
+    );
+
+    // conversation_attention_events
+    raw.exec(
+      /*sql*/ `CREATE INDEX IF NOT EXISTS idx_conv_attn_events_observed ON conversation_attention_events(observed_at)`,
+    );
+
+    // conversation_assistant_attention_state
+    raw.exec(
+      /*sql*/ `CREATE INDEX IF NOT EXISTS idx_conv_attn_state_latest_msg ON conversation_assistant_attention_state(latest_assistant_message_at)`,
+    );
+    raw.exec(
+      /*sql*/ `CREATE INDEX IF NOT EXISTS idx_conv_attn_state_last_seen ON conversation_assistant_attention_state(last_seen_assistant_message_at)`,
+    );
+
+    // actor_token_records
+    raw.exec(
+      /*sql*/ `CREATE UNIQUE INDEX IF NOT EXISTS idx_actor_tokens_active_device ON actor_token_records(guardian_principal_id, hashed_device_id) WHERE status = 'active'`,
+    );
+
+    // actor_refresh_token_records
+    raw.exec(
+      /*sql*/ `CREATE UNIQUE INDEX IF NOT EXISTS idx_refresh_tokens_active_device ON actor_refresh_token_records(guardian_principal_id, hashed_device_id) WHERE status = 'active'`,
+    );
+
+    log.info("Completed dropping assistant_id columns from all tables");
+  });
+}

package/src/memory/migrations/index.ts

@@ -78,6 +78,7 @@ export { migrateContactsAssistantId } from "./132-contacts-assistant-id.js";
 export { migrateAssistantContactMetadata } from "./133-assistant-contact-metadata.js";
 export { migrateContactsNotesColumn } from "./134-contacts-notes-column.js";
 export { migrateBackfillContactInteractionStats } from "./135-backfill-contact-interaction-stats.js";
+export { migrateDropAssistantIdColumns } from "./136-drop-assistant-id-columns.js";
 export {
   MIGRATION_REGISTRY,
   type MigrationRegistryEntry,

package/src/memory/migrations/registry.ts

@@ -123,11 +123,24 @@ export const MIGRATION_REGISTRY: MigrationRegistryEntry[] = [
       "Enforce NOT NULL on channel_guardian_bindings.guardian_principal_id after backfill",
   },
   {
-    key: "backfill_contact_interaction_stats",
+    key: "migration_contacts_notes_column_v1",
     version: 17,
+    description:
+      "Consolidate relationship/importance/response_expectation/preferred_tone into a single notes TEXT column, then drop the legacy columns",
+  },
+  {
+    key: "backfill_contact_interaction_stats",
+    version: 18,
     description:
       "Backfill contacts.last_interaction from the max lastSeenAt across each contact's channels",
   },
+  {
+    key: "migration_drop_assistant_id_columns_v1",
+    version: 19,
+    dependsOn: ["migration_normalize_assistant_id_to_self_v1"],
+    description:
+      "Drop assistant_id columns from all 16 daemon tables after normalization to single-tenant identity",
+  },
 ];
 
 export interface MigrationValidationResult {

package/src/memory/schema/calls.ts

@@ -6,7 +6,6 @@ import {
   text,
 } from "drizzle-orm/sqlite-core";
 
-import { DAEMON_INTERNAL_ASSISTANT_ID } from "../../runtime/assistant-scope.js";
 import { conversations } from "./conversations.js";
 
 export const callSessions = sqliteTable(
@@ -26,7 +25,6 @@ export const callSessions = sqliteTable(
     guardianVerificationSessionId: text("guardian_verification_session_id"),
     callerIdentityMode: text("caller_identity_mode"),
     callerIdentitySource: text("caller_identity_source"),
-    assistantId: text("assistant_id"),
     initiatedFromConversationId: text("initiated_from_conversation_id"),
     startedAt: integer("started_at"),
     endedAt: integer("ended_at"),
@@ -91,7 +89,6 @@ export const channelGuardianVerificationChallenges = sqliteTable(
   "channel_guardian_verification_challenges",
   {
     id: text("id").primaryKey(),
-    assistantId: text("assistant_id").notNull(),
     channel: text("channel").notNull(),
     challengeHash: text("challenge_hash").notNull(),
     expiresAt: integer("expires_at").notNull(),
@@ -128,9 +125,6 @@ export const channelGuardianApprovalRequests = sqliteTable(
     runId: text("run_id").notNull(),
     requestId: text("request_id"),
     conversationId: text("conversation_id").notNull(),
-    assistantId: text("assistant_id")
-      .notNull()
-      .default(DAEMON_INTERNAL_ASSISTANT_ID),
     channel: text("channel").notNull(),
     requesterExternalUserId: text("requester_external_user_id").notNull(),
     requesterChatId: text("requester_chat_id").notNull(),
@@ -151,7 +145,6 @@ export const channelGuardianRateLimits = sqliteTable(
   "channel_guardian_rate_limits",
   {
     id: text("id").primaryKey(),
-    assistantId: text("assistant_id").notNull(),
     channel: text("channel").notNull(),
     actorExternalUserId: text("actor_external_user_id").notNull(),
     actorChatId: text("actor_chat_id").notNull(),

package/src/memory/schema/contacts.ts

@@ -1,6 +1,5 @@
 import { index, integer, sqliteTable, text } from "drizzle-orm/sqlite-core";
 
-import { DAEMON_INTERNAL_ASSISTANT_ID } from "../../runtime/assistant-scope.js";
 import { conversations } from "./conversations.js";
 
 export const contacts = sqliteTable("contacts", {
@@ -13,7 +12,6 @@ export const contacts = sqliteTable("contacts", {
   updatedAt: integer("updated_at").notNull(),
   role: text("role").notNull().default("contact"), // 'guardian' | 'contact'
   principalId: text("principal_id"), // internal auth principal (nullable)
-  assistantId: text("assistant_id"), // which assistant this guardian is for (nullable, daemon default is DAEMON_INTERNAL_ASSISTANT_ID)
   contactType: text("contact_type").notNull().default("human"), // 'human' | 'assistant'
 });
 
@@ -69,9 +67,6 @@ export const assistantIngressInvites = sqliteTable(
   "assistant_ingress_invites",
   {
     id: text("id").primaryKey(),
-    assistantId: text("assistant_id")
-      .notNull()
-      .default(DAEMON_INTERNAL_ASSISTANT_ID),
     sourceChannel: text("source_channel").notNull(),
     tokenHash: text("token_hash").notNull(),
     createdBySessionId: text("created_by_session_id"),
@@ -103,9 +98,6 @@ export const assistantInboxThreadState = sqliteTable(
     conversationId: text("conversation_id")
       .primaryKey()
       .references(() => conversations.id, { onDelete: "cascade" }),
-    assistantId: text("assistant_id")
-      .notNull()
-      .default(DAEMON_INTERNAL_ASSISTANT_ID),
     sourceChannel: text("source_channel").notNull(),
     externalChatId: text("external_chat_id").notNull(),
     externalUserId: text("external_user_id"),

package/src/memory/schema/guardian.ts

@@ -1,15 +1,11 @@
 import { index, integer, sqliteTable, text } from "drizzle-orm/sqlite-core";
 
-import { DAEMON_INTERNAL_ASSISTANT_ID } from "../../runtime/assistant-scope.js";
 import { callPendingQuestions, callSessions } from "./calls.js";
 
 export const guardianActionRequests = sqliteTable(
   "guardian_action_requests",
   {
     id: text("id").primaryKey(),
-    assistantId: text("assistant_id")
-      .notNull()
-      .default(DAEMON_INTERNAL_ASSISTANT_ID),
     kind: text("kind").notNull(), // 'ask_guardian'
     sourceChannel: text("source_channel").notNull(), // 'voice'
     sourceConversationId: text("source_conversation_id").notNull(),
@@ -145,7 +141,6 @@ export const scopedApprovalGrants = sqliteTable(
   "scoped_approval_grants",
   {
     id: text("id").primaryKey(),
-    assistantId: text("assistant_id").notNull(),
     scopeMode: text("scope_mode").notNull(), // 'request_id' | 'tool_signature'
     requestId: text("request_id"),
     toolName: text("tool_name"),

package/src/memory/schema/infrastructure.ts

@@ -173,7 +173,6 @@ export const llmUsageEvents = sqliteTable(
 export const actorTokenRecords = sqliteTable("actor_token_records", {
   id: text("id").primaryKey(),
   tokenHash: text("token_hash").notNull(),
-  assistantId: text("assistant_id").notNull(),
   guardianPrincipalId: text("guardian_principal_id").notNull(),
   hashedDeviceId: text("hashed_device_id").notNull(),
   platform: text("platform").notNull(),
@@ -190,7 +189,6 @@ export const actorRefreshTokenRecords = sqliteTable(
     id: text("id").primaryKey(),
     tokenHash: text("token_hash").notNull(),
     familyId: text("family_id").notNull(),
-    assistantId: text("assistant_id").notNull(),
     guardianPrincipalId: text("guardian_principal_id").notNull(),
     hashedDeviceId: text("hashed_device_id").notNull(),
     platform: text("platform").notNull(),

package/src/memory/schema/notifications.ts

@@ -11,7 +11,6 @@ import { conversations } from "./conversations.js";
 
 export const notificationEvents = sqliteTable("notification_events", {
   id: text("id").primaryKey(),
-  assistantId: text("assistant_id").notNull(),
   sourceEventName: text("source_event_name").notNull(),
   sourceChannel: text("source_channel").notNull(),
   sourceSessionId: text("source_session_id").notNull(),
@@ -39,7 +38,6 @@ export const notificationDecisions = sqliteTable("notification_decisions", {
 
 export const notificationPreferences = sqliteTable("notification_preferences", {
   id: text("id").primaryKey(),
-  assistantId: text("assistant_id").notNull(),
   preferenceText: text("preference_text").notNull(),
   appliesWhenJson: text("applies_when_json").notNull().default("{}"),
   priority: integer("priority").notNull().default(0),
@@ -95,7 +93,6 @@ export const notificationDeliveries = sqliteTable(
     notificationDecisionId: text("notification_decision_id")
       .notNull()
       .references(() => notificationDecisions.id, { onDelete: "cascade" }),
-    assistantId: text("assistant_id").notNull(),
     channel: text("channel").notNull(),
     destination: text("destination").notNull(),
     status: text("status").notNull().default("pending"),
@@ -132,7 +129,6 @@ export const conversationAttentionEvents = sqliteTable(
     conversationId: text("conversation_id")
       .notNull()
       .references(() => conversations.id, { onDelete: "cascade" }),
-    assistantId: text("assistant_id").notNull(),
     sourceChannel: text("source_channel").notNull(),
     signalType: text("signal_type").notNull(),
     confidence: text("confidence").notNull(),
@@ -147,10 +143,7 @@ export const conversationAttentionEvents = sqliteTable(
       table.conversationId,
       table.observedAt,
     ),
-    index("idx_conv_attn_events_assistant_observed").on(
-      table.assistantId,
-      table.observedAt,
-    ),
+    index("idx_conv_attn_events_observed").on(table.observedAt),
     index("idx_conv_attn_events_channel_observed").on(
       table.sourceChannel,
       table.observedAt,
@@ -164,7 +157,6 @@ export const conversationAssistantAttentionState = sqliteTable(
     conversationId: text("conversation_id")
       .primaryKey()
       .references(() => conversations.id, { onDelete: "cascade" }),
-    assistantId: text("assistant_id").notNull(),
     latestAssistantMessageId: text("latest_assistant_message_id"),
     latestAssistantMessageAt: integer("latest_assistant_message_at"),
     lastSeenAssistantMessageId: text("last_seen_assistant_message_id"),
@@ -179,13 +171,7 @@ export const conversationAssistantAttentionState = sqliteTable(
     updatedAt: integer("updated_at").notNull(),
   },
   (table) => [
-    index("idx_conv_attn_state_assistant_latest_msg").on(
-      table.assistantId,
-      table.latestAssistantMessageAt,
-    ),
-    index("idx_conv_attn_state_assistant_last_seen").on(
-      table.assistantId,
-      table.lastSeenAssistantMessageAt,
-    ),
+    index("idx_conv_attn_state_latest_msg").on(table.latestAssistantMessageAt),
+    index("idx_conv_attn_state_last_seen").on(table.lastSeenAssistantMessageAt),
   ],
 );