lsm / neokai / 24707234971

83%

push

github

fix(copilot): call client.start() before caching CopilotClient (#1568)

getOrCreateClient() was creating a CopilotClient but never calling
.start(), so the client had no underlying CLI connection. This caused
a "Client not connected" warning on every daemon start when listModels()
was called against the unstarted client.

Convert getOrCreateClient() to async and await client.start() before
assigning to clientCache. If start() throws the cache stays undefined,
allowing the next call to retry with a fresh instance.

Add unit tests that verify start() is called on construction, the cache
is not populated on failure, and errors propagate through ensureServerStarted().

8948 of 12221 branches covered (73.22%)

Branch coverage included in aggregate %.

5 of 5 new or added lines in 1 file covered. (100.0%)

66480 of 75270 relevant lines covered (88.32%)

301.2 hits per line