@inkeep/agents-work-apps 0.48.1 → 0.48.2

package/dist/github/mcp/index.d.ts

@@ -1,11 +1,11 @@
 import { Hono } from "hono";
-import * as hono_types5 from "hono/types";
+import * as hono_types3 from "hono/types";
 
 //#region src/github/mcp/index.d.ts
 declare const app: Hono<{
   Variables: {
     toolId: string;
   };
-}, hono_types5.BlankSchema, "/">;
+}, hono_types3.BlankSchema, "/">;
 //#endregion
 export { app as default };

package/dist/github/routes/setup.d.ts

@@ -1,7 +1,7 @@
 import { Hono } from "hono";
-import * as hono_types6 from "hono/types";
+import * as hono_types4 from "hono/types";
 
 //#region src/github/routes/setup.d.ts
-declare const app: Hono<hono_types6.BlankEnv, hono_types6.BlankSchema, "/">;
+declare const app: Hono<hono_types4.BlankEnv, hono_types4.BlankSchema, "/">;
 //#endregion
 export { app as default };

package/dist/github/routes/tokenExchange.d.ts

@@ -1,7 +1,7 @@
 import { Hono } from "hono";
-import * as hono_types8 from "hono/types";
+import * as hono_types6 from "hono/types";
 
 //#region src/github/routes/tokenExchange.d.ts
-declare const app: Hono<hono_types8.BlankEnv, hono_types8.BlankSchema, "/">;
+declare const app: Hono<hono_types6.BlankEnv, hono_types6.BlankSchema, "/">;
 //#endregion
 export { app as default };

package/dist/github/routes/webhooks.d.ts

@@ -1,5 +1,5 @@
 import { Hono } from "hono";
