@sentry/junior 0.200.0 → 0.201.0

package/dist/app.js

@@ -1,15 +1,15 @@
 import {
   processConversationWork
-} from "./chunk-M7BHRB3G.js";
+} from "./chunk-VRD7FJZ2.js";
 import {
   bindSpawnAgent
-} from "./chunk-LICM66W2.js";
+} from "./chunk-4DDQVL3Q.js";
 import {
   createAgentInvocationWorker,
   enqueueAgentInvocation,
   listPendingAgentInvocationMailboxAppends,
   resolveAgentInvocationId
-} from "./chunk-EOOWKTQC.js";
+} from "./chunk-ZXSPPXX3.js";
 import {
   GET as GET2,
   GET2 as GET3,
@@ -37,12 +37,12 @@ import {
   startProcessingReaction,
   stripLeadingBotMention,
   wakePausedTurn
-} from "./chunk-V2VL335Y.js";
+} from "./chunk-GHZ677GD.js";
 import "./chunk-FA6POGLH.js";
 import {
   AgentRunError,
   executeTurn
-} from "./chunk-YVP6QNBZ.js";
+} from "./chunk-KS5RPVQI.js";
 import {
   buildDeliveredTurnStatePatch,
   completeAuthPauseTurn,
@@ -53,7 +53,7 @@ import {
   getTurnUserSlackMessageTs,
   persistAuthPauseTurnState,
   requireTurnFailureEventId
-} from "./chunk-ODSGDVHT.js";
+} from "./chunk-T4WKGKAW.js";
 import {
   claimDispatchMailboxAppend,
   confirmDispatchMailboxAppend,
@@ -73,11 +73,11 @@ import {
   processPluginTask,
   resolveTurnSessionRouting,
   scheduleSessionCompletedPluginTasks
-} from "./chunk-LUITHZZL.js";
+} from "./chunk-VQ2XYXLB.js";
 import {
   ConversationTurnLifecycleService,
   getTurnLifecycle
-} from "./chunk-5JDNDQHL.js";
+} from "./chunk-6CYKW2J7.js";
 import {
   coerceThreadConversationState,
   getLocationConfigurationService,
@@ -86,7 +86,7 @@ import {
   persistThreadRuntimeState,
   persistThreadState,
   persistThreadStateById
-} from "./chunk-ZPLAAHVP.js";
+} from "./chunk-IXGQ4NTO.js";
 import {
   GET,
   appendAndEnqueueWebMessage,
@@ -101,24 +101,24 @@ import {
   resolveSlackChannelTypeFromMessage,
   resolveSlackConversationContext,
   webActorFromEmail
-} from "./chunk-ADNSL3MX.js";
+} from "./chunk-7YNSWPVU.js";
 import {
   createWebAuthorization,
   deleteWebAuthorization
-} from "./chunk-752M26LC.js";
+} from "./chunk-RAWQD5DS.js";
 import {
   recordTaskExecution
-} from "./chunk-PSQ4SWI3.js";
+} from "./chunk-J4AWOSCT.js";
 import "./chunk-RC745GGZ.js";
 import {
   createVercelPluginTaskCallback,
   queueCallback,
   registerVercelPluginTaskDevConsumer
-} from "./chunk-AV552V2K.js";
+} from "./chunk-OZ2QJWQF.js";
 import {
   validatePluginEgressCredentialHooks,
   validatePluginRegistrations
-} from "./chunk-TF5TXGRD.js";
+} from "./chunk-APX6WBD7.js";
 import {
   defineJuniorPlugins,
   pluginCatalogConfigFromEnv,
@@ -141,6 +141,7 @@ import {
   ingestResourceEvent,
   isResourceEventConversationMessage,
   isResourceEventMailboxMetadata,
+  isVisionImageMediaType,
   latestReportedProgress,
   listThreadReplies,
   logScheduledTaskRunSkipped,
@@ -149,8 +150,10 @@ import {
   parseSandboxEgressCredentialToken,
   processWorkspaceSnapshotJob,
   publicArtifactHeaders,
+  readAttachmentVisionSummary,
   readPublicArtifact,
   recordFinishedTurnForAutomatedLimit,
+  recordSlackAttachment,
   renderAttachmentText,
   renderBlockText,
   renderTaskInput,
@@ -160,11 +163,11 @@ import {
   shouldEmitDevAgentTrace,
   shouldPropagateSandboxEgressTrace,
   unpublishedArtifactHeaders
-} from "./chunk-X2Q4H6JQ.js";
+} from "./chunk-X2PTNIN5.js";
 import {
   AuthorizationFlowDisabledError,
   PluginCredentialFailureError
-} from "./chunk-VLBLCJXK.js";
+} from "./chunk-ZPGGVW34.js";
 import "./chunk-6Y5B5PEX.js";
 import "./chunk-DIB4GX3G.js";
 import {
@@ -172,14 +175,14 @@ import {
   deleteMcpAuthSessionsForUserProvider,
   deleteMcpServerSessionId,
   deleteMcpStoredOAuthCredentials
-} from "./chunk-22Y5SBDM.js";
+} from "./chunk-5LXHEN3P.js";
 import {
   RetryableDeliveryError,
   createAgentRunner,
   formatProviderLabel,
   postSlackMessage,
   startOAuthFlow
-} from "./chunk-PIHLVU7T.js";
+} from "./chunk-3XPQV262.js";
 import {
   ConversationTurnBoundaryError,
   CooperativeTurnYieldError,
@@ -195,7 +198,7 @@ import {
   markTurnCompleted,
   markTurnFailed,
   startActiveTurn
-} from "./chunk-3WUBDFFL.js";
+} from "./chunk-UEIDAG2O.js";
 import {
   abandonTurnRecord,
   appendConversationMessages,
@@ -233,7 +236,7 @@ import {
   trimTrailingAssistantMessages,
   turnHasReply,
   upsertConversationMessage
-} from "./chunk-JLUCKXGI.js";
+} from "./chunk-6QQYL2XW.js";
 import {
   getExperimentalFeatures,
   isExperimentalFeatureEnabled,
@@ -252,11 +255,12 @@ import {
   saveScheduledTaskInLock,
   scheduledRunId,
   scheduledRunSchema,
+  storeAttachment,
   withScheduledTaskLock
-} from "./chunk-IAZBUDVU.js";
+} from "./chunk-Q3BTNRDK.js";
 import {
   parseSlackThreadId
-} from "./chunk-RG74Q34N.js";
+} from "./chunk-LINEEYW6.js";
 import {
   CONVERSATION_WORK_STALE_ENQUEUE_MS,
   appendAndEnqueueInboundMessage,
@@ -271,16 +275,16 @@ import {
   removeActiveConversation,
   requestConversationWork,
   stopConversationWork
-} from "./chunk-UR43NMUR.js";
+} from "./chunk-BQFWG7MB.js";
 import {
   buildDeterministicAssistantMessageId,
   buildDeterministicTurnId
 } from "./chunk-DA2KGZZI.js";
