stacklok / toolhive-studio / 29903783585

72%

push

github

fix(chat): hollow assistant messages and session-tied auto-titles (#2491)

* fix(chat): drop hollow assistant messages before model calls

Moonshot rejects transcripts that still contain empty assistant placeholders left by aborted mid-stream snapshots after switching threads.

Co-authored-by: Cursor <cursoragent@cursor.com>

* fix(chat): auto-title threads from session model on stream complete

Title generation now uses the thread's provider/model, disables OpenRouter reasoning, falls back to the first user message, and runs in main when a stream finishes so switching threads no longer leaves chats stuck on "New chat".

Co-authored-by: Cursor <cursoragent@cursor.com>

* chore(chat): drop unused title-service exports for knip

Co-authored-by: Cursor <cursoragent@cursor.com>

* fix(chat): refresh sidebar after async title write

Broadcast chat:thread:updated after successful auto-title so the LLM title reaches the sidebar; gate title gen on finished streams; log LLM failures with cause; share the fallback title contract across processes.

Co-authored-by: Cursor <cursoragent@cursor.com>

* fix(chat): fall back to removeListener when IPC on has no unsubscribe

Match useStreamingThreads so playground stream/title listeners do not leak when electronAPI.on returns a non-function.

Co-authored-by: Cursor <cursoragent@cursor.com>

* fix(chat): broadcast thread updated only when title was written

Avoid a no-op sidebar refresh after streams that skip or leave the title unchanged.

Co-authored-by: Cursor <cursoragent@cursor.com>

* fix(chat): address kantord review on hollow transcripts and titles

Coalesce adjacent user turns after hollow removal for strict role
alternation, stop padding tool-only assistants, and make main the sole
durable title writer while keeping renderer optimistic fallback guarded.
Also dedupe broadcast/listener helpers and remove unused title IPC.

Co-authored-by: Cursor <cursoragent@cursor.com>

---------

Co-authored-by: Cursor <cursor... (continued)

5592 of 8285 branches covered (67.5%)

100 of 109 new or added lines in 9 files covered. (91.74%)

8284 of 11454 relevant lines covered (72.32%)

137.39 hits per line