@sentry/junior 0.5.0 → 0.7.0

package/bin/junior.mjs

@@ -36,6 +36,15 @@ async function runInit(dir) {
   await runInitFn(dir);
 }
 
+async function runCheck(dir) {
+  const runCheckFn = await loadCliFunction(
+    "check",
+    "runCheck",
+    "Check module is unavailable; reinstall @sentry/junior and retry."
+  );
+  await runCheckFn(dir);
+}
+
 async function main() {
   const runCli = await loadCliFunction(
     "run",
@@ -44,7 +53,8 @@ async function main() {
   );
   const exitCode = await runCli(positionals, {
     runInit,
-    runSnapshotCreate
+    runSnapshotCreate,
+    runCheck
   });
   if (exitCode !== 0) {
     process.exit(exitCode);

package/dist/{bot-PFLNB5PC.js → bot-ZKMCCT3D.js}

@@ -4,10 +4,12 @@ import {
   createNormalizingStream,
   resetBotDepsForTests,
   setBotDepsForTests
-} from "./chunk-3ENACZ27.js";
-import "./chunk-BKYYVLVN.js";
-import "./chunk-PY4AI2GZ.js";
+} from "./chunk-JRKU55W5.js";
+import "./chunk-KT5HARSN.js";
+import "./chunk-RKOO42TW.js";
 import "./chunk-Z5E25LRN.js";
+import "./chunk-PY4AI2GZ.js";
+import "./chunk-VW26MOSO.js";
 export {
   appSlackRuntime,
   bot,

package/dist/{chunk-WM5Z766B.js → chunk-56WI5Q7P.js}

@@ -4,7 +4,7 @@ import {
   downloadPrivateSlackFile,
   getThreadMessageTopic,
   removeReactionFromMessage
-} from "./chunk-3ENACZ27.js";
+} from "./chunk-JRKU55W5.js";
 import {
   acquireQueueMessageProcessingOwnership,
   completeQueueMessageProcessingOwnership,
@@ -12,11 +12,12 @@ import {
   getQueueMessageProcessingState,
   getStateAdapter,
   refreshQueueMessageProcessingOwnership
-} from "./chunk-BKYYVLVN.js";
+} from "./chunk-RKOO42TW.js";
 import {
   createRequestContext,
   logError,
   logException,
+  logInfo,
   logWarn,
   setSpanStatus,
   withContext,
@@ -77,7 +78,9 @@ function createMessageOwnerToken() {
 }
 var QueueMessageOwnershipError = class extends Error {
   constructor(stage, dedupKey) {
-    super(`Queue message ownership lost during ${stage} for dedupKey=${dedupKey}`);
+    super(
+      `Queue message ownership lost during ${stage} for dedupKey=${dedupKey}`
+    );
     this.name = "QueueMessageOwnershipError";
   }
 };
@@ -89,6 +92,7 @@ var defaultProcessQueuedThreadMessageDeps = {
       emoji: "eyes"
     });
   },
+  logInfo,
   logWarn,
   processRuntime: processThreadMessageRuntime
 };
@@ -122,8 +126,25 @@ async function logThreadMessageFailure(payload, errorMessage) {
   );
 }
 async function processQueuedThreadMessage(payload, deps = defaultProcessQueuedThreadMessageDeps) {
-  const existingMessageState = await getQueueMessageProcessingState(payload.dedupKey);
+  const existingMessageState = await getQueueMessageProcessingState(
+    payload.dedupKey
+  );
   if (existingMessageState?.status === "completed") {
+    deps.logInfo(
+      "queue_message_skipped_completed",
+      {
+        slackThreadId: payload.normalizedThreadId,
+        slackChannelId: getPayloadChannelId(payload),
+        slackUserId: getPayloadUserId(payload)
+      },
+      {
+        "messaging.message.id": payload.message.id,
+        "app.queue.message_kind": payload.kind,
+        "app.queue.message_id": payload.queueMessageId,
+        "app.queue.processing_state": existingMessageState.status
+      },
+      "Skipping queue message because it is already completed"
+    );
     return;
   }
   const ownerToken = createMessageOwnerToken();
@@ -133,6 +154,22 @@ async function processQueuedThreadMessage(payload, deps = defaultProcessQueuedTh
     queueMessageId: payload.queueMessageId
   });
   if (claimResult === "blocked") {
+    deps.logInfo(
+      "queue_message_skipped_blocked",
+      {
+        slackThreadId: payload.normalizedThreadId,
+        slackChannelId: getPayloadChannelId(payload),
+        slackUserId: getPayloadUserId(payload)
+      },
+      {
+        "messaging.message.id": payload.message.id,
+        "app.queue.message_kind": payload.kind,
+        "app.queue.message_id": payload.queueMessageId,
+        "app.queue.claim_result": claimResult,
+        "app.queue.processing_state": "processing"
+      },
+      "Skipping queue message because another worker owns it"
+    );
     return;
   }
   const threadWasSerialized = isSerializedThread(payload.thread);
