@inkeep/agents-work-apps 0.51.0 → 0.53.0

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

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

package/dist/github/mcp/index.js

@@ -1,7 +1,7 @@
 import runDbClient_default from "../../db/runDbClient.js";
 import { githubMcpAuth } from "./auth.js";
 import { ReactionContentSchema } from "./schemas.js";
-import { commitFileChanges, commitNewFile, createIssueCommentReaction, createPullRequestReviewCommentReaction, deleteIssueCommentReaction, deletePullRequestReviewCommentReaction, fetchComments, fetchPrFileDiffs, fetchPrFiles, fetchPrInfo, formatFileDiff, generatePrMarkdown, getGitHubClientFromRepo, listIssueCommentReactions, listPullRequestReviewCommentReactions, visualizeUpdateOperations } from "./utils.js";
+import { commitFileChanges, commitNewFile, createIssueCommentReaction, createPullRequestReviewCommentReaction, deleteIssueCommentReaction, deletePullRequestReviewCommentReaction, fetchBranchChangedFiles, fetchComments, fetchPrFileDiffs, fetchPrFiles, fetchPrInfo, formatFileDiff, generatePrMarkdown, getGitHubClientFromRepo, listIssueCommentReactions, listIssueReactions, listPullRequestReviewCommentReactions, visualizeUpdateOperations } from "./utils.js";
 import { z } from "@hono/zod-openapi";
 import { getMcpToolRepositoryAccessWithDetails } from "@inkeep/agents-core";
 import { Hono } from "hono";
@@ -267,6 +267,65 @@ const getServer = async (toolId) => {
 			};
 		}
 	});
+	server.tool("get-changed-files-for-branch", `Get the list of files changed on a branch compared to a base ref, without requiring a pull request. Returns file paths, status, additions/deletions, and optionally patches and full file contents. ${getAvailableRepositoryString(repositoryAccess)}`, {
+		owner: z.string().describe("Repository owner name"),
+		repo: z.string().describe("Repository name"),
+		head: z.string().describe("The branch or ref to check for changes (e.g. \"feat/my-feature\")"),
+		base: z.string().optional().describe("The base branch or ref to compare against (defaults to the repository default branch)"),
+		file_paths: z.array(z.string()).optional().describe("Optional list of file path glob patterns to filter results"),
+		include_contents: z.boolean().default(false).describe("Whether to include full file contents for each changed file"),
+		include_patch: z.boolean().default(false).describe("Whether to include the patch/diff text for each changed file")
+	}, async ({ owner, repo, head, base, file_paths, include_contents, include_patch }) => {
+		try {
+			let githubClient;
+			try {
+				githubClient = getGitHubClientFromRepo(owner, repo, installationIdMap);
+			} catch (error) {
+				return {
+					content: [{
+						type: "text",
+						text: `Error accessing GitHub: ${error instanceof Error ? error.message : "Unknown error"}`
+					}],
+					isError: true
+				};
+			}
+			const baseRef = base ?? (await githubClient.rest.repos.get({
+				owner,
+				repo
+			})).data.default_branch;
+			const files = await fetchBranchChangedFiles(githubClient, owner, repo, baseRef, head, {
+				pathFilters: file_paths ?? [],
+				includeContents: include_contents,
+				includePatch: include_patch
+			});
+			if (files.length === 0) return { content: [{
+				type: "text",
+				text: `No changed files found between ${baseRef} and ${head} in ${owner}/${repo}.${file_paths?.length ? `\n\nFilters applied: ${file_paths.join(", ")}` : ""}`
+			}] };
+			const output = await formatFileDiff(0, files, include_contents);
+			return { content: [{
+				type: "text",
+				text: `## Changed files: ${baseRef}...${head} in ${owner}/${repo}\n\nFound ${files.length} changed file(s).\n\n` + output
+			}] };
+		} catch (error) {
+			if (error instanceof Error && "status" in error) {
+				if (error.status === 404) return {
+					content: [{
+						type: "text",
+						text: `Repository ${owner}/${repo} not found, or one of the refs ("${base ?? "default"}", "${head}") does not exist.`
+					}],
+					isError: true
+				};
+			}
+			return {
+				content: [{
+					type: "text",
+					text: `Error fetching changed files: ${error instanceof Error ? error.message : "Unknown error"}`
+				}],
+				isError: true
+			};
+		}
+	});
 	server.tool("create-branch", `Create a new branch in a repository. ${getAvailableRepositoryString(repositoryAccess)}`, {
 		owner: z.string().describe("Repository owner name"),
 		repo: z.string().describe("Repository name"),
@@ -607,13 +666,17 @@ const getServer = async (toolId) => {
 			};
 		}
 	});