-import "./chunk-RDWWCDA4.js";
+import "./chunk-73RVLY6A.js";
 import {
   getSandboxResourceConfig,
   setSandboxResourceConfig
-} from "./chunk-OIILBD56.js";
+} from "./chunk-I7F3JVYE.js";
 import "./chunk-PDO5BLNM.js";
 import "./chunk-UIE3R5XU.js";
 import {
@@ -289,7 +293,7 @@ import {
 } from "./chunk-X6GDDWQH.js";
 import {
   getTurnRequestDeadline
-} from "./chunk-4Q5IR5GA.js";
+} from "./chunk-FNAEGMGB.js";
 import {
   getVercelConversationWorkQueue,
   resolveConversationWorkQueueTopic,
@@ -310,6 +314,7 @@ import {
   createPluginLogger,
   createPluginState,
   downloadPrivateSlackFile,
+  findMatchingResourceEventSubscriptions,
   getPluginApiRoutes,
   getPluginRoutes,
   getPlugins,
@@ -325,7 +330,7 @@ import {
   verifyEventTaskCredentialSubject,
   verifyScheduledTaskCredentialSubject,
   verifySlackDirectCredentialSubject
-} from "./chunk-N5VHBUXF.js";
+} from "./chunk-JYJG6LAG.js";
 import {
   JUNIOR_THREAD_STATE_TTL_MS
 } from "./chunk-3Y47VMHV.js";
@@ -339,7 +344,7 @@ import {
   requireSlackDestination,
   withConversationEventLock,
   withConversationMutationLock
-} from "./chunk-JMRHEW6B.js";
+} from "./chunk-VGNL3QUM.js";
 import {
   juniorAgentBindings,
   juniorAgentInvocations,
@@ -347,15 +352,15 @@ import {
   juniorConversations,
   juniorSchedulerRuns,
   juniorSchedulerTasks
-} from "./chunk-K2QYWZ7F.js";
-import "./chunk-OVHO76WN.js";
+} from "./chunk-UJRD4NN4.js";
+import "./chunk-KKI5K43Z.js";
 import {
   MUTATION_LOCK_TTL_MS,
   acquireActiveLock,
   getStateAdapter,
   withActiveLock,
   withLock
-} from "./chunk-PMSENMWY.js";
+} from "./chunk-44W7JB5V.js";
 import {
   instructionProvenanceFor
 } from "./chunk-7NL7T6VU.js";
