lsm / neokai / 24972429555

82%

push

github

refactor(space): move per-node timeouts from runtime constants into workflow defs (#1649)

The runtime no longer carries an agent-role timeout table. Per-slot overrides
live with the workflow definition itself (`WorkflowNodeAgent.timeoutMs`); the
runtime's only timeout knob is `DEFAULT_NODE_TIMEOUT_MS = 4m30s`, sized to fit
within a model prompt-cache window.

- Add `WorkflowNodeAgent.timeoutMs` and round-trip it through the JSON config
  blob in `space_workflow_nodes` (no DDL change) and through export/import.
- New `resolveTimeoutForExecution()` helper reads the slot timeout from the
  workflow definition; returns `undefined` for "no override" so the runtime
  applies the default.
- Drop `CODER_/REVIEWER_/QA_/PLANNER_NODE_TIMEOUT_MS`, the `general → coder`
  string-keyed alias, and `resolveNodeTimeout()`.
- Add unit tests for the helper and a workflow-repo round-trip test.

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

9258 of 12427 branches covered (74.5%)

Branch coverage included in aggregate %.

22 of 22 new or added lines in 5 files covered. (100.0%)

66340 of 78708 relevant lines covered (84.29%)

299.26 hits per line