eslint-plugin-react-x 3.0.0-next.57 → 3.0.0-next.59

package/dist/index.js

@@ -68,7 +68,7 @@ const rules$7 = {
 //#endregion
 //#region package.json
 var name$6 = "eslint-plugin-react-x";
-var version = "3.0.0-next.57";
+var version = "3.0.0-next.59";
 
 //#endregion
 //#region src/utils/create-rule.ts
@@ -255,27 +255,27 @@ function create$62(context) {
 			if (ast.findParentNode(node, ast.isFunction) == null) continue;
 			if (reported.has(node)) continue;
 			context.report({
+				data: { name },
 				messageId: "component",
-				node,
-				data: { name }
+				node
 			});
 			reported.add(node);
 		}
 		for (const { name = "unknown", node } of cComponents) {
 			if (ast.findParentNode(node, ast.isFunction) == null) continue;
 			context.report({
+				data: { name },
 				messageId: "component",
-				node,
-				data: { name }
+				node
 			});
 		}
 		for (const { name, node } of hooks) {
 			if (ast.findParentNode(node, ast.isFunction) == null) continue;
 			if (reported.has(node)) continue;
 			context.report({
+				data: { name },
 				messageId: "hook",
-				node,
-				data: { name }
+				node
 			});
 			reported.add(node);
 		}
@@ -1337,13 +1337,13 @@ function create$60(context) {
 			const isProps = isPropsObject(rootId);
 			if (!isState && !isProps) return;
 			violations.push({
-				messageId: "mutatingArrayMethod",
-				node,
 				data: {
 					name: context.sourceCode.getText(object),
 					method: property.name
 				},
-				func: enclosingFn
+				func: enclosingFn,
+				messageId: "mutatingArrayMethod",
+				node
 			});
 		},
 		AssignmentExpression(node) {
@@ -1357,17 +1357,17 @@ function create$60(context) {
 			const isProps = isPropsObject(rootId);
 			if (!isState && !isProps) return;
 			violations.push({
-				messageId: "mutatingAssignment",
-				node,
 				data: { name: context.sourceCode.getText(node.left.object) },
-				func: enclosingFn
+				func: enclosingFn,
+				messageId: "mutatingAssignment",
+				node
 			});
 		},
 		"Program:exit"(program) {
 			const components = cCollector.ctx.getAllComponents(program);
 			const hooks = hCollector.ctx.getAllHooks(program);
 			const componentAndHookFns = new Set([...components.map((c) => c.node), ...hooks.map((h) => h.node)]);
-			for (const { messageId, node, data, func } of violations) {
+			for (const { data, func, messageId, node } of violations) {
 				let current = func;
 				let insideComponentOrHook = false;
 				while (current != null) {
@@ -1379,9 +1379,9 @@ function create$60(context) {
 				}
 				if (!insideComponentOrHook) continue;
 				context.report({
+					data,
 					messageId,
-					node,
-					data
+					node
 				});
 			}
 		}
@@ -1419,8 +1419,6 @@ function create$59(context) {
 			if (child.value === "$" && node.children.length === 2) continue;
 			const pos = child.loc.end;
 			context.report({
-				messageId: "default",
-				node: child,
 				loc: {
 					end: {
 						column: pos.column,
@@ -1431,11 +1429,13 @@ function create$59(context) {
 						line: pos.line
 					}
 				},
+				messageId: "default",
+				node: child,
 				suggest: [{
-					messageId: "removeDollarSign",
 					fix(fixer) {
 						return fixer.removeRange([child.range[1] - 1, child.range[1]]);
-					}
+					},
+					messageId: "removeDollarSign"
 				}]
 			});
 		}
@@ -1576,18 +1576,18 @@ function create$55(context) {
 		switch (true) {
 			case policy === 1 && value?.type === AST_NODE_TYPES.JSXExpressionContainer && value.expression.type === AST_NODE_TYPES.Literal && value.expression.value === true:
 				context.report({
-					messageId: "default",
-					node,
 					data: { message: `Omit attribute value for '${propName}'.` },
-					fix: (fixer) => fixer.removeRange([node.name.range[1], value.range[1]])
+					fix: (fixer) => fixer.removeRange([node.name.range[1], value.range[1]]),
+					messageId: "default",
+					node
 				});
 				break;
 			case policy === -1 && value == null:
 				context.report({
-					messageId: "default",
-					node: node.value ?? node,
 					data: { message: `Set attribute value for '${propName}'.` },
-					fix: (fixer) => fixer.insertTextAfter(node.name, `={true}`)
+					fix: (fixer) => fixer.insertTextAfter(node.name, `={true}`),
+					messageId: "default",
+					node: node.value ?? node
 				});
 				break;
 		}
@@ -1625,24 +1625,24 @@ function create$54(context) {
 		if (!core.isJsxFragmentElement(context, node, jsxConfig)) return;
 		if (node.openingElement.attributes.length > 0) return;
 		context.report({
-			messageId: "default",
-			node,
 			data: { message: "Use fragment shorthand syntax instead of 'Fragment' component." },
 			fix: (fixer) => {
 				const { closingElement, openingElement } = node;
 				if (closingElement == null) return [];
 				return [fixer.replaceTextRange([openingElement.range[0], openingElement.range[1]], "<>"), fixer.replaceTextRange([closingElement.range[0], closingElement.range[1]], "</>")];
-			}
+			},
+			messageId: "default",
+			node
 		});
 	} })).with(-1, () => defineRuleListener({ JSXFragment(node) {
 		context.report({
-			messageId: "default",
-			node,
 			data: { message: "Use 'Fragment' component instead of fragment shorthand syntax." },
 			fix: (fixer) => {
 				const { closingFragment, openingFragment } = node;
 				return [fixer.replaceTextRange([openingFragment.range[0], openingFragment.range[1]], `<${jsxFragmentFactory}>`), fixer.replaceTextRange([closingFragment.range[0], closingFragment.range[1]], `</${jsxFragmentFactory}>`)];
-			}
+			},
+			messageId: "default",
+			node
 		});
 	} })).otherwise(() => ({}));
 }
