lsm / neokai / 24761020454 / 3
82%
dev: 82%

DEFAULT BRANCH: dev
Ran
Files 98
Run time 4s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 43.16% (-0.06%) from 43.219%
24761020454.3

push

github

web-flow 
fix(space): restore node-agent MCP tools in workflow sessions after restart (#1586)

* fix(space): restore node-agent MCP tools in workflow sessions after restart

Adds mergeRuntimeMcpServers, detachRuntimeMcpServer, and restartQuery to
AgentSession so TaskAgentManager can attach/refresh MCP tools additively
without clobbering other runtime servers (space-agent-tools, db-query, etc.).

The reinjectNodeAgentMcpServer primitive now calls mergeRuntimeMcpServers +
restartQuery so the SDK picks up the fresh node-agent tool registry when
self-healing after a daemon restart or session rehydration.

Test mocks in 5 task-agent-manager test files are updated to implement the
new interface methods; two raw SQL workflow-node seeds now include an agents
config so resolveNodeAgents does not throw on reinject.

* test(agent): add unit tests for merge/detach MCP API and updateUserMcpServers

- `detachRuntimeMcpServer`: remove, no-op when absent, no DB persistence, rotate pattern
- `updateUserMcpServers` in SessionConfigHandler: preserves SDK-type servers, replaces
  subprocess servers, prevents user-supplied servers from overwriting runtime servers,
  persists to DB, emits session.updated
- config-handlers mock: wire `updateUserMcpServers` so handler tests don't throw

* fix(space): address review feedback on MCP tool loss fixes

- Fix missing config column in the third workflow-node SQL seed
  (task-agent-manager.test.ts line ~1071) for consistency with
  the other two seeds fixed in the original commit
- config.mcp.removeServer: return an error instead of silent success
  when the caller tries to remove a runtime SDK-managed server
- Document NEOKAI_DEBUG_MCP_INVARIANTS in .env.example
- Enumerate the 6 remaining setRuntimeMcpServers call sites in the
  @deprecated JSDoc so future migrations have a clear tracking point

9601 of 22245 relevant lines covered (43.16%)

22.64 hits per line

Source Files on job daemon-5-space-runtime - 24761020454.3