lsm / neokai / 27651318130

82%

Build Type

push

github

Committed by web-flow

Commit Message

fix(openai-chat-bridge): guard catch-block send() against closed controller (#2118)

streamChatToAnthropic's catch block called `send(errorSSE(...))` and
`send(messageStopSSE())` unconditionally. `send` does
`controller.enqueue(...)`, which throws `TypeError: Invalid state:
Controller is already closed` when the readable side has already been
torn down (client disconnect, or upstream returning data that closed
the stream before the error surfaced). That secondary TypeError
escaped the catch block and crashed the daemon.

The adjacent `finally { controller.close() }` already had this guard.
Mirror it on the catch-block sends so the daemon logs the upstream
error without dying.

Adds a regression test that closes the controller up front and feeds
an upstream stream that errors, then asserts `streamChatToAnthropic`
resolves instead of throwing.

Coverage Stats

10517 of 15352 branches covered (68.51%)

Branch coverage included in aggregate %.

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

1 existing line in 1 file now uncovered.

90464 of 108296 relevant lines covered (83.53%)

291.4 hits per line