@@ -378,17 +383,17 @@ import {
   setBotModelConfig,
   setProfiles,
   setSlackReactionConfig
-} from "./chunk-GFXDMDHR.js";
+} from "./chunk-PJIU4AGA.js";
 import {
   defaultModelId,
   modelIdForProfile
 } from "./chunk-NHJU2ZAL.js";
-import "./chunk-SY2RQ77O.js";
-import "./chunk-T6263C3N.js";
+import "./chunk-QHCI2E2A.js";
+import "./chunk-7V4KWIXL.js";
 import {
   pluginCatalogRuntime
-} from "./chunk-5VRPXF4T.js";
-import "./chunk-5Q6KP6LH.js";
+} from "./chunk-Y6EUAX5O.js";
+import "./chunk-H5SDC32F.js";
 import {
   credentialContextForActor
 } from "./chunk-CEMMLV5L.js";
@@ -424,13 +429,13 @@ import {
   withContext,
   withLogContext,
   withSpan
-} from "./chunk-7DO36XLF.js";
+} from "./chunk-SIMN6AUL.js";
 import {
   continueTrace
 } from "./chunk-PEQXYSSE.js";
 import {
   JUNIOR_VERSION
-} from "./chunk-OTTKTTVI.js";
+} from "./chunk-NN5Y32TJ.js";
 
 // src/app.ts
 import { createRequire } from "module";