@@ -209,42 +246,62 @@ async function processQueuedThreadMessage(payload, deps = defaultProcessQueuedTh
       queueMessageId: payload.queueMessageId
     });
     if (!failed && !(error instanceof QueueMessageOwnershipError)) {
-      throw new Error(`Failed to persist queue message failure state for dedupKey=${payload.dedupKey}: ${errorMessage}`);
+      throw new Error(
+        `Failed to persist queue message failure state for dedupKey=${payload.dedupKey}: ${errorMessage}`
+      );
     }
     throw error;
   }
 }
 
 // src/handlers/queue-callback.ts
-var callbackHandler = createQueueCallbackHandler(async (message, metadata) => {
-  if (metadata.topicName === getThreadMessageTopic()) {
-    const payload = {
-      ...message,
-      queueMessageId: metadata.messageId
-    };
-    await withSpan(
-      "queue.process_message",
-      "queue.process_message",
-      {
-        slackThreadId: payload.normalizedThreadId,
-        slackChannelId: payload.thread.channelId,
-        slackUserId: payload.message.author?.userId
-      },
-      async () => {
-        await processQueuedThreadMessage(payload);
-      },
-      {
-        "messaging.message.id": payload.message.id,
-        "app.queue.message_kind": payload.kind,
-        "app.queue.message_id": payload.queueMessageId,
-        "app.queue.delivery_count": metadata.deliveryCount,
-        "app.queue.topic": metadata.topicName
-      }
-    );
-    return;
+var callbackHandler = createQueueCallbackHandler(
+  async (message, metadata) => {
+    if (metadata.topicName === getThreadMessageTopic()) {
+      const payload = {
+        ...message,
+        queueMessageId: metadata.messageId
+      };
+      logInfo(
+        "queue_callback_received",
+        {
+          slackThreadId: payload.normalizedThreadId,
+          slackChannelId: payload.thread.channelId,
+          slackUserId: payload.message.author?.userId
+        },
+        {
+          "messaging.message.id": payload.message.id,
+          "app.queue.message_kind": payload.kind,
+          "app.queue.message_id": payload.queueMessageId,
+          "app.queue.delivery_count": metadata.deliveryCount,
+          "app.queue.topic": metadata.topicName
+        },
+        "Received queue callback payload"
+      );
+      await withSpan(
+        "queue.process_message",
+        "queue.process_message",
+        {
+          slackThreadId: payload.normalizedThreadId,
+          slackChannelId: payload.thread.channelId,
+          slackUserId: payload.message.author?.userId
+        },
+        async () => {
+          await processQueuedThreadMessage(payload);
+        },
+        {
+          "messaging.message.id": payload.message.id,
+          "app.queue.message_kind": payload.kind,
+          "app.queue.message_id": payload.queueMessageId,
+          "app.queue.delivery_count": metadata.deliveryCount,
+          "app.queue.topic": metadata.topicName
+        }
+      );
+      return;
+    }
+    throw new Error(`Unexpected queue topic: ${metadata.topicName}`);
   }
-  throw new Error(`Unexpected queue topic: ${metadata.topicName}`);
-});
+);
 async function POST(request) {
   const requestContext = createRequestContext(request, { platform: "queue" });
   return withContext(requestContext, async () => {