@vellumai/assistant 0.4.32 → 0.4.34

package/src/memory/channel-delivery-store.ts

@@ -22,6 +22,7 @@ export {
 export type { InboundResult, RecordInboundOptions } from "./delivery-crud.js";
 export {
   clearPayload,
+  deleteInbound,
   findMessageBySourceId,
   getLatestStoredPayload,
   linkMessage,

package/src/memory/conversation-attention-store.ts

@@ -32,7 +32,6 @@ export type Confidence = "explicit" | "inferred";
 export interface AttentionEvent {
   id: string;
   conversationId: string;
-  assistantId: string;
   sourceChannel: string;
   signalType: SignalType;
   confidence: Confidence;
@@ -45,7 +44,6 @@ export interface AttentionEvent {
 
 export interface AttentionState {
   conversationId: string;
-  assistantId: string;
   latestAssistantMessageId: string | null;
   latestAssistantMessageAt: number | null;
   lastSeenAssistantMessageId: string | null;
@@ -68,7 +66,6 @@ function rowToEvent(
   return {
     id: row.id,
     conversationId: row.conversationId,
-    assistantId: row.assistantId,
     sourceChannel: row.sourceChannel,
     signalType: row.signalType as SignalType,
     confidence: row.confidence as Confidence,
@@ -85,7 +82,6 @@ function rowToState(
 ): AttentionState {
   return {
     conversationId: row.conversationId,
-    assistantId: row.assistantId,
     latestAssistantMessageId: row.latestAssistantMessageId,
     latestAssistantMessageAt: row.latestAssistantMessageAt,
     lastSeenAssistantMessageId: row.lastSeenAssistantMessageId,
@@ -109,11 +105,10 @@ function rowToState(
  */
 export function projectAssistantMessage(params: {
   conversationId: string;
-  assistantId: string;
   messageId: string;
   messageAt: number;
 }): void {
-  const { conversationId, assistantId, messageId, messageAt } = params;
+  const { conversationId, messageId, messageAt } = params;
   const db = getDb();
   const now = Date.now();
 
@@ -129,7 +124,6 @@ export function projectAssistantMessage(params: {
     db.insert(conversationAssistantAttentionState)
       .values({
         conversationId,
-        assistantId,
         latestAssistantMessageId: messageId,
         latestAssistantMessageAt: messageAt,
         lastSeenAssistantMessageId: null,
@@ -175,7 +169,6 @@ export function projectAssistantMessage(params: {
  */
 export function recordConversationSeenSignal(params: {
   conversationId: string;
-  assistantId: string;
   sourceChannel: string;
   signalType: SignalType;
   confidence: Confidence;
@@ -186,7 +179,6 @@ export function recordConversationSeenSignal(params: {
 }): AttentionEvent {
   const {
     conversationId,
-    assistantId,
     sourceChannel,
     signalType,
     confidence,
@@ -205,7 +197,6 @@ export function recordConversationSeenSignal(params: {
   const event: typeof conversationAttentionEvents.$inferInsert = {
     id: eventId,
     conversationId,
-    assistantId,
     sourceChannel,
     signalType,
     confidence,
@@ -252,7 +243,6 @@ export function recordConversationSeenSignal(params: {
       tx.insert(conversationAssistantAttentionState)
         .values({
           conversationId,
-          assistantId,
           latestAssistantMessageId: latestMsgId,
           latestAssistantMessageAt: latestMsgAt,
           lastSeenAssistantMessageId: latestMsgId,
@@ -348,7 +338,6 @@ export function getAttentionStateByConversationIds(
 export type AttentionFilterState = "seen" | "unseen" | "all";
 
 export interface ListConversationAttentionParams {
-  assistantId: string;
   state?: AttentionFilterState;
   sourceChannel?: string;
   source?: string;
@@ -364,7 +353,6 @@ export function listConversationAttention(
   params: ListConversationAttentionParams,
 ): AttentionState[] {
   const {
-    assistantId,
     state: filterState = "all",
     sourceChannel,
     source,
@@ -373,9 +361,8 @@ export function listConversationAttention(
   } = params;
 
   const db = getDb();
-  const conditions = [
-    eq(conversationAssistantAttentionState.assistantId, assistantId),
-  ];
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  const conditions: any[] = [];
 
   if (sourceChannel) {
     conditions.push(eq(conversations.originChannel, sourceChannel));
@@ -415,7 +402,6 @@ export function listConversationAttention(
   let query = db
     .select({
       conversationId: conversationAssistantAttentionState.conversationId,
-      assistantId: conversationAssistantAttentionState.assistantId,
       latestAssistantMessageId:
         conversationAssistantAttentionState.latestAssistantMessageId,
       latestAssistantMessageAt:
@@ -448,8 +434,7 @@ export function listConversationAttention(
     );
   }
 
-  const rows = query
-    .where(and(...conditions))
+  const rows = (conditions.length > 0 ? query.where(and(...conditions)) : query)
     .orderBy(desc(conversationAssistantAttentionState.latestAssistantMessageAt))
     .limit(limit)
     .all();

package/src/memory/conversation-crud.ts

@@ -7,7 +7,6 @@ import { parseChannelId, parseInterfaceId } from "../channels/types.js";
 import { CHANNEL_IDS, INTERFACE_IDS, isChannelId } from "../channels/types.js";
 import { getConfig } from "../config/loader.js";
 import type { TrustContext } from "../daemon/session-runtime-assembly.js";
-import { DAEMON_INTERNAL_ASSISTANT_ID } from "../runtime/assistant-scope.js";
 import { getLogger } from "../util/logger.js";
 import { createRowMapper } from "../util/row-mapper.js";
 import { deleteOrphanAttachments } from "./attachments-store.js";
@@ -394,7 +393,6 @@ export async function addMessage(
     try {
       projectAssistantMessage({
         conversationId,
-        assistantId: DAEMON_INTERNAL_ASSISTANT_ID,
         messageId: message.id,
         messageAt: message.createdAt,
       });

package/src/memory/db-init.ts

@@ -36,6 +36,7 @@ import {
   migrateBackfillContactInteractionStats,
   migrateBackfillGuardianPrincipalId,
   migrateCallSessionMode,
+  migrateDropAssistantIdColumns,
   migrateCanonicalGuardianDeliveriesDestinationIndex,
   migrateCanonicalGuardianRequesterChatId,
   migrateChannelInboundDeliveredSegments,
@@ -56,6 +57,7 @@ import {
   migrateGuardianPrincipalIdNotNull,
   migrateGuardianVerificationPurpose,
   migrateGuardianVerificationSessions,
+  migrateInviteCodeHashColumn,
   migrateMessagesFtsBackfill,
   migrateNormalizePhoneIdentities,
   migrateNotificationDeliveryThreadDecision,
@@ -246,6 +248,9 @@ export function initializeDb(): void {
   // 27. Voice invite display metadata (friend_name, guardian_name) for personalized prompts
   migrateVoiceInviteDisplayMetadata(database);
 
+  // 27b. 6-digit invite code hash column for non-voice channel invite redemption
+  migrateInviteCodeHashColumn(database);
+
   // 28. Actor token records (hash-only actor token persistence)
   createActorTokenRecordsTable(database);
 
@@ -285,6 +290,9 @@ export function initializeDb(): void {
   // 39. Backfill contact interaction stats from channel lastSeenAt
   migrateBackfillContactInteractionStats(database);
 
+  // 40. Drop assistant_id columns from all 16 daemon tables
+  migrateDropAssistantIdColumns(database);
+
   validateMigrationState(database);
 
   if (process.env.BUN_TEST === "1") {

package/src/memory/delivery-crud.ts

@@ -125,6 +125,19 @@ export function recordInbound(
   };
 }
 
+/**
+ * Delete an inbound event record by its event ID. Used to roll back a
+ * dedup record when downstream processing (e.g. invite redemption) fails,
+ * so that webhook retries can re-attempt instead of short-circuiting as
+ * duplicates.
+ */
+export function deleteInbound(eventId: string): void {
+  const db = getDb();
+  db.delete(channelInboundEvents)
+    .where(eq(channelInboundEvents.id, eventId))
+    .run();
+}
+
 /**
  * Link an inbound event to the user message it created, so edits can
  * later find the correct message by source_message_id -> message_id.

package/src/memory/guardian-action-store.ts

@@ -10,7 +10,6 @@
 import { and, desc, eq, inArray, lt } from "drizzle-orm";
 import { v4 as uuid } from "uuid";
 
-import { DAEMON_INTERNAL_ASSISTANT_ID } from "../runtime/assistant-scope.js";
 import { getLogger } from "../util/logger.js";
 import { getDb, rawChanges } from "./db.js";
 import { guardianActionDeliveries, guardianActionRequests } from "./schema.js";
@@ -49,7 +48,6 @@ export type FollowupAction = "call_back" | "message_back" | "decline";
 
 export interface GuardianActionRequest {
   id: string;
-  assistantId: string;
   kind: string;
   sourceChannel: string;
   sourceConversationId: string;
@@ -101,7 +99,6 @@ function rowToRequest(
 ): GuardianActionRequest {
   return {
     id: row.id,
-    assistantId: row.assistantId,
     kind: row.kind,
     sourceChannel: row.sourceChannel,
     sourceConversationId: row.sourceConversationId,
@@ -165,7 +162,6 @@ function generateRequestCode(): string {
  * legacy guardian action rows continue to compile.
  */
 export function createGuardianActionRequest(params: {
-  assistantId?: string;
   kind: string;
   sourceChannel: string;
   sourceConversationId: string;
@@ -182,7 +178,6 @@ export function createGuardianActionRequest(params: {
 
   const row = {
     id,
-    assistantId: params.assistantId ?? DAEMON_INTERNAL_ASSISTANT_ID,
     kind: params.kind,
     sourceChannel: params.sourceChannel,
     sourceConversationId: params.sourceConversationId,
@@ -693,14 +688,12 @@ export function createGuardianActionDelivery(params: {
  * Used by inbound message routing to match incoming answers to deliveries.
  */
 export function getPendingDeliveriesByDestination(
-  assistantId: string,
   channel: string,
   chatId: string,
 ): GuardianActionDelivery[] {
   try {
     const db = getDb();
 
-    // Join deliveries with requests to filter by assistantId
     const rows = db
       .select({
         delivery: guardianActionDeliveries,
@@ -712,7 +705,6 @@ export function getPendingDeliveriesByDestination(
       )
       .where(
         and(
-          eq(guardianActionRequests.assistantId, assistantId),
           eq(guardianActionRequests.status, "pending"),
           eq(guardianActionDeliveries.destinationChannel, channel),
           eq(guardianActionDeliveries.destinationChatId, chatId),
@@ -784,7 +776,6 @@ export function getPendingDeliveriesByConversation(
  * Used by inbound message routing to match late guardian answers to expired requests.
  */
 export function getExpiredDeliveriesByDestination(
-  assistantId: string,
   channel: string,
   chatId: string,
 ): GuardianActionDelivery[] {
@@ -802,7 +793,6 @@ export function getExpiredDeliveriesByDestination(
       )
       .where(
         and(
-          eq(guardianActionRequests.assistantId, assistantId),
           eq(guardianActionRequests.status, "expired"),
           eq(guardianActionRequests.followupState, "none"),
           eq(guardianActionDeliveries.destinationChannel, channel),
@@ -877,7 +867,6 @@ export function getExpiredDeliveriesByConversation(
  * on channel paths (Telegram, SMS).
  */
 export function getFollowupDeliveriesByDestination(
-  assistantId: string,
   channel: string,
   chatId: string,
 ): GuardianActionDelivery[] {
@@ -895,7 +884,6 @@ export function getFollowupDeliveriesByDestination(
       )
       .where(
         and(
-          eq(guardianActionRequests.assistantId, assistantId),
           eq(guardianActionRequests.status, "expired"),
           eq(guardianActionRequests.followupState, "awaiting_guardian_choice"),
           eq(guardianActionDeliveries.destinationChannel, channel),

package/src/memory/guardian-approvals.ts

@@ -9,7 +9,6 @@
 import { and, count, desc, eq, gt, lte } from "drizzle-orm";
 import { v4 as uuid } from "uuid";
 
-import { DAEMON_INTERNAL_ASSISTANT_ID } from "../runtime/assistant-scope.js";
 import { getDb } from "./db.js";
 import { channelGuardianApprovalRequests } from "./schema.js";
 
@@ -29,7 +28,6 @@ export interface GuardianApprovalRequest {
   runId: string;
   requestId: string;
   conversationId: string;
-  assistantId: string;
   channel: string;
   requesterExternalUserId: string;
   requesterChatId: string;
@@ -57,7 +55,6 @@ function rowToApprovalRequest(
     runId: row.runId,
     requestId: row.requestId ?? row.runId,
     conversationId: row.conversationId,
-    assistantId: row.assistantId,
     channel: row.channel,
     requesterExternalUserId: row.requesterExternalUserId,
     requesterChatId: row.requesterChatId,
@@ -88,7 +85,6 @@ export function createApprovalRequest(params: {
   runId: string;
   requestId?: string;
   conversationId: string;
-  assistantId?: string;
   channel: string;
   requesterExternalUserId: string;
   requesterChatId: string;
@@ -108,7 +104,6 @@ export function createApprovalRequest(params: {
     runId: params.runId,
     requestId: params.requestId ?? null,
     conversationId: params.conversationId,
-    assistantId: params.assistantId ?? DAEMON_INTERNAL_ASSISTANT_ID,
     channel: params.channel,
     requesterExternalUserId: params.requesterExternalUserId,
     requesterChatId: params.requesterChatId,
@@ -172,34 +167,25 @@ export function getUnresolvedApprovalForRequest(
 /**
  * Find a pending guardian approval request by the guardian's chat ID.
  * Used when the guardian sends a decision from their chat.
- *
- * When `assistantId` is provided, the lookup is scoped to that assistant,
- * preventing cross-assistant approval consumption in shared guardian chats.
  */
 export function getPendingApprovalByGuardianChat(
   channel: string,
   guardianChatId: string,
-  assistantId?: string,
 ): GuardianApprovalRequest | null {
   const db = getDb();
   const now = Date.now();
 
-  const conditions = [
-    eq(channelGuardianApprovalRequests.channel, channel),
-    eq(channelGuardianApprovalRequests.guardianChatId, guardianChatId),
-    eq(channelGuardianApprovalRequests.status, "pending"),
-    gt(channelGuardianApprovalRequests.expiresAt, now),
-  ];
-  if (assistantId) {
-    conditions.push(
-      eq(channelGuardianApprovalRequests.assistantId, assistantId),
-    );
-  }
-
   const row = db
     .select()
     .from(channelGuardianApprovalRequests)
-    .where(and(...conditions))
+    .where(
+      and(
+        eq(channelGuardianApprovalRequests.channel, channel),
+        eq(channelGuardianApprovalRequests.guardianChatId, guardianChatId),
+        eq(channelGuardianApprovalRequests.status, "pending"),
+        gt(channelGuardianApprovalRequests.expiresAt, now),
+      ),
+    )
     .orderBy(desc(channelGuardianApprovalRequests.createdAt))
     .get();
 
@@ -216,28 +202,22 @@ export function getPendingApprovalByRequestAndGuardianChat(
   requestId: string,
   channel: string,
   guardianChatId: string,
-  assistantId?: string,
 ): GuardianApprovalRequest | null {
   const db = getDb();
   const now = Date.now();
 
-  const conditions = [
-    eq(channelGuardianApprovalRequests.requestId, requestId),
-    eq(channelGuardianApprovalRequests.channel, channel),
-    eq(channelGuardianApprovalRequests.guardianChatId, guardianChatId),
-    eq(channelGuardianApprovalRequests.status, "pending"),
-    gt(channelGuardianApprovalRequests.expiresAt, now),
-  ];
-  if (assistantId) {
-    conditions.push(
-      eq(channelGuardianApprovalRequests.assistantId, assistantId),
-    );
-  }
-
   const row = db
     .select()
     .from(channelGuardianApprovalRequests)
-    .where(and(...conditions))
+    .where(
+      and(
+        eq(channelGuardianApprovalRequests.requestId, requestId),
+        eq(channelGuardianApprovalRequests.channel, channel),
+        eq(channelGuardianApprovalRequests.guardianChatId, guardianChatId),
+        eq(channelGuardianApprovalRequests.status, "pending"),
+        gt(channelGuardianApprovalRequests.expiresAt, now),
+      ),
+    )
     .get();
 
   return row ? rowToApprovalRequest(row) : null;
@@ -247,34 +227,25 @@ export function getPendingApprovalByRequestAndGuardianChat(
  * Return all pending (non-expired) guardian approval requests for a given
  * guardian chat and channel. Used to detect ambiguity when a guardian sends
  * a plain-text decision while multiple approvals are pending.
- *
- * When `assistantId` is provided, the results are scoped to that assistant
- * to prevent cross-assistant approval consumption.
  */
 export function getAllPendingApprovalsByGuardianChat(
   channel: string,
   guardianChatId: string,
-  assistantId?: string,
 ): GuardianApprovalRequest[] {
   const db = getDb();
   const now = Date.now();
 
-  const conditions = [
-    eq(channelGuardianApprovalRequests.channel, channel),
-    eq(channelGuardianApprovalRequests.guardianChatId, guardianChatId),
-    eq(channelGuardianApprovalRequests.status, "pending"),
-    gt(channelGuardianApprovalRequests.expiresAt, now),
-  ];
-  if (assistantId) {
-    conditions.push(
-      eq(channelGuardianApprovalRequests.assistantId, assistantId),
-    );
-  }
-
   const rows = db
     .select()
     .from(channelGuardianApprovalRequests)
-    .where(and(...conditions))
+    .where(
+      and(
+        eq(channelGuardianApprovalRequests.channel, channel),
+        eq(channelGuardianApprovalRequests.guardianChatId, guardianChatId),
+        eq(channelGuardianApprovalRequests.status, "pending"),
+        gt(channelGuardianApprovalRequests.expiresAt, now),
+      ),
+    )
     .orderBy(desc(channelGuardianApprovalRequests.createdAt))
     .all();
 
@@ -326,11 +297,10 @@ export function updateApprovalDecision(
 // ---------------------------------------------------------------------------
 
 /**
- * List approval requests filtered by assistant, and optionally by channel,
- * conversation, and status. Returns a paginated list of escalations.
+ * List approval requests optionally filtered by channel, conversation, and
+ * status. Returns a paginated list of escalations.
  */
 export function listPendingApprovalRequests(params: {
-  assistantId?: string;
   channel?: string;
   conversationId?: string;
   status?: ApprovalRequestStatus;
@@ -339,12 +309,7 @@ export function listPendingApprovalRequests(params: {
 }): GuardianApprovalRequest[] {
   const db = getDb();
 
-  const conditions = [
-    eq(
-      channelGuardianApprovalRequests.assistantId,
-      params.assistantId ?? DAEMON_INTERNAL_ASSISTANT_ID,
-    ),
-  ];
+  const conditions: ReturnType<typeof eq>[] = [];
   if (params.channel) {
     conditions.push(
       eq(channelGuardianApprovalRequests.channel, params.channel),
@@ -447,26 +412,18 @@ export function resolveApprovalRequest(
  * Count pending approval requests for a given conversation.
  * Used by thread state projection to compute `pending_escalation_count`.
  */
-export function countPendingByConversation(
-  conversationId: string,
-  assistantId?: string,
-): number {
+export function countPendingByConversation(conversationId: string): number {
   const db = getDb();
 
-  const conditions = [
-    eq(channelGuardianApprovalRequests.conversationId, conversationId),
-    eq(channelGuardianApprovalRequests.status, "pending"),
-  ];
-  if (assistantId) {
-    conditions.push(
-      eq(channelGuardianApprovalRequests.assistantId, assistantId),
-    );
-  }
-
   const result = db
     .select({ count: count() })
     .from(channelGuardianApprovalRequests)
-    .where(and(...conditions))
+    .where(
+      and(
+        eq(channelGuardianApprovalRequests.conversationId, conversationId),
+        eq(channelGuardianApprovalRequests.status, "pending"),
+      ),
+    )
     .get();
 
   return result?.count ?? 0;
@@ -478,7 +435,6 @@ export function countPendingByConversation(
  * Retained for existing test fixtures that check legacy approval dedup.
  */
 export function findPendingAccessRequestForRequester(
-  assistantId: string,
   channel: string,
   requesterExternalUserId: string,
   toolName: string,
@@ -491,7 +447,6 @@ export function findPendingAccessRequestForRequester(
     .from(channelGuardianApprovalRequests)
     .where(
       and(
-        eq(channelGuardianApprovalRequests.assistantId, assistantId),
         eq(channelGuardianApprovalRequests.channel, channel),
         eq(
           channelGuardianApprovalRequests.requesterExternalUserId,

package/src/memory/guardian-rate-limits.ts

@@ -17,7 +17,6 @@ import { channelGuardianRateLimits } from "./schema.js";
 
 export interface VerificationRateLimit {
   id: string;
-  assistantId: string;
   channel: string;
   actorExternalUserId: string;
   actorChatId: string;
@@ -49,7 +48,6 @@ function rowToRateLimit(
   const timestamps = parseTimestamps(row.attemptTimestampsJson);
   return {
     id: row.id,
-    assistantId: row.assistantId,
     channel: row.channel,
     actorExternalUserId: row.actorExternalUserId,
     actorChatId: row.actorChatId,
@@ -69,7 +67,6 @@ function rowToRateLimit(
  * Get the rate-limit record for a given actor on a specific channel.
  */
 export function getRateLimit(
-  assistantId: string,
   channel: string,
   actorExternalUserId: string,
   actorChatId: string,
@@ -80,7 +77,6 @@ export function getRateLimit(
     .from(channelGuardianRateLimits)
     .where(
       and(
-        eq(channelGuardianRateLimits.assistantId, assistantId),
         eq(channelGuardianRateLimits.channel, channel),
         eq(channelGuardianRateLimits.actorExternalUserId, actorExternalUserId),
         eq(channelGuardianRateLimits.actorChatId, actorChatId),
@@ -101,7 +97,6 @@ export function getRateLimit(
  * accumulate indefinitely.
  */
 export function recordInvalidAttempt(
-  assistantId: string,
   channel: string,
   actorExternalUserId: string,
   actorChatId: string,
@@ -113,12 +108,7 @@ export function recordInvalidAttempt(
   const now = Date.now();
   const cutoff = now - windowMs;
 
-  const existing = getRateLimit(
-    assistantId,
-    channel,
-    actorExternalUserId,
-    actorChatId,
-  );
+  const existing = getRateLimit(channel, actorExternalUserId, actorChatId);
 
   if (existing) {
     // Keep only timestamps within the sliding window, then add the new one
@@ -158,7 +148,6 @@ export function recordInvalidAttempt(
   const lockedUntil = 1 >= maxAttempts ? now + lockoutMs : null;
   const row = {
     id,
-    assistantId,
     channel,
     actorExternalUserId,
     actorChatId,
@@ -181,7 +170,6 @@ export function recordInvalidAttempt(
  * successful verification).
  */
 export function resetRateLimit(
-  assistantId: string,
   channel: string,
   actorExternalUserId: string,
   actorChatId: string,
@@ -197,7 +185,6 @@ export function resetRateLimit(
     })
     .where(
       and(
-        eq(channelGuardianRateLimits.assistantId, assistantId),
         eq(channelGuardianRateLimits.channel, channel),
         eq(channelGuardianRateLimits.actorExternalUserId, actorExternalUserId),
         eq(channelGuardianRateLimits.actorChatId, actorChatId),