From 93fdc16e72a99c4c5622f1ca510ec876bd4937bc Mon Sep 17 00:00:00 2001 From: Diego Rodrigues de Sa e Souza Date: Wed, 2 Sep 2026 06:22:30 -0300 Subject: [PATCH] chore(lint): adopt eslint-plugin-react-hooks 7.1.1 (#12428) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * chore(lint): adopt eslint-plugin-react-hooks 7.1.1 The #12146 migration (284 react-hooks compiler-rule violations resolved in 8 batches) completed on 2026-09-01, unblocking the 7.1.1 adoption the pin test was holding back. Exact pin kept in both devDependencies and overrides; the pin test moves to 7.1.1 (the dependabot-level ignore from #12329 stays — a lint plugin coupled to the compiler rules always bumps via its own reviewed PR, never riding a group). * chore(lint): lockfile for the react-hooks 7.1.1 adoption Generated with a bare 'npm install --package-lock-only' (naming the package on the CLI rewrites the devDependency with a caret, which npm 11 then rejects against the exact override). Validated on the .113 with a fresh npm ci + cold NODE_OPTIONS=8G lint:json --max-warnings 0 → exit 0 (zero new violations from the 7.1.1 rule set) and the re-pinned version test green. --- package-lock.json | 10 +++++----- package.json | 4 ++-- tests/unit/eslint-react-hooks-version-pinned.test.ts | 4 ++-- 3 files changed, 9 insertions(+), 9 deletions(-) diff --git a/package-lock.json b/package-lock.json index 4b71b7e3b0..31a73d5464 100644 --- a/package-lock.json +++ b/package-lock.json @@ -132,7 +132,7 @@ "esbuild": "0.28.2", "eslint": "^10.9.0", "eslint-config-next": "16.3.3", - "eslint-plugin-react-hooks": "7.0.1", + "eslint-plugin-react-hooks": "7.1.1", "eslint-plugin-sonarjs": "^4.1.0", "espree": "^11.2.0", "fast-check": "^4.8.0", @@ -20913,9 +20913,9 @@ } }, "node_modules/eslint-plugin-react-hooks": { - "version": "7.0.1", - "resolved": "https://registry.npmjs.org/eslint-plugin-react-hooks/-/eslint-plugin-react-hooks-7.0.1.tgz", - "integrity": "sha512-O0d0m04evaNzEPoSW+59Mezf8Qt0InfgGIBJnpC0h3NH/WjUAR7BIKUfysC6todmtiZ/A0oUVS8Gce0WhBrHsA==", + "version": "7.1.1", + "resolved": "https://registry.npmjs.org/eslint-plugin-react-hooks/-/eslint-plugin-react-hooks-7.1.1.tgz", + "integrity": "sha512-f2I7Gw6JbvCexzIInuSbZpfdQ44D7iqdWX01FKLvrPgqxoE7oMj8clOfto8U6vYiz4yd5oKu39rRSVOe1zRu0g==", "dev": true, "license": "MIT", "dependencies": { @@ -20929,7 +20929,7 @@ "node": ">=18" }, "peerDependencies": { - "eslint": "^3.0.0 || ^4.0.0 || ^5.0.0 || ^6.0.0 || ^7.0.0 || ^8.0.0-0 || ^9.0.0" + "eslint": "^3.0.0 || ^4.0.0 || ^5.0.0 || ^6.0.0 || ^7.0.0 || ^8.0.0-0 || ^9.0.0 || ^10.0.0" } }, "node_modules/eslint-plugin-react/node_modules/resolve": { diff --git a/package.json b/package.json index 41c9a125b2..4e9f576908 100644 --- a/package.json +++ b/package.json @@ -396,7 +396,7 @@ "esbuild": "0.28.2", "eslint": "^10.9.0", "eslint-config-next": "16.3.3", - "eslint-plugin-react-hooks": "7.0.1", + "eslint-plugin-react-hooks": "7.1.1", "eslint-plugin-sonarjs": "^4.1.0", "espree": "^11.2.0", "fast-check": "^4.8.0", @@ -455,7 +455,7 @@ }, "overrides": { "onnxruntime-node": "1.24.3", - "eslint-plugin-react-hooks": "7.0.1", + "eslint-plugin-react-hooks": "7.1.1", "fast-xml-parser": "^5.10.1", "sharp": "^0.35.4", "postcss": "^8.5.18", diff --git a/tests/unit/eslint-react-hooks-version-pinned.test.ts b/tests/unit/eslint-react-hooks-version-pinned.test.ts index eb2072bc1d..a076347f89 100644 --- a/tests/unit/eslint-react-hooks-version-pinned.test.ts +++ b/tests/unit/eslint-react-hooks-version-pinned.test.ts @@ -13,7 +13,7 @@ import { fileURLToPath } from "node:url"; const ROOT = new URL("../../", import.meta.url); const PLUGIN = "eslint-plugin-react-hooks"; -const EXPECTED_VERSION = "7.0.1"; +const EXPECTED_VERSION = "7.1.1"; async function readJson(relative: string): Promise> { return JSON.parse(await readFile(fileURLToPath(new URL(relative, ROOT)), "utf8")); @@ -30,7 +30,7 @@ test("eslint-plugin-react-hooks is directly pinned to the locked version", async assert.equal( declared, EXPECTED_VERSION, - `${PLUGIN} must remain pinned to ${EXPECTED_VERSION} until the 7.1.1 lint migration` + `${PLUGIN} must remain pinned to ${EXPECTED_VERSION} so a group bump can never ride past the compiler-rules migration review` ); assert.ok(locked, `${PLUGIN} missing from package-lock.json`); assert.equal(