From 3648c021f8cc858926f51acb64b6ed325b361446 Mon Sep 17 00:00:00 2001 From: Cursor Agent Date: Wed, 21 Oct 2026 02:24:05 +0000 Subject: [PATCH] Name Slack MCP again, and both sources for Benny. /why searches a Slack MCP. Benny runs as a Factory automation, so it prefers Slack MCP or connector actions rather than a connector alone. Co-authored-by: Kaitian Xie --- automations/benny/skills/reproduce-and-fix-issues/SKILL.md | 2 +- automations/benny/skills/setup-benny/SKILL.md | 2 +- skills/why/references/sources/slack.md | 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/automations/benny/skills/reproduce-and-fix-issues/SKILL.md b/automations/benny/skills/reproduce-and-fix-issues/SKILL.md index fa6aeea..a1ffd50 100644 --- a/automations/benny/skills/reproduce-and-fix-issues/SKILL.md +++ b/automations/benny/skills/reproduce-and-fix-issues/SKILL.md @@ -119,7 +119,7 @@ Use the configured plain Unicode status strings. Keep status text short: - Draft pull request opened - Fix did not land -Prefer configured Slack connector actions. Use `BENNY_SLACK_BOT_TOKEN` only when the user configured it for a narrow missing capability such as editing this one status message. Never expose the token to a worker. +Prefer configured Slack MCP or connector actions. Use `BENNY_SLACK_BOT_TOKEN` only when the user configured it for a narrow missing capability such as editing this one status message. Never expose the token to a worker. If no operations channel is configured, keep detailed status in the automation run output. Do not substitute a source-channel root message. diff --git a/automations/benny/skills/setup-benny/SKILL.md b/automations/benny/skills/setup-benny/SKILL.md index 883e29a..87b15b7 100644 --- a/automations/benny/skills/setup-benny/SKILL.md +++ b/automations/benny/skills/setup-benny/SKILL.md @@ -127,7 +127,7 @@ The repro automation needs: - A pull request action that can open a draft pull request - The configured control-adapter skill -Prefer configured Slack connector actions for reads and posts. The optional `BENNY_SLACK_BOT_TOKEN` may fill a narrow gap such as editing one operations status message or downloading an attachment. Store the value in a secret manager or environment, not in YAML. +Prefer configured Slack MCP or connector actions for reads and posts. The optional `BENNY_SLACK_BOT_TOKEN` may fill a narrow gap such as editing one operations status message or downloading an attachment. Store the value in a secret manager or environment, not in YAML. Do not use undocumented integration endpoints. diff --git a/skills/why/references/sources/slack.md b/skills/why/references/sources/slack.md index 219b4c6..0b2c829 100644 --- a/skills/why/references/sources/slack.md +++ b/skills/why/references/sources/slack.md @@ -2,7 +2,7 @@ ## How to search it -Slack connector tools vary. Check which Slack connector is configured and inspect its tool schema first. If authentication fails, stop and report the gap. +Slack MCP tools vary. Check which Slack MCP is available and inspect its tool schema first. If authentication fails, stop and report the gap. 1. **Author-bounded search.** Messages from the PR author around the PR merge date. Limits scope dramatically and often hits gold. 3. **PR URL search.** Slack often links PRs when they're reviewed or discussed. Search for the PR URL (or just `/pull/`). @@ -14,4 +14,4 @@ Slack connector tools vary. Check which Slack connector is configured and inspec - **Unsearched DMs.** Many decisions happen in DMs that aren't searchable. You'll miss them. That's a known limitation. - **Speculative jokes as "decisions."** Slack is casual. "Lol just do the thing" isn't a decision, even if it preceded the commit. Look for considered discussion. - **Context collapse in single messages.** Without the thread, a single message often reads differently than in context. Always fetch threads. -- **Auth failures.** If the connector isn't authenticated, stop. Don't make up findings. Report that Slack wasn't searchable. +- **Auth failures.** If the MCP isn't authenticated, stop. Don't make up findings. Report that Slack wasn't searchable.