@sentry/junior 0.74.0 → 0.75.0

package/dist/{chunk-JL2SLRAT.js → chunk-UJ7OTHPO.js}

@@ -1,276 +1,31 @@
 import {
   parseDestination,
   sameDestination
-} from "./chunk-YRDS7VKO.js";
+} from "./chunk-WBZ4M5N5.js";
+import {
+  juniorConversations,
+  juniorDestinations,
+  juniorIdentities
+} from "./chunk-D7NFH5GD.js";
 import {
   getDefaultRedisStateAdapterFor,
   getStateAdapter
-} from "./chunk-F6HWCPOC.js";
+} from "./chunk-ZNNTSPNF.js";
 import {
   getChatConfig
-} from "./chunk-GM7HTXYC.js";
+} from "./chunk-FCZO7LAR.js";
 import {
   parseStoredSlackRequester
-} from "./chunk-CYUI7JU5.js";
+} from "./chunk-EIYL7I4S.js";
 import {
   isRecord,
   toOptionalNumber,
   toOptionalString
-} from "./chunk-3BYAPS6B.js";
+} from "./chunk-OK4KKR7B.js";
 
 // src/chat/state/ttl.ts
 var JUNIOR_THREAD_STATE_TTL_MS = 7 * 24 * 60 * 60 * 1e3;
 
