lsm / neokai / 24805936249

83%

push

github

feat(space): sync-attach space tools + surface runtime MCPs in Tool Modal (#1598)

* fix(space): synchronously attach space-agent-tools on session.create RPC

The previous fix (#1596) emitted session.created on daemonHub, but
TypedHub.dispatchLocally does not await async subscribers — it fires
handlers and swallows their promises. So attachSpaceToolsToMemberSession
could still lose the race with QueryOptionsBuilder.build(), which freezes
session.config.mcpServers at first-query start.

Mirror the pattern space.create already uses for setupSpaceAgentSession:
call spaceRuntimeService.attachSpaceToolsToMemberSession() directly from
the session.create RPC, awaited before returning. The daemonHub emit is
retained for non-critical subscribers (StateManager et al).

setupSessionHandlers is moved in rpc-handlers/index.ts to run after
spaceRuntimeService is constructed so the reference can be injected.

Tests: 3 new cases verify attachment fires with the right session, is
skipped when context.spaceId is absent, and that RPC still returns
when the attach call throws.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* feat(web): surface runtime-attached MCP servers in Tool Modal

Adds session.listRuntimeMcpServers RPC that filters session.config.mcpServers
to SDK-type entries (runtime-injected MCPs like space-agent-tools, db-query,
task-agent, node-agent, room-tools). Tool Modal now shows an "Agent Runtime
Tools" section so users can see which built-in MCPs are active for each
session, independent of session type.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>

8971 of 12185 branches covered (73.62%)

Branch coverage included in aggregate %.

36 of 38 new or added lines in 3 files covered. (94.74%)

67416 of 76211 relevant lines covered (88.46%)

302.44 hits per line