lsm / neokai / 25646115400

83%

push

github

feat: add update_task MCP tool for space agent and task agent (#1852)

* feat: add update_task MCP tool for space agent and task agent

Add an update_task tool to two MCP servers:

1. space-agent-tools — space agent can edit any task in the space
2. node-agent-tools (task agent variant only) — Task Agent can edit tasks it manages

NOT added to node agents (coder, reviewer, etc.) — task mutation is restricted
to coordination-level agents.

Implementation:
- Add UpdateTaskSchema + UpdateTaskInput to node-agent-tool-schemas.ts
- Add update_task handler to space-agent-tools.ts (look up task, verify space,
  call taskManager.updateTask, emit space.task.updated, log audit)
- Add conditional update_task handler to node-agent-tools.ts (gated by onUpdateTask
  callback; tool invisible when callback absent)
- Wire onUpdateTask callback in TaskAgentManager.buildNodeAgentMcpServerForSession
  so Task Agent sessions get the tool; regular node agents do not

Fields supported: title, description, priority, depends_on.
Excluded (runtime-managed): status, workflowRunId, activeSession, result, labels.

Tests:
- Space agent: update title/desc/priority/deps, multi-field, not-found,
  cross-space guard, event emission, MCP registration
- Node agent: delegates to callback, error when absent, MCP registration gating
- Task agent manager: onUpdateTask callback is wired and delegates to updateTask

* fix: restrict update_task to Task Agent, remove from node-agent path

- Remove onUpdateTask callback from NodeAgentToolsConfig and
  buildNodeAgentMcpServerForSession — this builder is used for ALL
  workflow node-agent sub-sessions (coder, reviewer, etc.), so passing
  the callback unconditionally broke the permission boundary.
- Add update_task directly to task-agent-tools.ts with taskId scoping:
  the handler rejects any task_id that does not match the session's
  bound task, preventing cross-task mutation.
- Emit space.task.updated event on successful updates (consistent with... (continued)

8347 of 11648 branches covered (71.66%)

Branch coverage included in aggregate %.

114 of 120 new or added lines in 3 files covered. (95.0%)

65408 of 77811 relevant lines covered (84.06%)

289.37 hits per line