Skip to content

feat: show Slack MCP Server status on App Home in Socket Mode#58

Merged
mwbrooks merged 3 commits intomainfrom
mwbrooks-app-home-mcp-status
Apr 17, 2026
Merged

feat: show Slack MCP Server status on App Home in Socket Mode#58
mwbrooks merged 3 commits intomainfrom
mwbrooks-app-home-mcp-status

Conversation

@mwbrooks
Copy link
Copy Markdown
Member

@mwbrooks mwbrooks commented Apr 16, 2026

Summary

This pull request always shows the Slack MCP Server connection status on the App Home, including when the app is running in Socket Mode.

  • Previously, the MCP status section was hidden when running in Socket Mode (no SLACK_CLIENT_ID). Now it displays as disconnected with a "Learn how to enable the Slack MCP Server" link pointing to the README.
  • Updates tests to match the new default behavior.

Preview

image

Testing

  • Open the App Home while running in Socket Mode (slack run or python3 app.py) and confirm the MCP status shows as disconnected with a learn-more link.
  • Open the App Home while running in OAuth mode and confirm connected/disconnected status still works as before.
  • Click the "Learn how to enable" link and confirm it navigates to the correct README section.

When running in Socket Mode, the App Home now displays the Slack MCP
Server as disconnected with a link to the README section on how to
enable it. Previously, the MCP status was hidden in Socket Mode.
@mwbrooks mwbrooks changed the title Show Slack MCP Server status on App Home in Socket Mode feat: show Slack MCP Server status on App Home in Socket Mode Apr 16, 2026
@mwbrooks mwbrooks added the enhancement New feature or request label Apr 17, 2026
@mwbrooks mwbrooks self-assigned this Apr 17, 2026
@mwbrooks mwbrooks marked this pull request as ready for review April 17, 2026 16:52
@mwbrooks mwbrooks requested a review from a team as a code owner April 17, 2026 16:52
Copy link
Copy Markdown
Member

@zimeg zimeg left a comment

Choose a reason for hiding this comment

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

🎉 LGTM! LGTM! We might want to merge this after slack-samples/bolt-python-starter-agent#9 but no blockers-

Comment on lines +42 to +44
mcp_section = next(t for t in section_texts if "Slack MCP Server" in t)
assert "disconnected" in mcp_section
assert "Learn how to enable" in mcp_section
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🧪 praise: Thanks as always for keeping our tests good @mwbrooks!

@mwbrooks
Copy link
Copy Markdown
Member Author

@zimeg Thank you, thank you! Tireless reviews on a Friday morning 📰 🙇🏻

@mwbrooks mwbrooks merged commit d44f7ef into main Apr 17, 2026
8 checks passed
@mwbrooks mwbrooks deleted the mwbrooks-app-home-mcp-status branch April 17, 2026 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants