@inkeep/agents-work-apps 0.48.0 → 0.48.2

package/dist/github/index.d.ts

@@ -4,10 +4,10 @@ import "./routes/setup.js";
 import "./routes/tokenExchange.js";
 import { WebhookVerificationResult, verifyWebhookSignature } from "./routes/webhooks.js";
 import { Hono } from "hono";
-import * as hono_types6 from "hono/types";
+import * as hono_types0 from "hono/types";
 
 //#region src/github/index.d.ts
-declare function createGithubRoutes(): Hono<hono_types6.BlankEnv, hono_types6.BlankSchema, "/">;
-declare const githubRoutes: Hono<hono_types6.BlankEnv, hono_types6.BlankSchema, "/">;
+declare function createGithubRoutes(): Hono<hono_types0.BlankEnv, hono_types0.BlankSchema, "/">;
+declare const githubRoutes: Hono<hono_types0.BlankEnv, hono_types0.BlankSchema, "/">;
 //#endregion
 export { GenerateInstallationAccessTokenResult, GenerateTokenError, GenerateTokenResult, GitHubAppConfig, InstallationAccessToken, InstallationInfo, LookupInstallationError, LookupInstallationForRepoResult, LookupInstallationResult, WebhookVerificationResult, clearConfigCache, createAppJwt, createGithubRoutes, determineStatus, fetchInstallationDetails, fetchInstallationRepositories, generateInstallationAccessToken, getGitHubAppConfig, getGitHubAppName, getStateSigningSecret, getWebhookSecret, githubRoutes, isGitHubAppConfigured, isGitHubAppNameConfigured, isStateSigningConfigured, isWebhookConfigured, lookupInstallationForRepo, validateGitHubAppConfigOnStartup, validateGitHubInstallFlowConfigOnStartup, validateGitHubWebhookConfigOnStartup, verifyWebhookSignature };

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

@@ -1,7 +1,7 @@
-import * as hono1 from "hono";
+import * as hono0 from "hono";
 
 //#region src/github/mcp/auth.d.ts
