stacklok / toolhive-studio / 25387222797

68%

main: 68%

Pull #2192

github

fix(playground): preserve reasoning + tool-call disclosure across virtualization

`ReasoningComponent` and `ToolCallComponent` kept their expand/collapse
state in local `useState`, so when virtualized rows recycled in long
threads the user would lose the expanded reasoning pane or tool-result
view as soon as they scrolled away and back.

Lifts the four boolean flags (one for reasoning, three independent
slots for the tool-call card) into a small module-level store backed
by `useSyncExternalStore`. Each subscriber pulls a key-scoped
snapshot, so a toggle on one row never re-renders the others.

The components accept an optional `disclosureKey` and fall back to
`useId()` when absent, so unrelated callsites and existing tests are
unaffected. `<AssistantMessage>` threads `${message.id}:${partIndex}`
as the key, which is stable across remounts of the same row.

Addresses the remaining Copilot review thread on #2192.

4681 of 7451 branches covered (62.82%)

100 of 142 new or added lines in 13 files covered. (70.42%)

6990 of 10322 relevant lines covered (67.72%)

120.86 hits per line