-// src/chat/sql/neon.ts
-import { AsyncLocalStorage } from "async_hooks";
-import {
-  Pool
-} from "@neondatabase/serverless";
-import { drizzle } from "drizzle-orm/neon-serverless";
-
-// src/chat/conversations/sql/schema/conversations.ts
-import { sql } from "drizzle-orm";
-import { index as index3, integer, jsonb as jsonb3, pgTable as pgTable3, text as text3 } from "drizzle-orm/pg-core";
-
-// src/chat/conversations/sql/schema/destinations.ts
-import { index, jsonb, pgTable, text, uniqueIndex } from "drizzle-orm/pg-core";
-
-// src/chat/conversations/sql/schema/timestamps.ts
-import { timestamp } from "drizzle-orm/pg-core";
-var timestamptz = (name) => timestamp(name, { withTimezone: true });
-
-// src/chat/conversations/sql/schema/destinations.ts
-var juniorDestinations = pgTable(
-  "junior_destinations",
-  {
-    id: text("id").primaryKey(),
-    provider: text("provider").notNull(),
-    providerTenantId: text("provider_tenant_id").notNull().default(""),
-    providerDestinationId: text("provider_destination_id").notNull(),
-    kind: text("kind").$type().notNull(),
-    parentDestinationId: text("parent_destination_id"),
-    displayName: text("display_name"),
-    visibility: text("visibility").$type().notNull().default("unknown"),
-    metadata: jsonb("metadata_json"),
-    createdAt: timestamptz("created_at").notNull(),
-    updatedAt: timestamptz("updated_at").notNull()
-  },
-  (table) => [
-    uniqueIndex("junior_destinations_provider_destination_uidx").on(
-      table.provider,
-      table.providerTenantId,
-      table.providerDestinationId
-    ),
-    index("junior_destinations_provider_kind_idx").on(
-      table.provider,
-      table.kind
-    )
-  ]
-);
-
-// src/chat/conversations/sql/schema/identities.ts
-import { index as index2, jsonb as jsonb2, pgTable as pgTable2, text as text2, uniqueIndex as uniqueIndex2 } from "drizzle-orm/pg-core";
-var juniorIdentities = pgTable2(
-  "junior_identities",
-  {
-    id: text2("id").primaryKey(),
-    kind: text2("kind").$type().notNull(),
-    provider: text2("provider").notNull(),
-    providerTenantId: text2("provider_tenant_id").notNull().default(""),
-    providerSubjectId: text2("provider_subject_id").notNull(),
-    displayName: text2("display_name"),
-    handle: text2("handle"),
-    email: text2("email"),
-    avatarUrl: text2("avatar_url"),
-    metadata: jsonb2("metadata_json"),
-    createdAt: timestamptz("created_at").notNull(),
-    updatedAt: timestamptz("updated_at").notNull()
-  },
-  (table) => [
-    uniqueIndex2("junior_identities_provider_subject_uidx").on(
-      table.provider,
-      table.providerTenantId,
-      table.providerSubjectId
-    ),
-    index2("junior_identities_kind_provider_idx").on(table.kind, table.provider)
-  ]
-);
-
-// src/chat/conversations/sql/schema/conversations.ts
-var juniorConversations = pgTable3(
-  "junior_conversations",
-  {
-    conversationId: text3("conversation_id").primaryKey(),
-    schemaVersion: integer("schema_version").notNull().default(1),
-    source: text3("source").$type(),
-    originType: text3("origin_type"),
-    originId: text3("origin_id"),
-    originRunId: text3("origin_run_id"),
-    destinationId: text3("destination_id").references(
-      () => juniorDestinations.id
-    ),
-    destination: jsonb3("destination_json").$type(),
-    actorIdentityId: text3("actor_identity_id").references(
-      () => juniorIdentities.id
-    ),
-    requesterIdentityId: text3("requester_identity_id").references(
-      () => juniorIdentities.id
-    ),
-    creatorIdentityId: text3("creator_identity_id").references(
-      () => juniorIdentities.id
-    ),
-    credentialSubjectIdentityId: text3(
-      "credential_subject_identity_id"
-    ).references(() => juniorIdentities.id),
-    requester: jsonb3("requester_json").$type(),
-    channelName: text3("channel_name"),
-    title: text3("title"),
-    createdAt: timestamptz("created_at").notNull(),
-    lastActivityAt: timestamptz("last_activity_at").notNull(),
-    updatedAt: timestamptz("updated_at").notNull(),
-    executionUpdatedAt: timestamptz("execution_updated_at"),
-    executionStatus: text3("execution_status").$type().notNull(),
-    runId: text3("run_id"),
-    lastCheckpointAt: timestamptz("last_checkpoint_at"),
-    lastEnqueuedAt: timestamptz("last_enqueued_at")
-  },
-  (table) => [
-    index3("junior_conversations_last_activity_idx").on(
-      table.lastActivityAt.desc(),
-      table.conversationId
-    ),
-    index3("junior_conversations_active_idx").using(
-      "btree",
-      sql`coalesce(${table.executionUpdatedAt}, ${table.updatedAt})`,
-      table.conversationId
-    ).where(sql`${table.executionStatus} <> 'idle'`),
-    index3("junior_conversations_destination_activity_idx").on(
-      table.destinationId,
-      table.lastActivityAt.desc()
-    ),
-    index3("junior_conversations_actor_activity_idx").on(
-      table.actorIdentityId,
-      table.lastActivityAt.desc()
-    ),
-    index3("junior_conversations_requester_activity_idx").on(
-      table.requesterIdentityId,
-      table.lastActivityAt.desc()
-    ),
-    index3("junior_conversations_origin_idx").on(
-      table.originType,
-      table.originId,
-      table.lastActivityAt.desc()
-    )
-  ]
-);
-
-// src/chat/conversations/sql/schema/migrations.ts
-import { pgTable as pgTable4, text as text4 } from "drizzle-orm/pg-core";
-var juniorSchemaMigrations = pgTable4("junior_schema_migrations", {
-  id: text4("id").primaryKey(),
-  checksum: text4("checksum").notNull(),
-  appliedAt: timestamptz("applied_at").notNull().defaultNow()
-});
-
-// src/chat/conversations/sql/schema.ts
-var schema = {
-  juniorConversations,
-  juniorDestinations,
-  juniorIdentities,
-  juniorSchemaMigrations
-};
-
-// src/chat/sql/schema.ts
-var juniorSqlSchema = {
-  ...schema
-};
-
-// src/chat/sql/neon.ts
-var NeonExecutor = class {
-  constructor(pool) {
-    this.pool = pool;
-  }
-  pool;
-  transactionClient = new AsyncLocalStorage();
-  db() {
-    return drizzle(this.queryClient(), {
-      schema: juniorSqlSchema
-    });
-  }
-  async execute(statement, params = []) {
-    await this.queryClient().query(statement, [...params]);
-  }
-  async query(statement, params = []) {
-    const result = await this.queryClient().query(statement, [
-      ...params
-    ]);
-    return result.rows;
-  }
-  async transaction(callback) {
-    const existingClient = this.transactionClient.getStore();
-    if (existingClient) {
-      return await callback();
-    }
-    const client = await this.pool.connect();
-    try {
-      await client.query("BEGIN");
-      const result = await this.transactionClient.run(client, callback);
-      await client.query("COMMIT");
-      return result;
-    } catch (error) {
-      await client.query("ROLLBACK");
-      throw error;
-    } finally {
-      client.release();
-    }
-  }
-  async withLock(lockName, callback) {
-    if (!lockName) {
-      throw new Error("SQL lock name is required");
-    }
-    const existingClient = this.transactionClient.getStore();
-    if (existingClient) {
-      await existingClient.query("SELECT pg_advisory_xact_lock(hashtext($1))", [
-        lockName
-      ]);
-      return await callback();
-    }
-    const client = await this.pool.connect();
-    try {
-      await client.query("BEGIN");
-      return await this.transactionClient.run(client, async () => {
-        try {
-          await client.query("SELECT pg_advisory_xact_lock(hashtext($1))", [
-            lockName
-          ]);
-          const result = await callback();
-          await client.query("COMMIT");
-          return result;
-        } catch (error) {
-          await client.query("ROLLBACK");
-          throw error;
-        }
-      });
-    } finally {
-      client.release();
-    }
-  }
-  async close() {
-    await this.pool.end();
-  }
-  queryClient() {
-    return this.transactionClient.getStore() ?? this.pool;
-  }
-};
-function createNeonJuniorSqlExecutor(args) {
-  return new NeonExecutor(
-    new Pool({
-      connectionString: args.connectionString,
-      max: 3
-    })
-  );
-}
-
 // src/chat/task-execution/state.ts
 import { randomUUID } from "crypto";
 var CONVERSATION_PREFIX = "junior:conversation";
