Update eslint dependencies (#3608)

* update eslint deps

* Add new rules

* No alternative for keyCode

* Use `RefObject`

* Fix types

Author: amanmahajan7

eslint.config.js

@@ -267,6 +267,7 @@ export default [
       'react/forbid-elements': 0,
       'react/forbid-foreign-prop-types': 0,
       'react/forbid-prop-types': 0,
+      'react/forward-ref-uses-ref': 1,
       'react/function-component-definition': [
         1,
         {
@@ -439,6 +440,7 @@ export default [
       '@typescript-eslint/no-base-to-string': 0,
       '@typescript-eslint/no-confusing-non-null-assertion': 0,
       '@typescript-eslint/no-confusing-void-expression': [1, { ignoreArrowShorthand: true }],
+      '@typescript-eslint/no-deprecated': 1,
       '@typescript-eslint/no-duplicate-enum-values': 1,
       '@typescript-eslint/no-duplicate-type-constituents': 1,
       '@typescript-eslint/no-dynamic-delete': 0,

package.json

@@ -75,20 +75,20 @@
     "@testing-library/react": "^16.0.0",
     "@testing-library/user-event": "^14.5.2",
     "@types/node": "^22.0.0",
-    "@types/react": "^18.3.3",
+    "@types/react": "^18.3.9",
     "@types/react-dom": "^18.3.0",
-    "@typescript-eslint/eslint-plugin": "^8.1.0",
-    "@typescript-eslint/parser": "^8.1.0",
+    "@typescript-eslint/eslint-plugin": "^8.7.0",
+    "@typescript-eslint/parser": "^8.7.0",
     "@vitejs/plugin-react": "^4.3.1",
     "@vitest/browser": "^2.1.1",
     "@vitest/coverage-v8": "^2.1.1",
     "@vitest/eslint-plugin": "^1.1.4",
     "@wyw-in-js/rollup": "^0.5.0",
     "@wyw-in-js/vite": "^0.5.0",
     "babel-plugin-optimize-clsx": "^2.6.2",
-    "eslint": "^9.9.0",
+    "eslint": "^9.11.1",
     "eslint-plugin-jest-dom": "^5.0.1",
-    "eslint-plugin-react": "^7.35.0",
+    "eslint-plugin-react": "^7.36.1",
     "eslint-plugin-react-hooks": "^4.6.2",
     "eslint-plugin-sonarjs": "^2.0.2",
     "eslint-plugin-testing-library": "^6.3.0",

src/DataGrid.tsx

@@ -591,6 +591,7 @@ function DataGrid<R, SR, K extends Key>(
     const isRowEvent = isTreeGrid && event.target === focusSinkRef.current;
     if (!isCellEvent && !isRowEvent) return;
 
+    // eslint-disable-next-line @typescript-eslint/no-deprecated
     const { keyCode } = event;
 
     if (

src/DragHandle.tsx

@@ -32,14 +32,19 @@ const cellDragHandleFrozenClassname = css`
 
 const cellDragHandleClassname = `rdg-cell-drag-handle ${cellDragHandle}`;
 
+// TODO: replace with RefObject once we drop support for React 18
+interface LatestDraggedOverRowIdxRef {
+  readonly current: number | undefined;
+}
+
 interface Props<R, SR> extends Pick<DataGridProps<R, SR>, 'rows' | 'onRowsChange'> {
   gridRowStart: number;
   column: CalculatedColumn<R, SR>;
   columnWidth: number | string;
   maxColIdx: number;
   isLastRow: boolean;
   selectedPosition: SelectCellState;
-  latestDraggedOverRowIdx: React.MutableRefObject<number | undefined>;
+  latestDraggedOverRowIdx: LatestDraggedOverRowIdxRef;
   isCellEditable: (position: Position) => boolean;
   onClick: () => void;
   onFill: (event: FillEvent<R>) => R;

src/TreeDataGrid.tsx

@@ -323,6 +323,7 @@ function TreeDataGrid<R, SR, K extends Key>(
     }
 
     // Prevent copy/paste on group rows
+    // eslint-disable-next-line @typescript-eslint/no-deprecated
     if (isCtrlKeyHeldDown(event) && (event.keyCode === 67 || event.keyCode === 86)) {
       event.preventGridDefault();
     }

src/utils/keyboardUtils.ts

@@ -54,6 +54,7 @@ export function isCtrlKeyHeldDown(e: React.KeyboardEvent): boolean {
 
 export function isDefaultCellInput(event: React.KeyboardEvent<HTMLDivElement>): boolean {
   const vKey = 86;
+  // eslint-disable-next-line @typescript-eslint/no-deprecated
   if (isCtrlKeyHeldDown(event) && event.keyCode !== vKey) return false;
   return !nonInputKeys.has(event.key);
 }