Skip to content

feat(mcp): workspace parity for bus_connect and thread_create (UL-8) - #78

Merged
bertheto merged 2 commits into
Killea:mainfrom
bertheto:feat/mcp-workspace-parity
Jul 28, 2026
Merged

feat(mcp): workspace parity for bus_connect and thread_create (UL-8)#78
bertheto merged 2 commits into
Killea:mainfrom
bertheto:feat/mcp-workspace-parity

Conversation

@bertheto

Copy link
Copy Markdown
Collaborator

Summary

  • Add optional workspace string parameter to MCP thread_create and bus_connect tool schemas.
  • Validate workspace paths with the same rules as REST (workspace must be an existing directory) and persist as cli_workspace thread metadata via MemoryStore.createThread.
  • Return workspace in MCP responses when set; bus_connect only applies workspace on new thread creation (ignored when joining an existing thread).

Unblocks UL-8 / UP-39 MCP workspace parity for RHUB orchestrator adoption.

Test plan

  • npx vitest run tests/integration/httpServer.test.ts (31/31 passed)
  • MCP thread_create stores workspace on new threads
  • MCP thread_create rejects invalid workspace paths
  • MCP bus_connect stores workspace when creating a thread

Notes

  • Kanban #278 remains open until post-merge adoption in RHUB orchestrator (out of scope for this PR).

bertheto added 2 commits July 29, 2026 00:51
Expose MCP parity with REST thread workspace handling (cli_workspace metadata)
so orchestrators can bind threads to an existing directory path.

Refs UL-8 / UP-39 workspace parity.
@bertheto
bertheto merged commit ee02103 into Killea:main Jul 28, 2026
1 check passed
bertheto added a commit to bertheto/AgentChatBus that referenced this pull request Jul 29, 2026
Sync vscode-agentchatbus web-ui assets (split-view, thread UI) and align
BUS_VERSION with PR Killea#78 MCP workspace parity so bundled VSIX exposes
workspace on bus_connect and thread_create.
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