fixup! fix(deps): update eslint

.changeset/eleven-apples-fly.md

@@ -0,0 +1,6 @@
+---
+"@rnx-kit/eslint-plugin": patch
+---
+
+Bumped `eslint-plugin-react` and `eslint-plugin-react-hooks` to better support
+ESLint 9.x

package.json

@@ -51,7 +51,7 @@
   },
   "packageManager": "[email protected]",
   "resolutions": {
-    "@microsoft/eslint-plugin-sdl/eslint-plugin-react": "^7.33.0",
+    "@microsoft/eslint-plugin-sdl/eslint-plugin-react": "^7.35.2",
     "@microsoft/eslint-plugin-sdl/eslint-plugin-security": "^1.4.0",
     "@react-native-community/cli": "^14.0.0",
     "@react-native-community/cli-platform-android": "^14.0.0",
@@ -148,9 +148,6 @@
       },
       "packages/eslint-plugin": {
         "ignoreDependencies": [
-          "@microsoft/eslint-plugin-sdl",
-          "eslint-plugin-node",
-          "eslint-plugin-react",
           "eslint-plugin-react-hooks"
         ]
       },

packages/eslint-config/index.js

@@ -1,21 +1,23 @@
-const { FlatCompat } = require("@eslint/eslintrc");
-const js = require("@eslint/js");
+const sdl = require("@microsoft/eslint-plugin-sdl");
 const rnx = require("@rnx-kit/eslint-plugin");
 
-const compat = new FlatCompat({
-  // Use `@rnx-kit/eslint-plugin` as base directory to ensure we get the same
-  // plugin instances
-  baseDirectory: require.resolve("@rnx-kit/eslint-plugin/package.json"),
-  recommendedConfig: js.configs.recommended,
-});
-
+/**
+ * Note that we don't directly use `sdl.configs.required` because:
+ *
+ *   1. It includes rules for Angular and Electron
+ *   2. Its `react` preset conflicts with our direct use of `eslint-plugin-react`
+ *
+ * https://github.com/microsoft/eslint-plugin-sdl/blob/957996315c80fdadcd1a9f7bb76fc4663d33ef1e/lib/index.js#L47-L54
+ */
 module.exports = [
+  ...sdl.configs.common,
+  ...sdl.configs.node,
   ...rnx.configs.strict,
   ...rnx.configs.stylistic,
-  ...compat.extends("plugin:@microsoft/sdl/required"),
   {
     rules: {
       "@typescript-eslint/consistent-type-definitions": ["error", "type"],
+      ...sdl.configs.react[0].rules,
     },
   },
 ];

packages/eslint-config/package.json

@@ -22,14 +22,14 @@
     "format": "prettier --write --log-level error README.md index.js package.json"
   },
   "dependencies": {
-    "@eslint/eslintrc": "^3.0.0",
-    "@eslint/js": "^9.0.0",
+    "@microsoft/eslint-plugin-sdl": "^1.0.0",
     "@rnx-kit/eslint-plugin": "workspace:*"
   },
   "peerDependencies": {
-    "eslint": ">=8.56.0"
+    "eslint": "^9.0.0"
   },
   "devDependencies": {
+    "eslint": "^9.0.0",
     "prettier": "^3.0.0"
   },
   "engines": {

packages/eslint-plugin/package.json

@@ -46,20 +46,19 @@
     "@eslint/eslintrc": "^3.0.0",
     "@react-native/eslint-plugin": "^0.75.0",
     "enhanced-resolve": "^5.8.3",
-    "eslint-plugin-react": "^7.33.0",
-    "eslint-plugin-react-hooks": "^4.6.0",
+    "eslint-plugin-react": "^7.35.2",
+    "eslint-plugin-react-hooks": "^5.0.0",
     "typescript-eslint": "^8.0.0"
   },
   "peerDependencies": {
     "eslint": ">=8.57.0"
   },
   "devDependencies": {
-    "@microsoft/eslint-plugin-sdl": "^0.2.0",
     "@rnx-kit/eslint-config": "*",
     "@rnx-kit/jest-preset": "*",
     "@rnx-kit/scripts": "*",
     "@rnx-kit/tsconfig": "*",
-    "@types/eslint": "^8.0.0",
+    "@types/eslint": "^9.0.0",
     "@types/eslint__eslintrc": "^2.1.1",
     "@types/eslint__js": "^8.0.0",
     "@types/estree": "*",
@@ -68,7 +67,6 @@
     "@typescript-eslint/parser": "^8.0.0",
     "@typescript-eslint/types": "^8.0.0",
     "eslint": "^9.0.0",
-    "eslint-plugin-node": "11.1.0",
     "jest": "^29.2.1",
     "prettier": "^3.0.0",
     "typescript": "^5.0.0"

packages/eslint-plugin/src/configs/react.js

@@ -0,0 +1,25 @@
+// @ts-check
+"use strict";
+
+const { FlatCompat } = require("@eslint/eslintrc");
+const js = (() => {
+  const path = require("node:path");
+  try {
+    const eslint = path.dirname(require.resolve("eslint/package.json"));
+    const eslintjs = require.resolve("@eslint/js", { paths: [eslint] });
+    return require(eslintjs);
+  } catch (_) {
+    return require("@eslint/js");
+  }
+})();
+const react = require("eslint-plugin-react");
+
+const compat = new FlatCompat({
+  baseDirectory: __dirname,
+  recommendedConfig: js.configs.recommended,
+});
+
+module.exports = [
+  ...compat.extends("plugin:react-hooks/recommended"),
+  react.configs.flat.recommended,
+];

packages/eslint-plugin/src/configs/recommended.js

@@ -1,24 +1,8 @@
 // @ts-check
 "use strict";
 
-const { FlatCompat } = require("@eslint/eslintrc");
-const js = (() => {
-  const path = require("node:path");
-  try {
-    const eslint = path.dirname(require.resolve("eslint/package.json"));
-    const eslintjs = require.resolve("@eslint/js", { paths: [eslint] });
-    return require(eslintjs);
-  } catch (_) {
-    return require("@eslint/js");
-  }
-})();
 const tseslint = require("typescript-eslint");
 
-const compat = new FlatCompat({
-  baseDirectory: __dirname,
-  recommendedConfig: js.configs.recommended,
-});
-
 /**
  * @param {string} spec
  * @returns {boolean}
@@ -32,9 +16,7 @@ function isInstalled(spec) {
 }
 
 const usesReact = isInstalled("react");
-const reactConfigs = usesReact
-  ? compat.extends("plugin:react-hooks/recommended", "plugin:react/recommended")
-  : [];
+const reactConfigs = usesReact ? require("./react") : [];
 
 module.exports = [
   ...tseslint.configs.recommended,

packages/jest-preset/eslint.config.js

@@ -27,6 +27,18 @@ module.exports = [
           },
         },
       ],
+      "no-unused-vars": [
+        "error",
+        {
+          args: "all",
+          argsIgnorePattern: "^_",
+          caughtErrors: "all",
+          caughtErrorsIgnorePattern: "^_",
+          destructuredArrayIgnorePattern: "^_",
+          varsIgnorePattern: "^_",
+          ignoreRestSiblings: true,
+        },
+      ],
     },
   },
 ];

packages/metro-resolver-symlinks/src/utils/patchMetro.ts

@@ -155,7 +155,7 @@ export function patchMetro(options: Options): void {
     req: { url: string },
     res: unknown
   ): Promise<void> {
-    // eslint-disable-next-line node/no-deprecated-api
+    // eslint-disable-next-line n/no-deprecated-api
     const urlObj = url.parse(decodeURI(req.url), true);
     let [, assetPath] =
       (urlObj &&