[Refactor] Consolidate Fast tools on Roomote MCP - #1611
Merged
daniel-lxs merged 7 commits intoAug 25, 2026
Conversation
Contributor
|
No code issues found. See task
Reviewed 0b3fbe0 |
…ote-mcp-dedup-37yzgn4wpofti
daniel-lxs
marked this pull request as ready for review
August 25, 2026 00:12
daniel-lxs
requested review from
brunobergher,
daniel-lxs and
mrubens
as code owners
August 25, 2026 00:12
…ote-mcp-dedup-37yzgn4wpofti
Contributor
Author
|
Fixed in |
daniel-lxs
marked this pull request as ready for review
August 25, 2026 00:52
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
Removed Fast-native
manage_tasks,get_chat_message_context, andget_chat_channel_messages. Fast now routes task inspection and chat-context requests throughintegration_callto the actor-authorized Roomote MCP catalog introduced in #1604.Native tools with Fast-specific orchestration or visibility semantics remain unchanged: task launch/message/cancel, chat replies/reactions, event retry/ignore, and
integration_call. Fast injects its Slack or Discord conversation provider into Roomote MCP chat lookups and defaults omitted channels to the active conversation target. Slack uses the current channel, while Discord uses the current thread when present. Slack channel history still defaults to the previous 24 hours whenoldestis omitted.Fast also accepts the shared all-repositories target for delegated launches. Discord treats that sentinel as the existing deployment-wide workspace target instead of attempting to resolve it as a configured environment.
Why this change was made
Fast previously maintained parallel native contracts for capabilities now exposed by the Roomote MCP. Consolidating them avoids duplicated schemas, authorization paths, and tests while preserving the scoped current-conversation behavior of the removed chat adapters.
Impact
Fast retains task inspection and chat-context capabilities through the Roomote MCP, using the same actor-resolved catalog as delegated tasks. Chat lookups without an explicit channel continue to target the active Slack channel or Discord thread, while explicit channel references and message links remain supported. Explicit all-repositories launches work consistently from Slack and Discord. The native surface is smaller and has one source of truth for shared deployment operations.
Related PRs