stacklok / toolhive-studio / 25110033920

70%

Build Type

push

github

Committed by web-flow

Commit Message

feat(playground): persist streaming responses and survive route changes (#2137)

* feat(playground): keep chat streams alive across route changes and persist mid-stream

* feat(playground): reconnect to in-flight chat streams via useChat({ resume: true })

* feat(playground): live streaming indicator and hover-only menu in threads sidebar

* feat(playground): show in-flight placeholder for token usage while streaming

* fix: adjust hydratation fetch and condense comments

* refactor: remove streamUIMessagesOverIPC export

* fix(playground): tighten stream subscription lifecycle and detach abort listener

* test(playground): cover useStreamingThreads hook

* docs(playground): note bufferedChunks lifetime in active-streams

* fix(playground): plug IPC listener leak in chat transport

events.on registers a wrapped listener under the hood, so removeListener by reference never matched and listeners accumulated for the lifetime of the renderer. Use the unsubscribe handle events.on returns instead and update the global mocks to surface that contract.

* fix(playground): harden chat stream lifecycle

Address review feedback on PR #2137:

- Reject duplicate runManagedStream calls for the same chatId; surface setup failures to the renderer via chat:stream:error.

- Abort the upstream on stream error and on before-quit so the LLM socket is torn down promptly (switched app.once → app.on so cancelled-then-retried quits still flush).

- Cap bufferedChunks at MAX_BUFFER_CHUNKS=5000; subscribeToStream returns null past the cap so late subscribers hydrate from SQLite.

- Broadcast a one-shot chat:stream:persist-error and toast a single warning when the snapshot write fails (disk full, SQLite locked, …).

- Extract buildActiveStream / startAssembler / pumpChunks / finalize{Success,Error} / teardownStream so runManagedStream reads as a workflow; replace the persistence-boundary switch with a ReadonlySet.

* test(playground): cover registry duplicate, persist-error, over... (continued)

Coverage Stats

4285 of 7075 branches covered (60.57%)

290 of 416 new or added lines in 10 files covered. (69.71%)

2 existing lines in 2 files now uncovered.

6569 of 9895 relevant lines covered (66.39%)

121.76 hits per line