eslint-plugin-react-x 5.18.5 → 5.18.7

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.18.5";
+var version = "5.18.7";
 
 //#endregion
 //#region src/utils/create-rule.ts
@@ -1163,12 +1163,12 @@ function resolveGlobalOrigin(context, node, seen = /* @__PURE__ */ new Set()) {
 	if (seen.has(expression)) return null;
 	seen.add(expression);
 	if (expression.type === AST_NODE_TYPES.MemberExpression) return resolveGlobalOrigin(context, expression.object, seen);
-	if (expression.type !== AST_NODE_TYPES.Identifier) return null;
+	if (!Check.isIdentifier(expression)) return null;
 	if (isGlobalVariable(context, expression)) return expression;
 	const variable = findVariable(context.sourceCode.getScope(expression), expression);
 	const definition = variable?.defs.length === 1 ? variable.defs[0] : null;
 	if (definition?.type !== DefinitionType.Variable) return null;
-	if (definition.node.id.type !== AST_NODE_TYPES.Identifier || definition.node.init == null) return null;
+	if (!Check.isIdentifier(definition.node.id) || definition.node.init == null) return null;
 	if (definition.node.parent.kind !== "const") return null;
 	const initializer = Extract.unwrap(definition.node.init);
 	if (initializer.type !== AST_NODE_TYPES.Identifier && initializer.type !== AST_NODE_TYPES.MemberExpression) return null;
@@ -1197,7 +1197,7 @@ function getAssignmentTargets(node) {
 function resolveToFunction(context, node, seen = /* @__PURE__ */ new Set()) {
 	const expression = Extract.unwrap(node);
 	if (Check.isFunction(expression)) return expression;
-	if (expression.type !== AST_NODE_TYPES.Identifier || seen.has(expression)) return null;
+	if (!Check.isIdentifier(expression) || seen.has(expression)) return null;
 	seen.add(expression);
 	const resolved = resolve(context, expression);
 	if (resolved == null) return null;
@@ -1242,7 +1242,7 @@ function create$49(context) {
 		directEffects.set(enclosing, effects);
 	}
 	function recordWrite(node, target) {
-		if (target.type === AST_NODE_TYPES.Identifier) {
+		if (Check.isIdentifier(target)) {
 			if (!isGlobalVariable(context, target)) return;
 			recordEffect(node, "mutatingGlobal", { name: target.name });
 			return;
@@ -1341,7 +1341,7 @@ const NAVIGATION_HOOKS = /* @__PURE__ */ new Set([
 function resolveToFunctionNode(context, node, seen = /* @__PURE__ */ new Set()) {
 	const expr = Extract.unwrap(node);
 	if (Check.isFunction(expr)) return expr;
-	if (expr.type !== AST_NODE_TYPES.Identifier || seen.has(expr)) return null;
+	if (!Check.isIdentifier(expr) || seen.has(expr)) return null;
 	seen.add(expr);
 	const resolved = resolve(context, expr);
 	return resolved == null ? null : resolveToFunctionNode(context, resolved, seen);
@@ -1352,7 +1352,7 @@ function resolveVariableOrigin(context, variable, seen = /* @__PURE__ */ new Set
 	const def = variable.defs.length === 1 ? variable.defs[0] : null;
 	if (def?.type !== DefinitionType.Variable || def.node.init == null) return variable;
 	const init = Extract.unwrap(def.node.init);
-	if (init.type !== AST_NODE_TYPES.Identifier) return variable;
+	if (!Check.isIdentifier(init)) return variable;
 	const source = findVariable(context.sourceCode.getScope(init), init);
 	return source == null ? variable : resolveVariableOrigin(context, source, seen);
 }
@@ -1360,12 +1360,12 @@ function isRefLikeName$1(name) {
 	return name === "ref" || name.endsWith("Ref");
 }
 function hasRefLikeNameInChain(node) {
-	if (node.type === AST_NODE_TYPES.Identifier) return isRefLikeName$1(node.name);
+	if (Check.isIdentifier(node)) return isRefLikeName$1(node.name);
 	if (node.type !== AST_NODE_TYPES.MemberExpression) return false;
-	return node.property.type === AST_NODE_TYPES.Identifier ? isRefLikeName$1(node.property.name) || hasRefLikeNameInChain(node.object) : hasRefLikeNameInChain(node.object);
+	return Check.isIdentifier(node.property) ? isRefLikeName$1(node.property.name) || hasRefLikeNameInChain(node.object) : hasRefLikeNameInChain(node.object);
 }
 function isInitializedFromCall(context, node, isCall) {
-	const root = node.type === AST_NODE_TYPES.Identifier ? node : Extract.getIdentifierAt(node, 0);
+	const root = Check.isIdentifier(node) ? node : Extract.getIdentifierAt(node, 0);
 	if (root == null) return false;
 	const variable = findVariable(context.sourceCode.getScope(root), root);
 	if (variable == null) return false;
@@ -1480,7 +1480,7 @@ function isGlobalOrModuleVariable(variable) {
 	return variable.defs.length === 0 || variable.scope.type === ScopeType.global || variable.scope.type === ScopeType.module;
 }
 function isRefMutation(context, mutation) {
-	if (mutation.target.type === AST_NODE_TYPES.Identifier) return isRefLikeName$1(mutation.target.name);
+	if (Check.isIdentifier(mutation.target)) return isRefLikeName$1(mutation.target.name);
 	return isRefLikeChain(context, mutation.target);
 }
 function inferMutableFunctions(context, mutations) {
@@ -1604,7 +1604,7 @@ function create$47(context) {
 			if (currMethod == null || isStatic) return;
 			const [setState, hasThisState = false] = setStateStack.at(-1) ?? [];
 			if (setState == null || hasThisState) return;
-			if (node.property.type !== AST_NODE_TYPES.Identifier || node.property.name !== "state") return;
+			if (!Check.isIdentifier(node.property, "state")) return;
 			context.report({
 				messageId: "default",
 				node
@@ -1630,7 +1630,7 @@ function create$47(context) {
 			const [setState, hasThisState = false] = setStateStack.at(-1) ?? [];
 			if (setState == null || hasThisState) return;
 			if (node.init == null || node.init.type !== AST_NODE_TYPES.ThisExpression || node.id.type !== AST_NODE_TYPES.ObjectPattern) return;
-			if (!node.id.properties.some((prop) => prop.type === AST_NODE_TYPES.Property && !prop.computed && prop.key.type === AST_NODE_TYPES.Identifier && prop.key.name === "state")) return;
+			if (!node.id.properties.some((prop) => prop.type === AST_NODE_TYPES.Property && !prop.computed && Check.isIdentifier(prop.key, "state"))) return;
 			context.report({
 				messageId: "default",
 				node
@@ -1679,7 +1679,7 @@ function isArrayIndexReference(context, node) {
 	if (call.type !== AST_NODE_TYPES.CallExpression) return false;
 	const callee = Extract.unwrap(call.callee);
 	if (callee.type !== AST_NODE_TYPES.MemberExpression) return false;
-	if (callee.property.type !== AST_NODE_TYPES.Identifier) return false;
+	if (!Check.isIdentifier(callee.property)) return false;
 	const indexPosition = INDEX_PARAM_POSITIONS$1.get(callee.property.name);
 	if (indexPosition == null) return false;
 	const callbackPosition = core.isChildrenMap(context, callee) || core.isChildrenForEach(context, callee) ? 1 : 0;
@@ -1695,7 +1695,7 @@ function isArrayIndexReference(context, node) {
 * @returns The identifiers found in the expression.
 */
 function getIdentifiersFromBinaryExpression(side) {
-	if (side.type === AST_NODE_TYPES.Identifier) return [side];
+	if (Check.isIdentifier(side)) return [side];
 	if (side.type === AST_NODE_TYPES.BinaryExpression) return [...getIdentifiersFromBinaryExpression(side.left), ...getIdentifiersFromBinaryExpression(side.right)];
 	return [];
 }
@@ -1716,21 +1716,15 @@ var no_array_index_key_default = createRule({
 	create: create$46,
 	defaultOptions: []
 });
-/**
-* Gets the value of an object property named 'key' (e.g. in `createElement('div', { key: ... })`).
-* @param property The object literal member to inspect.
-* @returns The value of the 'key' property, or `null` if the member is not one.
-*/
-function getKeyPropValue(property) {
-	if (property.type !== AST_NODE_TYPES.Property || property.computed) return null;
-	return property.key.type === AST_NODE_TYPES.Identifier && property.key.name === "key" || property.key.type === AST_NODE_TYPES.Literal && property.key.value === "key" ? property.value : null;
-}
 function create$46(context) {
 	function isArrayIndex(node) {
-		return node.type === AST_NODE_TYPES.Identifier && isArrayIndexReference(context, node);
+		return Check.isIdentifier(node) && isArrayIndexReference(context, node);
 	}
-	function isCreateOrCloneElementCall(node) {
-		return core.isCreateElementCall(context, node) || core.isCloneElementCall(context, node);
+	function getPropsObject(node) {
+		const props = node.arguments[1];
+		if (core.isCreateElementCall(context, node)) return core.getCreateElementPropsObject(context, node);
+		if (!core.isCloneElementCall(context, node)) return null;
+		return props?.type === AST_NODE_TYPES.ObjectExpression ? props : null;
 	}
 	/**
 	* Checks an expression used as a 'key' value, recursing into the branches
@@ -1772,12 +1766,13 @@ function create$46(context) {
 	}
 	return {
 		CallExpression(node) {
-			if (!isCreateOrCloneElementCall(node)) return;
-			const [, props] = node.arguments;
-			if (props?.type !== AST_NODE_TYPES.ObjectExpression) return;
-			for (const property of props.properties) {
-				const value = getKeyPropValue(property);
-				if (value == null) continue;
+			const propsObject = getPropsObject(node);
+			if (propsObject == null) return;
+			for (const property of propsObject.properties) {
+				if (property.type !== AST_NODE_TYPES.Property) continue;
+				if (property.computed) continue;
+				if (Extract.getPropertyName(property, "max") !== "key") continue;
+				const value = property.value;
 				for (const desc of visitKeyExpression(value)) context.report(desc);
 			}
 		},
@@ -2146,7 +2141,7 @@ function create$34(context) {
 //#region src/rules/no-direct-mutation-state/no-direct-mutation-state.ts
 const RULE_NAME$33 = "no-direct-mutation-state";
 function isConstructorFunction(node) {
-	return Check.isOneOf([AST_NODE_TYPES.FunctionDeclaration, AST_NODE_TYPES.FunctionExpression])(node) && Check.isPropertyOrMethod(node.parent) && node.parent.key.type === AST_NODE_TYPES.Identifier && node.parent.key.name === "constructor";
+	return Check.isOneOf([AST_NODE_TYPES.FunctionDeclaration, AST_NODE_TYPES.FunctionExpression])(node) && Check.isPropertyOrMethod(node.parent) && Check.isIdentifier(node.parent.key) && node.parent.key.name === "constructor";
 }
 var no_direct_mutation_state_default = createRule({
 	meta: {
@@ -2214,7 +2209,7 @@ function create$32(context) {
 					const call = Traverse.findParent(jsxElement, (n) => {
 						if (n.type !== AST_NODE_TYPES.CallExpression) return false;
 						const callee = Extract.unwrap(n.callee);
-						return callee.type === AST_NODE_TYPES.MemberExpression && callee.property.type === AST_NODE_TYPES.Identifier && callee.property.name === "map";
+						return callee.type === AST_NODE_TYPES.MemberExpression && Check.isIdentifier(callee.property) && callee.property.name === "map";
 					});
 					const iter = Traverse.findParent(jsxElement, Check.isFunction, (n) => n === call);
 					if (!Check.isFunction(iter)) return;
@@ -2293,7 +2288,7 @@ function couldFix(context, node) {
 	const callee = Extract.unwrap(node.callee);
 	switch (callee.type) {
 		case AST_NODE_TYPES.Identifier: return isInitializedFromReact(callee.name, initialScope, importSource);
-		case AST_NODE_TYPES.MemberExpression: return callee.object.type === AST_NODE_TYPES.Identifier && isInitializedFromReact(callee.object.name, initialScope, importSource);
+		case AST_NODE_TYPES.MemberExpression: return Check.isIdentifier(callee.object) && isInitializedFromReact(callee.object.name, initialScope, importSource);
 		default: return false;
 	}
 }
@@ -3259,7 +3254,7 @@ var no_missing_key_default = createRule({
 function getIteratorCallback(node) {
 	const callee = Extract.unwrap(node.callee);
 	if (callee.type !== AST_NODE_TYPES.MemberExpression) return null;
-	if (callee.property.type !== AST_NODE_TYPES.Identifier) return null;
+	if (!Check.isIdentifier(callee.property)) return null;
 	const position = INDEX_PARAM_POSITIONS.get(callee.property.name);
 	if (position == null) return null;
 	const callback = node.arguments[position];
@@ -3482,23 +3477,10 @@ function getWrapperCallBoundName(context, node) {
 			parent = parent.parent;
 		}
 	}
-	if (parent.type !== AST_NODE_TYPES.VariableDeclarator || parent.id.type !== AST_NODE_TYPES.Identifier) return null;
+	if (parent.type !== AST_NODE_TYPES.VariableDeclarator || !Check.isIdentifier(parent.id)) return null;
 	return parent.id.name;
 }
 /**
-* Check if the node is inside `createElement`'s props argument
-* @param context The rule context
-* @param node The AST node to check
-* @returns `true` if the node is inside `createElement`'s props
-*/
-function isInsideCreateElementProps(context, node) {
-	const call = Traverse.findParent(node, core.isCreateElementCall(context));
-	if (call == null) return false;
-	const prop = Traverse.findParent(node, Check.is(AST_NODE_TYPES.ObjectExpression));
-	if (prop == null) return false;
-	return prop === call.arguments[1];
-}
-/**
 * Check if the node is inside a JSX attribute value
 * @param node The AST node to check
 * @returns `true` if the node is inside a JSX attribute value
@@ -3560,7 +3542,7 @@ function create$22(context) {
 				});
 				continue;
 			}
-			if (isInsideCreateElementProps(context, component)) {
+			if (core.isInsideCreateElementProps(context, component)) {
 				context.report({
 					data: {
 						name,
@@ -3740,7 +3722,7 @@ function containsUseComments(context, node) {
 	return context.sourceCode.getCommentsInside(node).some(({ value }) => /use\([\s\S]*?\)/u.test(value) || /use[A-Z0-9]\w*\([\s\S]*?\)/u.test(value));
 }
 function isTestMock(node) {
-	return node != null && node.type === AST_NODE_TYPES.MemberExpression && node.object.type === AST_NODE_TYPES.Identifier && node.property.type === AST_NODE_TYPES.Identifier && node.property.name === "mock";
+	return node != null && node.type === AST_NODE_TYPES.MemberExpression && Check.isIdentifier(node.object) && Check.isIdentifier(node.property) && node.property.name === "mock";
 }
 function isTestMockCallback(node) {
 	return node != null && Check.isFunction(node) && node.parent.type === AST_NODE_TYPES.CallExpression && isTestMock(node.parent.callee) && node.parent.arguments[1] === node;
@@ -3980,14 +3962,14 @@ var no_unstable_default_props_default = createRule({
 function extractIdentifier(node) {
 	if (node.type === AST_NODE_TYPES.NewExpression) {
 		const callee = Extract.unwrap(node.callee);
-		if (callee.type === AST_NODE_TYPES.Identifier) return callee.name;
+		if (Check.isIdentifier(callee)) return callee.name;
 	}
 	if (node.type === AST_NODE_TYPES.CallExpression) {
 		const callee = Extract.unwrap(node.callee);
-		if (callee.type === AST_NODE_TYPES.Identifier) return callee.name;
+		if (Check.isIdentifier(callee)) return callee.name;
 		if (callee.type === AST_NODE_TYPES.MemberExpression) {
 			const { object } = callee;
-			if (object.type === AST_NODE_TYPES.Identifier) return object.name;
+			if (Check.isIdentifier(object)) return object.name;
 		}
 	}
 	return null;
@@ -4655,7 +4637,7 @@ function resolveBuiltinObjectName(context, node, seen = /* @__PURE__ */ new Set(
 	if (def.type === DefinitionType.ImplicitGlobalVariable) return node.name;
 	if (def.type === DefinitionType.Variable && def.node.init != null) {
 		const init = Extract.unwrap(def.node.init);
-		if (init.type === AST_NODE_TYPES.Identifier) return resolveBuiltinObjectName(context, init, seen);
+		if (Check.isIdentifier(init)) return resolveBuiltinObjectName(context, init, seen);
 		if (init.type === AST_NODE_TYPES.MemberExpression) {
 			const rootId = Extract.getIdentifierAt(init, 0);
 			if (rootId != null) return resolveBuiltinObjectName(context, rootId, seen);
@@ -4687,7 +4669,7 @@ function create$7(context) {
 		CallExpression(node) {
 			const expr = Extract.unwrap(node.callee);
 			switch (true) {
-				case expr.type === AST_NODE_TYPES.Identifier: {
+				case Check.isIdentifier(expr): {
 					const builtinName = resolveBuiltinObjectName(context, expr);
 					if (builtinName == null) return;
 					const globalThisImpure = IMPURE_FUNCS.get("globalThis");
@@ -4700,7 +4682,7 @@ function create$7(context) {
 					});
 					break;
 				}
-				case expr.type === AST_NODE_TYPES.MemberExpression && expr.property.type === AST_NODE_TYPES.Identifier: {
+				case expr.type === AST_NODE_TYPES.MemberExpression && Check.isIdentifier(expr.property): {
 					const rootId = Extract.getIdentifierAt(expr.object, 0);
 					if (rootId == null) return;
 					const objectName = resolveBuiltinObjectName(context, rootId);
@@ -4720,7 +4702,7 @@ function create$7(context) {
 		},
 		NewExpression(node) {
 			const expr = Extract.unwrap(node.callee);
-			if (expr.type !== AST_NODE_TYPES.Identifier) return;
+			if (!Check.isIdentifier(expr)) return;
 			const builtinName = resolveBuiltinObjectName(context, expr);
 			if (builtinName == null) return;
 			if (!IMPURE_CTORS.has(builtinName)) return;
@@ -4775,7 +4757,7 @@ function createBindingResolver(context) {
 	}
 	function getBindingValue(node) {
 		const value = Extract.unwrap(node);
-		if (value.type === AST_NODE_TYPES.Identifier) {
+		if (Check.isIdentifier(value)) {
 			const variable = getVariable(value);
 			return variable == null ? { kind: "unknown" } : {
 				kind: "variable",
@@ -4787,7 +4769,7 @@ function createBindingResolver(context) {
 			node: value
 		};
 		if (value.type === AST_NODE_TYPES.CallExpression && (core.isUseRefLikeCall(value, additionalRefHooks) || core.isCreateRefCall(context, value))) return { kind: "ref" };
-		if (value.type === AST_NODE_TYPES.MemberExpression && value.property.type === AST_NODE_TYPES.Identifier) {
+		if (value.type === AST_NODE_TYPES.MemberExpression && Check.isIdentifier(value.property)) {
 			if (isRefLikeName(value.property.name)) return { kind: "ref" };
 		}
 		return { kind: "unknown" };
@@ -4855,14 +4837,14 @@ function createBindingResolver(context) {
 	function resolveCallable(node, position) {
 		const callee = Extract.unwrap(node);
 		if (isFunctionExpressionLike(callee)) return callee;
-		if (callee.type === AST_NODE_TYPES.Identifier) {
+		if (Check.isIdentifier(callee)) {
 			const variable = getVariable(callee);
 			return variable == null ? null : resolveFunction(variable, position);
 		}
-		if (callee.type !== AST_NODE_TYPES.MemberExpression || callee.property.type !== AST_NODE_TYPES.Identifier) return null;
+		if (callee.type !== AST_NODE_TYPES.MemberExpression || !Check.isIdentifier(callee.property)) return null;
 		const object = Extract.unwrap(callee.object);
 		const property = callee.property.name;
-		if (object.type !== AST_NODE_TYPES.Identifier) return null;
+		if (!Check.isIdentifier(object)) return null;
 		const variable = getVariable(object);
 		if (variable == null) return null;
 		const event = getLatestValue(memberBindings.get(variable)?.get(property), position);
@@ -4873,13 +4855,13 @@ function createBindingResolver(context) {
 	}
 	function getRefTarget(node) {
 		const object = Extract.unwrap(node.object);
-		if (object.type === AST_NODE_TYPES.Identifier) {
+		if (Check.isIdentifier(object)) {
 			const variable = getVariable(object);
 			if (variable == null) return null;
 			const identity = resolveRef(variable, node.range[0]);
 			return identity == null ? null : { identity };
 		}
-		if (object.type === AST_NODE_TYPES.MemberExpression && object.property.type === AST_NODE_TYPES.Identifier) {
+		if (object.type === AST_NODE_TYPES.MemberExpression && Check.isIdentifier(object.property)) {
 			if (isRefLikeName(object.property.name)) return { identity: null };
 		}
 		return null;
@@ -4887,12 +4869,12 @@ function createBindingResolver(context) {
 	function getNullBranch(test, identity) {
 		return getNullCheckBranch(test, (candidate) => {
 			if (candidate.type !== AST_NODE_TYPES.MemberExpression) return false;
-			if (candidate.property.type !== AST_NODE_TYPES.Identifier || candidate.property.name !== "current") return false;
+			if (!Check.isIdentifier(candidate.property, "current")) return false;
 			return getRefTarget(candidate)?.identity === identity;
 		}, (candidate) => {
 			if (candidate.type === AST_NODE_TYPES.Literal) return candidate.value == null;
 			if (candidate.type === AST_NODE_TYPES.UnaryExpression && candidate.operator === "void") return true;
-			if (candidate.type !== AST_NODE_TYPES.Identifier || candidate.name !== "undefined") return false;
+			if (!Check.isIdentifier(candidate, "undefined")) return false;
 			const variable = getVariable(candidate);
 			return variable == null || variable.defs.length === 0;
 		});
@@ -5078,13 +5060,13 @@ function create$6(context) {
 		AssignmentExpression(node) {
 			if (node.operator !== "=") return;
 			const left = Extract.unwrap(node.left);
-			if (left.type === AST_NODE_TYPES.Identifier) {
+			if (Check.isIdentifier(left)) {
 				addIdentifierBinding(left, node.right, node.range[0]);
 				return;
 			}
-			if (left.type !== AST_NODE_TYPES.MemberExpression || left.property.type !== AST_NODE_TYPES.Identifier) return;
+			if (left.type !== AST_NODE_TYPES.MemberExpression || !Check.isIdentifier(left.property)) return;
 			const object = Extract.unwrap(left.object);
-			if (object.type === AST_NODE_TYPES.Identifier) addMemberBinding(object, left.property.name, node.right, node.range[0]);
+			if (Check.isIdentifier(object)) addMemberBinding(object, left.property.name, node.right, node.range[0]);
 		},
 		CallExpression(node) {
 			calls.push(node);
@@ -5095,11 +5077,11 @@ function create$6(context) {
 		JSXAttribute(node) {
 			if (node.name.type !== AST_NODE_TYPES.JSXIdentifier || node.name.name !== "ref" || node.value?.type !== AST_NODE_TYPES.JSXExpressionContainer) return;
 			const expression = Extract.unwrap(node.value.expression);
-			if (expression.type !== AST_NODE_TYPES.Identifier) return;
+			if (!Check.isIdentifier(expression)) return;
 			addJsxRef(expression);
 		},
 		MemberExpression(node) {
-			if (node.property.type !== AST_NODE_TYPES.Identifier || node.property.name !== "current") return;
+			if (!Check.isIdentifier(node.property, "current")) return;
 			refAccesses.push(getRefAccess(node));
 		},
 		"Program:exit"(program) {
@@ -5178,7 +5160,7 @@ function create$6(context) {
 				for (const argument of callArguments) {
 					if (argument.type === AST_NODE_TYPES.SpreadElement) continue;
 					const value = Extract.unwrap(argument);
-					if (value.type !== AST_NODE_TYPES.Identifier) continue;
+					if (!Check.isIdentifier(value)) continue;
 					const variable = getVariable(value);
 					if (variable == null || resolveRef(variable, value.range[0]) == null) continue;
 					context.report({
@@ -5189,7 +5171,7 @@ function create$6(context) {
 			}
 		},
 		VariableDeclarator(node) {
-			if (node.id.type !== AST_NODE_TYPES.Identifier || node.init == null) return;
+			if (!Check.isIdentifier(node.id) || node.init == null) return;
 			addIdentifierBinding(node.id, node.init, node.range[0]);
 		}
 	});
@@ -7338,7 +7320,7 @@ function last(array) {
 */
 function getNestedIdentifiers(node) {
 	const identifiers = [];
-	if (node.type === AST_NODE_TYPES.Identifier) identifiers.push(node);
+	if (Check.isIdentifier(node)) identifiers.push(node);
 	if ("arguments" in node) {
 		const chunk = node.arguments.flatMap(getNestedIdentifiers);
 		identifiers.push(...chunk);
@@ -7415,7 +7397,7 @@ function getNestedIdentifiers(node) {
 }
 function isHookDecl(node) {
 	if (node.type !== AST_NODE_TYPES.VariableDeclarator) return false;
-	if (node.id.type !== AST_NODE_TYPES.Identifier) return false;
+	if (!Check.isIdentifier(node.id)) return false;
 	const init = node.init;
 	if (init == null || init.type !== AST_NODE_TYPES.CallExpression) return false;
 	const name = Extract.getCalleeName(init);
@@ -7430,7 +7412,7 @@ function isInitializedFromRef(context, name, initialScope, seen = /* @__PURE__ *
 		const init = node.init;
 		if (init == null) continue;
 		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.MemberExpression && Check.isIdentifier(init.object) && (init.object.name === "ref" || init.object.name.endsWith("Ref")): return true;
 			case init.type === AST_NODE_TYPES.CallExpression && core.isUseRefLikeCall(init, additionalRefHooks): return true;
 			case init.type === AST_NODE_TYPES.CallExpression: return getNestedIdentifiers(init).some((id) => isInitializedFromRef(context, id.name, context.sourceCode.getScope(id), seen));
 		}
@@ -7469,7 +7451,7 @@ function isRefInExpression(context, node) {
 * @returns The actual CallExpression node
 */
 function getSetStateCallExpression(node) {
-	return node.type === AST_NODE_TYPES.Identifier && node.parent.type === AST_NODE_TYPES.CallExpression ? node.parent : node;
+	return Check.isIdentifier(node) && node.parent.type === AST_NODE_TYPES.CallExpression ? node.parent : node;
 }
 
 //#endregion
@@ -7540,7 +7522,7 @@ function create$5(context) {
 		if (!isUseStateCall(initNode)) return false;
 		const variableNodeParent = initNode.parent;
 		if (!("id" in variableNodeParent) || variableNodeParent.id?.type !== AST_NODE_TYPES.ArrayPattern) return true;
-		return variableNodeParent.id.elements.findIndex((e) => e?.type === AST_NODE_TYPES.Identifier && e.name === id.name) === at;
+		return variableNodeParent.id.elements.findIndex((e) => e != null && Check.isIdentifier(e, id.name)) === at;
 	}
 	function isSetStateCall(node) {
 		const callee = Extract.unwrap(node.callee);
@@ -7657,6 +7639,8 @@ function create$5(context) {
 		},
 		"Program:exit"() {
 			const getSetStateCalls = (context, id) => {
+				const variable = findVariable(context.sourceCode.getScope(id), id);
+				if (variable != null && variable.defs.some((def) => def.type === DefinitionType.Parameter)) return [];
 				const node = resolve(context, id);
 				switch (node?.type) {
 					case AST_NODE_TYPES.ArrowFunctionExpression:
@@ -7676,7 +7660,7 @@ function create$5(context) {
 			}
 			for (const { callee } of trackedFnCalls) {
 				const unwrappedCallee = Extract.unwrap(callee);
-				if (unwrappedCallee.type !== AST_NODE_TYPES.Identifier) continue;
+				if (!Check.isIdentifier(unwrappedCallee)) continue;
 				const setStateCalls = getSetStateCalls(context, unwrappedCallee);
 				for (const setStateCall of setStateCalls) {
 					if (isRefGatedContext(context, getSetStateCallExpression(setStateCall))) continue;
@@ -7729,8 +7713,8 @@ function isInsideEventHandler(node, stopAt) {
 function isComponentOrHookLikeFunction(node) {
 	const id = core.getFunctionId(node);
 	if (id == null) return false;
-	if (id.type === AST_NODE_TYPES.Identifier) return core.isFunctionComponentName(id.name) || core.isHookName(id.name);
-	if (id.type === AST_NODE_TYPES.MemberExpression && id.property.type === AST_NODE_TYPES.Identifier) return core.isFunctionComponentName(id.property.name) || core.isHookName(id.property.name);
+	if (Check.isIdentifier(id)) return core.isFunctionComponentName(id.name) || core.isHookName(id.name);
+	if (id.type === AST_NODE_TYPES.MemberExpression && Check.isIdentifier(id.property)) return core.isFunctionComponentName(id.property.name) || core.isHookName(id.property.name);
 	return false;
 }
 
@@ -7763,7 +7747,7 @@ function create$4(context) {
 		if (!isUseStateCall(initNode)) return false;
 		const initNodeParent = initNode.parent;
 		if (!("id" in initNodeParent) || initNodeParent.id?.type !== AST_NODE_TYPES.ArrayPattern) return true;
-		return initNodeParent.id.elements.findIndex((e) => e?.type === AST_NODE_TYPES.Identifier && e.name === topLevelId.name) === at;
+		return initNodeParent.id.elements.findIndex((e) => e != null && Check.isIdentifier(e, topLevelId.name)) === at;
 	}
 	function isSetStateCall(node) {
 		const callee = Extract.unwrap(node.callee);
@@ -8000,7 +7984,7 @@ function isEvalCall(node) {
 		case AST_NODE_TYPES.Identifier: return callee.name === "eval";
 		case AST_NODE_TYPES.MemberExpression:
 			if (!Check.isIdentifier(Extract.unwrap(callee.object), "globalThis")) return false;
-			return !callee.computed && callee.property.type === AST_NODE_TYPES.Identifier && callee.property.name === "eval";
+			return !callee.computed && Check.isIdentifier(callee.property, "eval");
 		default: return false;
 	}
 }
@@ -8121,7 +8105,7 @@ function create$1(context) {
 		simpleTraverse(callback.body, { enter(node) {
 			if (node.type !== AST_NODE_TYPES.AssignmentExpression) return;
 			const left = Extract.unwrap(node.left);
-			if (left.type !== AST_NODE_TYPES.Identifier) return;
+			if (!Check.isIdentifier(left)) return;
 			if (Traverse.findParent(node, Check.isFunction, (n) => n === callback) != null) return;
 			const scope = context.sourceCode.getScope(left);
 			const variable = findVariable(scope, left);
@@ -8153,7 +8137,7 @@ function create$1(context) {
 			if (firstParam == null) return;
 			context.report({
 				messageId: "noParameters",
-				node: firstParam.type === AST_NODE_TYPES.Identifier ? firstParam : callback
+				node: Check.isIdentifier(firstParam) ? firstParam : callback
 			});
 		}
 		if (callback.async || callback.generator) context.report({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "5.18.5",
+  "version": "5.18.7",
   "description": "A set of composable ESLint rules for libraries and frameworks that use React as a UI runtime.",
   "keywords": [
     "react",
@@ -36,35 +36,35 @@
     "dist"
   ],
   "dependencies": {
-    "@typescript-eslint/scope-manager": "^8.67.0",
-    "@typescript-eslint/type-utils": "^8.67.0",
-    "@typescript-eslint/types": "^8.67.0",
-    "@typescript-eslint/typescript-estree": "^8.67.0",
-    "@typescript-eslint/utils": "^8.67.0",
+    "@eslint-react/ast": "5.18.7",
+    "@eslint-react/core": "5.18.7",
+    "@eslint-react/eslint": "5.18.7",
+    "@eslint-react/jsx": "5.18.7",
+    "@eslint-react/shared": "5.18.7",
+    "@eslint-react/var": "5.18.7",
+    "@typescript-eslint/scope-manager": "^8.69.0",
+    "@typescript-eslint/type-utils": "^8.69.0",
+    "@typescript-eslint/types": "^8.69.0",
+    "@typescript-eslint/typescript-estree": "^8.69.0",
+    "@typescript-eslint/utils": "^8.69.0",
     "compare-versions": "^6.1.1",
     "string-ts": "^2.3.1",
     "ts-api-utils": "^2.5.0",
-    "ts-pattern": "^5.9.0",
-    "@eslint-react/ast": "5.18.5",
-    "@eslint-react/eslint": "5.18.5",
-    "@eslint-react/shared": "5.18.5",
-    "@eslint-react/jsx": "5.18.5",
-    "@eslint-react/var": "5.18.5",
-    "@eslint-react/core": "5.18.5"
+    "ts-pattern": "^5.9.0"
   },
   "devDependencies": {
+    "@local/configs": "0.0.0",
+    "@local/eff": "0.0.0",
     "@types/react": "^19.2.18",
-    "@types/react-dom": "^19.2.4",
+    "@types/react-dom": "^19.2.5",
     "dedent": "^1.7.2",
-    "eslint": "^10.8.1",
+    "eslint": "^10.9.1",
     "react": "^19.2.8",
     "react-dom": "^19.2.8",
     "tsdown": "^0.22.14",
     "tsl": "^1.0.30",
     "tsl-dx": "^0.13.3",
-    "typescript": "6.0.3",
-    "@local/configs": "0.0.0",
-    "@local/eff": "0.0.0"
+    "typescript": "6.0.3"
   },
   "peerDependencies": {
     "eslint": "*",