lsm / neokai / 23113575934

84%

dev: 84%

push

github

Dev -> Main (#361)

* feat(tasks): make failed state non-terminal (allow messages + retry)

Allow tasks in needs_attention status to receive human messages and
auto-restart via the established reviveTaskForMessage path, aligning
the RPC handler with the agent-tool path.

Changes:
- routeHumanMessageToGroup: pure routing only — blocks all terminated
  groups (completedAt set), no longer handles needs_attention restart
- task-handlers.ts: intercepts needs_attention before routing:
  setTaskStatus('review') → reviveTaskForMessage() → rollback on failure
- room-agent-tools.ts: same pattern for cancelled guard + revival
- TaskOperator interface: trimmed to just getTask (setTaskStatus removed)
- Tests: 9 focused routing tests + 2 revival tests with rollback assertion
- ci: allow any branch to merge into main (policy relaxation)

* fix: pass taskManager as parameter to avoid accessing private field

- routeHumanMessageToGroup now accepts taskManager as a parameter
  instead of accessing runtime.taskManager (which is private)
- Added TaskOperator interface for the required method signatures
- Updated TaskManagerLike to include updateTaskStatus

Fixes typecheck error from PR #301 review

* fix: align needs_attention handling between RPC and agent-tool paths

Previously routeHumanMessageToGroup did a full metadata wipe
(resetGroupForRestart) for needs_attention tasks, while the agent-tool path
used the lightweight reviveGroup + reviveTaskForMessage which preserves
conversation history and metadata.

Fix the divergence:
- Remove needs_attention restart logic from routeHumanMessageToGroup entirely.
  The function now only routes messages to *active* groups; all terminated
  groups are blocked with a clear error.
- Add needs_attention pre-handling to task-handlers.ts (human RPC path) that
  mirrors room-agent-tools.ts: setTaskStatus('review') +
  runtime.reviveTaskForMessage(), with rollback on failure.

This also eliminates the double-rollback bug (failGroup sett... (continued)

3014 of 3634 branches covered (82.94%)

Branch coverage included in aggregate %.

177 of 185 new or added lines in 9 files covered. (95.68%)

26500 of 31394 relevant lines covered (84.41%)

244.67 hits per line