fix: Restore missing hook lint rules

Author: pulpdrew

packages/app/eslint.config.mjs

@@ -122,12 +122,23 @@ export default [
       ...nextPlugin.configs.recommended.rules,
       ...nextPlugin.configs['core-web-vitals'].rules,
       ...reactHooksPlugin.configs.recommended.rules,
-      ...eslintReactPlugin.configs.recommended.rules,
-      // Disable rules from eslint-plugin-react-hooks that have equivalent rules in @eslint-react
-      ...eslintReactPlugin.configs['disable-conflict-eslint-plugin-react-hooks'].rules,
       ...eslintReactPlugin.configs['recommended-type-checked'].rules,
-      'react-hooks/set-state-in-effect': 'warn',
-      'react-hooks/exhaustive-deps': 'error',
+      '@eslint-react/exhaustive-deps': 'error',
+      
+      // @eslint-react version is experimental, we'll use react-hooks/set-state-in-render instead
+      '@eslint-react/set-state-in-render': 'off',
+      'react-hooks/set-state-in-render': 'error',
+      
+      // Disable rules from eslint-plugin-react-hooks that have equivalent rules enabled in @eslint-react
+      'react-hooks/rules-of-hooks': 'off',
+      'react-hooks/static-components': 'off',
+      'react-hooks/use-memo': 'off',
+      'react-hooks/component-hook-factories': 'off',
+      'react-hooks/error-boundaries': 'off',
+      'react-hooks/set-state-in-effect': 'off',
+      'react-hooks/unsupported-syntax': 'off',
+      'react-hooks/exhaustive-deps': 'off',
+      
       'react-hook-form/no-use-watch': 'error',
       '@eslint-react/no-unstable-default-props': 'error',
       '@typescript-eslint/ban-ts-comment': 'warn',

packages/app/src/DBSearchPage.tsx

@@ -1247,7 +1247,7 @@ function DBSearchPage() {
       updateRelativeTimeInputValue(interval);
     }
     // we only want this to run on initial mount
-    // eslint-disable-next-line react-hooks/exhaustive-deps
+    // eslint-disable-next-line @eslint-react/exhaustive-deps
   }, [updateRelativeTimeInputValue, searchedConfig.source, isReady]);
 
   useLiveUpdate({

packages/app/src/components/DBEditTimeChartForm/EditTimeChartForm.tsx

@@ -397,7 +397,6 @@ export default function EditTimeChartForm({
 
   // Emulate the date range picker auto-searching similar to dashboards
   useEffect(() => {
-    // eslint-disable-next-line react-hooks/set-state-in-effect
     setQueriedConfig((config: ChartConfigWithDateRange | undefined) => {
       if (config == null) {
         return config;

packages/app/src/components/DBRowTable.tsx

@@ -858,7 +858,7 @@ export const RawLogTable = memo(
         // Conditions not met, clear flag
         tableSearch.clearShouldScrollToMatch();
       }
-      // eslint-disable-next-line react-hooks/exhaustive-deps
+      // eslint-disable-next-line @eslint-react/exhaustive-deps
     }, [
       tableSearch.shouldScrollToMatch,
       tableSearch.currentMatchIndex,
@@ -1531,7 +1531,7 @@ function DBSqlRowTableComponent({
     if (prevSourceId && prevSourceId !== sourceId) {
       _onSortingChange(null);
     }
-    // eslint-disable-next-line react-hooks/exhaustive-deps
+    // eslint-disable-next-line @eslint-react/exhaustive-deps
   }, [sourceId]);
 
   // Sync local orderBy state with initialSortBy when it changes

packages/app/src/components/DBTable/TableSearchInput.tsx

@@ -141,6 +141,7 @@ export const TableSearchInput = ({
 
   // Handle keyboard shortcuts (Cmd+F, Escape)
   const handleKeyDown = useCallback(
+    // eslint-disable-next-line react-hooks/preserve-manual-memoization
     (e: KeyboardEvent) => {
       // Detect Cmd+F (Mac) or Ctrl+F (Windows/Linux)
       if ((e.metaKey || e.ctrlKey) && e.key === 'f') {
@@ -155,7 +156,7 @@ export const TableSearchInput = ({
         handleClose();
       }
     },
-    // eslint-disable-next-line react-hooks/exhaustive-deps
+    // eslint-disable-next-line @eslint-react/exhaustive-deps
     [isVisible, handleClose, handleShow],
   );
 

packages/app/src/components/TimePicker/TimePicker.tsx

@@ -135,7 +135,7 @@ const TimePickerComponent = ({
       }
     }
     // only run when dateRange changes or opened state changes
-    // eslint-disable-next-line react-hooks/exhaustive-deps
+    // eslint-disable-next-line @eslint-react/exhaustive-deps
   }, [dateRange, opened, mode]);
 
   const handleRelativeSearch = React.useCallback(

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,
     [],
   );

packages/app/src/hooks/useTableSearch.ts

@@ -97,7 +97,7 @@ export function useTableSearch({
       shouldScrollToMatchRef.current = true;
     }
     // Otherwise keep the current match index as is (loading more data shouldn't change position)
-    // eslint-disable-next-line react-hooks/exhaustive-deps
+    // eslint-disable-next-line @eslint-react/exhaustive-deps
   }, [debouncedSearchQuery, rows, searchableColumns]);
 
   // Handle visibility changes
@@ -146,6 +146,7 @@ export function useTableSearch({
     shouldScrollToMatchRef.current = false;
   }, []);
 
+  // eslint-disable-next-line react-hooks/refs
   return {
     searchQuery: debouncedSearchQuery,
     inputValue,
@@ -154,6 +155,7 @@ export function useTableSearch({
     currentMatchIndex,
     isSearchVisible,
     setIsSearchVisible,
+    // eslint-disable-next-line react-hooks/refs
     shouldScrollToMatch: shouldScrollToMatchRef.current,
     clearShouldScrollToMatch,
     handlePreviousMatch,

packages/app/src/searchFilters.tsx

@@ -420,7 +420,7 @@ export const useSearchPageFilterState = ({
       setFilters(parsedQuery.filters);
     }
     // only react to changes in parsed query
-    // eslint-disable-next-line react-hooks/exhaustive-deps
+    // eslint-disable-next-line @eslint-react/exhaustive-deps
   }, [parsedQuery.filters]);
 
   const updateFilterQuery = React.useCallback(