Skip to content

[Refactor] Consolidate Fast tools on Roomote MCP - #1611

Merged
daniel-lxs merged 7 commits into
developfrom
refactor/fast-roomote-mcp-dedup-37yzgn4wpofti
Aug 25, 2026
Merged

[Refactor] Consolidate Fast tools on Roomote MCP#1611
daniel-lxs merged 7 commits into
developfrom
refactor/fast-roomote-mcp-dedup-37yzgn4wpofti

Conversation

@roomote-roomote

@roomote-roomote roomote-roomote Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

​Opened on behalf of @daniel-lxs. Follow up by mentioning @roomote-roomote or in the web UI.

What changed

Removed Fast-native manage_tasks, get_chat_message_context, and get_chat_channel_messages. Fast now routes task inspection and chat-context requests through integration_call to 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 when oldest is 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

@roomote-community

roomote-community Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

No code issues found. See task

  • Chat lookups through integration_call no longer receive the Fast conversation provider, so raw channel IDs fail. (packages/cloud-agents/src/server/fast-agent/fast-agent-service.ts:913)
  • Fast chat lookups no longer default to the current conversation channel, so calls without an explicit channel fail. (packages/cloud-agents/src/server/fast-agent/fast-agent-service.ts:938)
  • Advertising the all-repositories target breaks Fast launches from Discord because the sentinel is resolved as a concrete environment. (apps/api/src/handlers/discord/fast-agent.ts:141)

Reviewed 0b3fbe0

Comment thread packages/cloud-agents/src/server/fast-agent/fast-agent-service.ts
@daniel-lxs
daniel-lxs marked this pull request as ready for review August 25, 2026 00:12
Comment thread packages/cloud-agents/src/server/fast-agent/fast-agent-service.ts
@roomote-roomote
roomote-roomote Bot marked this pull request as draft August 25, 2026 00:23
@roomote-roomote

roomote-roomote Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor Author

Fixed in 0b3fbe0f: Discord Fast launches now treat __all_repositories__ as the deployment-wide workspace target instead of resolving it as a concrete environment. Added a focused adapter regression test; related Fast launch tests, affected package typechecks, and the full pre-push gate pass.

@daniel-lxs
daniel-lxs marked this pull request as ready for review August 25, 2026 00:52
@daniel-lxs
daniel-lxs merged commit 1883ca2 into develop Aug 25, 2026
16 checks passed
@daniel-lxs
daniel-lxs deleted the refactor/fast-roomote-mcp-dedup-37yzgn4wpofti branch August 25, 2026 01:08
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.

1 participant