From 73b746cbcf198bc90afb98a4dc06745cf88d9eee Mon Sep 17 00:00:00 2001
From: Aaron Knudtson <87577305+knudtty@users.noreply.github.com>
Date: Wed, 8 Apr 2026 15:44:32 -0400
Subject: [PATCH] chore: automatic fixes from the linter (#2074)

## Summary

I'm tired of seeing automatic lint fixes so I ran `make dev-lint`
---
 packages/app/src/DBChartPage.tsx                          | 1 -
 packages/app/src/DOMPlayer.tsx                            | 1 -
 packages/app/src/components/DBHeatmapChart.tsx            | 2 +-
 packages/app/src/components/SQLEditor/SQLEditor.tsx       | 2 +-
 packages/app/src/components/SQLEditor/SQLInlineEditor.tsx | 1 -
 packages/app/src/hooks/useResizable.tsx                   | 2 +-
 packages/app/src/hooks/useStableCallback.ts               | 2 +-
 packages/app/src/timeQuery.ts                             | 4 +---
 packages/app/src/useQueryParam.tsx                        | 1 -
 packages/app/src/utils.ts                                 | 2 +-
 10 files changed, 6 insertions(+), 12 deletions(-)

diff --git a/packages/app/src/DBChartPage.tsx b/packages/app/src/DBChartPage.tsx
index 82f2e5e2..b77c7602 100644
--- a/packages/app/src/DBChartPage.tsx
+++ b/packages/app/src/DBChartPage.tsx
@@ -166,7 +166,6 @@ function AIAssistant({
       </Group>
       <Collapse in={opened}>
         {opened && (
-          // eslint-disable-next-line react-hooks/refs
           <form onSubmit={handleSubmit(onSubmit)}>
             <Group mb="md">
               <SourceSelectControlled
diff --git a/packages/app/src/DOMPlayer.tsx b/packages/app/src/DOMPlayer.tsx
index 3e9c9b35..97485cd0 100644
--- a/packages/app/src/DOMPlayer.tsx
+++ b/packages/app/src/DOMPlayer.tsx
@@ -264,7 +264,6 @@ export default function DOMPlayer({
       );
     }
 
-    // eslint-disable-next-line react-hooks/immutability
     updatePlayerTimeRafRef.current = requestAnimationFrame(updatePlayerTime);
   }, []);
 
diff --git a/packages/app/src/components/DBHeatmapChart.tsx b/packages/app/src/components/DBHeatmapChart.tsx
index f548a3b7..04c35db6 100644
--- a/packages/app/src/components/DBHeatmapChart.tsx
+++ b/packages/app/src/components/DBHeatmapChart.tsx
@@ -953,7 +953,7 @@ function Heatmap({
       },
       plugins: [
         // legendAsTooltipPlugin()
-        // eslint-disable-next-line react-hooks/refs -- mouseInsideRef is read at event time, not during render
+
         highlightDataPlugin({
           proximity: 20,
           yFormatter: tickFormatter,
diff --git a/packages/app/src/components/SQLEditor/SQLEditor.tsx b/packages/app/src/components/SQLEditor/SQLEditor.tsx
index ba140961..d445a0e4 100644
--- a/packages/app/src/components/SQLEditor/SQLEditor.tsx
+++ b/packages/app/src/components/SQLEditor/SQLEditor.tsx
@@ -98,7 +98,7 @@ export default function SQLEditor({
         minHeight={'100px'}
         extensions={[
           createCodeMirrorStyleTheme(),
-          // eslint-disable-next-line react-hooks/refs
+
           compartmentRef.current.of(
             clickhouseSql({
               upperCaseKeywords: true,
diff --git a/packages/app/src/components/SQLEditor/SQLInlineEditor.tsx b/packages/app/src/components/SQLEditor/SQLInlineEditor.tsx
index 9e8dfaff..94224486 100644
--- a/packages/app/src/components/SQLEditor/SQLInlineEditor.tsx
+++ b/packages/app/src/components/SQLEditor/SQLInlineEditor.tsx
@@ -242,7 +242,6 @@ export default function SQLInlineEditor({
       // Enable line wrapping when multiline is allowed (regardless of focus)
       ...(allowMultiline ? [EditorView.lineWrapping] : []),
 
-      // eslint-disable-next-line react-hooks/refs
       compartmentRef.current.of(
         clickhouseSql({
           upperCaseKeywords: true,
diff --git a/packages/app/src/hooks/useResizable.tsx b/packages/app/src/hooks/useResizable.tsx
index fcc4ac51..bb56bfc0 100644
--- a/packages/app/src/hooks/useResizable.tsx
+++ b/packages/app/src/hooks/useResizable.tsx
@@ -52,7 +52,7 @@ function useResizable(
 
   const endResize = useCallback(() => {
     document.removeEventListener('mousemove', handleResize);
-    // eslint-disable-next-line react-hooks/immutability
+
     document.removeEventListener('mouseup', endResize);
   }, [handleResize]);
 
diff --git a/packages/app/src/hooks/useStableCallback.ts b/packages/app/src/hooks/useStableCallback.ts
index 2ca50d1b..fc978fad 100644
--- a/packages/app/src/hooks/useStableCallback.ts
+++ b/packages/app/src/hooks/useStableCallback.ts
@@ -10,7 +10,7 @@ export const useStableCallback = <T extends (...args: any[]) => any>(
   });
 
   return useCallback(
-    // eslint-disable-next-line react-hooks/use-memo, @typescript-eslint/no-unsafe-type-assertion
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-type-assertion
     ((...args: Parameters<T>) => callbackRef.current(...args)) as T,
     [],
   );
diff --git a/packages/app/src/timeQuery.ts b/packages/app/src/timeQuery.ts
index 175fa351..f8f24f3d 100644
--- a/packages/app/src/timeQuery.ts
+++ b/packages/app/src/timeQuery.ts
@@ -243,13 +243,12 @@ export function useTimeQuery({
       liveTailTimeRange == null &&
       tempLiveTailTimeRange == null &&
       !isInputTimeQueryLive(inputTimeQuery) &&
-      // eslint-disable-next-line react-hooks/refs
       inputTimeQueryDerivedTimeQueryRef.current != null
     ) {
       // Use the input time query, allows users to specify relative time ranges
       // via url ex. /logs?tq=Last+30+minutes
       // return inputTimeQueryDerivedTimeQuery as [Date, Date];
-      // eslint-disable-next-line react-hooks/refs
+
       return inputTimeQueryDerivedTimeQueryRef.current;
     } else if (
       isReady &&
@@ -345,7 +344,6 @@ export function useTimeQuery({
     ],
   );
 
-  // eslint-disable-next-line react-hooks/refs
   return {
     isReady, // Don't search until we know what we want to do
     isLive,
diff --git a/packages/app/src/useQueryParam.tsx b/packages/app/src/useQueryParam.tsx
index 56840865..0f4f3ef7 100644
--- a/packages/app/src/useQueryParam.tsx
+++ b/packages/app/src/useQueryParam.tsx
@@ -28,7 +28,6 @@ export const QueryParamProvider = ({
 
   const setState = useCallback(
     (state: Record<string, any>) => {
-      // eslint-disable-next-line react-hooks/immutability
       setCache(oldCache => {
         const newCache = {
           ...oldCache,
diff --git a/packages/app/src/utils.ts b/packages/app/src/utils.ts
index e20b5b1d..1aa0b572 100644
--- a/packages/app/src/utils.ts
+++ b/packages/app/src/utils.ts
@@ -643,7 +643,7 @@ export const usePrevious = <T>(value: T): T | undefined => {
   useEffect(() => {
     ref.current = value;
   });
-  // eslint-disable-next-line react-hooks/refs
+
   return ref.current;
 };