-	server.tool("add-comment-reaction", `Add a reaction to a comment on a pull request. Supports general pull request comments and inline PR review comments. ${getAvailableRepositoryString(repositoryAccess)}`, {
+	server.tool("add-reaction", `Add a reaction to a pull request body, a general PR comment, or an inline PR review comment. ${getAvailableRepositoryString(repositoryAccess)}`, {
 		owner: z.string().describe("Repository owner name"),
 		repo: z.string().describe("Repository name"),
-		comment_id: z.number().describe("The ID of the comment to react to"),
-		comment_type: z.enum(["issue_comment", "review_comment"]).describe("The type of comment: \"issue_comment\" for general pull request comments, \"review_comment\" for inline PR review comments"),
+		target_id: z.number().describe("The target to react to: a comment ID for issue_comment/review_comment, or the pull request number for pull_request"),
+		target_type: z.enum([
+			"pull_request",
+			"issue_comment",
+			"review_comment"
+		]).describe("The type of target: \"pull_request\" for the PR body itself, \"issue_comment\" for general pull request comments, \"review_comment\" for inline PR review comments"),
 		reaction: ReactionContentSchema.describe("The reaction emoji to add: +1, -1, laugh, hooray, confused, heart, rocket, or eyes")
-	}, async ({ owner, repo, comment_id, comment_type, reaction }) => {
+	}, async ({ owner, repo, target_id, target_type, reaction }) => {
 		try {
 			let githubClient;
 			try {
@@ -627,9 +690,23 @@ const getServer = async (toolId) => {
 					isError: true
 				};
 			}
+			let result;
+			if (target_type === "pull_request") {
+				const { data } = await githubClient.rest.reactions.createForIssue({
+					owner,
+					repo,
+					issue_number: target_id,
+					content: reaction
+				});
+				result = {
+					id: data.id,
+					content: data.content
+				};
+			} else if (target_type === "issue_comment") result = await createIssueCommentReaction(githubClient, owner, repo, target_id, reaction);
+			else result = await createPullRequestReviewCommentReaction(githubClient, owner, repo, target_id, reaction);
 			return { content: [{
 				type: "text",
-				text: `Successfully added ${reaction} reaction to ${comment_type} comment ${comment_id} in ${owner}/${repo}\n\nReaction ID: ${(comment_type === "issue_comment" ? await createIssueCommentReaction(githubClient, owner, repo, comment_id, reaction) : await createPullRequestReviewCommentReaction(githubClient, owner, repo, comment_id, reaction)).id}`
+				text: `Successfully added ${reaction} reaction to ${target_type === "pull_request" ? `PR #${target_id}` : `${target_type} ${target_id}`} in ${owner}/${repo}\n\nReaction ID: ${result.id}`
 			}] };
 		} catch (error) {
 			if (error instanceof Error && "status" in error) {
@@ -637,21 +714,21 @@ const getServer = async (toolId) => {
 				if (apiError.status === 404) return {
 					content: [{
 						type: "text",
-						text: `Comment ${comment_id} not found in ${owner}/${repo}.`
+						text: `Target ${target_id} (${target_type}) not found in ${owner}/${repo}.`
 					}],
 					isError: true
 				};
 				if (apiError.status === 422) return {
 					content: [{
 						type: "text",
-						text: `Invalid reaction. Ensure the reaction type is valid and the comment exists.`
+						text: `Invalid reaction. Ensure the reaction type is valid and the target exists.`
 					}],
 					isError: true
 				};
 				if (apiError.status === 403) return {
 					content: [{
 						type: "text",
-						text: `Access denied when adding reaction to comment ${comment_id} in ${owner}/${repo}. Your GitHub App may not have sufficient permissions to create reactions.`
+						text: `Access denied when adding reaction to ${target_type} ${target_id} in ${owner}/${repo}. Your GitHub App may not have sufficient permissions to create reactions.`
 					}],
 					isError: true
 				};
@@ -665,13 +742,17 @@ const getServer = async (toolId) => {
 			};
 		}
 	});
-	server.tool("remove-comment-reaction", `Remove a reaction from a comment on a pull request. Requires the reaction ID (returned when adding a reaction or available from comment data). ${getAvailableRepositoryString(repositoryAccess)}`, {
+	server.tool("remove-reaction", `Remove a reaction from a pull request body, a general PR comment, or an inline PR review comment. Requires the reaction ID (returned when adding a reaction or from list-reactions). ${getAvailableRepositoryString(repositoryAccess)}`, {
 		owner: z.string().describe("Repository owner name"),
 		repo: z.string().describe("Repository name"),
-		comment_id: z.number().describe("The ID of the comment the reaction belongs to"),
-		comment_type: z.enum(["issue_comment", "review_comment"]).describe("The type of comment: \"issue_comment\" for general pull request comments, \"review_comment\" for inline PR review comments"),
+		target_id: z.number().describe("The target the reaction belongs to: a comment ID for issue_comment/review_comment, or the pull request number for pull_request"),
+		target_type: z.enum([
+			"pull_request",
+			"issue_comment",
+			"review_comment"
+		]).describe("The type of target: \"pull_request\" for the PR body itself, \"issue_comment\" for general pull request comments, \"review_comment\" for inline PR review comments"),
 		reaction_id: z.number().describe("The ID of the reaction to remove")
-	}, async ({ owner, repo, comment_id, comment_type, reaction_id }) => {
+	}, async ({ owner, repo, target_id, target_type, reaction_id }) => {
 		try {
 			let githubClient;
 			try {
@@ -685,18 +766,24 @@ const getServer = async (toolId) => {
 					isError: true
 				};
 			}
-			if (comment_type === "issue_comment") await deleteIssueCommentReaction(githubClient, owner, repo, comment_id, reaction_id);
-			else await deletePullRequestReviewCommentReaction(githubClient, owner, repo, comment_id, reaction_id);
+			if (target_type === "pull_request") await githubClient.rest.reactions.deleteForIssue({
+				owner,
+				repo,
+				issue_number: target_id,
+				reaction_id
+			});
+			else if (target_type === "issue_comment") await deleteIssueCommentReaction(githubClient, owner, repo, target_id, reaction_id);
+			else await deletePullRequestReviewCommentReaction(githubClient, owner, repo, target_id, reaction_id);
 			return { content: [{
 				type: "text",
-				text: `Successfully removed reaction ${reaction_id} from ${comment_type} comment ${comment_id} in ${owner}/${repo}`
+				text: `Successfully removed reaction ${reaction_id} from ${target_type === "pull_request" ? `PR #${target_id}` : `${target_type} ${target_id}`} in ${owner}/${repo}`
 			}] };
 		} catch (error) {
 			if (error instanceof Error && "status" in error) {
 				if (error.status === 404) return {
 					content: [{
 						type: "text",
-						text: `Comment ${comment_id} or reaction ${reaction_id} not found in ${owner}/${repo}.`
+						text: `Target ${target_id} (${target_type}) or reaction ${reaction_id} not found in ${owner}/${repo}.`
 					}],
 					isError: true
 				};
@@ -710,12 +797,16 @@ const getServer = async (toolId) => {
 			};
 		}
 	});
-	server.tool("list-comment-reactions", `List all reactions on a comment, including each reaction's ID (needed for removal). Supports both general issue/PR comments and inline PR review comments. ${getAvailableRepositoryString(repositoryAccess)}`, {
+	server.tool("list-reactions", `List all reactions on a pull request body, a general PR comment, or an inline PR review comment. Returns each reaction's ID (needed for removal). ${getAvailableRepositoryString(repositoryAccess)}`, {
 		owner: z.string().describe("Repository owner name"),
 		repo: z.string().describe("Repository name"),
-		comment_id: z.number().describe("The ID of the comment to list reactions for"),
-		comment_type: z.enum(["issue_comment", "review_comment"]).describe("The type of comment: \"issue_comment\" for general pull request comments, \"review_comment\" for inline PR review comments")
-	}, async ({ owner, repo, comment_id, comment_type }) => {
+		target_id: z.number().describe("The target to list reactions for: a comment ID for issue_comment/review_comment, or the pull request number for pull_request"),
+		target_type: z.enum([
+			"pull_request",
+			"issue_comment",
+			"review_comment"
+		]).describe("The type of target: \"pull_request\" for the PR body itself, \"issue_comment\" for general pull request comments, \"review_comment\" for inline PR review comments")
+	}, async ({ owner, repo, target_id, target_type }) => {
 		try {
 			let githubClient;
 			try {
@@ -729,22 +820,26 @@ const getServer = async (toolId) => {
 					isError: true
 				};
 			}
-			const reactions = comment_type === "issue_comment" ? await listIssueCommentReactions(githubClient, owner, repo, comment_id) : await listPullRequestReviewCommentReactions(githubClient, owner, repo, comment_id);
+			let reactions = [];
+			if (target_type === "pull_request") reactions = await listIssueReactions(githubClient, owner, repo, target_id);
+			else if (target_type === "issue_comment") reactions = await listIssueCommentReactions(githubClient, owner, repo, target_id);
+			else reactions = await listPullRequestReviewCommentReactions(githubClient, owner, repo, target_id);
+			const label = target_type === "pull_request" ? `PR #${target_id}` : `${target_type} ${target_id}`;
 			if (reactions.length === 0) return { content: [{
 				type: "text",
-				text: `No reactions found on ${comment_type} comment ${comment_id} in ${owner}/${repo}.`
+				text: `No reactions found on ${label} in ${owner}/${repo}.`
 			}] };
 			const formatted = reactions.map((r) => `• ${r.content} by @${r.user} (reaction_id: ${r.id})`).join("\n");
 			return { content: [{
 				type: "text",
-				text: `Found ${reactions.length} reaction(s) on ${comment_type} comment ${comment_id} in ${owner}/${repo}:\n\n${formatted}`
+				text: `Found ${reactions.length} reaction(s) on ${label} in ${owner}/${repo}:\n\n${formatted}`
 			}] };
 		} catch (error) {
 			if (error instanceof Error && "status" in error) {
 				if (error.status === 404) return {
 					content: [{
 						type: "text",
-						text: `Comment ${comment_id} not found in ${owner}/${repo}.`
+						text: `Target ${target_id} (${target_type}) not found in ${owner}/${repo}.`
 					}],
 					isError: true
 				};

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

@@ -152,6 +152,15 @@ declare function fetchPrFiles(octokit: Octokit, owner: string, repo: string, prN
  * Get file-based diffs with all commit messages that impacted each file.
  */
 declare function fetchPrFileDiffs(octokit: Octokit, owner: string, repo: string, prNumber: number): Promise<ChangedFile[]>;
+/**
+ * Fetch files changed on a branch compared to a base ref, without requiring a PR.
+ * Uses the GitHub Compare API (`repos.compareCommitsWithBasehead`).
+ */
+declare function fetchBranchChangedFiles(octokit: Octokit, owner: string, repo: string, base: string, head: string, options?: {
+  pathFilters?: string[];
+  includeContents?: boolean;
+  includePatch?: boolean;
+}): Promise<ChangedFile[]>;
 /**
  * Fetch all PR comments (both issue comments and review comments)
  */
@@ -245,6 +254,7 @@ interface ReactionDetail {
 }
 declare function listIssueCommentReactions(octokit: Octokit, owner: string, repo: string, commentId: number): Promise<ReactionDetail[]>;
 declare function listPullRequestReviewCommentReactions(octokit: Octokit, owner: string, repo: string, commentId: number): Promise<ReactionDetail[]>;
+declare function listIssueReactions(octokit: Octokit, owner: string, repo: string, issueNumber: number): Promise<ReactionDetail[]>;
 declare function formatFileDiff(pullRequestNumber: number, files: ChangedFile[], includeContents?: boolean): Promise<string>;
 //#endregion
-export { CommitData, LLMUpdateOperation, PullCommit, ReactionDetail, applyOperation, applyOperations, commitFileChanges, commitNewFile, createIssueCommentReaction, createPullRequestReviewCommentReaction, deleteIssueCommentReaction, deletePullRequestReviewCommentReaction, fetchComments, fetchCommitDetails, fetchPrCommits, fetchPrFileDiffs, fetchPrFiles, fetchPrInfo, formatFileDiff, generatePrMarkdown, getFilePathsInRepo, getGitHubClientFromInstallationId, getGitHubClientFromRepo, listIssueCommentReactions, listPullRequestReviewCommentReactions, validateLineNumbers, visualizeUpdateOperations };
+export { CommitData, LLMUpdateOperation, PullCommit, ReactionDetail, applyOperation, applyOperations, commitFileChanges, commitNewFile, createIssueCommentReaction, createPullRequestReviewCommentReaction, deleteIssueCommentReaction, deletePullRequestReviewCommentReaction, fetchBranchChangedFiles, fetchComments, fetchCommitDetails, fetchPrCommits, fetchPrFileDiffs, fetchPrFiles, fetchPrInfo, formatFileDiff, generatePrMarkdown, getFilePathsInRepo, getGitHubClientFromInstallationId, getGitHubClientFromRepo, listIssueCommentReactions, listIssueReactions, listPullRequestReviewCommentReactions, validateLineNumbers, visualizeUpdateOperations };

package/dist/github/mcp/utils.js

@@ -265,6 +265,92 @@ async function fetchPrFileDiffs(octokit, owner, repo, prNumber) {
 	}
 }
 /**
+* Fetch files changed on a branch compared to a base ref, without requiring a PR.
+* Uses the GitHub Compare API (`repos.compareCommitsWithBasehead`).
+*/
+async function fetchBranchChangedFiles(octokit, owner, repo, base, head, options = {}) {
+	const { pathFilters = [], includeContents = false, includePatch = false } = options;
+	logger.info({
+		owner,
+		repo,
+		base,
+		head,
+		pathFilters,
+		includeContents,
+		includePatch
+	}, `Fetching changed files between ${base}...${head}`);
+	const totalCommits = (await octokit.rest.repos.compareCommitsWithBasehead({
+		owner,
+		repo,
+		basehead: `${base}...${head}`,
+		per_page: 1
+	})).data.total_commits;
+	const collectedFiles = [];
+	let page = 1;
+	const perPage = 100;
+	while (true) {
+		const files = (await octokit.rest.repos.compareCommitsWithBasehead({
+			owner,
+			repo,
+			basehead: `${base}...${head}`,
+			per_page: perPage,
+			page
+		})).data.files ?? [];
+		if (files.length === 0) break;
+		for (const file of files) {
+			if (pathFilters.length > 0 && !pathFilters.some((filter) => minimatch(file.filename, filter))) continue;
+			collectedFiles.push({
+				commit_messages: [],
+				path: file.filename,
+				status: file.status,
+				additions: file.additions,
+				deletions: file.deletions,
+				patch: includePatch ? file.patch : void 0,
+				previousPath: file.previous_filename
+			});
+		}
+		if (files.length < perPage) break;
+		page++;
+	}
+	if (includeContents) {
+		const BATCH_SIZE = 10;
+		const filesToFetch = collectedFiles.filter((f) => f.status !== "removed");
+		for (let i = 0; i < filesToFetch.length; i += BATCH_SIZE) {
+			const batch = filesToFetch.slice(i, i + BATCH_SIZE);
+			await Promise.all(batch.map(async (changedFile) => {
+				try {
+					const { data: content } = await octokit.rest.repos.getContent({
+						owner,
+						repo,
+						path: changedFile.path,
+						ref: head
+					});
+					if ("content" in content && content.encoding === "base64") changedFile.contents = Buffer.from(content.content, "base64").toString("utf-8");
+				} catch (error) {
+					logger.warn({
+						owner,
+						repo,
+						base,
+						head,
+						file: changedFile.path
+					}, `Failed to fetch contents for ${changedFile.path}: ${error}`);
+				}
+			}));
+		}
+	}
+	logger.info({
+		owner,
+		repo,
+		base,
+		head,
+		totalCommits,
+		pathFilters,
+		includeContents,
+		fileCount: collectedFiles.length
+	}, `Found ${collectedFiles.length} changed files between ${base}...${head} (${totalCommits} commits)`);
+	return collectedFiles;
+}
+/**
 * Fetch all PR comments (both issue comments and review comments)
 */
 async function fetchComments(octokit, owner, repo, prNumber) {
@@ -672,6 +758,21 @@ async function listPullRequestReviewCommentReactions(octokit, owner, repo, comme
 	});
 	return reactions;
 }
+async function listIssueReactions(octokit, owner, repo, issueNumber) {
+	const reactions = [];
+	for await (const response of octokit.paginate.iterator(octokit.rest.reactions.listForIssue, {
+		owner,
+		repo,
+		issue_number: issueNumber,
+		per_page: 100
+	})) for (const r of response.data) reactions.push({
+		id: r.id,
+		content: r.content,
+		user: r.user?.login ?? "unknown",
+		createdAt: r.created_at
+	});
+	return reactions;
+}
 async function formatFileDiff(pullRequestNumber, files, includeContents = false) {
 	let output = `## File Patches for PR #${pullRequestNumber}\n\n`;
 	output += `Found ${files.length} file(s) matching the requested paths.\n\n`;
@@ -694,4 +795,4 @@ async function formatFileDiff(pullRequestNumber, files, includeContents = false)
 }
 
 //#endregion
-export { applyOperation, applyOperations, commitFileChanges, commitNewFile, createIssueCommentReaction, createPullRequestReviewCommentReaction, deleteIssueCommentReaction, deletePullRequestReviewCommentReaction, fetchComments, fetchCommitDetails, fetchPrCommits, fetchPrFileDiffs, fetchPrFiles, fetchPrInfo, formatFileDiff, generatePrMarkdown, getFilePathsInRepo, getGitHubClientFromInstallationId, getGitHubClientFromRepo, listIssueCommentReactions, listPullRequestReviewCommentReactions, validateLineNumbers, visualizeUpdateOperations };
+export { applyOperation, applyOperations, commitFileChanges, commitNewFile, createIssueCommentReaction, createPullRequestReviewCommentReaction, deleteIssueCommentReaction, deletePullRequestReviewCommentReaction, fetchBranchChangedFiles, fetchComments, fetchCommitDetails, fetchPrCommits, fetchPrFileDiffs, fetchPrFiles, fetchPrInfo, formatFileDiff, generatePrMarkdown, getFilePathsInRepo, getGitHubClientFromInstallationId, getGitHubClientFromRepo, listIssueCommentReactions, listIssueReactions, listPullRequestReviewCommentReactions, validateLineNumbers, visualizeUpdateOperations };

package/dist/slack/dispatcher.js

@@ -1,7 +1,7 @@
 import { getLogger } from "../logger.js";
 import { findWorkspaceConnectionByTeamId } from "./services/nango.js";
-import { getSlackClient } from "./services/client.js";
 import { sendResponseUrlMessage } from "./services/events/utils.js";
+import { getSlackClient } from "./services/client.js";
 import { SLACK_SPAN_KEYS } from "./tracer.js";
 import { handleAppMention } from "./services/events/app-mention.js";
 import { handleMessageShortcut, handleOpenAgentSelectorModal, handleOpenFollowUpModal, handleToolApproval } from "./services/events/block-actions.js";
@@ -35,6 +35,15 @@ async function dispatchSlackEvent(eventType, payload, options, span) {
 			}, "Ignoring bot message");
 			return { outcome };
 		}
+		if (event?.edited) {
+			outcome = "ignored_edited_message";
+			span.setAttribute(SLACK_SPAN_KEYS.OUTCOME, outcome);
+			logger.info({
+				teamId,
+				innerEventType
+			}, "Ignoring edited message");
+			return { outcome };
+		}
 		if (event?.type === "app_mention" && event.channel && event.user && teamId) {
 			outcome = "handled";
 			span.setAttribute(SLACK_SPAN_KEYS.OUTCOME, outcome);

package/dist/slack/routes/events.js

@@ -70,7 +70,7 @@ app.post("/events", async (c) => {
 			retryReason
 		}, "Acknowledging Slack retry without re-processing");
 		span.end();
-		return c.json({ ok: true });
+		return c.body(null, 200);
 	});
 	const waitUntil = await getWaitUntil();
 	return tracer.startActiveSpan(SLACK_SPAN_NAMES.WEBHOOK, async (span) => {
@@ -128,7 +128,7 @@ app.post("/events", async (c) => {
 				return c.json(result.response);
 			}
 			span.end();
-			return c.json({ ok: true });
+			return c.body(null, 200);
 		} catch (error) {
 			outcome = "error";
 			span.setAttribute(SLACK_SPAN_KEYS.OUTCOME, outcome);

package/dist/slack/routes/oauth.js

@@ -6,7 +6,7 @@ import { getSlackClient, getSlackTeamInfo, getSlackUserInfo } from "../services/
 import { getBotTokenForTeam, setBotTokenForTeam } from "../services/workspace-tokens.js";
 import "../services/index.js";
 import { OpenAPIHono, z } from "@hono/zod-openapi";
-import { createWorkAppSlackWorkspace } from "@inkeep/agents-core";
+import { createWorkAppSlackWorkspace, isUniqueConstraintError, listWorkAppSlackWorkspacesByTenant } from "@inkeep/agents-core";
 import * as crypto$1 from "node:crypto";
 import { createProtectedRoute, noAuth } from "@inkeep/agents-core/middleware";
 
@@ -96,6 +96,7 @@ app.openapi(createProtectedRoute({
 		"chat:write",
 		"chat:write.public",
 		"commands",
+		"files:write",
 		"groups:history",
 		"groups:read",
 		"im:history",
@@ -212,6 +213,26 @@ app.openapi(createProtectedRoute({
 			appId: tokenData.app_id,
 			installedAt: (/* @__PURE__ */ new Date()).toISOString()
 		};
+		if (tenantId && workspaceData.teamId) {
+			let existingWorkspaces;
+			try {
+				existingWorkspaces = await listWorkAppSlackWorkspacesByTenant(runDbClient_default)(tenantId);
+			} catch (err) {
+				logger.error({
+					err,
+					tenantId
+				}, "Failed to check existing workspaces");
+				return c.redirect(`${dashboardUrl}?error=workspace_check_failed`);
+			}
+			if (existingWorkspaces.some((w) => w.slackTeamId !== workspaceData.teamId)) {
+				logger.warn({
+					tenantId,
+					newTeamId: workspaceData.teamId,
+					existingTeamIds: existingWorkspaces.map((w) => w.slackTeamId)
+				}, "Tenant already has a different Slack workspace, rejecting installation");
+				return c.redirect(`${dashboardUrl}?error=workspace_limit_reached`);
+			}
+		}
 		if (workspaceData.teamId && workspaceData.botToken) {
 			clearWorkspaceConnectionCache(workspaceData.teamId);
 			const nangoResult = await storeWorkspaceInstallation({
@@ -252,8 +273,7 @@ app.openapi(createProtectedRoute({
 						tenantId
 					}, "Persisted workspace installation to database");
 				} catch (dbError) {
-					const dbErrorMessage = dbError instanceof Error ? dbError.message : String(dbError);
-					if (dbErrorMessage.includes("duplicate key") || dbErrorMessage.includes("unique constraint")) logger.info({
+					if (isUniqueConstraintError(dbError)) logger.info({
 						teamId: workspaceData.teamId,
 						tenantId
 					}, "Workspace already exists in database");
@@ -261,7 +281,6 @@ app.openapi(createProtectedRoute({
 						const pgCode = dbError && typeof dbError === "object" && "code" in dbError ? dbError.code : void 0;
 						logger.error({
 							err: dbError,
-							dbErrorMessage,
 							pgCode,
 							teamId: workspaceData.teamId,
 							tenantId,

package/dist/slack/routes/users.js

@@ -3,7 +3,7 @@ import runDbClient_default from "../../db/runDbClient.js";
 import { createConnectSession } from "../services/nango.js";
 import "../services/index.js";
 import { OpenAPIHono, z } from "@hono/zod-openapi";
-import { createWorkAppSlackUserMapping, deleteWorkAppSlackUserMapping, findWorkAppSlackUserMapping, findWorkAppSlackUserMappingByInkeepUserId, verifySlackLinkToken } from "@inkeep/agents-core";
+import { createWorkAppSlackUserMapping, deleteWorkAppSlackUserMapping, findWorkAppSlackUserMapping, findWorkAppSlackUserMappingByInkeepUserId, isUniqueConstraintError, verifySlackLinkToken } from "@inkeep/agents-core";
 import { createProtectedRoute, inheritedWorkAppsAuth } from "@inkeep/agents-core/middleware";
 
 //#region src/slack/routes/users.ts
@@ -165,7 +165,7 @@ app.openapi(createProtectedRoute({
 			tenantId
 		});
 	} catch (error) {
-		if (error instanceof Error && (error.message.includes("duplicate key") || error.message.includes("unique constraint")) || typeof error === "object" && error !== null && "cause" in error && typeof error.cause === "object" && error.cause?.code === "23505") {
+		if (isUniqueConstraintError(error)) {
 			logger.info({ userId: body.userId }, "Concurrent link resolved — mapping already exists");
 			return c.json({ success: true });
 		}