@@ -339,12 +94,12 @@ function normalizeInput(value) {
   if (!isRecord(value)) {
     return void 0;
   }
-  const text5 = toOptionalString(value.text);
-  if (!text5) {
+  const text = toOptionalString(value.text);
+  if (!text) {
     return void 0;
   }
   return {
-    text: text5,
+    text,
     authorId: toOptionalString(value.authorId),
     attachments: Array.isArray(value.attachments) ? [...value.attachments] : void 0,
     metadata: normalizeMetadata(value.metadata)
@@ -589,9 +344,9 @@ function parseRedisIndexEntries(values) {
     return [];
   }
   const entries = [];
-  for (let index4 = 0; index4 < values.length; index4 += 2) {
-    const conversationId = toOptionalString(values[index4]);
-    const score = typeof values[index4 + 1] === "number" ? values[index4 + 1] : Number(values[index4 + 1]);
+  for (let index = 0; index < values.length; index += 2) {
+    const conversationId = toOptionalString(values[index]);
+    const score = typeof values[index + 1] === "number" ? values[index + 1] : Number(values[index + 1]);
     if (!conversationId || !Number.isFinite(score)) {
       continue;
     }
@@ -728,16 +483,16 @@ async function getConversationIndexStore(state) {
   return emulatedConversationIndexStore(state);
 }
 async function upsertIndexEntry(args) {
-  const index4 = await getConversationIndexStore(args.state);
-  await index4.upsert({
+  const index = await getConversationIndexStore(args.state);
+  await index.upsert({
     conversationId: args.conversationId,
     indexKey: args.indexKey,
     score: args.score
   });
 }
 async function removeIndexEntry(args) {
-  const index4 = await getConversationIndexStore(args.state);
-  await index4.remove({
+  const index = await getConversationIndexStore(args.state);
+  await index.remove({
     conversationId: args.conversationId,
     indexKey: args.indexKey
   });
@@ -1088,7 +843,20 @@ async function drainConversationMailbox(args) {
   if (pending.length === 0) {
     return [];
   }
-  await args.inject(pending);
+  const acknowledgedIds = await args.inject(pending);
+  const offeredIds = new Set(
+    pending.map((message) => message.inboundMessageId)
+  );
+  for (const inboundMessageId of acknowledgedIds ?? []) {
+    if (!offeredIds.has(inboundMessageId)) {
+      throw new Error(
+        `Conversation mailbox acknowledgement is not pending for ${args.conversationId}`
+      );
+    }
+  }
+  const drainedIds = new Set(
+    acknowledgedIds ?? pending.map((message) => message.inboundMessageId)
+  );
   await withConversationMutation(args, async (state) => {
     const current = await readConversation(state, args.conversationId);
     if (!current || current.execution.lease?.token !== args.leaseToken) {
@@ -1096,9 +864,6 @@ async function drainConversationMailbox(args) {
         `Conversation lease is not held for ${args.conversationId}`
       );
     }
-    const drainedIds = new Set(
-      pending.map((message) => message.inboundMessageId)
-    );
     const pendingMessages2 = current.execution.pendingMessages.filter(
       (message) => !drainedIds.has(message.inboundMessageId)
     );
@@ -1115,7 +880,7 @@ async function drainConversationMailbox(args) {
       )
     );
   });
-  return pending;
+  return pending.filter((message) => drainedIds.has(message.inboundMessageId));
 }
 async function markConversationMessagesInjected(args) {
   const nowMs = args.nowMs ?? now();
@@ -1254,8 +1019,8 @@ async function removeActiveConversation(args) {
 }
 async function listActiveConversationIds(args = {}) {
   const state = await getConnectedState(args.state);
-  const index4 = await getConversationIndexStore(state);
-  const entries = await index4.list({
+  const index = await getConversationIndexStore(state);
+  const entries = await index.list({
     indexKey: CONVERSATION_ACTIVE_INDEX_KEY,
     limit: args.limit,
     order: "asc",
@@ -1265,8 +1030,8 @@ async function listActiveConversationIds(args = {}) {
 }
 async function listConversationsByActivity(args = {}) {
   const state = await getConnectedState(args.state);
-  const index4 = await getConversationIndexStore(state);
-  const entries = await index4.list({
+  const index = await getConversationIndexStore(state);
+  const entries = await index.list({
     indexKey: CONVERSATION_BY_ACTIVITY_INDEX_KEY,
     limit: args.limit ?? CONVERSATION_ACTIVITY_INDEX_MAX_LENGTH,
     offset: args.offset,
@@ -1306,7 +1071,7 @@ function createStateConversationStore(state) {
 
 // src/chat/conversations/sql/store.ts
 import { randomUUID as randomUUID2 } from "crypto";
-import { asc, desc, eq, sql as sql2 } from "drizzle-orm";
+import { asc, desc, eq, sql } from "drizzle-orm";
 
 // src/chat/conversations/sql/migrations.ts
 import { createHash } from "crypto";
@@ -1803,9 +1568,9 @@ var SqlStore = class {
   /** Upsert the conversation row while preserving previously discovered nullable metadata fields. */
   async upsertConversation(args) {
     const { conversation } = args;
-    const incomingExecutionVersion = sql2`coalesce(excluded.execution_updated_at, excluded.updated_at)`;
-    const currentExecutionVersion = sql2`coalesce(${juniorConversations.executionUpdatedAt}, ${juniorConversations.updatedAt})`;
-    const incomingExecutionIsFresh = sql2`${incomingExecutionVersion} >= ${currentExecutionVersion}`;
+    const incomingExecutionVersion = sql`coalesce(excluded.execution_updated_at, excluded.updated_at)`;
+    const currentExecutionVersion = sql`coalesce(${juniorConversations.executionUpdatedAt}, ${juniorConversations.updatedAt})`;
+    const incomingExecutionIsFresh = sql`${incomingExecutionVersion} >= ${currentExecutionVersion}`;
     const destinationId = await this.upsertDestination(
       destinationUpsertFromDestination({
         channelName: conversation.channelName,
@@ -1849,27 +1614,27 @@ var SqlStore = class {
     }).onConflictDoUpdate({
       target: juniorConversations.conversationId,
       set: {
-        source: sql2`coalesce(excluded.source, ${juniorConversations.source})`,
-        originType: sql2`coalesce(excluded.origin_type, ${juniorConversations.originType})`,
-        originId: sql2`coalesce(excluded.origin_id, ${juniorConversations.originId})`,
-        originRunId: sql2`coalesce(excluded.origin_run_id, ${juniorConversations.originRunId})`,
-        destinationId: sql2`coalesce(excluded.destination_id, ${juniorConversations.destinationId})`,
-        destination: sql2`coalesce(excluded.destination_json, ${juniorConversations.destination})`,
-        actorIdentityId: sql2`coalesce(excluded.actor_identity_id, ${juniorConversations.actorIdentityId})`,
-        requesterIdentityId: sql2`coalesce(excluded.requester_identity_id, ${juniorConversations.requesterIdentityId})`,
-        creatorIdentityId: sql2`coalesce(excluded.creator_identity_id, ${juniorConversations.creatorIdentityId})`,
-        credentialSubjectIdentityId: sql2`coalesce(excluded.credential_subject_identity_id, ${juniorConversations.credentialSubjectIdentityId})`,
-        requester: sql2`coalesce(excluded.requester_json, ${juniorConversations.requester})`,
-        channelName: sql2`coalesce(excluded.channel_name, ${juniorConversations.channelName})`,
-        title: sql2`coalesce(excluded.title, ${juniorConversations.title})`,
-        createdAt: sql2`least(${juniorConversations.createdAt}, excluded.created_at)`,
-        lastActivityAt: sql2`greatest(${juniorConversations.lastActivityAt}, excluded.last_activity_at)`,
-        updatedAt: sql2`greatest(${juniorConversations.updatedAt}, excluded.updated_at)`,
-        executionUpdatedAt: sql2`case when ${incomingExecutionIsFresh} then excluded.execution_updated_at else ${juniorConversations.executionUpdatedAt} end`,
-        executionStatus: sql2`case when ${incomingExecutionIsFresh} then excluded.execution_status else ${juniorConversations.executionStatus} end`,
-        runId: sql2`case when ${incomingExecutionIsFresh} then excluded.run_id else ${juniorConversations.runId} end`,
-        lastCheckpointAt: sql2`case when ${incomingExecutionIsFresh} then excluded.last_checkpoint_at else ${juniorConversations.lastCheckpointAt} end`,
-        lastEnqueuedAt: sql2`case when ${incomingExecutionIsFresh} then excluded.last_enqueued_at else ${juniorConversations.lastEnqueuedAt} end`
+        source: sql`coalesce(excluded.source, ${juniorConversations.source})`,
+        originType: sql`coalesce(excluded.origin_type, ${juniorConversations.originType})`,
+        originId: sql`coalesce(excluded.origin_id, ${juniorConversations.originId})`,
+        originRunId: sql`coalesce(excluded.origin_run_id, ${juniorConversations.originRunId})`,
+        destinationId: sql`coalesce(excluded.destination_id, ${juniorConversations.destinationId})`,
+        destination: sql`coalesce(excluded.destination_json, ${juniorConversations.destination})`,
+        actorIdentityId: sql`coalesce(excluded.actor_identity_id, ${juniorConversations.actorIdentityId})`,
+        requesterIdentityId: sql`coalesce(excluded.requester_identity_id, ${juniorConversations.requesterIdentityId})`,
+        creatorIdentityId: sql`coalesce(excluded.creator_identity_id, ${juniorConversations.creatorIdentityId})`,
+        credentialSubjectIdentityId: sql`coalesce(excluded.credential_subject_identity_id, ${juniorConversations.credentialSubjectIdentityId})`,
+        requester: sql`coalesce(excluded.requester_json, ${juniorConversations.requester})`,
+        channelName: sql`coalesce(excluded.channel_name, ${juniorConversations.channelName})`,
+        title: sql`coalesce(excluded.title, ${juniorConversations.title})`,
+        createdAt: sql`least(${juniorConversations.createdAt}, excluded.created_at)`,
+        lastActivityAt: sql`greatest(${juniorConversations.lastActivityAt}, excluded.last_activity_at)`,
+        updatedAt: sql`greatest(${juniorConversations.updatedAt}, excluded.updated_at)`,
+        executionUpdatedAt: sql`case when ${incomingExecutionIsFresh} then excluded.execution_updated_at else ${juniorConversations.executionUpdatedAt} end`,
+        executionStatus: sql`case when ${incomingExecutionIsFresh} then excluded.execution_status else ${juniorConversations.executionStatus} end`,
+        runId: sql`case when ${incomingExecutionIsFresh} then excluded.run_id else ${juniorConversations.runId} end`,
+        lastCheckpointAt: sql`case when ${incomingExecutionIsFresh} then excluded.last_checkpoint_at else ${juniorConversations.lastCheckpointAt} end`,
+        lastEnqueuedAt: sql`case when ${incomingExecutionIsFresh} then excluded.last_enqueued_at else ${juniorConversations.lastEnqueuedAt} end`
       }
     });
   }
@@ -1897,13 +1662,13 @@ var SqlStore = class {
         juniorIdentities.providerSubjectId
       ],
       set: {
-        kind: sql2`excluded.kind`,
-        displayName: sql2`coalesce(excluded.display_name, ${juniorIdentities.displayName})`,
-        handle: sql2`coalesce(excluded.handle, ${juniorIdentities.handle})`,
-        email: sql2`coalesce(excluded.email, ${juniorIdentities.email})`,
-        avatarUrl: sql2`coalesce(excluded.avatar_url, ${juniorIdentities.avatarUrl})`,
-        metadata: sql2`coalesce(excluded.metadata_json, ${juniorIdentities.metadata})`,
-        updatedAt: sql2`excluded.updated_at`
+        kind: sql`excluded.kind`,
+        displayName: sql`coalesce(excluded.display_name, ${juniorIdentities.displayName})`,
+        handle: sql`coalesce(excluded.handle, ${juniorIdentities.handle})`,
+        email: sql`coalesce(excluded.email, ${juniorIdentities.email})`,
+        avatarUrl: sql`coalesce(excluded.avatar_url, ${juniorIdentities.avatarUrl})`,
+        metadata: sql`coalesce(excluded.metadata_json, ${juniorIdentities.metadata})`,
+        updatedAt: sql`excluded.updated_at`
       }
     }).returning({ id: juniorIdentities.id });
     return rows[0]?.id;
@@ -1931,11 +1696,11 @@ var SqlStore = class {
         juniorDestinations.providerDestinationId
       ],
       set: {
-        kind: sql2`excluded.kind`,
-        displayName: sql2`coalesce(excluded.display_name, ${juniorDestinations.displayName})`,
-        visibility: sql2`excluded.visibility`,
-        metadata: sql2`coalesce(excluded.metadata_json, ${juniorDestinations.metadata})`,
-        updatedAt: sql2`excluded.updated_at`
+        kind: sql`excluded.kind`,
+        displayName: sql`coalesce(excluded.display_name, ${juniorDestinations.displayName})`,
+        visibility: sql`excluded.visibility`,
+        metadata: sql`coalesce(excluded.metadata_json, ${juniorDestinations.metadata})`,
+        updatedAt: sql`excluded.updated_at`
       }
     }).returning({ id: juniorDestinations.id });
     return rows[0]?.id;
@@ -1947,7 +1712,6 @@ function createSqlStore(executor) {
 
 export {
   JUNIOR_THREAD_STATE_TTL_MS,
-  createNeonJuniorSqlExecutor,
   CONVERSATION_WORK_CHECK_IN_INTERVAL_MS,
   CONVERSATION_WORK_STALE_ENQUEUE_MS,
   getConversation,