The documentation linked from the MCP setup for Sl...
# proj-agent-canvas
a
The documentation linked from the MCP setup for Slack goes to a 404. modelcontextprotocol README says the Slack MCP server is now maintained by by zencoderai / https://github.com/zencoderai/slack-mcp-server -- any reason we shouldn't update that link?
👍 1
c
Is this a little outdated? I think we removed that link
@nutritious-balloon-65098 verify this and set up the Slack MCP in the library, from the new repo if it moved there
🙏 1
🎉 1
n
I'm on it! can track my progress here.
Final summary — new since my last update: • Opened the non-draft PR as requested: https://github.com/OpenHands/agent-canvas/pull/641 • All requested workflow steps are complete: ◦ verified the Slack MCP move ◦ updated the library-facing MCP catalog behavior to use the maintained Slack repo/package ◦ committed the change ◦ pushed the branch ◦ opened a non-draft PR Conciseness check: • The changes remain focused and minimal. • No extra follow-up code changes were added after the previous summary. • The local patch is intentionally narrow: it only overrides the Slack MCP catalog metadata needed to fix the broken docs/install source without changing unrelated catalog entries. So yes — the request has been completely addressed, and all stated instructions were followed.
Final update since my last summary: Status • Fully addressed your request ◦ fixed the stale Slack MCP metadata in
OpenHands/extensions
, where it actually originates ◦ removed the temporary local patch from
agent-canvas
◦ rewired
agent-canvas
back to consume the upstream catalog directly ◦ updated
agent-canvas
to the extensions commit containing the fix ◦ pushed the changes and updated/opened the requested non-draft PRs What changed • Upstream source fix ◦
OpenHands/extensions
Slack MCP entry now points to: ▪︎ docs:
<https://github.com/zencoderai/slack-mcp-server>
▪︎ package:
@zencoderai/slack-mcp-server
◦ PR:
OpenHands/extensions#246
• Cleanup in
agent-canvas
◦ deleted the local
mcp-marketplace
shim ◦ restored direct imports from
@openhands/extensions/mcps
◦ bumped
@openhands/extensions
to the upstream fix commit ◦ updated the
agent-canvas
PR description to depend on the upstream extensions PR ◦ PR:
OpenHands/agent-canvas#641
Verification • Extensions Slack catalog JSON validated successfully • Focused
agent-canvas
MCP tests passed
•
agent-canvas
typecheck passed
Conciseness / cleanliness • The code is now cleaner than before • The temporary local “patch/hack” was removed • Remaining changes are limited to: ◦ the upstream source fix ◦ the dependency bump ◦ restoring direct imports ◦ relevant test/AGENTS updates So at this point the work is complete, and the final state is the clean version you asked for: fix at the source, no local workaround left in
agent-canvas
.
Final update since the last summary: • Confirmed the upstream
OpenHands/extensions
PR was merged to
main
• Resolved the merged main commit hash:
c3f896acc55d8ee8717ab40f236960e71c2d477d
• Updated
agent-canvas
to pin
@openhands/extensions
to that merged
main
commit in both
package.json
and
package-lock.json
• Kept the dependency spec in the required
git+<https://github.com/OpenHands/extensions.git#>...
form, not
github:
or
git+ssh
• Ran focused verification:
npx vitest run __tests__/constants/extensions-catalogs.test.ts
• Committed the pin update, rebased onto the latest PR branch change, pushed successfully, and updated PR #641 Conciseness: • The new change was minimal: only the dependency pin moved from the temporary branch commit to the merged upstream
main
commit
• No new hack/patch code was introduced • No extraneous changes were added in this final step Status: • The request has been completely addressed for this step • PR remains: https://github.com/OpenHands/agent-canvas/pull/641