Skip to content

Restore Slack MCP, and name both Benny sources - #23

Merged
caelaxie merged 1 commit into
mainfrom
cursor/slack-mcp-or-connector-9106
Oct 21, 2026
Merged

caelaxie merged 1 commit into
mainfrom
cursor/slack-mcp-or-connector-9106

Conversation

@caelaxie

Copy link
Copy Markdown
Contributor

Closes #20

Is a Factory Slack connector the expected source for Benny?

It is one expected source, not the only one. Benny runs as a Factory automation, and Factory automations use the run identity's connector access for Slack. Droid also supports Slack through MCP, and /why already tells the agent to inspect configured MCP servers and connectors. Connector alone was too narrow, so Benny says "Slack MCP or connector".

skills/why/references/sources/slack.md says Slack MCP again. mcp_auth stays dropped; that tool name is Cursor's. The auth-failure line says "If the MCP isn't authenticated".

Open in Web Open in Cursor 

/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 <caelaxie@users.noreply.github.com>

@caelaxie caelaxie left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

verdict: pass, merging.

  • min changes: skills/why/references/sources/slack.md L5/L17 now match upstream 88d4751 minus only the cursor-specific mcp_auth sentence, exactly #20's smallest fix. benny's two lines take the "Slack MCP or connector" option #20 offered, and the body answers the question (factory automations use connectors, droid also has mcp).
  • layout/target: no cursor files or shims touched.
  • parity: no version change needed.
  • proof: text-only skill change; ci Validate plugin (droid marketplace add + install + list + installed-copy checks) green at 3648c02.
  • linkage: Closes #20. not a draft.

@caelaxie
caelaxie merged commit c9cb93b into main Oct 21, 2026
1 check passed
@cursor

cursor Bot commented Oct 21, 2026

Copy link
Copy Markdown

Harness proof. Command output is unedited.

HOME=/tmp/droid-proof-slack
which=/tmp/droid-proof-slack/.local/bin/droid
0.245.0
===== droid plugin marketplace add =====
Successfully added marketplace: droid-pstack@cursor-slack-mcp-or-connector-9106
===== droid plugin install =====
Successfully installed pstack@droid-pstack@cursor-slack-mcp-or-connector-9106 (3648c02)
===== droid plugin list =====
Installed plugins:
Active:
  pstack@droid-pstack@cursor-slack-mcp-or-connector-9106  [user]  3648c02
$ git -C /tmp/droid-proof-slack/.factory/plugins/cache/droid-pstack@cursor-slack-mcp-or-connector-9106-bcc7c52eb073/pstack-7171b73fae68/3648c021f8cc rev-parse HEAD
3648c021f8cc858926f51acb64b6ed325b361446
(exit 0)
$ echo expected
3648c021f8cc858926f51acb64b6ed325b361446
$ git -C /workspace diff --name-status 3a60c3580a01bba100804e6efe3395d9f5dd924e cursor/slack-mcp-or-connector-9106
M	automations/benny/skills/reproduce-and-fix-issues/SKILL.md
M	automations/benny/skills/setup-benny/SKILL.md
M	skills/why/references/sources/slack.md
(exit 0)
MATCH automations/benny/skills/reproduce-and-fix-issues/SKILL.md
MATCH automations/benny/skills/setup-benny/SKILL.md
MATCH skills/why/references/sources/slack.md
===== droid exec =====
Skill "why" activated
Error during droid execution: Authentication failed. Please log in using /login or set a valid FACTORY_API_KEY environment variable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

audit: slack mcp renamed to connector

2 participants