-import * as hono_types3 from "hono/types";
+import * as hono_types8 from "hono/types";
 
 //#region src/github/routes/webhooks.d.ts
 interface WebhookVerificationResult {
@@ -7,6 +7,6 @@ interface WebhookVerificationResult {
   error?: string;
 }
 declare function verifyWebhookSignature(payload: string, signature: string | undefined, secret: string): WebhookVerificationResult;
-declare const app: Hono<hono_types3.BlankEnv, hono_types3.BlankSchema, "/">;
+declare const app: Hono<hono_types8.BlankEnv, hono_types8.BlankSchema, "/">;
 //#endregion
 export { WebhookVerificationResult, app as default, verifyWebhookSignature };

package/dist/slack/routes/events.js

@@ -63,6 +63,19 @@ app.post("/commands", async (c) => {
 	return c.json(response);
 });
 app.post("/events", async (c) => {
+	const retryNum = c.req.header("x-slack-retry-num");
+	const retryReason = c.req.header("x-slack-retry-reason");
+	if (retryNum) return tracer.startActiveSpan(`${SLACK_SPAN_NAMES.WEBHOOK} retry`, (span) => {
+		span.setAttribute(SLACK_SPAN_KEYS.OUTCOME, "ignored_slack_retry");
+		span.setAttribute("slack.retry_num", retryNum);
+		if (retryReason) span.setAttribute("slack.retry_reason", retryReason);
+		logger.info({
+			retryNum,
+			retryReason
+		}, "Acknowledging Slack retry without re-processing");
+		span.end();
+		return c.json({ ok: true });
+	});
 	return tracer.startActiveSpan(SLACK_SPAN_NAMES.WEBHOOK, async (span) => {
 		let outcome = "ignored_unknown_event";
 		try {

package/dist/slack/services/events/streaming.js

@@ -14,6 +14,8 @@ import { SLACK_SPAN_KEYS, SLACK_SPAN_NAMES, setSpanWithError, tracer } from "../
 const logger = getLogger("slack-streaming");
 const STREAM_TIMEOUT_MS = 12e4;
 const CHATSTREAM_OP_TIMEOUT_MS = 1e4;
+/** Shorter timeout for best-effort cleanup in error paths to bound total error handling time. */
+const CLEANUP_TIMEOUT_MS = 3e3;
 /**
 * Wrap a promise with a timeout to prevent indefinite blocking on Slack API calls.
 */
@@ -184,6 +186,7 @@ async function streamAgentResponse(params) {
 			thread_ts: threadTs
 		});
 		try {
+			let agentCompleted = false;
 			while (true) {
 				const { done, value } = await reader.read();
 				if (done) break;
@@ -196,7 +199,13 @@ async function streamAgentResponse(params) {
 					if (!jsonStr || jsonStr === "[DONE]") continue;
 					try {
 						const data = JSON.parse(jsonStr);
-						if (data.type === "data-operation") continue;
+						if (data.type === "data-operation") {
+							if (data.data?.type === "completion") {
+								agentCompleted = true;
+								break;
+							}
+							continue;
+						}
 						if (data.type === "text-start" || data.type === "text-end") continue;
 						if (data.type === "text-delta" && data.delta) {
 							fullText += data.delta;
@@ -211,10 +220,21 @@ async function streamAgentResponse(params) {
 						}
 					} catch {}
 				}
+				if (agentCompleted) break;
 			}
 			clearTimeout(timeoutId);
 			const contextBlock = createContextBlock({ agentName });
-			await withTimeout(streamer.stop({ blocks: [contextBlock] }), CHATSTREAM_OP_TIMEOUT_MS, "streamer.stop");
+			try {
+				await withTimeout(streamer.stop({ blocks: [contextBlock] }), CHATSTREAM_OP_TIMEOUT_MS, "streamer.stop");
+			} catch (stopError) {
+				span.setAttribute(SLACK_SPAN_KEYS.STREAM_FINALIZATION_FAILED, true);
+				logger.warn({
+					stopError,
+					channel,
+					threadTs,
+					responseLength: fullText.length
+				}, "Failed to finalize chatStream — content was already delivered");
+			}
 			if (thinkingMessageTs) try {
 				await slackClient.chat.delete({
 					channel,
@@ -235,8 +255,26 @@ async function streamAgentResponse(params) {
 		} catch (streamError) {
 			clearTimeout(timeoutId);
 			if (streamError instanceof Error) setSpanWithError(span, streamError);
+			if (fullText.length > 0) {
+				span.setAttribute(SLACK_SPAN_KEYS.CONTENT_ALREADY_DELIVERED, true);
+				logger.warn({
+					streamError,
+					channel,
+					threadTs,
+					responseLength: fullText.length
+				}, "Error during Slack streaming after content was already delivered — suppressing user-facing error");
+				await withTimeout(streamer.stop(), CLEANUP_TIMEOUT_MS, "streamer.stop-cleanup").catch((e) => logger.warn({ error: e }, "Failed to stop streamer during error cleanup"));
+				if (thinkingMessageTs) try {
+					await slackClient.chat.delete({
+						channel,
+						ts: thinkingMessageTs
+					});
+				} catch {}
+				span.end();
+				return { success: true };
+			}
 			logger.error({ streamError }, "Error during Slack streaming");
-			await withTimeout(streamer.stop(), CHATSTREAM_OP_TIMEOUT_MS, "streamer.stop").catch((e) => logger.warn({ error: e }, "Failed to stop streamer during error cleanup"));
+			await withTimeout(streamer.stop(), CLEANUP_TIMEOUT_MS, "streamer.stop-cleanup").catch((e) => logger.warn({ error: e }, "Failed to stop streamer during error cleanup"));
 			if (thinkingMessageTs) try {
 				await slackClient.chat.delete({
 					channel,

package/dist/slack/tracer.d.ts

@@ -34,7 +34,9 @@ declare const SLACK_SPAN_KEYS: {
   readonly IS_BOT_MESSAGE: "slack.is_bot_message";
   readonly HAS_QUERY: "slack.has_query";
   readonly IS_IN_THREAD: "slack.is_in_thread";
+  readonly STREAM_FINALIZATION_FAILED: "slack.stream_finalization_failed";
+  readonly CONTENT_ALREADY_DELIVERED: "slack.content_already_delivered";
 };
-type SlackOutcome = 'handled' | 'ignored_bot_message' | 'ignored_unknown_event' | 'ignored_no_action_match' | 'url_verification' | 'validation_error' | 'signature_invalid' | 'error';
+type SlackOutcome = 'handled' | 'ignored_bot_message' | 'ignored_unknown_event' | 'ignored_no_action_match' | 'ignored_slack_retry' | 'url_verification' | 'validation_error' | 'signature_invalid' | 'error';
 //#endregion
 export { SLACK_SPAN_KEYS, SLACK_SPAN_NAMES, SlackOutcome, setSpanWithError, tracer };

package/dist/slack/tracer.js

@@ -32,7 +32,9 @@ const SLACK_SPAN_KEYS = {
 	OUTCOME: "slack.outcome",
 	IS_BOT_MESSAGE: "slack.is_bot_message",
 	HAS_QUERY: "slack.has_query",
-	IS_IN_THREAD: "slack.is_in_thread"
+	IS_IN_THREAD: "slack.is_in_thread",
+	STREAM_FINALIZATION_FAILED: "slack.stream_finalization_failed",
+	CONTENT_ALREADY_DELIVERED: "slack.content_already_delivered"
 };
 
 //#endregion

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@inkeep/agents-work-apps",
-  "version": "0.48.1",
+  "version": "0.48.2",
   "description": "First party integrations for Inkeep Agents",
   "type": "module",
   "license": "SEE LICENSE IN LICENSE.md",
@@ -33,7 +33,7 @@
     "jose": "^6.1.0",
     "minimatch": "^10.1.1",
     "slack-block-builder": "^2.8.0",
-    "@inkeep/agents-core": "0.48.1"
+    "@inkeep/agents-core": "0.48.2"
   },
   "peerDependencies": {
     "@hono/zod-openapi": "^1.1.5",