@@ -1684,9 +1684,9 @@ function create$53(context) {
 function debugReport(context, node, name) {
 	if (process.env["ESLINT_REACT_DEBUG"] !== "1") return;
 	context.report({
+		data: { name },
 		messageId: "default",
-		node,
-		data: { name }
+		node
 	});
 }
 
@@ -2084,9 +2084,9 @@ function create$43(context) {
 		for (const { name = "anonymous", node: component } of ctx.getAllComponents(program)) {
 			if (component.body.body.some((m) => core.isComponentDidCatch(m) || core.isGetDerivedStateFromError(m))) continue;
 			context.report({
+				data: { name },
 				messageId: "default",
-				node: component,
-				data: { name }
+				node: component
 			});
 		}
 	} });
@@ -2137,12 +2137,12 @@ function create$41(context) {
 		for (const { node: component } of ctx.getAllComponents(program)) {
 			const { body } = component.body;
 			for (const member of body) if (core.isComponentWillMount(member)) context.report({
-				messageId: "default",
-				node: member,
 				fix(fixer) {
 					if (!("key" in member)) return null;
 					return fixer.replaceText(member.key, "UNSAFE_componentWillMount");
-				}
+				},
+				messageId: "default",
+				node: member
 			});
 		}
 	} });
@@ -2170,12 +2170,12 @@ function create$40(context) {
 		for (const { node: component } of ctx.getAllComponents(program)) {
 			const { body } = component.body;
 			for (const member of body) if (core.isComponentWillReceiveProps(member)) context.report({
-				messageId: "default",
-				node: member,
 				fix(fixer) {
 					if (!("key" in member)) return null;
 					return fixer.replaceText(member.key, "UNSAFE_componentWillReceiveProps");
-				}
+				},
+				messageId: "default",
+				node: member
 			});
 		}
 	} });
@@ -2203,12 +2203,12 @@ function create$39(context) {
 		for (const { node: component } of ctx.getAllComponents(program)) {
 			const { body } = component.body;
 			for (const member of body) if (core.isComponentWillUpdate(member)) context.report({
-				messageId: "default",
-				node: member,
 				fix(fixer) {
 					if (!("key" in member)) return null;
 					return fixer.replaceText(member.key, "UNSAFE_componentWillUpdate");
-				}
+				},
+				messageId: "default",
+				node: member
 			});
 		}
 	} });
