lsm / neokai / 22554288852

84%

Build Type

push

github

Committed by web-flow

Commit Message

fix: prevent SDK startup timeout after model switch mid-session (#172)

* fix: prevent SDK startup timeout after model switch mid-session

Three bugs caused model switching to break sessions with substantial history:

1. Session file validation gap: validateAndRepairSDKSession() only ran via
   ensureQueryStarted() (user message path), never during model switch
   restart() or Reset Agent reset(). Corrupted session files from interrupted
   SDK compaction were never repaired.

2. Race condition in runQuery() finally block: shared state cleanup (abort
   controller, timers) ran before the stale query check, causing the old
   query's finally block to abort the new query's abort controller.

3. No recovery from startup timeout: when SDK startup timed out (15s),
   sdkSessionId was never cleared, so every subsequent restart/reset
   attempted to resume the same problematic session file.

Fixes:
- Add validateAndRepairSDKSession() to restart() and reset() paths
- Move isStaleQuery check before shared state cleanup in finally block
- Clear sdkSessionId on startup timeout for automatic recovery
- Add error/circuit breaker clearing to restart()

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

* style: fix biome formatting in query-runner.ts

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

---------

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

Run Details

2561 of 3019 branches covered (84.83%)

Branch coverage included in aggregate %.

27 of 27 new or added lines in 2 files covered. (100.0%)

98 existing lines in 2 files now uncovered.

20255 of 23979 relevant lines covered (84.47%)

265.89 hits per line