lsm / neokai / 25261971329

82%

Build Type

push

github

Committed by web-flow

Commit Message

Guard idle node agents with last-message checks (#1755)

* Guard idle node agents with last-message checks

Inspect the last persisted SDK message for idle node-agent sessions.
Non-terminal messages (unresolved tool_use, thinking-only, no message)
trigger retry-to-pending up to a limit, then block/escalate. Terminal
messages allow normal workflow progression. Use rowid for deterministic
insertion-order tie-breaking in getLastSDKMessage.

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

* Fix getLastSDKMessage ordering and test SDK table bootstrap

Use rowid DESC (monotonic insertion order) instead of random UUID id
DESC for deterministic tie-breaking in getLastSDKMessage. Clear
agentSessionId on auto-completed executions so the non-terminal idle
handler does not re-process them. Add sdk_messages table to runtime
test fixtures that exercise idle execution paths.

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

* Exhaust blocked retry budget on non-terminal idle escalation

When handleNonTerminalIdleExecutions blocks a run after exhausting its
retry budget, set blockedRetryCounts to MAX_BLOCKED_RUN_RETRIES so
that attemptBlockedRunRecovery does not immediately auto-retry the
blocked execution. This preserves the escalation signal for the Space
Agent instead of re-spawning agents that will likely idle again.

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

* Filter getLastSDKMessage to top-level SDK rows only

Exclude subagent/tool-linked messages (those with parent_tool_use_id)
from getLastSDKMessage so that a subagent result cannot shadow a
stalled main-thread message and cause a false terminal classification.

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

* Reset nonTerminalIdleCounts on recovery and exclude unsent user rows

Clear nonTerminalIdleCounts for a run's executions when
recoverWorkflowBackedTask recovers the run, so manual recovery starts
with a fresh retry budget. Exclude deferred/enqueued user messages... (continued)

Coverage Stats

7843 of 10812 branches covered (72.54%)

Branch coverage included in aggregate %.

180 of 212 new or added lines in 4 files covered. (84.91%)

1 existing line in 1 file now uncovered.

61119 of 85907 relevant lines covered (71.15%)

253.41 hits per line