@@ -2248,14 +2248,14 @@ function create$38(context) {
 			messageId: "default",
 			node,
 			suggest: [{
-				messageId: "replace",
 				fix(fixer) {
 					if (!core.isComponentNameLoose(contextSelfName)) return null;
 					const openingElement = node.openingElement;
 					const closingElement = node.closingElement;
 					if (closingElement == null) return fixer.replaceText(openingElement.name, contextFullName);
 					return [fixer.replaceText(openingElement.name, contextFullName), fixer.replaceText(closingElement.name, contextFullName)];
-				}
+				},
+				messageId: "replace"
 			}]
 		});
 	} });
@@ -2371,9 +2371,9 @@ function create$35(context) {
 			for (const { hasDuplicate, keys } of keyedEntries.values()) {
 				if (!hasDuplicate) continue;
 				for (const key of keys) context.report({
+					data: { value: context.sourceCode.getText(key) },
 					messageId: "default",
-					node: key,
-					data: { value: context.sourceCode.getText(key) }
+					node: key
 				});
 			}
 		}
@@ -2407,8 +2407,8 @@ function create$34(context) {
 		if (!core.isForwardRefCall(context, node)) return;
 		const id = ast.getFunctionId(node);
 		const suggest = canFix(context, node) ? [{
-			messageId: "replace",
-			fix: getFix(context, node)
+			fix: getFix(context, node),
+			messageId: "replace"
 		}] : [];
 		context.report({
 			messageId: "default",
@@ -2580,16 +2580,16 @@ function create$32(context) {
 			if (left.type === AST_NODE_TYPES.UnaryExpression && left.operator === "!") return getReportDescriptor(right);
 			const initialScope = context.sourceCode.getScope(left);
 			if (ast.isIdentifier(left, "NaN") || getStaticValue(left, initialScope)?.value === "NaN") return {
+				data: { value: context.sourceCode.getText(left) },
 				messageId: "default",
-				node: left,
-				data: { value: context.sourceCode.getText(left) }
+				node: left
 			};
 			const leftTypeVariants = getTypeVariants(unionConstituents(getConstrainedTypeAtLocation(services, left)));
 			if (Array.from(leftTypeVariants.values()).every((type) => allowedVariants.some((allowed) => allowed === type))) return getReportDescriptor(right);
 			return {
+				data: { value: context.sourceCode.getText(left) },
 				messageId: "default",
-				node: left,
-				data: { value: context.sourceCode.getText(left) }
+				node: left
 			};
 		}).with({ type: AST_NODE_TYPES.ConditionalExpression }, ({ alternate, consequent }) => {
 			return getReportDescriptor(consequent) ?? getReportDescriptor(alternate);
@@ -2622,7 +2622,7 @@ function create$31(context) {
 		hint: core.DEFAULT_COMPONENT_DETECTION_HINT
 	});
 	return defineRuleListener(visitor, { "Program:exit"(program) {
-		for (const { node, displayName, flag } of ctx.getAllComponents(program)) {
+		for (const { displayName, flag, node } of ctx.getAllComponents(program)) {
 			const id = ast.getFunctionId(node);
 			const isMemoOrForwardRef = (flag & (core.ComponentFlag.ForwardRef | core.ComponentFlag.Memo)) > 0n;
 			if (id != null) continue;
@@ -2678,8 +2678,6 @@ function create$30(context) {
 					const object = left.object;
 					return isAssignmentTargetEqual(context, id, object);
 				})) context.report({
-					messageId: "default",
-					node: id,
 					fix(fixer) {
 						if (id.type !== AST_NODE_TYPES.Identifier || id.parent !== call.parent) return [];
 						return fixer.insertTextAfter(context.sourceCode.getTokenAfter(call) ?? call, [
@@ -2693,7 +2691,9 @@ function create$30(context) {
 							JSON.stringify(id.name),
 							";"
 						].join(""));
-					}
+					},
+					messageId: "default",
+					node: id
 				});
 			}
 		}
@@ -2864,55 +2864,55 @@ function create$27(context) {
 			if (name == null) continue;
 			if (isInsideJSXAttributeValue(component)) {
 				context.report({
-					messageId: "default",
-					node: component,
 					data: {
 						name,
 						suggestion: "Move it to the top level or pass it as a prop."
-					}
+					},
+					messageId: "default",
+					node: component
 				});
 				continue;
 			}
 			if (isInsideCreateElementProps(context, component)) {
 				context.report({
-					messageId: "default",
-					node: component,
 					data: {
 						name,
 						suggestion: "Move it to the top level or pass it as a prop."
-					}
+					},
+					messageId: "default",
+					node: component
 				});
 				continue;
 			}
 			if (findEnclosingComponent(component) != null) {
 				context.report({
-					messageId: "default",
-					node: component,
 					data: {
 						name,
 						suggestion: component.parent.type === AST_NODE_TYPES.Property ? "Move it to the top level or pass it as a prop." : "Move it to the top level."
-					}
+					},
+					messageId: "default",
+					node: component
 				});
 				continue;
 			}
 			if (isInsideRenderMethod(component)) context.report({
-				messageId: "default",
-				node: component,
 				data: {
 					name,
 					suggestion: "Move it to the top level."
-				}
+				},
+				messageId: "default",
+				node: component
 			});
 		}
 		for (const { name = "unknown", node: component } of cComponents) {
 			if (findEnclosingComponent(component) == null) continue;
 			context.report({
-				messageId: "default",
-				node: component,
 				data: {
 					name,
 					suggestion: component.parent.type === AST_NODE_TYPES.Property ? "Move it to the top level or pass it as a prop." : "Move it to the top level."
-				}
+				},
+				messageId: "default",
+				node: component
 			});
 		}
 	} });
