@sentry/junior 0.155.0 → 0.157.0

package/dist/{chunk-V57LZDHN.js → chunk-MGJW3SW4.js}

@@ -4,30 +4,30 @@ import {
   isConversationScopedChannel,
   isDmChannel,
   normalizeSlackConversationId
-} from "./chunk-TKJ7277U.js";
+} from "./chunk-MDZQ62PB.js";
 import {
   getConversationEventStore,
   getDb
-} from "./chunk-JJX77MB7.js";
+} from "./chunk-G2FN3I66.js";
 import {
   juniorConversationAnnotations,
   juniorConversationEvents,
   juniorIdentities,
   juniorUsers
-} from "./chunk-773XC7P4.js";
+} from "./chunk-3GS7O66Z.js";
 import {
   SANDBOX_WORKSPACE_ROOT
 } from "./chunk-EM225ITJ.js";
 import {
   getStateAdapter
-} from "./chunk-SDRJ2LE3.js";
+} from "./chunk-I3OQVVMV.js";
 import {
   botConfig,
   completeObject,
   embedTexts,
   getSlackBotToken,
   standardModelId
-} from "./chunk-XFZLQANQ.js";
+} from "./chunk-UJYA75N5.js";
 import {
   isActorUserId,
   parseActorUserId,
@@ -40,7 +40,7 @@ import {
   logInfo,
   logWarn,
   normalizeIdentityEmail
-} from "./chunk-7NYEB2TM.js";
+} from "./chunk-MKA3T2R6.js";
 
 // src/chat/plugins/agent-hooks.ts
 import {

package/dist/{chunk-7NYEB2TM.js → chunk-MKA3T2R6.js}

@@ -9,7 +9,7 @@ import {
 } from "./chunk-6GWA276C.js";
 import {
   getDeploymentTelemetryAttributes
-} from "./chunk-EORR34UB.js";
+} from "./chunk-F7VOY4VQ.js";
 
 // src/chat/logging.ts
 import path from "path";

package/dist/{chunk-RQ62VKM7.js → chunk-MXO6VFB3.js}

@@ -1,6 +1,6 @@
 import {
   createAndEnqueueAgentInvocation
-} from "./chunk-Y7ZUBP2K.js";
+} from "./chunk-7J65DOBL.js";
 import {
   actorFromRun,
   toolInvocationDestination

package/dist/{chunk-AENPRXCG.js → chunk-OWASYHHV.js}

@@ -7,7 +7,7 @@ import {
   PluginCredentialFailureError,
   agentNameSchema,
   createPluginAuthOrchestration
-} from "./chunk-CPQBTTI6.js";
+} from "./chunk-WNNYOPXX.js";
 import {
   authorizationId
 } from "./chunk-DIB4GX3G.js";
@@ -32,7 +32,7 @@ import {
   searchSlackUsers,
   selectPluginGrant,
   toMcpProviderError
-} from "./chunk-2AHVM3HK.js";
+} from "./chunk-AFQ47JQ7.js";
 import {
   RetryableDeliveryError,
   actorFromRun,
@@ -48,7 +48,7 @@ import {
   getSlackMessagePermalink,
   resolveBaseUrl,
   uploadFilesToConversation
-} from "./chunk-KBT2I5ER.js";
+} from "./chunk-VDAEXEBB.js";
 import {
   parseSandboxEgressAuthRequiredSignal,
   parseSandboxEgressPermissionDeniedSignal,
@@ -74,7 +74,7 @@ import {
   runNonInteractiveCommand,
   throwSandboxOperationError,
   wrapSandboxSetupError
-} from "./chunk-37G5HJ2X.js";
+} from "./chunk-GO2W3VZY.js";
 import {
   SCHEDULED_TASK_SYSTEM_ACTOR,
   compactEventTask,
@@ -105,11 +105,12 @@ import {
   resolveLocalScheduleAtMs,
   saveEventTask,
   scheduledTaskAttributes,
+  storeAttachments,
   writableEventTask
-} from "./chunk-LQT3SWFA.js";
+} from "./chunk-HJ3EZHIX.js";
 import {
   parseSlackThreadId
-} from "./chunk-APHDJYN7.js";
+} from "./chunk-VYDM7HUF.js";
 import {
   AGENTS_REMOVAL_NOTICE,
   AGENTS_REPLACEMENT_NOTICE,
@@ -179,7 +180,7 @@ import {
   stripRuntimeTurnContext,
   trimTrailingAssistantMessages,
   truncateText
-} from "./chunk-J2SYI36O.js";
+} from "./chunk-GEY6IFXH.js";
 import {
   JUNIOR_THREAD_STATE_TTL_MS
 } from "./chunk-3Y47VMHV.js";
