@sentry/junior 0.186.0 → 0.188.0

package/dist/{chunk-OZG7E22Y.js → chunk-QDHJTKPE.js}

@@ -2,29 +2,29 @@ import {
   coerceThreadConversationState,
   getPersistedThreadState,
   persistThreadStateById
-} from "./chunk-CM6RTYSO.js";
+} from "./chunk-CIG6WI7Q.js";
 import {
   buildDeterministicTurnId,
   clearPendingAuth,
   markTurnClosed,
   markTurnCompleted
-} from "./chunk-VOGOTEW7.js";
+} from "./chunk-Z5SKZE5B.js";
 import {
   hydrateConversationMessages,
   markConversationMessage
-} from "./chunk-H7X2G2KS.js";
+} from "./chunk-6YGFBU3A.js";
 import {
   ProviderError,
   findProviderError,
   getProviderErrorAttributes,
   getProviderErrorUserMessage
-} from "./chunk-AD64YMEC.js";
+} from "./chunk-Q7O74WOO.js";
 import {
   parseSlackMessageTs
 } from "./chunk-N7TMSOZH.js";
 import {
   buildTurnFailureResponse
-} from "./chunk-3CVEE6LA.js";
+} from "./chunk-QIMN3FXL.js";
 
 // src/chat/runtime/delivered-turn-state.ts
 function buildDeliveredTurnStatePatch(args) {

package/dist/{chunk-3CVEE6LA.js → chunk-QIMN3FXL.js}

@@ -12,7 +12,7 @@ import {
 } from "./chunk-PEQXYSSE.js";
 import {
   JUNIOR_VERSION
-} from "./chunk-YXSBZRP7.js";
+} from "./chunk-32XSSFZV.js";
 
 // src/deployment.ts
 var JUNIOR_HEARTBEAT_ROUTE = "/api/internal/heartbeat";

package/dist/{chunk-64WF2GQ3.js → chunk-QIOUTXTZ.js}

@@ -12,7 +12,7 @@ import {
   logInfo,
   logWarn,
   setSpanAttributes
-} from "./chunk-3CVEE6LA.js";
+} from "./chunk-QIMN3FXL.js";
 
 // src/chat/plugins/manifest.ts
 import { z } from "zod";

package/dist/{chunk-VXL2LBPK.js → chunk-QVH6FW3T.js}

@@ -4,7 +4,7 @@ import {
 } from "./chunk-VFUK3X5B.js";
 import {
   DEFAULT_SQL_STATEMENT_TIMEOUT_MS
-} from "./chunk-AD64YMEC.js";
+} from "./chunk-Q7O74WOO.js";
 
 // src/db/neon.ts
 import { AsyncLocalStorage } from "async_hooks";