@@ -3010,13 +3010,13 @@ function create$25(context) {
 	if (!context.sourceCode.text.includes("shouldComponentUpdate")) return {};
 	const { ctx, visitor } = core.useComponentCollectorLegacy(context);
 	return defineRuleListener(visitor, { "Program:exit"(program) {
-		for (const { name = "PureComponent", node: component, flag } of ctx.getAllComponents(program)) {
+		for (const { name = "PureComponent", flag, node: component } of ctx.getAllComponents(program)) {
 			if ((flag & core.ComponentFlag.PureComponent) === 0n) continue;
 			const { body } = component.body;
 			for (const member of body) if (isShouldComponentUpdate(member)) context.report({
+				data: { componentName: name },
 				messageId: "default",
-				node: member,
-				data: { componentName: name }
+				node: member
 			});
 		}
 	} });
@@ -3186,9 +3186,9 @@ function checkForUsageInsideUseEffect$1(sourceCode, node) {
 		if (effectSet.size > 1) return;
 	}
 	return {
+		data: { name: node.parent.id.name },
 		messageId: "noUnnecessaryUseCallbackInsideUseEffect",
-		node,
-		data: { name: node.parent.id.name }
+		node
 	};
 }
 
@@ -3270,9 +3270,9 @@ function checkForUsageInsideUseEffect(sourceCode, node) {
 		if (effectSet.size > 1) return;
 	}
 	return {
+		data: { name: node.parent.id.name },
 		messageId: "noUnnecessaryUseMemoInsideUseEffect",
-		node,
-		data: { name: node.parent.id.name }
+		node
 	};
 }
 
