stacklok / toolhive-studio / 24515551334

70%

Build Type

push

github

Committed by web-flow

Commit Message

feat(playground): thread ID in URL with sub-route architecture (#2003)

* feat(playground): add sub-routes with thread ID in URL

Convert /playground to a layout route with an Outlet. Add a /playground/
index route that resolves the last-viewed thread (stored active ID or most
recent) and redirects to /playground/chat/$threadId. Extract PlaygroundContent
into the new /playground/chat/$threadId route which reads the thread from
params and wires thread navigation via useNavigate.

* refactor(playground): make usePlaygroundThreads URL-driven

Accept activeThreadId as input instead of managing it as internal state.
Persist it to IPC via a side-effect. createThread and deleteThread now
return the new/next thread ID so callers can navigate programmatically.
Remove selectThread — navigation is the caller's responsibility.

* test(playground): update tests for sub-route and URL-driven thread selection

Reduce playground layout route tests to the permission gate only. Add a new
test suite for the /playground/chat/$threadId route covering loading state,
sidebar/ChatInterface prop wiring, and threadId-from-param behaviour. Update
usePlaygroundThreads tests for the new hook signature, createThread/deleteThread
return values, and IPC persistence side-effect.

* fix(playground): move index redirect to component effect for permission safety

The beforeLoad hook runs before the parent layout renders, bypassing the
permission gate. Move thread resolution and redirect logic into a component
useEffect so it only executes after the layout allows rendering. Also handle
createChatThread failure explicitly with error logging instead of rendering
a blank page.

* fix(playground): use ref in deleteThread to avoid stale closure

Read from threadsRef.current instead of the captured threads closure so
concurrent state updates from refreshThread are not overwritten. Add tests
for the index route covering all redirect branches: stored active thread,
stale ID fallback, no stored ID... (continued)

Coverage Stats

3542 of 5901 branches covered (60.02%)

61 of 70 new or added lines in 5 files covered. (87.14%)

4 existing lines in 2 files now uncovered.

5665 of 8630 relevant lines covered (65.64%)

121.33 hits per line