eslint-plugin-react-x 5.7.8-next.0 → 5.7.9-beta.0

package/dist/index.js

@@ -143,7 +143,7 @@ const rules$6 = {
 //#endregion
 //#region package.json
 var name$6 = "eslint-plugin-react-x";
-var version = "5.7.8-next.0";
+var version = "5.7.9-beta.0";
 
 //#endregion
 //#region src/utils/create-rule.ts
@@ -169,11 +169,26 @@ var error_boundaries_default = createRule({
 	create: create$50,
 	defaultOptions: []
 });
+/**
+* Finds the nearest TryStatement whose `try` block (not catch/finally)
+* encloses the given node.
+* @param node The node to check.
+* @returns The enclosing TryStatement, or null if none is found.
+*/
 function getEnclosingTryBlock(node) {
-	const tryStmt = Traverse.findParent(node, is(AST_NODE_TYPES.TryStatement));
-	if (tryStmt == null) return null;
-	if (Traverse.findParent(node, (n) => n === tryStmt.block) != null) return tryStmt;
-	return getEnclosingTryBlock(tryStmt);
+	let current = node.parent;
+	while (current != null) {
+		if (current.type === AST_NODE_TYPES.TryStatement) {
+			let n = node;
+			while (n != null && n !== current) {
+				if (n === current.block) return current;
+				n = n.parent;
+			}
+		}
+		if (current.type === "Program") return null;
+		current = current.parent;
+	}
+	return null;
 }
 function create$50(context) {
 	if (!context.sourceCode.text.includes("try")) return {};
@@ -1282,6 +1297,7 @@ const MUTATING_ARRAY_METHODS = new Set([
 /**
 * Check if a name is ref-like ("ref" or ends with "Ref").
 * Refs are mutable by design and exempted from immutability checks.
+* @param name
 */
 function isRefLikeName(name) {
 	return name === "ref" || name.endsWith("Ref");
@@ -1289,6 +1305,7 @@ function isRefLikeName(name) {
 /**
 * Check if any identifier or property name in a member-expression chain
 * is ref-like.
+* @param node
 */
 function hasRefLikeNameInChain(node) {
 	if (node.type === AST_NODE_TYPES.Identifier) return isRefLikeName(node.name);
@@ -2616,16 +2633,16 @@ function create$27(context) {
 	* @param node The AST node to inspect
 	* @returns A report descriptor if a problem is found, otherwise `null`
 	*/
-	function getReportDescriptor(node) {
+	function getReportDescriptor(node, visited = /* @__PURE__ */ new Set()) {
 		if (node == null) return null;
-		if (is(AST_NODE_TYPES.JSXExpressionContainer)(node)) return getReportDescriptor(node.expression);
+		if (is(AST_NODE_TYPES.JSXExpressionContainer)(node)) return getReportDescriptor(node.expression, visited);
 		if (Check.isJSX(node)) return null;
-		if (Check.isTypeExpression(node)) return getReportDescriptor(node.expression);
+		if (Check.isTypeExpression(node)) return getReportDescriptor(node.expression, visited);
 		return match(node).with({
 			type: AST_NODE_TYPES.LogicalExpression,
 			operator: "&&"
 		}, ({ left, right }) => {
-			if (left.type === AST_NODE_TYPES.UnaryExpression && left.operator === "!") return getReportDescriptor(right);
+			if (left.type === AST_NODE_TYPES.UnaryExpression && left.operator === "!") return getReportDescriptor(right, visited);
 			const initialScope = context.sourceCode.getScope(left);
 			if (Check.isIdentifier("NaN")(left) || getStaticValue(left, initialScope)?.value === "NaN") return {
 				data: { value: context.sourceCode.getText(left) },
@@ -2634,17 +2651,19 @@ function create$27(context) {
 			};
 			const leftType = getConstrainedTypeAtLocation(services, left);
 			const leftTypeVariants = core.getTypeVariants(unionConstituents(leftType));
-			if (Array.from(leftTypeVariants.values()).every((type) => allowedVariants.some((allowed) => allowed === type))) return getReportDescriptor(right);
+			if (Array.from(leftTypeVariants.values()).every((type) => allowedVariants.some((allowed) => allowed === type))) return getReportDescriptor(right, visited);
 			return {
 				data: { value: context.sourceCode.getText(left) },
 				messageId: "default",
 				node: left
 			};
 		}).with({ type: AST_NODE_TYPES.ConditionalExpression }, ({ alternate, consequent }) => {
-			return getReportDescriptor(consequent) ?? getReportDescriptor(alternate);
+			return getReportDescriptor(consequent, visited) ?? getReportDescriptor(alternate, visited);
 		}).with({ type: AST_NODE_TYPES.Identifier }, (n) => {
+			if (visited.has(n.name)) return null;
+			visited.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) }) }, getReportDescriptor).otherwise(() => null);
+			return match(variableDefNode).with({ init: P.select({ type: P.not(AST_NODE_TYPES.VariableDeclaration) }) }, (init) => getReportDescriptor(init, visited)).otherwise(() => null);
 		}).otherwise(() => null);
 	}
 	return merge({ JSXExpressionContainer: flow(getReportDescriptor, report$1(context)) });
@@ -6674,7 +6693,9 @@ function isHookDecl(node) {
 		default: return false;
 	}
 }
-function isInitializedFromRef(context, name, initialScope) {
+function isInitializedFromRef(context, name, initialScope, visited = /* @__PURE__ */ new Set()) {
+	if (visited.has(name)) return false;
+	visited.add(name);
 	for (const { node } of findVariable(initialScope, name)?.defs ?? []) {
 		if (node.type !== AST_NODE_TYPES.VariableDeclarator) continue;
 		const init = node.init;
@@ -6682,7 +6703,7 @@ function isInitializedFromRef(context, name, initialScope) {
 		switch (true) {
 			case init.type === AST_NODE_TYPES.MemberExpression && init.object.type === AST_NODE_TYPES.Identifier && (init.object.name === "ref" || init.object.name.endsWith("Ref")): return true;
 			case init.type === AST_NODE_TYPES.CallExpression && core.isUseRefCall(context, init): return true;
-			case init.type === AST_NODE_TYPES.CallExpression: return getNestedIdentifiers(init).some((id) => isInitializedFromRef(context, id.name, context.sourceCode.getScope(id)));
+			case init.type === AST_NODE_TYPES.CallExpression: return getNestedIdentifiers(init).some((id) => isInitializedFromRef(context, id.name, context.sourceCode.getScope(id), visited));
 		}
 	}
 	return false;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "5.7.8-next.0",
+  "version": "5.7.9-beta.0",
   "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.7.8-next.0",
-    "@eslint-react/core": "5.7.8-next.0",
-    "@eslint-react/jsx": "5.7.8-next.0",
-    "@eslint-react/eslint": "5.7.8-next.0",
-    "@eslint-react/shared": "5.7.8-next.0",
-    "@eslint-react/var": "5.7.8-next.0"
+    "@eslint-react/ast": "5.7.9-beta.0",
+    "@eslint-react/eslint": "5.7.9-beta.0",
+    "@eslint-react/core": "5.7.9-beta.0",
+    "@eslint-react/shared": "5.7.9-beta.0",
+    "@eslint-react/jsx": "5.7.9-beta.0",
+    "@eslint-react/var": "5.7.9-beta.0"
   },
   "devDependencies": {
     "@types/react": "^19.2.14",
@@ -63,8 +63,8 @@
     "tsl": "^1.0.30",
     "tsl-dx": "^0.12.1",
     "typescript": "^6.0.3",
-    "@local/configs": "0.0.0",
-    "@local/eff": "3.0.0-beta.72"
+    "@local/eff": "3.0.0-beta.72",
+    "@local/configs": "0.0.0"
   },
   "peerDependencies": {
     "eslint": "^10.3.0",