@@ -3297,16 +3297,16 @@ var no_unnecessary_use_prefix_default = createRule({
 function create$19(context) {
 	const { ctx, visitor } = core.useHookCollector(context);
 	return defineRuleListener(visitor, { "Program:exit"(program) {
-		for (const { id, name, node, hookCalls } of ctx.getAllHooks(program)) {
+		for (const { id, name, hookCalls, node } of ctx.getAllHooks(program)) {
 			if (hookCalls.length > 0) continue;
 			if (ast.isFunctionEmpty(node)) continue;
 			if (WELL_KNOWN_HOOKS.includes(name)) continue;
 			if (containsUseComments(context, node)) continue;
 			if (ast.findParentNode(node, ast.isViMockCallback) != null) continue;
 			context.report({
+				data: { name },
 				messageId: "default",
-				node: id ?? node,
-				data: { name }
+				node: id ?? node
 			});
 		}
 	} });
@@ -3436,18 +3436,18 @@ function create$15(context) {
 			getOrElseUpdate(constructions, functionEntry.node, () => []).push(construction);
 		},
 		"Program:exit"(program) {
-			for (const { node: component, directives } of ctx.getAllComponents(program)) {
+			for (const { directives, node: component } of ctx.getAllComponents(program)) {
 				if (compilationMode === "annotation" && directives.some((d) => d.directive === "use memo")) continue;
 				for (const construction of constructions.get(component) ?? []) {
 					const { kind, node: constructionNode } = construction;
 					const suggestion = kind === "function" ? "Consider wrapping it in a useCallback hook." : "Consider wrapping it in a useMemo hook.";
 					context.report({
-						messageId: "unstableContextValue",
-						node: constructionNode,
 						data: {
 							kind: ast.getHumanReadableKind(constructionNode),
 							suggestion
-						}
+						},
+						messageId: "unstableContextValue",
+						node: constructionNode
 					});
 				}
 			}
@@ -3525,9 +3525,9 @@ function create$14(context, [options]) {
 						if (identifier != null && safePatterns.some((pattern) => pattern.test(identifier))) continue;
 					}
 					context.report({
+						data: { kind: ast.getHumanReadableKind(right) },
 						messageId: "default",
-						node: right,
-						data: { kind: ast.getHumanReadableKind(right) }
+						node: right
 					});
 				}
 			}
@@ -3595,12 +3595,12 @@ function create$13(context) {
 			if (methodName == null) continue;
 			if ((usages?.has(methodName) ?? false) || LIFECYCLE_METHODS.has(methodName)) continue;
 			context.report({
-				messageId: "default",
-				node: def,
 				data: {
 					className: id != null ? context.sourceCode.getText(id) : "Component",
 					methodName
-				}
+				},
+				messageId: "default",
+				node: def
 			});
 		}
 	}
@@ -3757,9 +3757,9 @@ function reportUnusedProp(context, services, prop) {
 	if (declarationNode == null) return;
 	const nodeToReport = declarationNode.type === AST_NODE_TYPES.TSPropertySignature ? declarationNode.key : declarationNode;
 	context.report({
+		data: { name: prop.name },
 		messageId: "default",
-		node: nodeToReport,
-		data: { name: prop.name }
+		node: nodeToReport
 	});
 }
 
@@ -3795,12 +3795,12 @@ function create$11(context) {
 		const currentClass = classStack.pop();
 		if (currentClass == null || !core.isClassComponent(currentClass)) return;
 		const id = ast.getClassId(currentClass);
-		const { node: defNode, isUsed = false } = stateDefs.get(currentClass) ?? {};
+		const { isUsed = false, node: defNode } = stateDefs.get(currentClass) ?? {};
 		if (defNode == null || isUsed) return;
 		context.report({
+			data: { className: id != null ? context.sourceCode.getText(id) : "Component" },
 			messageId: "default",
-			node: defNode,
-			data: { className: id != null ? context.sourceCode.getText(id) : "Component" }
+			node: defNode
 		});
 	}
 	function methodEnter(node) {
@@ -3811,15 +3811,15 @@ function create$11(context) {
 			if (core.isGetDerivedStateFromProps(node) && isMatching({ params: [P.nonNullable, ...P.array()] })(node.value)) {
 				const defNode = stateDefs.get(currentClass)?.node;
 				stateDefs.set(currentClass, {
-					node: defNode,
-					isUsed: true
+					isUsed: true,
+					node: defNode
 				});
 			}
 			return;
 		}
 		if (ast.getPropertyName(node.key) === "state") stateDefs.set(currentClass, {
-			node: node.key,
-			isUsed: false
+			isUsed: false,
+			node: node.key
 		});
 	}
 	function methodExit() {
@@ -3840,8 +3840,8 @@ function create$11(context) {
 			if (currentConstructor == null || !currentClass.body.body.includes(currentConstructor)) return;
 			const isUsed = stateDefs.get(currentClass)?.isUsed ?? false;
 			stateDefs.set(currentClass, {
-				node: node.left,
-				isUsed
+				isUsed,
+				node: node.left
 			});
 		},
 		ClassDeclaration: classEnter,
@@ -3859,8 +3859,8 @@ function create$11(context) {
 			if (!currentClass.body.body.includes(currentMethod)) return;
 			const defNode = stateDefs.get(currentClass)?.node;
 			stateDefs.set(currentClass, {
-				node: defNode,
-				isUsed: true
+				isUsed: true,
+				node: defNode
 			});
 		},
 		MethodDefinition: methodEnter,
@@ -3883,8 +3883,8 @@ function create$11(context) {
 			})) return;
 			const defNode = stateDefs.get(currentClass)?.node;
 			stateDefs.set(currentClass, {
-				node: defNode,
-				isUsed: true
+				isUsed: true,
+				node: defNode
 			});
 		}
 	});