@@ -5104,13 +5109,9 @@ function hasPotentialImageAttachment(attachments) {
   return countPotentialImageAttachments(attachments) > 0;
 }
 function countPotentialImageAttachments(attachments) {
-  return attachments?.filter((attachment) => {
-    if (attachment.type === "image") {
-      return true;
-    }
-    const mimeType = attachment.mimeType ?? "";
-    return attachment.type === "file" && mimeType.startsWith("image/");
-  }).length ?? 0;
+  return attachments?.filter(
+    (attachment) => isVisionImageMediaType(attachment.mimeType ?? "")
+  ).length ?? 0;
 }
 function isVisionEnabled() {
   return Boolean(botConfig.visionModelId);
@@ -5126,6 +5127,10 @@ function buildImageAttachmentPromptText(args) {
     "<image-attachment>",
     `filename: ${args.filename ?? "unnamed"}`,
     `media_type: ${args.mediaType}`,
+    ...args.attachmentId ? [
+      `attachment_id: ${args.attachmentId}`,
+      "Use loadAttachment with this id when the task needs the original file."
+    ] : [],
     "<summary>",
     args.summary,
     "</summary>",
@@ -5137,9 +5142,12 @@ function buildImageAttachmentFailurePromptText(args) {
     "<image-attachment>",
     `filename: ${args.filename ?? "unnamed"}`,
     `media_type: ${args.mediaType}`,
+    ...args.attachmentId ? [
+      `attachment_id: ${args.attachmentId}`,
+      "Use loadAttachment with this id to access the original file."
+    ] : [],
     "<analysis-error>",
     args.message,
-    "The image contents are unavailable.",
     "</analysis-error>",
     "</image-attachment>"
   ].join("\n");
@@ -5182,6 +5190,24 @@ async function summarizeImageWithVision(args) {
 function truncateVisionSummary(summary) {
   return summary.slice(0, MAX_VISION_SUMMARY_CHARS);
 }
+function buildStoredAttachmentPromptText(args) {
+  return [
+    "<attachment>",
+    `filename: ${args.filename ?? "unnamed"}`,
+    `media_type: ${args.mediaType}`,
+    `attachment_id: ${args.attachmentId}`,
+    "Use loadAttachment with this id to access the original file.",
+    "</attachment>"
+  ].join("\n");
+}
+function getSlackFileId(attachment) {
+  const metadataId = attachment.fetchMetadata?.fileId;
+  if (metadataId) return metadataId;
+  const match = (attachment.url ?? attachment.fetchMetadata?.url ?? "").match(
+    /(?:^|[-/])(F[A-Z0-9]+)(?:[-/]|$)/i
+  );
+  return match?.[1];
+}
 function getCachedImageSummaries(args) {
   if (!args.conversation || !args.messageTs) {
     return [];
@@ -5213,55 +5239,101 @@ async function resolveUserAttachmentsWithDeps(attachments, context, deps) {
   });
   let nextCachedImageSummaryIndex = 0;
   for (const attachment of attachments) {
-    if (results.length >= MAX_USER_ATTACHMENTS) break;
-    if (attachment.type !== "image" && attachment.type !== "file") continue;
     const mediaType = attachment.mimeType ?? "application/octet-stream";
-    const isImageAttachment = attachment.type === "image" || mediaType.startsWith("image/");
-    if (isImageAttachment && !isVisionEnabled()) {
+    const isImageAttachment = isVisionImageMediaType(mediaType);
+    let data;
+    try {
+      if (attachment.fetchData) {
+        data = await attachment.fetchData();
+      } else if (attachment.data instanceof Buffer) {
+        data = attachment.data;
+      } else {
+        continue;
+      }
+    } catch (error) {
+      logWarn("attachment.resolution.failed", {
+        "exception.message": error instanceof Error ? error.message : String(error),
+        "app.file.mime_type": mediaType
+      });
       continue;
     }
-    try {
-      const resolvedAttachment = {
-        mediaType,
-        filename: attachment.name
-      };
-      if (isImageAttachment) {
-        const cachedSummary = cachedImageSummaries[nextCachedImageSummaryIndex];
-        nextCachedImageSummaryIndex += 1;
-        if (cachedSummary) {
-          resolvedAttachment.promptText = buildImageAttachmentPromptText({
-            filename: attachment.name,
-            mediaType,
-            summary: cachedSummary
-          });
-          results.push(resolvedAttachment);
-          continue;
-        }
-        let imageData = null;
-        if (attachment.fetchData) {
-          imageData = await attachment.fetchData();
-        } else if (attachment.data instanceof Buffer) {
-          imageData = attachment.data;
-        }
-        if (!imageData) {
-          throw createImageAttachmentProcessingError({
-            filename: attachment.name
-          });
-        }
-        if (imageData.byteLength > MAX_USER_ATTACHMENT_BYTES) {
-          throw createImageAttachmentProcessingError({
-            filename: attachment.name
-          });
-        }
+    let attachmentId;
+    const slackFileId = getSlackFileId(attachment);
+    if (context.conversationId && deps.attachmentStorage) {
+      try {
+        const stored = await storeAttachment({
+          conversationId: context.conversationId,
+          db: getSqlExecutor(),
+          file: {
+            bytes: data.byteLength,
+            data,
+            filename: attachment.name ?? "attachment",
+            mimeType: mediaType,
+            path: attachment.name ?? "attachment"
+          },
+          ...slackFileId ? { source: { id: slackFileId, provider: "slack" } } : void 0,
+          storage: deps.attachmentStorage
+        });
+        attachmentId = stored.id;
+      } catch (error) {
+        logWarn("attachment.storage.failed", {
+          "exception.message": error instanceof Error ? error.message : String(error),
+          "app.file.mime_type": mediaType,
+          ...attachment.name ? { "file.name": attachment.name } : void 0
+        });
+      }
+    }
+    if (results.length >= MAX_USER_ATTACHMENTS || attachment.type !== "image" && attachment.type !== "file") {
+      continue;
+    }
+    const resolvedAttachment = {
+      attachmentId,
+      mediaType,
+      filename: attachment.name
+    };
+    if (data.byteLength > MAX_USER_ATTACHMENT_BYTES) {
+      logWarn("attachment.size_limit.skipped", {
+        "file.size": data.byteLength,
+        "app.file.mime_type": mediaType
+      });
+      if (attachmentId) {
+        resolvedAttachment.promptText = buildStoredAttachmentPromptText({
+          attachmentId,
+          filename: attachment.name,
+          mediaType
+        });
+        results.push(resolvedAttachment);
+      }
+      continue;
+    }
+    if (isImageAttachment && isVisionEnabled()) {
+      const storedVisionSummary = attachmentId ? await readAttachmentVisionSummary({
+        attachmentId,
+        conversationId: context.conversationId,
+        db: getSqlExecutor()
+      }) : void 0;
+      const cachedSummary = storedVisionSummary ?? cachedImageSummaries[nextCachedImageSummaryIndex];
+      nextCachedImageSummaryIndex += 1;
+      if (cachedSummary) {
+        resolvedAttachment.promptText = buildImageAttachmentPromptText({
+          attachmentId,
+          filename: attachment.name,
+          mediaType,
+          summary: cachedSummary
+        });
+        results.push(resolvedAttachment);
+        continue;
+      }
+      try {
         const summary = await summarizeImageWithVision({
           completeText: deps.completeText,
-          imageData,
+          imageData: data,
           mimeType: mediaType,
           maxTokens: 220,
           prompt: [
-            "Extract concise, factual context from this user-provided image.",
-            "Focus on visible text, UI state, charts, diagrams, errors, names, and other concrete details useful for answering the user's current request.",
-            "Do not speculate.",
+            "Describe this image with concise facts that help answer the user's request.",
+            "Include visible text, UI state, charts, diagrams, errors, and names.",
+            "Do not guess.",
             "Return plain text only."
           ].join(" "),
           metadata: {
@@ -5277,32 +5349,22 @@ async function resolveUserAttachmentsWithDeps(attachments, context, deps) {
             filename: attachment.name
           });
         }
+        const truncatedSummary = truncateVisionSummary(summary);
         resolvedAttachment.promptText = buildImageAttachmentPromptText({
+          attachmentId,
           filename: attachment.name,
           mediaType,
-          summary: truncateVisionSummary(summary)
+          summary: truncatedSummary
         });
-        results.push(resolvedAttachment);
-        continue;
-      }
-      let data = null;
-      if (attachment.fetchData) {
-        data = await attachment.fetchData();
-      } else if (attachment.data instanceof Buffer) {
-        data = attachment.data;
-      }
-      if (!data) continue;
-      if (data.byteLength > MAX_USER_ATTACHMENT_BYTES) {
-        logWarn("attachment.size_limit.skipped", {
-          "file.size": data.byteLength,
-          "app.file.mime_type": mediaType
-        });
-        continue;
-      }
-      resolvedAttachment.data = data;
-      results.push(resolvedAttachment);
-    } catch (error) {
-      if (isImageAttachment) {
+        if (attachmentId) {
+          await recordSlackAttachment({
+            attachmentId,
+            conversationId: context.conversationId,
+            db: getSqlExecutor(),
+            visionSummary: truncatedSummary
+          });
+        }
+      } catch (error) {
         const attachmentError = error instanceof ImageAttachmentProcessingError ? error : createImageAttachmentProcessingError({
           filename: attachment.name
         });
@@ -5311,22 +5373,26 @@ async function resolveUserAttachmentsWithDeps(attachments, context, deps) {
           "app.file.mime_type": mediaType,
           ...attachment.name ? { "file.name": attachment.name } : void 0
         });
-        results.push({
-          mediaType,
+        resolvedAttachment.promptText = buildImageAttachmentFailurePromptText({
+          attachmentId,
           filename: attachment.name,
-          promptText: buildImageAttachmentFailurePromptText({
-            filename: attachment.name,
-            mediaType,
-            message: attachmentError.message
-          })
+          mediaType,
+          message: attachmentError.message
         });
-        continue;
       }
-      logWarn("attachment.resolution.failed", {
-        "exception.message": error instanceof Error ? error.message : String(error),
-        "app.file.mime_type": mediaType
+      results.push(resolvedAttachment);
+      continue;
+    }
+    if (isImageAttachment && attachmentId) {
+      resolvedAttachment.promptText = buildStoredAttachmentPromptText({
+        attachmentId,
+        filename: attachment.name,
+        mediaType
       });
+    } else {
+      resolvedAttachment.data = data;
     }
+    results.push(resolvedAttachment);
   }
   return results;
 }
@@ -5430,7 +5496,7 @@ async function hydrateConversationVisionContextWithDeps(conversation, context, d
     const imageFiles = (reply.files ?? []).filter((file) => {
       const mimeType = toOptionalString(file.mimetype);
       return Boolean(
-        toOptionalString(file.id) && mimeType?.startsWith("image/")
+        toOptionalString(file.id) && mimeType && isVisionImageMediaType(mimeType)
       );
     }).slice(0, MAX_MESSAGE_IMAGE_ATTACHMENTS);
     if (imageFiles.length === 0) {
@@ -6622,6 +6688,7 @@ function createJuniorRuntimeServices(overrides = {}) {
     autoCompactionTriggerTokens: overrides.contextCompactor?.autoCompactionTriggerTokens
   });
   const visionContext = createVisionContextService({
+    attachmentStorage: overrides.visionContext?.attachmentStorage,
     completeText: overrides.visionContext?.completeText ?? completeText,
     listThreadReplies: overrides.visionContext?.listThreadReplies ?? listThreadReplies,
     downloadFile: overrides.visionContext?.downloadFile ?? downloadPrivateSlackFile
@@ -7076,6 +7143,7 @@ function createSlackTurn(deps) {
                 timestampMs: queued.message.metadata.dateSent.getTime(),
                 omittedImageAttachmentCount: !isVisionEnabled() && hasPotentialImageAttachment(attachments) ? countPotentialImageAttachments(attachments) : 0,
                 attachments: await deps.resolveUserAttachments(attachments, {
+                  conversationId,
                   threadId,
                   actorId: queued.message.author.userId,
                   channelId,
@@ -7301,6 +7369,7 @@ function createSlackTurn(deps) {
         const userAttachments = await deps.resolveUserAttachments(
           turnAttachments,
           {
+            conversationId,
             threadId,
             actorId: slackActorId,
             channelId,
@@ -9897,6 +9966,21 @@ function createResourceEventTeamIdResolver() {
 function createResourceEventAppPublisher(args) {
   const resolveEventTaskTeamId = createResourceEventTeamIdResolver();
   return {
+    async hasMatch(event) {
+      const work = args.conversationWork();
+      const eventTaskTeamId = await resolveEventTaskTeamId();
+      const [subscriptions, tasks] = await Promise.all([
+        findMatchingResourceEventSubscriptions({
+          data: event.data,
+          eventType: event.eventType,
+          namespace: event.namespace,
+          identifier: event.identifier,
+          state: work.state
+        }),
+        eventTaskTeamId ? findMatchingEventTasks(getDb(), event, eventTaskTeamId) : Promise.resolve([])
+      ]);
+      return subscriptions.length > 0 || tasks.length > 0;
+    },
     async neededMatchKeys(input) {
       const work = args.conversationWork();
       const eventTaskTeamId = await resolveEventTaskTeamId();
@@ -12444,7 +12528,8 @@ async function createApp(options) {
   });
   const runtimeServiceOverrides = {
     agentRunner,
-    sandbox: { tracePropagation }
+    sandbox: { tracePropagation },
+    visionContext: { attachmentStorage }
   };
   let conversationWorkOptions;
   const getConversationWorkOptions = () => {

package/dist/{catalog-runtime-CIO3ERVQ.js → catalog-runtime-7TIXWSYG.js}

@@ -1,14 +1,14 @@
 import {
   pluginCatalogRuntime
-} from "./chunk-5VRPXF4T.js";
-import "./chunk-5Q6KP6LH.js";
+} from "./chunk-Y6EUAX5O.js";
+import "./chunk-H5SDC32F.js";
 import "./chunk-CEMMLV5L.js";
 import "./chunk-N7TMSOZH.js";
 import "./chunk-DWPN6SCH.js";
 import "./chunk-TXKUKMVO.js";
-import "./chunk-7DO36XLF.js";
+import "./chunk-SIMN6AUL.js";
 import "./chunk-PEQXYSSE.js";
-import "./chunk-OTTKTTVI.js";
+import "./chunk-NN5Y32TJ.js";
 export {
   pluginCatalogRuntime
 };