Problem
All use // eslint-disable-line react-hooks/exhaustive-deps to suppress missing dependency warnings in useEffect hooks. This can cause stale closures and UI bugs.
Impact
Stale state references, especially for connection-related logic where the active connection changes frequently.
Affected Files
src/components/editor/SQLEditor.tsxsrc/components/editor/FormatterSettingsDialog.tsxsrc/components/import/ImportDialog.tsxsrc/components/layout/ConnectionTabs.tsxsrc/components/ai/AIChatPanel.tsx
Suggested Fix
Add the missing dependencies or use useCallback/useRef to stabilize the referenced values. Audit each case to determine if it is a genuine stabilization pattern or a bug.
Severity: Low - React best practices
Problem
All use
// eslint-disable-line react-hooks/exhaustive-depsto suppress missing dependency warnings inuseEffecthooks. This can cause stale closures and UI bugs.Impact
Stale state references, especially for connection-related logic where the active connection changes frequently.
Affected Files
src/components/editor/SQLEditor.tsxsrc/components/editor/FormatterSettingsDialog.tsxsrc/components/import/ImportDialog.tsxsrc/components/layout/ConnectionTabs.tsxsrc/components/ai/AIChatPanel.tsxSuggested Fix
Add the missing dependencies or use
useCallback/useRefto stabilize the referenced values. Audit each case to determine if it is a genuine stabilization pattern or a bug.Severity: Low - React best practices