stacklok / toolhive-studio / 25860028129

70%

Build Type

push

github

Committed by web-flow

Commit Message

fix(deps): update react monorepo (#2056)

* fix(deps): update react monorepo

* fix(chat): satisfy react-hooks lint rules from plugin v7.1.x

The eslint-plugin-react-hooks 7.1 bump in this PR enables stricter
`set-state-in-effect` and `refs` rules. Refactor the flagged sites to
the React-recommended patterns: derive state during render, sync via
the "adjust state on prop change" pattern, log-only effects, and write
refs from effects rather than during render.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* fix(chat): satisfy react-hooks rules in queue auto-flush

The queue auto-flush in `useChatStreaming` was still tripping two of
the new react-hooks v7.1 rules introduced by this PR:

- `prevQueueStatusRef.current` was being written during render in the
  thread-switch branch. Drop the reset — clearing `queuedMessage` in
  the same render already prevents the race the regression test guards.
- `setQueuedMessage(null)` inside the auto-flush effect tripped
  `set-state-in-effect`. Move the dispatch into `useEffectEvent` so the
  state update is non-reactive code, and gate it on
  `queueOwnerThreadId === currentThreadId` for an extra safety net
  against the thread-switch race.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Samuele Verzi <samu@stacklok.com>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Coverage Stats

5038 of 7754 branches covered (64.97%)

33 of 44 new or added lines in 5 files covered. (75.0%)

2 existing lines in 1 file now uncovered.

7516 of 10671 relevant lines covered (70.43%)

117.27 hits per line