@@ -195,7 +196,7 @@ import {
   getSlackToolContext,
   readActorIdentity,
   resolveChannelCapabilities
-} from "./chunk-V57LZDHN.js";
+} from "./chunk-MGJW3SW4.js";
 import {
   SlackActionError,
   downloadPrivateSlackFile,
@@ -204,7 +205,7 @@ import {
   isConversationScopedChannel,
   normalizeSlackConversationId,
   withSlackRetries
-} from "./chunk-TKJ7277U.js";
+} from "./chunk-MDZQ62PB.js";
 import {
   KNOWN_CONVERSATION_EVENT_TYPES,
   contextProvenance,
@@ -216,10 +217,10 @@ import {
   instructionActors,
   instructionProvenanceFor,
   sameActorIdentity
-} from "./chunk-JJX77MB7.js";
+} from "./chunk-G2FN3I66.js";
 import {
   juniorIdentities
-} from "./chunk-773XC7P4.js";
+} from "./chunk-3GS7O66Z.js";
 import {
   SANDBOX_DATA_ROOT,
   SANDBOX_SKILLS_ROOT,
@@ -231,7 +232,7 @@ import {
 } from "./chunk-EM225ITJ.js";
 import {
   getStateAdapter
-} from "./chunk-SDRJ2LE3.js";
+} from "./chunk-I3OQVVMV.js";
 import {
   ACTIVE_TURN_COMPACTION_SUMMARY_PREFIX,
   COMPACTION_SUMMARY_PREFIX,
@@ -264,7 +265,7 @@ import {
   resolveGatewayCredential,
   resolveGatewayModel,
   unwrapCurrentInstruction
-} from "./chunk-XFZLQANQ.js";
+} from "./chunk-UJYA75N5.js";
 import {
   canExposeConversationPayload,
   resolveConversationPrivacy,
@@ -274,16 +275,16 @@ import {
   toGenAiMessagesTraceAttributes,
   toGenAiPayloadMetadata,
   toGenAiPayloadTraceAttributes
-} from "./chunk-2WLJDM23.js";
+} from "./chunk-D7BNWRYL.js";
 import {
   discoverSkills,
   findSkillByName,
   loadSkillsByName,
   parseSkillInvocation
-} from "./chunk-JIC33O5L.js";
+} from "./chunk-RQBRGPUF.js";
 import {
   pluginCatalogRuntime
-} from "./chunk-TVXAN43I.js";
+} from "./chunk-6MI43QZO.js";
 import {
   CredentialUnavailableError,
   OAuthProviderError,
@@ -291,7 +292,7 @@ import {
   getOAuthProviderErrorAttributes,
   resolveAuthTokenPlaceholder,
   resolvePluginCommandEnv
-} from "./chunk-RN3AWYEP.js";
+} from "./chunk-77QB5FNI.js";
 import {
   credentialUserSubjectId
 } from "./chunk-BDUENQD3.js";
