eslint-plugin-react-x 5.12.0 → 5.12.1

package/dist/index.js

@@ -144,7 +144,7 @@ const rules$6 = {
 //#endregion
 //#region package.json
 var name$6 = "eslint-plugin-react-x";
-var version = "5.12.0";
+var version = "5.12.1";
 
 //#endregion
 //#region src/utils/create-rule.ts
@@ -1751,17 +1751,6 @@ function create$47(context) {
 //#endregion
 //#region src/rules/no-array-index-key/lib.ts
 /**
-* Creates a report function for the given rule context.
-* @param context The ESLint rule context.
-* @returns A function that can be used to report violations.
-*/
-function report$2(context) {
-	return (descriptor) => {
-		if (descriptor == null) return;
-		context.report(descriptor);
-	};
-}
-/**
 * Iterator-like methods that pass the item's index to their callback,
 * mapped to the position of the index parameter in the callback's parameter list.
 * `map` and `forEach` also cover `Children.map` and `Children.forEach`,
@@ -1858,7 +1847,7 @@ function create$46(context) {
 	* @param node The key value expression to check.
 	* @returns The report descriptors for the violations found.
 	*/
-	function checkKeyExpression(node) {
+	function visitKeyExpression(node) {
 		switch (node.type) {
 			case AST_NODE_TYPES.Identifier: return isArrayIndex(node) ? [{
 				messageId: "default",
@@ -1872,8 +1861,8 @@ function create$46(context) {
 				messageId: "default",
 				node: identifier
 			}));
-			case AST_NODE_TYPES.ConditionalExpression: return [...checkKeyExpression(node.consequent), ...checkKeyExpression(node.alternate)];
-			case AST_NODE_TYPES.LogicalExpression: return [...checkKeyExpression(node.left), ...checkKeyExpression(node.right)];
+			case AST_NODE_TYPES.ConditionalExpression: return [...visitKeyExpression(node.consequent), ...visitKeyExpression(node.alternate)];
+			case AST_NODE_TYPES.LogicalExpression: return [...visitKeyExpression(node.left), ...visitKeyExpression(node.right)];
 			case AST_NODE_TYPES.CallExpression: {
 				const callee = Extract.unwrap(node.callee);
 				if (callee.type === AST_NODE_TYPES.MemberExpression && Extract.getPropertyName(callee.property) === "toString" && isArrayIndex(callee.object)) return [{
@@ -1898,12 +1887,12 @@ function create$46(context) {
 			for (const property of props.properties) {
 				const value = getKeyPropValue(property);
 				if (value == null) continue;
-				checkKeyExpression(value).forEach(report$2(context));
+				for (const desc of visitKeyExpression(value)) context.report(desc);
 			}
 		},
 		"JSXAttribute[name.name='key']"(node) {
 			if (node.value?.type !== AST_NODE_TYPES.JSXExpressionContainer) return;
-			checkKeyExpression(node.value.expression).forEach(report$2(context));
+			for (const desc of visitKeyExpression(node.value.expression)) context.report(desc);
 		}
 	};
 }
@@ -2599,7 +2588,7 @@ function create$28(context) {
 * @param context The ESLint rule context.
 * @returns A function that can be used to report violations.
 */
-function report$1(context) {
+function report(context) {
 	return (descriptor) => {
 		if (descriptor == null) return;
 		context.report(descriptor);
@@ -2643,16 +2632,16 @@ function create$27(context) {
 	* @param seen A set of visited identifier names to prevent infinite recursion.
 	* @returns A report descriptor if a problem is found, otherwise `null`
 	*/
-	function getReportDescriptor(node, seen = /* @__PURE__ */ new Set()) {
+	function visit(node, seen = /* @__PURE__ */ new Set()) {
 		if (node == null) return null;
-		if (Check.is(AST_NODE_TYPES.JSXExpressionContainer)(node)) return getReportDescriptor(node.expression, seen);
+		if (Check.is(AST_NODE_TYPES.JSXExpressionContainer)(node)) return visit(node.expression, seen);
 		if (Check.isJSX(node)) return null;
-		if (Check.isTypeExpression(node)) return getReportDescriptor(node.expression, seen);
+		if (Check.isTypeExpression(node)) return visit(node.expression, seen);
 		return match(node).with({
 			type: AST_NODE_TYPES.LogicalExpression,
 			operator: "&&"
 		}, ({ left, right }) => {
-			if (left.type === AST_NODE_TYPES.UnaryExpression && left.operator === "!") return getReportDescriptor(right, seen);
+			if (left.type === AST_NODE_TYPES.UnaryExpression && left.operator === "!") return visit(right, seen);
 			const initialScope = context.sourceCode.getScope(left);
 			if (Check.isIdentifier(left, "NaN") || getStaticValue(left, initialScope)?.value === "NaN") return {
 				data: { value: context.sourceCode.getText(left) },
@@ -2661,22 +2650,22 @@ function create$27(context) {
 			};
 			const leftType = getConstrainedTypeAtLocation(services, left);
 			const leftTypeVariants = core.getTypeVariants(unionConstituents(leftType));
-			if (Array.from(leftTypeVariants.values()).every((t) => allowedTypeVariants.some((a) => a === t))) return getReportDescriptor(right, seen);
+			if (Array.from(leftTypeVariants.values()).every((t) => allowedTypeVariants.some((a) => a === t))) return visit(right, seen);
 			return {
 				data: { value: context.sourceCode.getText(left) },
 				messageId: "default",
 				node: left
 			};
 		}).with({ type: AST_NODE_TYPES.ConditionalExpression }, ({ alternate, consequent }) => {
-			return getReportDescriptor(consequent, seen) ?? getReportDescriptor(alternate, seen);
+			return visit(consequent, seen) ?? visit(alternate, seen);
 		}).with({ type: AST_NODE_TYPES.Identifier }, (n) => {
 			if (seen.has(n.name)) return null;
 			seen.add(n.name);
 			const variableDefNode = findVariable(context.sourceCode.getScope(n), n.name)?.defs.at(0)?.node;
-			return match(variableDefNode).with({ init: P.select({ type: P.not(AST_NODE_TYPES.VariableDeclaration) }) }, (init) => getReportDescriptor(init, seen)).otherwise(() => null);
+			return match(variableDefNode).with({ init: P.select({ type: P.not(AST_NODE_TYPES.VariableDeclaration) }) }, (init) => visit(init, seen)).otherwise(() => null);
 		}).otherwise(() => null);
 	}
-	return { JSXExpressionContainer: flow(getReportDescriptor, report$1(context)) };
+	return { JSXExpressionContainer: flow(visit, report(context)) };
 }
 
 //#endregion
@@ -2791,17 +2780,6 @@ const INDEX_PARAM_POSITIONS = /* @__PURE__ */ new Map([
 	["map", 0]
 ]);
 /**
-* Creates a report function for the given rule context.
-* @param context The ESLint rule context.
-* @returns A function that can be used to report violations.
-*/
-function report(context) {
-	return (descriptor) => {
-		if (descriptor == null) return;
-		context.report(descriptor);
-	};
-}
-/**
 * Gets the nested return statements in the node that are within the same function
 * @param node The AST node
 * @returns The nested return statements in the node
@@ -2857,7 +2835,7 @@ function create$24(context) {
 	* @param node The expression to check.
 	* @returns The report descriptors for the violations found.
 	*/
-	function checkItemExpression(node) {
+	function visitItemExpression(node) {
 		switch (node.type) {
 			case AST_NODE_TYPES.JSXElement: return hasAttribute(context, node, "key") ? [] : [{
 				messageId: "default",
@@ -2867,8 +2845,8 @@ function create$24(context) {
 				messageId: "unexpectedFragmentSyntax",
 				node
 			}];
-			case AST_NODE_TYPES.ConditionalExpression: return [...checkItemExpression(node.consequent), ...checkItemExpression(node.alternate)];
-			case AST_NODE_TYPES.LogicalExpression: return [...checkItemExpression(node.left), ...checkItemExpression(node.right)];
+			case AST_NODE_TYPES.ConditionalExpression: return [...visitItemExpression(node.consequent), ...visitItemExpression(node.alternate)];
+			case AST_NODE_TYPES.LogicalExpression: return [...visitItemExpression(node.left), ...visitItemExpression(node.right)];
 			default: return [];
 		}
 	}
@@ -2878,16 +2856,16 @@ function create$24(context) {
 	* @param node The callback function to check.
 	* @returns The report descriptors for the violations found.
 	*/
-	function checkIteratorCallback(node) {
-		if (node.body.type !== AST_NODE_TYPES.BlockStatement) return checkItemExpression(node.body);
-		return getNestedReturnStatements$1(node.body).flatMap((stmt) => stmt.argument == null ? [] : checkItemExpression(stmt.argument));
+	function visitIteratorCallback(node) {
+		if (node.body.type !== AST_NODE_TYPES.BlockStatement) return visitItemExpression(node.body);
+		return getNestedReturnStatements$1(node.body).flatMap((stmt) => stmt.argument == null ? [] : visitItemExpression(stmt.argument));
 	}
 	return {
 		ArrayExpression(node) {
 			if (childrenToArrayDepth > 0) return;
 			for (const element of node.elements) {
 				if (element == null || element.type === AST_NODE_TYPES.SpreadElement) continue;
-				checkItemExpression(element).forEach(report(context));
+				for (const desc of visitItemExpression(element)) context.report(desc);
 			}
 		},
 		CallExpression(node) {
@@ -2898,7 +2876,7 @@ function create$24(context) {
 			if (childrenToArrayDepth > 0) return;
 			const callback = getIteratorCallback(node);
 			if (callback == null) return;
-			checkIteratorCallback(callback).forEach(report(context));
+			for (const desc of visitIteratorCallback(callback)) context.report(desc);
 		},
 		"CallExpression:exit"(node) {
 			if (core.isChildrenToArrayCall(context, node)) childrenToArrayDepth -= 1;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "5.12.0",
+  "version": "5.12.1",
   "description": "A set of composable ESLint rules for libraries and frameworks that use React as a UI runtime.",
   "keywords": [
     "react",
@@ -45,12 +45,12 @@
     "string-ts": "^2.3.1",
     "ts-api-utils": "^2.5.0",
     "ts-pattern": "^5.9.0",
-    "@eslint-react/ast": "5.12.0",
-    "@eslint-react/core": "5.12.0",
-    "@eslint-react/eslint": "5.12.0",
-    "@eslint-react/jsx": "5.12.0",
-    "@eslint-react/shared": "5.12.0",
-    "@eslint-react/var": "5.12.0"
+    "@eslint-react/ast": "5.12.1",
+    "@eslint-react/jsx": "5.12.1",
+    "@eslint-react/core": "5.12.1",
+    "@eslint-react/var": "5.12.1",
+    "@eslint-react/shared": "5.12.1",
+    "@eslint-react/eslint": "5.12.1"
   },
   "devDependencies": {
     "@types/react": "^19.2.17",