-declare const githubMcpAuth: () => hono1.MiddlewareHandler<{
+declare const githubMcpAuth: () => hono0.MiddlewareHandler<{
   Variables: {
     toolId: string;
   };

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

@@ -1,11 +1,11 @@
 import { Hono } from "hono";
-import * as hono_types1 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_types1.BlankSchema, "/">;
+}, hono_types3.BlankSchema, "/">;
 //#endregion
 export { app as default };

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

@@ -36,8 +36,8 @@ declare const ChangedFileSchema: z.ZodObject<{
   path: z.ZodString;
   status: z.ZodEnum<{
     added: "added";
-    removed: "removed";
     modified: "modified";
+    removed: "removed";
     renamed: "renamed";
     copied: "copied";
     changed: "changed";

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

@@ -1,7 +1,7 @@
 import { Hono } from "hono";
-import * as hono_types2 from "hono/types";
+import * as hono_types4 from "hono/types";
 
 //#region src/github/routes/setup.d.ts
-declare const app: Hono<hono_types2.BlankEnv, hono_types2.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_types0 from "hono/types";
+import * as hono_types6 from "hono/types";
 
 //#region src/github/routes/tokenExchange.d.ts
-declare const app: Hono<hono_types0.BlankEnv, hono_types0.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_types4 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_types4.BlankEnv, hono_types4.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/client.js

@@ -1,5 +1,5 @@
 import { getLogger } from "../../logger.js";
-import { WebClient, retryPolicies } from "@slack/web-api";
+import { WebClient } from "@slack/web-api";
 
 //#region src/slack/services/client.ts
 /**
@@ -29,7 +29,7 @@ async function paginateSlack({ fetchPage, extractItems, getNextCursor, limit })
 * @returns Configured Slack WebClient instance
 */
 function getSlackClient(token) {
-	return new WebClient(token, { retryConfig: retryPolicies.fiveRetriesInFiveMinutes });
+	return new WebClient(token);
 }
 /**
 * Fetch user profile information from Slack.

package/dist/slack/services/events/app-mention.js

@@ -71,6 +71,7 @@ async function handleAppMention(params) {
 		const replyThreadTs = threadTs || messageTs;
 		const isInThread = Boolean(threadTs && threadTs !== messageTs);
 		const hasQuery = Boolean(text && text.trim().length > 0);
+		let thinkingMessageTs;
 		try {
 			const [agentConfig, existingLink] = await Promise.all([resolveChannelAgentConfig(teamId, channel, workspaceConnection), findCachedUserMapping(tenantId, slackUserId, teamId)]);
 			if (!agentConfig) {
@@ -167,11 +168,11 @@ _Using: ${agentDisplayName}_`
 					slackTeamId: teamId,
 					slackUserId
 				});
-				const ackMessage$1 = await slackClient.chat.postMessage({
+				thinkingMessageTs = (await slackClient.chat.postMessage({
 					channel,
 					thread_ts: threadTs,
 					text: `_${agentDisplayName} is reading this thread..._`
-				});
+				})).ts || void 0;
 				const conversationId$1 = generateSlackConversationId({
 					teamId,
 					threadTs,
@@ -203,7 +204,7 @@ Respond naturally as if you're joining the conversation to help.`;
 					slackClient,
 					channel,
 					threadTs,
-					thinkingMessageTs: ackMessage$1.ts || "",
+					thinkingMessageTs: thinkingMessageTs || "",
 					slackUserId,
 					teamId,
 					jwtToken: slackUserToken$1,
@@ -227,11 +228,11 @@ Respond naturally as if you're joining the conversation to help.`;
 				slackTeamId: teamId,
 				slackUserId
 			});
-			const ackMessage = await slackClient.chat.postMessage({
+			thinkingMessageTs = (await slackClient.chat.postMessage({
 				channel,
 				thread_ts: replyThreadTs,
 				text: `_${agentDisplayName} is preparing a response..._`
-			});
+			})).ts || void 0;
 			const conversationId = generateSlackConversationId({
 				teamId,
 				threadTs: replyThreadTs,
@@ -249,7 +250,7 @@ Respond naturally as if you're joining the conversation to help.`;
 				slackClient,
 				channel,
 				threadTs: replyThreadTs,
-				thinkingMessageTs: ackMessage.ts || "",
+				thinkingMessageTs: thinkingMessageTs || "",
 				slackUserId,
 				teamId,
 				jwtToken: slackUserToken,
@@ -268,6 +269,12 @@ Respond naturally as if you're joining the conversation to help.`;
 				teamId
 			}, "Failed in app mention handler");
 			if (error instanceof Error) setSpanWithError(span, error);
+			if (thinkingMessageTs) try {
+				await slackClient.chat.delete({
+					channel,
+					ts: thinkingMessageTs
+				});
+			} catch {}
 			const userMessage = getUserFriendlyErrorMessage(classifyError(error));
 			try {
 				await slackClient.chat.postEphemeral({

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.
 */
@@ -78,12 +80,12 @@ async function streamAgentResponse(params) {
 		} catch (fetchError) {
 			clearTimeout(timeoutId);
 			if (fetchError.name === "AbortError") {
-				const errorType = SlackErrorType.TIMEOUT;
-				const errorMessage = getUserFriendlyErrorMessage(errorType, agentName);
+				const errorType$1 = SlackErrorType.TIMEOUT;
+				const errorMessage$1 = getUserFriendlyErrorMessage(errorType$1, agentName);
 				await slackClient.chat.postMessage({
 					channel,
 					thread_ts: threadTs,
-					text: errorMessage
+					text: errorMessage$1
 				});
 				if (thinkingMessageTs) try {
 					await slackClient.chat.delete({
@@ -94,13 +96,30 @@ async function streamAgentResponse(params) {
 				span.end();
 				return {
 					success: false,
-					errorType,
-					errorMessage
+					errorType: errorType$1,
+					errorMessage: errorMessage$1
 				};
 			}
+			const errorType = classifyError(fetchError);
+			const errorMessage = getUserFriendlyErrorMessage(errorType, agentName);
+			await slackClient.chat.postMessage({
+				channel,
+				thread_ts: threadTs,
+				text: errorMessage
+			}).catch((e) => logger.warn({ error: e }, "Failed to send fetch error notification"));
+			if (thinkingMessageTs) try {
+				await slackClient.chat.delete({
+					channel,
+					ts: thinkingMessageTs
+				});
+			} catch {}
 			if (fetchError instanceof Error) setSpanWithError(span, fetchError);
 			span.end();
-			throw fetchError;
+			return {
+				success: false,
+				errorType,
+				errorMessage
+			};
 		}
 		if (!response.ok) {
 			clearTimeout(timeoutId);
@@ -167,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;
@@ -179,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;
@@ -194,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,
@@ -218,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.0",
+  "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.0"
+    "@inkeep/agents-core": "0.48.2"
   },
   "peerDependencies": {
     "@hono/zod-openapi": "^1.1.5",