@@ -3919,14 +3919,14 @@ function create$10(context) {
 			messageId: "default",
 			node: node.callee,
 			suggest: [{
-				messageId: "replace",
 				fix(fixer) {
 					switch (node.callee.type) {
 						case AST_NODE_TYPES.Identifier: return fixer.replaceText(node.callee, "use");
 						case AST_NODE_TYPES.MemberExpression: return fixer.replaceText(node.callee.property, "use");
 					}
 					return null;
-				}
+				},
+				messageId: "replace"
 			}]
 		});
 	} });
@@ -3976,20 +3976,19 @@ function create$9(context, [option]) {
 	* Check if a fragment node is useless and should be reported
 	*/
 	function checkNode(context, node) {
-		if (node.type === AST_NODE_TYPES.JSXElement && core.getJsxAttribute(context, node)("key") != null) return;
 		if (core.isJsxHostElement(context, node.parent)) context.report({
-			messageId: "default",
-			node,
 			data: { reason: "placed inside a host component" },
-			fix: getFix(context, node)
+			fix: getFix(context, node),
+			messageId: "default",
+			node
 		});
 		if (node.children.length === 0) {
 			if (allowEmptyFragment) return;
 			context.report({
-				messageId: "default",
-				node,
 				data: { reason: "contains less than two children" },
-				fix: getFix(context, node)
+				fix: getFix(context, node),
+				messageId: "default",
+				node
 			});
 			return;
 		}
@@ -3998,28 +3997,28 @@ function create$9(context, [option]) {
 			case allowExpressions && !isChildElement && node.children.length === 1 && core.isJsxText(node.children.at(0)): return;
 			case !allowExpressions && isChildElement:
 				context.report({
-					messageId: "default",
-					node,
 					data: { reason: "contains less than two children" },
-					fix: getFix(context, node)
+					fix: getFix(context, node),
+					messageId: "default",
+					node
 				});
 				return;
 			case !allowExpressions && !isChildElement && node.children.length === 1:
 				context.report({
-					messageId: "default",
-					node,
 					data: { reason: "contains less than two children" },
-					fix: getFix(context, node)
+					fix: getFix(context, node),
+					messageId: "default",
+					node
 				});
 				return;
 		}
 		const nonPaddingChildren = node.children.filter((child) => !isPaddingSpaces(child));
 		const firstNonPaddingChild = nonPaddingChildren.at(0);
 		if (nonPaddingChildren.length === 0 || nonPaddingChildren.length === 1 && firstNonPaddingChild?.type !== AST_NODE_TYPES.JSXExpressionContainer) context.report({
-			messageId: "default",
-			node,
 			data: { reason: "contains less than two children" },
-			fix: getFix(context, node)
+			fix: getFix(context, node),
+			messageId: "default",
+			node
 		});
 	}
 	function getFix(context, node) {
@@ -4042,6 +4041,8 @@ function create$9(context, [option]) {
 	return defineRuleListener({
 		JSXElement(node) {
 			if (!core.isJsxFragmentElement(context, node, jsxConfig)) return;
+			if (core.getJsxAttribute(context, node)("key") != null) return;
+			if (core.getJsxAttribute(context, node)("ref") != null) return;
 			checkNode(context, node);
 		},
 		JSXFragment(node) {
@@ -4128,8 +4129,6 @@ function create$7(context) {
 	return defineRuleListener({ [`ImportDeclaration[source.value="${importSource}"] ImportDefaultSpecifier`](node) {
 		const hasOtherSpecifiers = node.parent.specifiers.length > 1;
 		context.report({
-			messageId: "default",
-			node: hasOtherSpecifiers ? node : node.parent,
 			data: { importSource },
 			fix(fixer) {
 				const importDeclarationText = context.sourceCode.getText(node.parent);
@@ -4140,7 +4139,9 @@ function create$7(context) {
 				if (!hasOtherSpecifiers) return fixer.replaceText(node.parent, `${importStringPrefix} * as ${node.local.name} from ${importSourceQuoted}${semi}`);
 				const specifiers = importDeclarationText.slice(importDeclarationText.indexOf("{"), importDeclarationText.indexOf("}") + 1);
 				return fixer.replaceText(node.parent, [`${importStringPrefix} * as ${node.local.name} from ${importSourceQuoted}${semi}`, `${importStringPrefix} ${specifiers} from ${importSourceQuoted}${semi}`].join("\n"));
-			}
+			},
+			messageId: "default",
+			node: hasOtherSpecifiers ? node : node.parent
 		});
 	} });
 }
@@ -4177,8 +4178,8 @@ function create$6(context) {
 			const func = ast.findParentNode(node, ast.isFunction);
 			if (func == null) return;
 			cExprs.push({
-				node,
-				func
+				func,
+				node
 			});
 		},
 		NewExpression(node) {
@@ -4188,20 +4189,20 @@ function create$6(context) {
 			const func = ast.findParentNode(node, ast.isFunction);
 			if (func == null) return;
 			nExprs.push({
-				node,
-				func
+				func,
+				node
 			});
 		},
 		"Program:exit"(node) {
 			const components = cCollector.ctx.getAllComponents(node);
 			const hooks = hCollector.ctx.getAllHooks(node);
 			const funcs = [...components, ...hooks];
-			for (const { node, func } of [...cExprs, ...nExprs]) {
+			for (const { func, node } of [...cExprs, ...nExprs]) {
 				if (!funcs.some((f) => f.node === func)) continue;
 				context.report({
+					data: { name: context.sourceCode.getText(node) },
 					messageId: "default",
-					node,
-					data: { name: context.sourceCode.getText(node) }
+					node
 				});
 			}
 		}
@@ -4248,6 +4249,8 @@ function create$5(context) {
 	/**
 	* Check if a test expression is a null check on `ref.current` for a given ref name.
 	* Matches forms like `ref.current === null`, `null === ref.current`, and their != variants.
+	* @param test
+	* @param refName
 	*/
 	function isRefCurrentNullCheck(test, refName) {
 		if (test.type !== AST_NODE_TYPES.BinaryExpression) return false;
@@ -4268,8 +4271,8 @@ function create$5(context) {
 		MemberExpression(node) {
 			if (!ast.isIdentifier(node.property, "current")) return;
 			refAccesses.push({
-				node,
-				isWrite: match(node.parent).with({ type: AST_NODE_TYPES.AssignmentExpression }, (p) => p.left === node).with({ type: AST_NODE_TYPES.UpdateExpression }, (p) => p.argument === node).otherwise(() => false)
+				isWrite: match(node.parent).with({ type: AST_NODE_TYPES.AssignmentExpression }, (p) => p.left === node).with({ type: AST_NODE_TYPES.UpdateExpression }, (p) => p.argument === node).otherwise(() => false),
+				node
 			});
 		},
 		"Program:exit"(program) {
@@ -4277,7 +4280,7 @@ function create$5(context) {
 			const hooks = hCollector.ctx.getAllHooks(program);
 			const funcs = new Set([...comps.map((c) => c.node), ...hooks.map((h) => h.node)]);
 			const isCompOrHookFn = (n) => ast.isFunction(n) && funcs.has(n);
-			for (const { node, isWrite } of refAccesses) {
+			for (const { isWrite, node } of refAccesses) {
 				const obj = node.object;
 				if (obj.type !== AST_NODE_TYPES.Identifier) continue;
 				switch (true) {
@@ -6620,9 +6623,9 @@ function create$4(context) {
 						}
 						if (isArgumentUsingRefValue(context, args0)) return;
 						context.report({
+							data: { name: context.sourceCode.getText(node.callee) },
 							messageId: "default",
-							node,
-							data: { name: context.sourceCode.getText(node.callee) }
+							node
 						});
 						return;
 					}
@@ -6674,26 +6677,26 @@ function create$4(context) {
 				return [];
 			};
 			for (const [, calls] of setStateInEffectSetup) for (const call of calls) context.report({
+				data: { name: call.name },
 				messageId: "default",
-				node: call,
-				data: { name: call.name }
+				node: call
 			});
 			for (const { callee } of trackedFnCalls) {
 				if (!("name" in callee)) continue;
 				const { name } = callee;
 				const setStateCalls = getSetStateCalls(name, context.sourceCode.getScope(callee));
 				for (const setStateCall of setStateCalls) context.report({
+					data: { name: getCallName(setStateCall) },
 					messageId: "default",
-					node: setStateCall,
-					data: { name: getCallName(setStateCall) }
+					node: setStateCall
 				});
 			}
 			for (const id of setupFnIds) {
 				const setStateCalls = getSetStateCalls(id.name, context.sourceCode.getScope(id));
 				for (const setStateCall of setStateCalls) context.report({
+					data: { name: getCallName(setStateCall) },
 					messageId: "default",
-					node: setStateCall,
-					data: { name: getCallName(setStateCall) }
+					node: setStateCall
 				});
 			}
 		}
@@ -6814,9 +6817,9 @@ function create$3(context) {
 			if (isInsideConditional(node, componentFn)) return;
 			if (componentHasEarlyReturn.current) return;
 			context.report({
+				data: { name: context.sourceCode.getText(node.callee) },
 				messageId: "default",
-				node,
-				data: { name: context.sourceCode.getText(node.callee) }
+				node
 			});
 		},
 		ReturnStatement(node) {
@@ -6872,8 +6875,8 @@ function create$2(context) {
 			const func = ast.findParentNode(node, ast.isFunction);
 			if (func == null) return;
 			evalCalls.push({
-				node,
-				func
+				func,
+				node
 			});
 		},
 		"JSXElement :function"(node) {
@@ -6892,14 +6895,14 @@ function create$2(context) {
 			const components = cCollector.ctx.getAllComponents(node);
 			const hooks = hCollector.ctx.getAllHooks(node);
 			const funcs = [...components, ...hooks];
-			for (const { node, func } of evalCalls) {
+			for (const { func, node } of evalCalls) {
 				if (!funcs.some((f) => f.node === func)) continue;
 				context.report({
 					messageId: "eval",
 					node
 				});
 			}
-			for (const { node, func } of withStmts) {
+			for (const { func, node } of withStmts) {
 				if (!funcs.some((f) => f.node === func)) continue;
 				context.report({
 					messageId: "with",
@@ -6911,8 +6914,8 @@ function create$2(context) {
 			const func = ast.findParentNode(node, ast.isFunction);
 			if (func == null) return;
 			withStmts.push({
-				node,
-				func
+				func,
+				node
 			});
 		}
 	});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "3.0.0-next.57",
+  "version": "3.0.0-next.59",
   "description": "A set of composable ESLint rules for libraries and frameworks that use React as a UI runtime.",
   "keywords": [
     "react",
@@ -45,11 +45,11 @@
     "string-ts": "^2.3.1",
     "ts-api-utils": "^2.4.0",
     "ts-pattern": "^5.9.0",
-    "@eslint-react/ast": "3.0.0-next.57",
-    "@eslint-react/eff": "3.0.0-next.57",
-    "@eslint-react/shared": "3.0.0-next.57",
-    "@eslint-react/var": "3.0.0-next.57",
-    "@eslint-react/core": "3.0.0-next.57"
+    "@eslint-react/ast": "3.0.0-next.59",
+    "@eslint-react/core": "3.0.0-next.59",
+    "@eslint-react/eff": "3.0.0-next.59",
+    "@eslint-react/shared": "3.0.0-next.59",
+    "@eslint-react/var": "3.0.0-next.59"
   },
   "devDependencies": {
     "@types/react": "^19.2.14",