stacklok / toolhive-studio / 25500470366

68%

main: 71%

Build Type

Pull #2207

github

Committed by samuv

Commit Message

fix(playground): seed URL-driven draft id into the sidebar threads list

After deferring thread persistence to the first message, a fresh /playground visit (or a deep link to /playground/chat/<draft-id>) loaded zero DB threads, so hasThreads stayed false: the sidebar disappeared and ChatInterface received threadId={undefined}, which fell into the legacy useThreadManagement auto-create path and called createChatThread() over IPC — re-introducing the empty-row write the previous commit removed.

Add a small effect in usePlaygroundThreads that runs after the initial DB load and prepends a { pending: true } entry for the URL-driven activeThreadId when it is not present in the fetched list. Symmetrical for the fresh-visit and deep-link cases. Tests cover the seeded-draft, already-in-DB, and null-id branches; two existing tests that incidentally relied on the pre-fix mismatch behavior are aligned to use a matching DB id.

Addresses Copilot review on this PR.

Pull Request Pull Request #2207: fix(playground): defer empty chat thread persistence until first message

Coverage Stats

4696 of 7463 branches covered (62.92%)

33 of 34 new or added lines in 6 files covered. (97.06%)

2 existing lines in 2 files now uncovered.

7012 of 10334 relevant lines covered (67.85%)

120.86 hits per line