@@ -137,13 +137,19 @@ var juniorConversations = pgTable4(
     schemaVersion: integer2("schema_version").notNull().default(1),
     source: text4("source").$type(),
     /** Structured session Source locator; set-once on the conversation root. */
+    // TODO(dcramer): Remove source_json after every stored Conversation has a
+    // complete Location and all readers use it for provider context.
     sessionSource: jsonb2("source_json").$type(),
     originType: text4("origin_type"),
     originId: text4("origin_id"),
     originRunId: text4("origin_run_id"),
+    // TODO(dcramer): Rename destination_id to location_id after all deployed
+    // readers and writers use the singular Conversation Location contract.
     destinationId: text4("destination_id").references(
       () => juniorDestinations.id
     ),
+    // TODO(dcramer): Drop destination_json after every row uses location_id and
+    // no supported worker writes the legacy JSON value.
     destination: jsonb2("destination_json").$type(),
     actorIdentityId: text4("actor_identity_id").references(
       () => juniorIdentities.id
@@ -422,6 +428,8 @@ var juniorConversationBindings = pgTable10(
     conversationId: text10("conversation_id").notNull().references(() => juniorConversations.conversationId),
     provider: text10("provider").notNull(),
     providerTenantId: text10("provider_tenant_id").notNull().default(""),
+    // TODO(dcramer): Rename provider_destination_id with the Location SQL
+    // migration. It identifies the provider place that contains the thread.
     providerDestinationId: text10("provider_destination_id").notNull(),
     providerConversationId: text10("provider_conversation_id").notNull(),
     createdAt: timestamptz("created_at").notNull()
@@ -660,7 +668,8 @@ var juniorSchedulerTasks = pgTable17(
   {
     id: text17("id").primaryKey(),
     teamId: text17("team_id").notNull(),
-    // TODO(v0.128.0): Remove after v0.127.x runtimes no longer write this column.
+    // TODO(dcramer): Remove after v0.127.x runtimes are unsupported and no
+    // supported runtime writes this column.
     creatorSlackUserId: text17("creator_slack_user_id"),
     creatorIdentityId: text17("creator_identity_id"),
     status: text17("status").notNull(),
@@ -672,7 +681,8 @@ var juniorSchedulerTasks = pgTable17(
     record: jsonb7("record").$type().notNull()
   },
   (table) => [
-    // TODO(v0.128.0): Remove with creatorSlackUserId.
+    // TODO(dcramer): Remove after no supported runtime reads or writes the
+    // legacy creatorSlackUserId column.
     index13("junior_scheduler_tasks_creator_idx").on(table.teamId, table.creatorSlackUserId, table.createdAtMs, table.id).where(sql6`${table.status} <> 'deleted'`),
     index13("junior_scheduler_tasks_creator_identity_idx").on(table.creatorIdentityId, table.createdAtMs, table.id).where(
       sql6`${table.status} <> 'deleted' AND ${table.creatorIdentityId} IS NOT NULL`

package/dist/{chunk-GSB3NTNS.js → chunk-RMB7CERI.js}

@@ -7,7 +7,7 @@ import {
   PluginCredentialFailureError,
   agentNameSchema,
   createPluginAuthOrchestration
-} from "./chunk-TXLBLGNQ.js";
+} from "./chunk-6JZGXLL2.js";
 import {
   parseSandboxEgressAuthRequiredSignal,
   parseSandboxEgressPermissionDeniedSignal,
@@ -38,7 +38,7 @@ import {
   searchSlackUsers,
   selectPluginGrant,
   toMcpProviderError
-} from "./chunk-QU3URVWG.js";
+} from "./chunk-W2DDJCPS.js";
 import {
   addReactionToMessage,
   deliverOAuthAuthorization,
@@ -46,7 +46,7 @@ import {
   getSlackMessagePermalink,
   resolveBaseUrl,
   uploadFilesToConversation
-} from "./chunk-VQZYIVYN.js";
+} from "./chunk-UVUXFRYN.js";
 import {
   SCHEDULED_TASK_SYSTEM_ACTOR,
   WORKSPACE_SNAPSHOT_FAILED_EVENT,
@@ -87,10 +87,10 @@ import {
   workspaceSnapshotFinishedEvent,
   workspaceSnapshotWatch,
   writableEventTask
-} from "./chunk-DHYZ4VKG.js";
+} from "./chunk-VFZAUQ5V.js";
 import {
   parseSlackThreadId
-} from "./chunk-7EOWT62G.js";
+} from "./chunk-CVBDVIF4.js";
 import {
   RetryableDeliveryError,
   actorFromRun,
@@ -98,7 +98,7 @@ import {
   isAgentRunFeatureDisabled,
   surfaceFromRun,
   toolInvocationDestination
-} from "./chunk-INQHUE72.js";
+} from "./chunk-6FGN7ZEG.js";
 import {
   CooperativeTurnYieldError,
   NO_REPLY_MARKER,
@@ -111,7 +111,7 @@ import {
   isNoReplyMarker,
   isTurnInputCommitLostError,
   sanitizeAssistantText
-} from "./chunk-VOGOTEW7.js";
+} from "./chunk-Z5SKZE5B.js";
 import {
   AGENTS_REMOVAL_NOTICE,
   AGENTS_REPLACEMENT_NOTICE,
@@ -171,7 +171,7 @@ import {
   stripRuntimeTurnContext,
   trimTrailingAssistantMessages,
   truncateText
-} from "./chunk-H7X2G2KS.js";
+} from "./chunk-6YGFBU3A.js";
 import {
   bindSandboxFileSystem,
   createSandboxSession,
@@ -182,7 +182,7 @@ import {
   resolve,
   setSnapshotSpanAttributes,
   stopSession
-} from "./chunk-CV5CPRVG.js";
+} from "./chunk-XJIPMHKI.js";
 import {
   privateTraceResultAttributes
 } from "./chunk-UIE3R5XU.js";
@@ -195,7 +195,7 @@ import {
 } from "./chunk-DKIJ2RP5.js";
 import {
   getTurnRequestDeadline
-} from "./chunk-BRKB5R7P.js";
+} from "./chunk-WGZJ5SU4.js";
 import {
   RESOURCE_SUBSCRIPTION_DEFAULT_TTL_MS,
   RESOURCE_SUBSCRIPTION_MAX_TTL_MS,
@@ -239,19 +239,19 @@ import {
   updateWorkspace,
   withSlackRetries,
   workspaceRepoCheckoutPath
-} from "./chunk-JFPY6XFF.js";
+} from "./chunk-QCGNHSMA.js";
 import {
   getConversationEventStore,
   getConversationMessageSearchStore,
   getConversationStore,
   getDb,
   getSqlExecutor
-} from "./chunk-43LODHJM.js";
+} from "./chunk-IUUQWIG2.js";
 import {
   juniorArtifacts,
   juniorIdentities,
   juniorSchedulerRuns
-} from "./chunk-VXL2LBPK.js";
+} from "./chunk-QVH6FW3T.js";
 import {
   SANDBOX_DATA_ROOT,
   SANDBOX_REPOS_ROOT,
@@ -282,12 +282,12 @@ import {
   throwSandboxOperationError,
   workspaceSnapshotNotReadyUserMessage,
   wrapSandboxSetupError
-} from "./chunk-HWXP6RHG.js";
+} from "./chunk-X55FOVZO.js";
 import {
   fenceLock,
   getStateAdapter,
   withLock
-} from "./chunk-DOP3FIQC.js";
+} from "./chunk-ZX7RSBQ6.js";
 import {
   KNOWN_CONVERSATION_EVENT_TYPES,
   contextProvenance,
@@ -320,7 +320,7 @@ import {
   resolveGatewayCredential,
   resolveGatewayModel,
   unwrapCurrentInstruction
-} from "./chunk-AD64YMEC.js";
+} from "./chunk-Q7O74WOO.js";
 import {
   ModelProfileNotConfiguredError,
   TURN_REASONING_LEVELS,
@@ -337,16 +337,16 @@ import {
   toGenAiMessagesTraceAttributes,
   toGenAiPayloadMetadata,
   toGenAiPayloadTraceAttributes
-} from "./chunk-FFYDRLXW.js";
+} from "./chunk-FIUCHYT6.js";
 import {
   discoverSkills,
   findSkillByName,
   loadSkillsByName,
   parseSkillInvocation
-} from "./chunk-HKFLRIBD.js";
+} from "./chunk-3HEZWDS5.js";
 import {
   pluginCatalogRuntime
-} from "./chunk-NRMAOGRS.js";
+} from "./chunk-NZLZ4KYZ.js";
 import {
   CredentialUnavailableError,
   OAuthProviderError,
@@ -354,7 +354,7 @@ import {
   getOAuthProviderErrorAttributes,
   resolveAuthTokenPlaceholder,
   resolvePluginCommandEnv
-} from "./chunk-64WF2GQ3.js";
+} from "./chunk-QIOUTXTZ.js";
 import {
   credentialContextForActor,
   credentialUserSubjectId
@@ -390,7 +390,7 @@ import {
   summarizeMessageText,
   withLogContext,
   withSpan
-} from "./chunk-3CVEE6LA.js";
+} from "./chunk-QIMN3FXL.js";
 import {
   startInactiveSpan,
   withActiveSpan
@@ -888,12 +888,13 @@ function buildTurnResult(input) {
   const rawPrimaryText = sanitizeAssistantText(
     terminalAssistantMessages.map((message) => extractAssistantText(message)).join("\n\n")
   ).trim();
-  const exactNoReplyMarker = isNoReplyMarker(rawPrimaryText);
-  const mixedNoReplyMarker = !exactNoReplyMarker && containsNoReplyMarker(rawPrimaryText);
-  const noReplyRequested = exactNoReplyMarker || mixedNoReplyMarker;
-  const primaryText = noReplyRequested ? "" : terminalAssistantMessages.map((message) => decideReply(message)).filter(
+  const primaryText = terminalAssistantMessages.map((message) => decideReply(message)).filter(
     (output) => output.kind === "deliver"
   ).map((output) => output.text).join("\n\n");
+  const terminalTexts = terminalAssistantMessages.map(
+    (message) => sanitizeAssistantText(extractAssistantText(message))
+  );
+  const noReplyRequested = !primaryText && terminalTexts.some((text) => isNoReplyMarker(text)) && terminalTexts.every((text) => !text || isNoReplyMarker(text));
   const toolErrorCount = toolResults.filter((result) => result.isError).length;
   const reactionPerformed = toolResults.some(
     (result) => !isToolResultError(result) && normalizeToolNameFromResult(result) === "addReaction"
@@ -903,7 +904,7 @@ function buildTurnResult(input) {
   const stopReason = typeof lastAssistant?.stopReason === "string" ? lastAssistant.stopReason : void 0;
   const errorMessage = typeof lastAssistant?.errorMessage === "string" ? lastAssistant.errorMessage : void 0;
   const isProviderError = stopReason === "error";
-  if (exactNoReplyMarker) {
+  if (noReplyRequested) {
     const markerCategory = reactionPerformed ? "reaction" : "none";
     const markerAttributes = {
       "app.ai.no_reply_marker": true,
@@ -913,7 +914,7 @@ function buildTurnResult(input) {
     if (!isProviderError) {
       logInfo("ai.no_reply_marker.accepted", markerAttributes);
     }
-  } else if (mixedNoReplyMarker) {
+  } else if (containsNoReplyMarker(rawPrimaryText)) {
     logWarn("ai.no_reply_marker.mixed_text", {
       "app.ai.no_reply_marker": true,
       "app.ai.no_reply_marker_mode": "mixed"
@@ -13504,7 +13505,7 @@ function createSearchConversationEventsTool(context) {
       const targetRootConversationId = await resolveRootConversationId(
         conversationStore,
         conversationId,
-        target.lineage?.parentConversationId
+        target.parentConversationId
       );
       const targetRoot = targetRootConversationId === conversationId ? target : await conversationStore.get({
         conversationId: targetRootConversationId
@@ -13542,7 +13543,7 @@ async function resolveAccessScope(conversationStore, context, currentConversatio
   const currentRootConversationId = current ? await resolveRootConversationId(
     conversationStore,
     currentConversationId,
-    current.lineage?.parentConversationId
+    current.parentConversationId
   ) : currentConversationId;
   if (context.source.platform === "slack") {
     return {
@@ -13580,10 +13581,10 @@ async function resolveRootConversationId(conversationStore, conversationId, pare
     if (!parent) {
       return conversationId;
     }
-    if (!parent.lineage?.parentConversationId) {
+    if (!parent.parentConversationId) {
       return parent.conversationId;
     }
-    cursor = parent.lineage.parentConversationId;
+    cursor = parent.parentConversationId;
   }
   return conversationId;
 }
@@ -17389,9 +17390,6 @@ async function wireAgentTools(args) {
   const runSource = args.run.source;
   const credentialUserId = args.run.credentialContext ? credentialUserSubjectId(args.run.credentialContext) : void 0;
   const userTokenStore = createUserTokenStore();
-  const conversation = await getConversationStore().get({
-    conversationId: args.run.conversationId
-  });
   const pluginHooks = createPluginHookRunner({
     actor: args.currentActor,
     actors: args.currentActors
@@ -17478,7 +17476,7 @@ async function wireAgentTools(args) {
   );
   const commonToolRuntimeContext = {
     conversationId: args.run.conversationId,
-    ...conversation?.location ? { locationId: conversation.location.id } : void 0,
+    ...args.run.source.location ? { locationId: args.run.source.location.id } : void 0,
     userText: args.userInput,
     configuration: args.configurationValues,
     egress: createPluginEgress({
@@ -19127,7 +19125,7 @@ async function executeAgentRunInPrivacyContext(run, conversationPrivacy, runLogC
         return;
       }
       try {
-        await delivery(message);
+        await delivery.send(message);
         acceptedToolFreeAssistant = true;
       } catch (error) {
         assistantMessageDeliveryError = new AssistantMessageDeliveryError(

package/dist/{chunk-EWRPUXE3.js → chunk-SC67CH3C.js}

@@ -1,15 +1,15 @@
 import {
   AgentRunError,
   executeTurn
-} from "./chunk-GQMQCY7H.js";
+} from "./chunk-ERZMACYA.js";
 import {
   ConversationTurnLifecycleService
-} from "./chunk-STOCCPWH.js";
+} from "./chunk-W42ENMJW.js";
 import {
   getPersistedSandboxState,
   getPersistedThreadState,
   persistThreadStateById
-} from "./chunk-CM6RTYSO.js";
+} from "./chunk-CIG6WI7Q.js";
 import {
   AgentInvocationBusyError,
   AgentInvocationLimitError,
@@ -18,32 +18,32 @@ import {
   agentBindingSchema,
   agentInvocationSchema,
   createAgentInvocationSchema
-} from "./chunk-TXLBLGNQ.js";
+} from "./chunk-6JZGXLL2.js";
 import {
   appendAndEnqueueInboundMessage
-} from "./chunk-2IBDVE62.js";
+} from "./chunk-MQDIIFZM.js";
 import {
   TurnInputCommitLostError,
   getAssistantReplyText,
   getConversationTurnBoundaryError,
   isTurnInputCommitLostError
-} from "./chunk-VOGOTEW7.js";
+} from "./chunk-Z5SKZE5B.js";
 import {
   failTurnRecord,
   getTerminalAssistantMessages,
   getTurnRecord,
   openConversationProjection,
   saveTurnCheckpoint
-} from "./chunk-H7X2G2KS.js";
+} from "./chunk-6YGFBU3A.js";
 import {
   getConversationEventStore,
   getConversationStore,
   getSqlExecutor
-} from "./chunk-43LODHJM.js";
+} from "./chunk-IUUQWIG2.js";
 import {
   juniorAgentBindings,
   juniorAgentInvocations
-} from "./chunk-VXL2LBPK.js";
+} from "./chunk-QVH6FW3T.js";
 
 // src/chat/agent-invocations/work.ts
 import { z } from "zod";
@@ -518,6 +518,7 @@ function createAgentInvocationWorker(agentRunner) {
     );
     let sandboxRef;
     let history;
+    let location;
     try {
       if (invocation.childConversationId !== context.conversationId) {
         throw new Error(
@@ -556,6 +557,9 @@ function createAgentInvocationWorker(agentRunner) {
       ]);
       sandboxRef = getPersistedSandboxState(persisted);
       history = projection.messages;
+      location = (await getConversationStore().get({
+        conversationId: invocation.parentConversationId
+      }))?.location;
     } catch (error) {
       if (isInvocationInputCommitLost(error)) {
         return { status: "lost_lease" };
@@ -591,12 +595,12 @@ function createAgentInvocationWorker(agentRunner) {
           destination: invocation.destination,
           destinationVisibility: invocation.destinationVisibility,
           publishExternally: context.publishExternally,
-          source: invocation.source,
+          source: location ? { ...invocation.source, location } : invocation.source,
           surface: "internal",
-          // TODO(#881, #883): Child runs may still need a path to force
-          // interactive auth when a delegated tool requires credentials the
-          // parent already has authority to request. Today background children
-          // hard-fail instead of pausing for an OAuth link.
+          // TODO(dcramer): Issues #881 and #883 track a path for child runs to
+          // force interactive auth when a delegated tool requires credentials
+          // the parent can request. Today background children hard-fail instead
+          // of pausing for an OAuth link.
           disabledFeatures: ["handoff", "interactive-auth", "subagents"],
           reasoning: invocation.reasoningLevel,
           state: {

package/dist/{chunk-VQZYIVYN.js → chunk-UVUXFRYN.js}

@@ -5,19 +5,19 @@ import {
   isDmChannel,
   normalizeSlackConversationId,
   withSlackRetries
-} from "./chunk-JFPY6XFF.js";
+} from "./chunk-QCGNHSMA.js";
 import {
   parseDestination
-} from "./chunk-43LODHJM.js";
+} from "./chunk-IUUQWIG2.js";
 import {
   getStateAdapter
-} from "./chunk-DOP3FIQC.js";
+} from "./chunk-ZX7RSBQ6.js";
 import {
   normalizeSlackEmojiName
-} from "./chunk-AD64YMEC.js";
+} from "./chunk-Q7O74WOO.js";
 import {
   pluginCatalogRuntime
-} from "./chunk-NRMAOGRS.js";
+} from "./chunk-NZLZ4KYZ.js";
 import {
   parseSlackMessageTs,
   parseSlackUserId
@@ -26,7 +26,7 @@ import {
   isRecord,
   logInfo,
   logWarn
-} from "./chunk-3CVEE6LA.js";
+} from "./chunk-QIMN3FXL.js";
 
 // src/chat/slack/outbound.ts
 var MAX_SLACK_MESSAGE_TEXT_CHARS = 4e4;