Enforce reader ownership across chat scroll work - #458
Merged
hamzamerzic merged 1 commit intoAug 1, 2026
Annotations
10 warnings
|
Run frontend unit tests:
frontend/src/components/ChatView/ChatView.jsx#L1289
React Hook useLayoutEffect has missing dependencies: 'messagesRef', 'offsetRef', 'pendingQueue', and 'setServerRunningState'. Either include them or remove the dependency array
|
|
Run frontend unit tests:
frontend/src/components/ChatView/ChatView.jsx#L1231
React Hook useCallback has a missing dependency: 'setServerRunningState'. Either include it or remove the dependency array
|
|
Run frontend unit tests:
frontend/src/components/ChatView/ChatView.jsx#L900
React Hook useCallback has missing dependencies: 'isStreamingRef', 'pendingQueue', and 'setServerRunningState'. Either include them or remove the dependency array
|
|
Run frontend unit tests:
frontend/src/components/ChatView/ChatView.jsx#L839
React Hook useCallback has missing dependencies: 'isStreamingRef', 'messagesRef', 'offsetRef', 'pendingQueue', and 'setServerRunningState'. Either include them or remove the dependency array
|
|
Run frontend unit tests:
frontend/src/components/ChatView/ChatView.jsx#L522
React Hook useEffect has missing dependencies: 'handleComposerInputChange' and 'inputValueRef'. Either include them or remove the dependency array
|
|
Run frontend unit tests:
frontend/src/components/ChatView/ChatSettingsPanel.jsx#L647
The 'hiddenIds' logical expression could make the dependencies of useMemo Hook (at line 684) change on every render. Move it inside the useMemo callback. Alternatively, wrap the initialization of 'hiddenIds' in its own useMemo() Hook
|
|
Run frontend unit tests:
frontend/src/components/ChatView/ChatSettingsPanel.jsx#L528
React Hook useCallback has a missing dependency: 'chatId'. Either include it or remove the dependency array
|
|
Run frontend unit tests:
frontend/src/components/ChatView/ActivityStretch.jsx#L248
React Hook useEffect has a missing dependency: 'detailRef'. Either include it or remove the dependency array
|
|
Run frontend unit tests:
frontend/src/components/ChatEmbed/ChatEmbed.jsx#L250
The ref value 'pendingContextResolversRef.current' will likely have changed by the time this effect cleanup function runs. If this ref points to a node rendered by React, copy 'pendingContextResolversRef.current' to a variable inside the effect, and use that variable in the cleanup function
|
|
Run frontend unit tests:
frontend/src/App.jsx#L211
React Hook useEffect has missing dependencies: 'resumeStep' and 'savedResumeStep'. Either include them or remove the dependency array
|
background
wait
wait-all
cancel
parallel
Loading