@@ -319,7 +320,7 @@ import {
   summarizeMessageText,
   withLogContext,
   withSpan
-} from "./chunk-7NYEB2TM.js";
+} from "./chunk-MKA3T2R6.js";
 import {
   startInactiveSpan,
   withActiveSpan
@@ -10538,12 +10539,13 @@ function createSlackMessageAddReactionTool(context, state) {
 import { createHash as createHash4 } from "crypto";
 import { z as z42 } from "zod";
 var sendFilesResultSchema = juniorToolOutputSchema.extend({
-  target: z42.string().min(1),
-  channel_id: z42.string().min(1),
   deduplicated: z42.boolean().optional(),
-  file_count: z42.number().int().nonnegative(),
-  file_ids: z42.array(z42.string().min(1)).optional(),
-  thread_ts: z42.string().min(1)
+  attachment_refs: z42.array(
+    z42.object({
+      id: z42.string().min(1),
+      name: z42.string().min(1)
+    })
+  )
 });
 function normalizeFiles(files) {
   return files.map((file) => ({
@@ -10564,7 +10566,7 @@ function fileOperationInput(files) {
     sha256: fileContentDigest(file.data)
   }));
 }
-function createSendFilesTool(context, state, materializeFile) {
+function createSendFilesTool(context, state, materializeFile, attachments) {
   return zodTool({
     annotations: {
       destructiveHint: false,
@@ -10606,22 +10608,26 @@ function createSendFilesTool(context, state, materializeFile) {
           deduplicated: true
         });
       }
+      const stored = attachments ? await storeAttachments({
+        conversationId: attachments.conversationId,
+        db: attachments.db,
+        files: materializedFiles,
+        storage: attachments.storage
+      }) : [];
       const uploads = materializedFiles.map((file) => ({
         data: file.data,
         filename: file.filename
       }));
-      const uploaded = await uploadFilesToConversation({
+      await uploadFilesToConversation({
         channelId: activeChannelId,
         files: uploads,
         threadTs
       });
-      const fileIds = uploaded?.files?.map((file) => file.id).filter((id) => Boolean(id));
       const response = {
-        target: `${activeChannelId}:${threadTs}`,
-        channel_id: activeChannelId,
-        thread_ts: threadTs,
-        file_count: uploads.length,
-        ...fileIds ? { file_ids: fileIds } : {}
+        attachment_refs: stored.map((attachment, index) => ({
+          id: attachment.id,
+          name: materializedFiles[index].filename
+        }))
       };
       state.setOperationResult(operationKey, response);
       return response;
@@ -13318,7 +13324,12 @@ function createTools(availableSkills, hooks = {}, context, options = {}) {
       tools.sendFiles = createSendFilesTool(
         slackContext,
         state,
-        (input) => readSandboxFileUpload(context.workspace, input)
+        (input) => readSandboxFileUpload(context.workspace, input),
+        context.conversationId && context.attachmentStorage ? {
+          conversationId: context.conversationId,
+          db: getSqlExecutor(),
+          storage: context.attachmentStorage
+        } : void 0
       );
     }
     tools.slackChannelListMessages = createSlackChannelListMessagesTool(slackContext);
@@ -14995,17 +15006,34 @@ async function requestBodyBytes(request) {
   }
   return await request.arrayBuffer();
 }
-function isGrantSelectionBodyVisible(input) {
-  return input.provider === "github" && input.upstreamUrl.hostname.toLowerCase() === "api.github.com" && input.upstreamUrl.pathname.toLowerCase().endsWith("/graphql");
+function githubBodyInspection(input) {
+  if (input.provider !== "github" || input.requestMethod.toUpperCase() !== "POST" || input.upstreamUrl.hostname.toLowerCase() !== "api.github.com") {
+    return void 0;
+  }
+  const pathname = input.upstreamUrl.pathname.toLowerCase();
+  if (pathname.endsWith("/graphql")) {
+    return "graphql";
+  }
+  return /^\/repos\/[^/]+\/[^/]+\/pulls\/[^/]+\/reviews(?:\/[^/]+\/events)?$/.test(
+    pathname
+  ) ? "pull-request-review" : void 0;
+}
+function grantSelectionBodyTooLargeMessage(inspection) {
+  return inspection === "graphql" ? "GitHub GraphQL request body is too large for Junior to inspect before issuing credentials." : "GitHub pull request review request body is too large for Junior to inspect before issuing credentials.";
 }
 function grantSelectionBodyText(input) {
   if (input.body === void 0) {
     return void 0;
   }
   if (input.body.byteLength > GRANT_SELECTION_BODY_TEXT_LIMIT_BYTES) {
-    if (!input.operation && input.provider === "github" && input.request.method.toUpperCase() === "POST" && input.upstreamUrl.hostname.toLowerCase() === "api.github.com" && input.upstreamUrl.pathname.toLowerCase().endsWith("/graphql")) {
+    const inspection = githubBodyInspection({
+      provider: input.provider,
+      requestMethod: input.request.method,
+      upstreamUrl: input.upstreamUrl
+    });
+    if (inspection) {
       throw new EgressPolicyDenied(
-        "GitHub GraphQL request body is too large for Junior to inspect before issuing credentials."
+        grantSelectionBodyTooLargeMessage(inspection)
       );
     }
     return void 0;
@@ -15163,8 +15191,9 @@ async function executeCredentialedEgressRequest(input) {
     request,
     upstreamUrl
   } = input;
-  const bodyForGrantSelection = isGrantSelectionBodyVisible({
+  const bodyForGrantSelection = githubBodyInspection({
     provider,
+    requestMethod: request.method,
     upstreamUrl
   }) ? await requestBodyBytes(request) : void 0;
   let grantSelection;
@@ -15172,7 +15201,6 @@ async function executeCredentialedEgressRequest(input) {
     grantSelection = await selectSandboxEgressGrant({
       bodyText: grantSelectionBodyText({
         body: bodyForGrantSelection,
-        ...operation ? { operation } : {},
         provider,
         request,
         upstreamUrl
@@ -15940,7 +15968,8 @@ async function wireAgentTools(args) {
   });
   const toolRuntimeContext = {
     ...commonToolRuntimeContext,
-    ...toolRoute
+    ...toolRoute,
+    attachmentStorage: args.run.environment?.attachmentStorage
   };
   const actionReview = createToolActionReview({
     context: {

package/dist/{chunk-UWTXAYBP.js → chunk-PP4UQTH3.js}

@@ -2,7 +2,7 @@ import {
   coerceThreadConversationState,
   getPersistedThreadState,
   persistThreadStateById
-} from "./chunk-4K5N6HTX.js";
+} from "./chunk-5TIH5GOE.js";
 import {
   pluginTaskId,
   pluginTaskParamsSchema,
@@ -22,7 +22,7 @@ import {
   markTurnCompleted,
   normalizeToolNameFromResult,
   stripRuntimeTurnContext
-} from "./chunk-J2SYI36O.js";
+} from "./chunk-GEY6IFXH.js";
 import {
   JUNIOR_THREAD_STATE_TTL_MS
 } from "./chunk-3Y47VMHV.js";
@@ -33,25 +33,25 @@ import {
   createPluginModel,
   createPluginState,
   getPlugins
-} from "./chunk-V57LZDHN.js";
+} from "./chunk-MGJW3SW4.js";
 import {
   getConversationStore,
   getDb,
   sameActorIdentity
-} from "./chunk-JJX77MB7.js";
+} from "./chunk-G2FN3I66.js";
 import {
   juniorConversations,
   juniorTaskExecutions
-} from "./chunk-773XC7P4.js";
+} from "./chunk-3GS7O66Z.js";
 import {
   getStateAdapter
-} from "./chunk-SDRJ2LE3.js";
+} from "./chunk-I3OQVVMV.js";
 import {
   ProviderError,
   findProviderError,
   getProviderErrorAttributes,
   getProviderErrorUserMessage
-} from "./chunk-XFZLQANQ.js";
+} from "./chunk-UJYA75N5.js";
 import {
   credentialSubjectSchema
 } from "./chunk-BDUENQD3.js";
@@ -61,7 +61,7 @@ import {
 import {
   buildTurnFailureResponse,
   logWarn
-} from "./chunk-7NYEB2TM.js";
+} from "./chunk-MKA3T2R6.js";
 
 // src/chat/runtime/delivered-turn-state.ts
 function buildDeliveredTurnStatePatch(args) {
@@ -87,7 +87,7 @@ import { pluginRunContextSchema } from "@sentry/junior-plugin-api";
 
 // src/chat/services/turn-session-routing.ts
 async function resolveTurnSessionRouting(args) {
-  const conversationStore = args.conversationStore ?? (await import("./db-WT7P56AH.js")).getConversationStore();
+  const conversationStore = args.conversationStore ?? (await import("./db-VQAV4PJ3.js")).getConversationStore();
   const conversation = await conversationStore.get({
     conversationId: args.conversationId
   });

package/dist/{chunk-JIC33O5L.js → chunk-RQBRGPUF.js}

@@ -1,9 +1,9 @@
 import {
   pluginCatalogRuntime
-} from "./chunk-TVXAN43I.js";
+} from "./chunk-6MI43QZO.js";
 import {
   logWarn
-} from "./chunk-7NYEB2TM.js";
+} from "./chunk-MKA3T2R6.js";
 import {
   skillRoots
 } from "./chunk-H4HBKISO.js";

package/dist/{chunk-NWSKFYOH.js → chunk-S4PFOWRA.js}

@@ -1,7 +1,7 @@
 import {
   FUNCTION_TIMEOUT_BUFFER_SECONDS,
   getChatConfig
-} from "./chunk-XFZLQANQ.js";
+} from "./chunk-UJYA75N5.js";
 
 // src/chat/runtime/request-deadline.ts
 import { AsyncLocalStorage } from "async_hooks";

package/dist/{chunk-NKFRRZIT.js → chunk-TA5R6BME.js}

@@ -3,15 +3,15 @@ import {
 } from "./chunk-FA6POGLH.js";
 import {
   getTurnRequestDeadline
-} from "./chunk-NWSKFYOH.js";
+} from "./chunk-S4PFOWRA.js";
 import {
   buildSentryConversationUrl,
   deleteWebAuthorization
-} from "./chunk-KTVHAHUV.js";
+} from "./chunk-4M25PSFB.js";
 import {
   ensureConversationWake,
   requestConversationWork
-} from "./chunk-ELOTIL5F.js";
+} from "./chunk-BLCVTEE7.js";
 import {
   buildDeliveredTurnStatePatch,
   finalizeFailedTurnReplyWithEvent,
@@ -24,7 +24,7 @@ import {
   requireTurnFailureEventId,
   resolveTurnSessionRouting,
   scheduleSessionCompletedPluginTasks
-} from "./chunk-UWTXAYBP.js";
+} from "./chunk-PP4UQTH3.js";
 import {
   ConversationTurnLifecycleService,
   coerceThreadConversationState,
@@ -33,7 +33,7 @@ import {
   getPersistedSandboxState,
   getPersistedThreadState,
   persistThreadStateById
-} from "./chunk-4K5N6HTX.js";
+} from "./chunk-5TIH5GOE.js";
 import {
   getVercelConversationWorkQueue
 } from "./chunk-WR76IVA5.js";
@@ -47,7 +47,7 @@ import {
   lookupSlackUserProfile,
   resolvePluginOAuthAccount,
   runAgentWithTimeout
-} from "./chunk-2AHVM3HK.js";
+} from "./chunk-AFQ47JQ7.js";
 import {
   RetryableDeliveryError
 } from "./chunk-RF6J3DIO.js";
@@ -63,13 +63,13 @@ import {
   removeReactionFromMessage,
   resolveBaseUrl,
   truncateStatusText
-} from "./chunk-KBT2I5ER.js";
+} from "./chunk-VDAEXEBB.js";
 import {
   parseSlackThreadId,
   readSlackRawMessageContext,
   resolveSlackChannelIdFromMessage,
   resolveSlackChannelIdFromThreadId
-} from "./chunk-APHDJYN7.js";
+} from "./chunk-VYDM7HUF.js";
 import {
   CooperativeTurnYieldError,
   TurnSliceLimitExceededError,
@@ -96,29 +96,29 @@ import {
   recordTurnSummary,
   saveTurnCheckpoint,
   turnHasReply
-} from "./chunk-J2SYI36O.js";
+} from "./chunk-GEY6IFXH.js";
 import {
   getDashboardConversationLink,
   getPluginSlackConversationLink
-} from "./chunk-V57LZDHN.js";
+} from "./chunk-MGJW3SW4.js";
 import {
   SlackActionError,
   getHeaderString,
   getSlackClient,
   isDmChannel,
   normalizeSlackConversationId
-} from "./chunk-TKJ7277U.js";
+} from "./chunk-MDZQ62PB.js";
 import {
   getConversationEventStore,
   getSqlExecutor,
   requireSlackDestination,
   upsertIdentity,
   upsertLinkedIdentity
-} from "./chunk-JJX77MB7.js";
+} from "./chunk-G2FN3I66.js";
 import {
   acquireActiveLock,
   getStateAdapter
-} from "./chunk-SDRJ2LE3.js";
+} from "./chunk-I3OQVVMV.js";
 import {
   botConfig,
   escapeXml,
@@ -126,18 +126,18 @@ import {
   getRuntimeMetadata,
   normalizeSlackEmojiName,
   standardModelId
-} from "./chunk-XFZLQANQ.js";
+} from "./chunk-UJYA75N5.js";
 import {
   discoverSkills
-} from "./chunk-JIC33O5L.js";
+} from "./chunk-RQBRGPUF.js";
 import {
   pluginCatalogRuntime
-} from "./chunk-TVXAN43I.js";
+} from "./chunk-6MI43QZO.js";
 import {
   buildOAuthTokenRequest,
   hasRequiredOAuthScope,
   parseOAuthTokenResponse
-} from "./chunk-RN3AWYEP.js";
+} from "./chunk-77QB5FNI.js";
 import {
   createSlackResumeActor,
   isUserActor,
@@ -153,7 +153,7 @@ import {
   setTags,
   toOptionalString,
   withLogContext
-} from "./chunk-7NYEB2TM.js";
+} from "./chunk-MKA3T2R6.js";
 import {
   homeDir
 } from "./chunk-H4HBKISO.js";
@@ -1432,6 +1432,7 @@ async function resumeSlackTurnInContext(args) {
       const recordedMessageId = recordDeliveredAssistantMessage({
         conversation: deliveryState2.conversation,
         sessionId: deliveryState2.sessionId,
+        source: "slack",
         text,
         userMessageId: deliveryState2.userMessageId
       });

package/dist/{chunk-XFZLQANQ.js → chunk-UJYA75N5.js}

@@ -3,7 +3,7 @@ import {
   toCanonicalInputMessage,
   toCanonicalOutputMessage,
   toGenAiMessagesTraceAttributes
-} from "./chunk-2WLJDM23.js";
+} from "./chunk-D7BNWRYL.js";
 import {
   extractGenAiUsageAttributes,
   logException,
@@ -11,7 +11,7 @@ import {
   normalizeGenAiFinishReason,
   serializeGenAiAttribute,
   withSpan
-} from "./chunk-7NYEB2TM.js";
+} from "./chunk-MKA3T2R6.js";
 
 // src/chat/services/provider-error.ts
 var ProviderError = class extends Error {