stacklok / toolhive-studio / 24734052733

67%

main: 71%

Build Type

Pull #2062

github

Committed by samuv

Commit Message

refactor(chat): delegate chat scroll restoration to TanStack Router

Enable router-level scrollRestoration keyed by location.pathname and
register the chat messages container with data-scroll-restoration-id so
TSR persists and restores its scroll position across navigations. Apply
overflow-anchor: auto on the scroll container and each MCP app iframe
wrapper so async height changes don't shove the viewport.

useAutoScroll is rewritten around the new contract: its responsibilities
shrink to following the bottom while streaming, the scroll-to-bottom
button, and an initial 5s settling window that re-applies the saved
scroll target as MCP iframes finish their size handshake. Distinguishes
overflow-anchor-driven scroll events from genuine user input (wheel /
touch / key / pointer) so the settling window isn't cancelled by the
first iframe resize. The in-component loading overlay is removed — the
route's pendingComponent now owns that state.

Pull Request Pull Request #2062: feat(playground): preload chat thread data in the route loader and delegate scroll restoration to TanStack Router

Coverage Stats

3876 of 6366 branches covered (60.89%)

110 of 117 new or added lines in 8 files covered. (94.02%)

22 existing lines in 1 file now uncovered.

6025 of 9056 relevant lines covered (66.53%)

118.88 hits per line