@agent-native/core 0.132.2 → 0.133.0

package/dist/integrations/adapters/slack.js

@@ -5,7 +5,7 @@ import { getRequestContext } from "../../server/request-context.js";
 import { getIntegrationRequestContext } from "../../server/request-context.js";
 import { consumeIntegrationAwaitingInput } from "../awaiting-input-store.js";
 import { createIntegrationControl } from "../controls-store.js";
-import { getActiveIntegrationInstallationByKey, getActiveIntegrationInstallationForTenant, listIntegrationInstallations, resolveIntegrationTokenBundle, } from "../installations-store.js";
+import { getActiveIntegrationInstallationByKey, listActiveIntegrationInstallationsForTenant, listIntegrationInstallations, resolveIntegrationTokenBundle, } from "../installations-store.js";
 import { hasActivePendingTask } from "../pending-tasks-store.js";
 import { slackInstallationKey } from "../slack-oauth.js";
 /** Slack's max message length */
@@ -510,13 +510,16 @@ export function slackAdapter(options = {}) {
                     channelId: target.destination,
                     threadTs: target.threadRef,
                     teamId: target.tenantId,
+                    installationKey: target.installationKey,
                 },
                 tenantId: target.tenantId,
                 timestamp: Date.now(),
             };
             const token = await resolveBotToken(targetContext);
             if (!token) {
-                console.error("[slack] SLACK_BOT_TOKEN not configured");
+                console.error("[slack] no bot token for outbound target" +
+                    (target.tenantId ? ` (tenant ${target.tenantId})` : "") +
+                    "; set SLACK_BOT_TOKEN or pass installationKey to name the app");
                 return;
             }
             const chunks = splitNonEmptyMessage(message.text, SLACK_MAX_LENGTH);
@@ -631,17 +634,31 @@ async function resolveManagedSlackBotToken(incoming) {
     const enterpriseId = typeof incoming.platformContext.enterpriseId === "string"
         ? incoming.platformContext.enterpriseId
         : undefined;
+    const installationKeyHint = typeof incoming.platformContext.installationKey === "string"
+        ? incoming.platformContext.installationKey
+        : undefined;
     if (!teamId && !enterpriseId)
         return undefined;
     try {
-        let installation = apiAppId
-            ? await getActiveIntegrationInstallationByKey("slack", slackInstallationKey({ teamId, enterpriseId, apiAppId }))
+        let installation = installationKeyHint
+            ? await getActiveIntegrationInstallationByKey("slack", installationKeyHint)
             : null;
-        if (!installation && !apiAppId && teamId) {
-            installation = await getActiveIntegrationInstallationForTenant("slack", teamId);
+        if (!installation && apiAppId) {
+            installation = await getActiveIntegrationInstallationByKey("slack", slackInstallationKey({ teamId, enterpriseId, apiAppId }));
         }
-        if (!installation && !apiAppId && enterpriseId) {
-            installation = await getActiveIntegrationInstallationForTenant("slack", enterpriseId);
+        if (!installation && !apiAppId) {
+            // Without an app id the tenant can match several connected Slack apps.
+            // Sending as an arbitrary one posts under the wrong bot identity, so
+            // only proceed when the tenant resolves to exactly one installation.
+            const tenant = teamId ?? enterpriseId;
+            const candidates = await listActiveIntegrationInstallationsForTenant("slack", tenant);
+            if (candidates.length > 1) {
+                console.error(`[slack] ${candidates.length} connected Slack apps for tenant ${tenant}; ` +
+                    `cannot choose one without an app id. Pass installationKey on the outbound target. ` +
+                    `Candidates: ${candidates.map((c) => c.installationKey).join(", ")}`);
+                return undefined;
+            }
+            installation = candidates[0] ?? null;
         }
         const key = installation?.installationKey ??
             slackInstallationKey({ teamId, enterpriseId, apiAppId });