eslint-plugin-react-x 5.24.6 → 5.24.8

package/dist/index.js

@@ -135,7 +135,7 @@ const rules$6 = {
 //#endregion
 //#region package.json
 var name$6 = "eslint-plugin-react-x";
-var version = "5.24.6";
+var version = "5.24.8";
 //#endregion
 //#region src/utils/create-rule.ts
 function getDocsUrl(ruleName) {
@@ -1335,15 +1335,16 @@ function resolveToFunctionNode(context, node, seen = /* @__PURE__ */ new Set())
 	const resolved = resolveOrigin(context, expr);
 	return resolved == null ? null : resolveToFunctionNode(context, resolved, seen);
 }
-function resolveVariableOrigin(context, variable, seen = /* @__PURE__ */ new Set()) {
+function resolveVariableOrigin(context, variable, mode = "alias", seen = /* @__PURE__ */ new Set()) {
 	if (seen.has(variable)) return variable;
 	seen.add(variable);
 	const def = variable.defs.length === 1 ? variable.defs[0] : null;
 	if (def?.type !== DefinitionType.Variable || def.node.init == null) return variable;
+	if (mode === "binding" && def.node.id.type === AST_NODE_TYPES.Identifier) return variable;
 	const init = Extract.unwrap(def.node.init);
 	if (!Check.isIdentifier(init)) return variable;
 	const source = findVariable(context.sourceCode.getScope(init), init);
-	return source == null ? variable : resolveVariableOrigin(context, source, seen);
+	return source == null ? variable : resolveVariableOrigin(context, source, mode, seen);
 }
 function isRefLikeName$1(name) {
 	return name === "ref" || name.endsWith("Ref");
@@ -1467,10 +1468,10 @@ function createFactCollector$5() {
 }
 //#endregion
 //#region src/rules/immutability/origins.ts
-function classifyFrozenOrigin(context, variable, components, seen = /* @__PURE__ */ new Set()) {
+function classifyFrozenOrigin(context, variable, components, mode = "alias", seen = /* @__PURE__ */ new Set()) {
 	if (seen.has(variable)) return null;
 	seen.add(variable);
-	const origin = resolveVariableOrigin(context, variable);
+	const origin = resolveVariableOrigin(context, variable, mode);
 	const def = origin.defs.length === 1 ? origin.defs[0] : null;
 	if (def == null) return null;
 	if (isComponentPropsDefinition(def, components)) return {
@@ -1485,7 +1486,7 @@ function classifyFrozenOrigin(context, variable, components, seen = /* @__PURE__
 		if (root == null || !Check.isIdentifier(root)) return null;
 		const source = findVariable(context.sourceCode.getScope(root), root);
 		if (source == null) return null;
-		const inner = classifyFrozenOrigin(context, source, components, seen);
+		const inner = classifyFrozenOrigin(context, source, components, mode, seen);
 		return inner == null ? null : {
 			kind: "iterator",
 			name: origin.name,
@@ -1515,7 +1516,7 @@ function classifyFrozenOrigin(context, variable, components, seen = /* @__PURE__
 				if (!Check.isIdentifier(argument)) continue;
 				const source = findVariable(context.sourceCode.getScope(argument), argument);
 				if (source == null) continue;
-				const inner = classifyFrozenOrigin(context, source, components, seen);
+				const inner = classifyFrozenOrigin(context, source, components, mode, seen);
 				if (inner != null) return {
 					kind: "shallow-copy",
 					name: origin.name,
@@ -1570,7 +1571,7 @@ function inferDirectMutations(context, mutations, components) {
 		if (isRefMutation(context, mutation)) continue;
 		const variable = findVariable(context.sourceCode.getScope(mutation.root), mutation.root);
 		if (variable == null) continue;
-		const origin = classifyFrozenOrigin(context, variable, components);
+		const origin = classifyFrozenOrigin(context, variable, components, mutation.kind === "binding" ? "binding" : "alias");
 		if (origin == null) continue;
 		if (mutation.kind === "binding") {
 			switch (origin.kind) {
@@ -4875,7 +4876,7 @@ var ForkContext = class ForkContext {
 	makeNext(begin, end) {
 		return makeSegments(this, begin, end, CodePathSegment.newNext);
 	}
-	makeUnreachable(begin, end) {
+	makeUnreachable(begin = NaN, end = NaN) {
 		return makeSegments(this, begin, end, CodePathSegment.newUnreachable);
 	}
 	makeDisconnected(begin, end) {
@@ -5253,7 +5254,7 @@ var CodePathState = class {
 		if (returned && !returned.empty) getReturnContext(this).returnedForkContext.add(leavingSegments);
 		if (!thrown.empty) getThrowContext(this).thrownForkContext.add(leavingSegments);
 		this.forkContext.replaceHead(normalSegments);
-		if (!context.lastOfTryIsReachable && !context.lastOfCatchIsReachable) this.forkContext.makeUnreachable(-1, -1);
+		if (!context.lastOfTryIsReachable && !context.lastOfCatchIsReachable) this.forkContext.makeUnreachable();
 	}
 	makeCatchBlock() {
 		const context = this.tryContext;
@@ -6765,9 +6766,9 @@ function create$5(context) {
 //#endregion
 //#region src/rules/set-state-in-render/helpers.ts
 const isComponentOrHookName = or(core.isFunctionComponentName, core.isHookName);
-function isComponentOrHookLikeFunction(node) {
+function isComponentOrHookLikeFunction(context, node) {
 	const id = core.getFunctionId(node);
-	if (id == null) return false;
+	if (id == null) return core.isFunctionComponentWrapperCallback(context, node);
 	if (Check.isIdentifier(id)) return isComponentOrHookName(id.name);
 	if (id.type === AST_NODE_TYPES.MemberExpression && Check.isIdentifier(id.property)) return isComponentOrHookName(id.property.name);
 	return false;
@@ -6788,18 +6789,19 @@ function isInsideConditional(node, stopAt) {
 }
 //#endregion
 //#region src/rules/set-state-in-render/collect.ts
-function createFactCollector$2(additionalStateHooks) {
+function createFactCollector$2(context) {
+	const { additionalStateHooks } = getSettingsFromContext(context);
 	const facts = {
 		calls: [],
 		earlyReturns: []
 	};
 	const functionEntries = [];
-	const componentFnRef = { current: null };
+	const componentFnStack = [];
 	function isUseStateCall(node) {
 		return core.isUseStateLikeCall(node, additionalStateHooks);
 	}
 	function getFunctionKind(node) {
-		if (isComponentOrHookLikeFunction(node)) return "component";
+		if (isComponentOrHookLikeFunction(context, node)) return "component";
 		const parent = Traverse.findParent(node, not(Check.isTypeExpression)) ?? node.parent;
 		if (parent.type === AST_NODE_TYPES.CallExpression && parent.callee !== node) return "callback";
 		return "other";
@@ -6813,10 +6815,10 @@ function createFactCollector$2(additionalStateHooks) {
 					kind,
 					node
 				});
-				if (kind === "component") componentFnRef.current = node;
+				if (kind === "component") componentFnStack.push(node);
 			},
 			":function:exit"(node) {
-				if (functionEntries.at(-1)?.kind === "component" && componentFnRef.current === node) componentFnRef.current = null;
+				if (functionEntries.at(-1)?.kind === "component" && componentFnStack.at(-1) === node) componentFnStack.pop();
 				functionEntries.pop();
 			},
 			CallExpression(node) {
@@ -6824,14 +6826,14 @@ function createFactCollector$2(additionalStateHooks) {
 				if (entry == null) return;
 				if (isUseStateCall(node)) return;
 				facts.calls.push({
-					componentFunction: componentFnRef.current,
+					componentFunction: componentFnStack.at(-1) ?? null,
 					enclosingFunction: entry.node,
 					enclosingFunctionKind: entry.kind,
 					node
 				});
 			},
 			ReturnStatement(node) {
-				const componentFn = componentFnRef.current;
+				const componentFn = componentFnStack.at(-1);
 				if (componentFn == null) return;
 				const entry = functionEntries.at(-1);
 				if (entry == null || entry.node !== componentFn) return;
@@ -6930,8 +6932,7 @@ var set_state_in_render_default = createRule({
 	create: create$4
 });
 function create$4(context) {
-	const { additionalStateHooks } = getSettingsFromContext(context);
-	const facts = createFactCollector$2(additionalStateHooks);
+	const facts = createFactCollector$2(context);
 	return merge(facts.visitor, { "Program:exit"() {
 		for (const violation of inferViolations$1(context, facts.facts)) context.report({
 			data: { name: violation.name },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "5.24.6",
+  "version": "5.24.8",
   "description": "A set of composable ESLint rules for libraries and frameworks that use React as a UI runtime.",
   "keywords": [
     "react",
@@ -36,16 +36,16 @@
     "dist"
   ],
   "dependencies": {
-    "@eslint-react/ast": "5.24.6",
-    "@eslint-react/core": "5.24.6",
-    "@eslint-react/eslint": "5.24.6",
-    "@eslint-react/jsx": "5.24.6",
-    "@eslint-react/shared": "5.24.6",
-    "@eslint-react/var": "5.24.6",
-    "@typescript-eslint/scope-manager": "^8.71.0",
-    "@typescript-eslint/types": "^8.71.0",
-    "@typescript-eslint/typescript-estree": "^8.71.0",
-    "@typescript-eslint/utils": "^8.71.0",
+    "@eslint-react/ast": "5.24.8",
+    "@eslint-react/core": "5.24.8",
+    "@eslint-react/eslint": "5.24.8",
+    "@eslint-react/jsx": "5.24.8",
+    "@eslint-react/shared": "5.24.8",
+    "@eslint-react/var": "5.24.8",
+    "@typescript-eslint/scope-manager": "^8.71.1",
+    "@typescript-eslint/types": "^8.71.1",
+    "@typescript-eslint/typescript-estree": "^8.71.1",
+    "@typescript-eslint/utils": "^8.71.1",
     "compare-versions": "^6.1.1",
     "string-ts": "^2.3.1",
     "ts-api-utils": "^2.5.0",