eslint-plugin-react-x 5.19.1 → 5.20.0

package/dist/index.js

@@ -145,7 +145,7 @@ const rules$6 = {
 //#endregion
 //#region package.json
 var name$6 = "eslint-plugin-react-x";
-var version = "5.19.1";
+var version = "5.20.0";
 
 //#endregion
 //#region src/utils/create-rule.ts
@@ -1581,11 +1581,12 @@ function createImmutabilityCollector() {
 /**
 * Classify whether a variable ultimately holds a value that must be treated as
 * immutable: a component's props, a state value returned from `useState`-like or
-* `useReducer` calls, or a shallow copy (spread literal) of either.
+* `useReducer` calls, a shallow copy (spread literal) of either, or a `for...of`
+* iterator variable whose iterated collection resolves to one of those.
 * @param context The rule context.
 * @param variable The variable to classify.
 * @param components The confirmed function component nodes in the file.
-* @param seen Variables already visited during spread recursion.
+* @param seen Variables already visited during spread/iterator recursion.
 * @returns The frozen origin, or `null` when the variable is not derived from one.
 */
 function classifyFrozenOrigin(context, variable, components, seen = /* @__PURE__ */ new Set()) {
@@ -1599,8 +1600,21 @@ function classifyFrozenOrigin(context, variable, components, seen = /* @__PURE__
 		name: origin.name
 	};
 	if (def.type !== DefinitionType.Variable) return null;
-	const init = def.node.init == null ? null : Extract.unwrap(def.node.init);
-	if (init == null) return null;
+	if (def.node.init == null) {
+		const loop = def.node.parent.parent;
+		if (loop.type !== AST_NODE_TYPES.ForOfStatement || loop.left !== def.node.parent) return null;
+		const root = Extract.getIdentifierAt(loop.right, 0);
+		if (root == null) return null;
+		const source = findVariable(context.sourceCode.getScope(root), root);
+		if (source == null) return null;
+		const inner = classifyFrozenOrigin(context, source, components, seen);
+		return inner == null ? null : {
+			kind: "iterator",
+			name: origin.name,
+			original: inner.name
+		};
+	}
+	const init = Extract.unwrap(def.node.init);
 	switch (init.type) {
 		case AST_NODE_TYPES.CallExpression: {
 			const hook = getStateHookName(context, init);
@@ -1704,6 +1718,12 @@ function inferDirectMutations(context, mutations, components) {
 					detail: `It is a shallow copy of '${origin.original}'; mutating nested values through it mutates '${origin.original}' in place.`,
 					node: mutation.node
 				});
+				break;
+			case "iterator": directMutations.push({
+				name: origin.name,
+				detail: `It is an element of '${origin.original}' and must be treated as immutable.`,
+				node: mutation.node
+			});
 		}
 	}
 	return directMutations;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "5.19.1",
+  "version": "5.20.0",
   "description": "A set of composable ESLint rules for libraries and frameworks that use React as a UI runtime.",
   "keywords": [
     "react",
@@ -36,12 +36,12 @@
     "dist"
   ],
   "dependencies": {
-    "@eslint-react/ast": "5.19.1",
-    "@eslint-react/core": "5.19.1",
-    "@eslint-react/eslint": "5.19.1",
-    "@eslint-react/jsx": "5.19.1",
-    "@eslint-react/shared": "5.19.1",
-    "@eslint-react/var": "5.19.1",
+    "@eslint-react/ast": "5.20.0",
+    "@eslint-react/core": "5.20.0",
+    "@eslint-react/eslint": "5.20.0",
+    "@eslint-react/jsx": "5.20.0",
+    "@eslint-react/shared": "5.20.0",
+    "@eslint-react/var": "5.20.0",
     "@typescript-eslint/scope-manager": "^8.70.0",
     "@typescript-eslint/type-utils": "^8.70.0",
     "@typescript-eslint/types": "^8.70.0",
@@ -63,7 +63,7 @@
     "react-dom": "^19.3.0",
     "tsdown": "^0.23.0",
     "tsl": "^1.0.30",
-    "tsl-dx": "^0.13.3",
+    "tsl-dx": "^0.13.5",
     "typescript": "6.0.3"
   },
   "peerDependencies": {