eslint-plugin-react-x 2.7.2-beta.2 → 2.7.2-beta.3

package/dist/index.js

@@ -1,4 +1,4 @@
-import { DEFAULT_ESLINT_REACT_SETTINGS, WEBSITE_URL, coerceSettings, getConfigAdapters, getSettingsFromContext, report, toRegExp } from "@eslint-react/shared";
+import { DEFAULT_ESLINT_REACT_SETTINGS, WEBSITE_URL, coerceSettings, defineRuleListener, getConfigAdapters, getSettingsFromContext, report, toRegExp } from "@eslint-react/shared";
 import { AST_NODE_TYPES } from "@typescript-eslint/types";
 import { ESLintUtils } from "@typescript-eslint/utils";
 import { P, isMatching, match } from "ts-pattern";
@@ -68,7 +68,7 @@ const rules$7 = {
 //#endregion
 //#region package.json
 var name$6 = "eslint-plugin-react-x";
-var version = "2.7.2-beta.2";
+var version = "2.7.2-beta.3";
 
 //#endregion
 //#region src/utils/create-rule.ts
@@ -937,20 +937,17 @@ var no_class_component_default = createRule({
 });
 function create$45(context) {
 	if (!context.sourceCode.text.includes("Component")) return {};
-	const { ctx, listeners } = useComponentCollectorLegacy(context);
-	return {
-		...listeners,
-		"Program:exit"(program) {
-			for (const { name: name$9 = "anonymous", node: component } of ctx.getAllComponents(program)) {
-				if (component.body.body.some((m) => isComponentDidCatch(m) || isGetDerivedStateFromError(m))) continue;
-				context.report({
-					messageId: "noClassComponent",
-					node: component,
-					data: { name: name$9 }
-				});
-			}
+	const { ctx, visitors } = useComponentCollectorLegacy(context);
+	return defineRuleListener(visitors, { "Program:exit"(program) {
+		for (const { name: name$9 = "anonymous", node: component } of ctx.getAllComponents(program)) {
+			if (component.body.body.some((m) => isComponentDidCatch(m) || isGetDerivedStateFromError(m))) continue;
+			context.report({
+				messageId: "noClassComponent",
+				node: component,
+				data: { name: name$9 }
+			});
 		}
-	};
+	} });
 }
 
 //#endregion
@@ -993,23 +990,20 @@ var no_component_will_mount_default = createRule({
 });
 function create$43(context) {
 	if (!context.sourceCode.text.includes("componentWillMount")) return {};
-	const { ctx, listeners } = useComponentCollectorLegacy(context);
-	return {
-		...listeners,
-		"Program:exit"(program) {
-			for (const { node: component } of ctx.getAllComponents(program)) {
-				const { body } = component.body;
-				for (const member of body) if (isComponentWillMount(member)) context.report({
-					messageId: "noComponentWillMount",
-					node: member,
-					fix(fixer) {
-						if (!("key" in member)) return null;
-						return fixer.replaceText(member.key, "UNSAFE_componentWillMount");
-					}
-				});
-			}
+	const { ctx, visitors } = useComponentCollectorLegacy(context);
+	return defineRuleListener(visitors, { "Program:exit"(program) {
+		for (const { node: component } of ctx.getAllComponents(program)) {
+			const { body } = component.body;
+			for (const member of body) if (isComponentWillMount(member)) context.report({
+				messageId: "noComponentWillMount",
+				node: member,
+				fix(fixer) {
+					if (!("key" in member)) return null;
+					return fixer.replaceText(member.key, "UNSAFE_componentWillMount");
+				}
+			});
 		}
-	};
+	} });
 }
 
 //#endregion
@@ -1029,23 +1023,20 @@ var no_component_will_receive_props_default = createRule({
 });
 function create$42(context) {
 	if (!context.sourceCode.text.includes("componentWillReceiveProps")) return {};
-	const { ctx, listeners } = useComponentCollectorLegacy(context);
-	return {
-		...listeners,
-		"Program:exit"(program) {
-			for (const { node: component } of ctx.getAllComponents(program)) {
-				const { body } = component.body;
-				for (const member of body) if (isComponentWillReceiveProps(member)) context.report({
-					messageId: "noComponentWillReceiveProps",
-					node: member,
-					fix(fixer) {
-						if (!("key" in member)) return null;
-						return fixer.replaceText(member.key, "UNSAFE_componentWillReceiveProps");
-					}
-				});
-			}
+	const { ctx, visitors } = useComponentCollectorLegacy(context);
+	return defineRuleListener(visitors, { "Program:exit"(program) {
+		for (const { node: component } of ctx.getAllComponents(program)) {
+			const { body } = component.body;
+			for (const member of body) if (isComponentWillReceiveProps(member)) context.report({
+				messageId: "noComponentWillReceiveProps",
+				node: member,
+				fix(fixer) {
+					if (!("key" in member)) return null;
+					return fixer.replaceText(member.key, "UNSAFE_componentWillReceiveProps");
+				}
+			});
 		}
-	};
+	} });
 }
 
 //#endregion
@@ -1065,23 +1056,20 @@ var no_component_will_update_default = createRule({
 });
 function create$41(context) {
 	if (!context.sourceCode.text.includes("componentWillUpdate")) return {};
-	const { ctx, listeners } = useComponentCollectorLegacy(context);
-	return {
-		...listeners,
-		"Program:exit"(program) {
-			for (const { node: component } of ctx.getAllComponents(program)) {
-				const { body } = component.body;
-				for (const member of body) if (isComponentWillUpdate(member)) context.report({
-					messageId: "noComponentWillUpdate",
-					node: member,
-					fix(fixer) {
-						if (!("key" in member)) return null;
-						return fixer.replaceText(member.key, "UNSAFE_componentWillUpdate");
-					}
-				});
-			}
+	const { ctx, visitors } = useComponentCollectorLegacy(context);
+	return defineRuleListener(visitors, { "Program:exit"(program) {
+		for (const { node: component } of ctx.getAllComponents(program)) {
+			const { body } = component.body;
+			for (const member of body) if (isComponentWillUpdate(member)) context.report({
+				messageId: "noComponentWillUpdate",
+				node: member,
+				fix(fixer) {
+					if (!("key" in member)) return null;
+					return fixer.replaceText(member.key, "UNSAFE_componentWillUpdate");
+				}
+			});
 		}
-	};
+	} });
 }
 
 //#endregion
@@ -1492,26 +1480,23 @@ var no_missing_component_display_name_default = createRule({
 });
 function create$32(context) {
 	if (!context.sourceCode.text.includes("memo") && !context.sourceCode.text.includes("forwardRef")) return {};
-	const { ctx, listeners } = useComponentCollector(context, {
+	const { ctx, visitors } = useComponentCollector(context, {
 		collectDisplayName: true,
 		collectHookCalls: false,
 		hint: DEFAULT_COMPONENT_DETECTION_HINT
 	});
-	return {
-		...listeners,
-		"Program:exit"(program) {
-			for (const { node, displayName, flag } of ctx.getAllComponents(program)) {
-				const id = AST.getFunctionId(node);
-				const isMemoOrForwardRef = (flag & (ComponentFlag.ForwardRef | ComponentFlag.Memo)) > 0n;
-				if (id != null) continue;
-				if (!isMemoOrForwardRef) continue;
-				if (displayName == null) context.report({
-					messageId: "noMissingComponentDisplayName",
-					node
-				});
-			}
+	return defineRuleListener(visitors, { "Program:exit"(program) {
+		for (const { node, displayName, flag } of ctx.getAllComponents(program)) {
+			const id = AST.getFunctionId(node);
+			const isMemoOrForwardRef = (flag & (ComponentFlag.ForwardRef | ComponentFlag.Memo)) > 0n;
+			if (id != null) continue;
+			if (!isMemoOrForwardRef) continue;
+			if (displayName == null) context.report({
+				messageId: "noMissingComponentDisplayName",
+				node
+			});
 		}
-	};
+	} });
 }
 
 //#endregion
@@ -1721,66 +1706,42 @@ var no_nested_component_definitions_default = createRule({
 function create$28(context) {
 	const fCollector = useComponentCollector(context, { hint: ComponentDetectionHint.SkipArrayMapCallback | ComponentDetectionHint.SkipNullLiteral | ComponentDetectionHint.SkipUndefined | ComponentDetectionHint.SkipBooleanLiteral | ComponentDetectionHint.SkipStringLiteral | ComponentDetectionHint.SkipNumberLiteral | ComponentDetectionHint.StrictLogical | ComponentDetectionHint.StrictConditional });
 	const cCollector = useComponentCollectorLegacy(context);
-	return {
-		...fCollector.listeners,
-		...cCollector.listeners,
-		"Program:exit"(program) {
-			const fComponents = [...fCollector.ctx.getAllComponents(program)];
-			const cComponents = [...cCollector.ctx.getAllComponents(program)];
-			const isFunctionComponent = (node) => {
-				return AST.isFunction(node) && fComponents.some((component) => component.node === node);
-			};
-			const isClassComponent$1 = (node) => {
-				return AST.isClass(node) && cComponents.some((component) => component.node === node);
-			};
-			for (const { name: name$9, node: component } of fComponents) {
-				if (name$9 == null) continue;
-				if (isDirectValueOfRenderPropertyLoose(component)) continue;
-				if (isInsideJSXAttributeValue(component)) {
-					if (!isDeclaredInRenderPropLoose(component)) context.report({
-						messageId: "noNestedComponentDefinitions",
-						node: component,
-						data: {
-							name: name$9,
-							suggestion: "Move it to the top level or pass it as a prop."
-						}
-					});
-					continue;
-				}
-				if (isInsideCreateElementProps(context, component)) {
-					context.report({
-						messageId: "noNestedComponentDefinitions",
-						node: component,
-						data: {
-							name: name$9,
-							suggestion: "Move it to the top level or pass it as a prop."
-						}
-					});
-					continue;
-				}
-				const parentComponent = AST.findParentNode(component, isFunctionComponent);
-				if (parentComponent != null && !isDirectValueOfRenderPropertyLoose(parentComponent)) {
-					context.report({
-						messageId: "noNestedComponentDefinitions",
-						node: component,
-						data: {
-							name: name$9,
-							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."
-						}
-					});
-					continue;
-				}
-				if (isInsideRenderMethod(component)) context.report({
+	return defineRuleListener(fCollector.visitors, cCollector.visitors, { "Program:exit"(program) {
+		const fComponents = [...fCollector.ctx.getAllComponents(program)];
+		const cComponents = [...cCollector.ctx.getAllComponents(program)];
+		const isFunctionComponent = (node) => {
+			return AST.isFunction(node) && fComponents.some((component) => component.node === node);
+		};
+		const isClassComponent$1 = (node) => {
+			return AST.isClass(node) && cComponents.some((component) => component.node === node);
+		};
+		for (const { name: name$9, node: component } of fComponents) {
+			if (name$9 == null) continue;
+			if (isDirectValueOfRenderPropertyLoose(component)) continue;
+			if (isInsideJSXAttributeValue(component)) {
+				if (!isDeclaredInRenderPropLoose(component)) context.report({
+					messageId: "noNestedComponentDefinitions",
+					node: component,
+					data: {
+						name: name$9,
+						suggestion: "Move it to the top level or pass it as a prop."
+					}
+				});
+				continue;
+			}
+			if (isInsideCreateElementProps(context, component)) {
+				context.report({
 					messageId: "noNestedComponentDefinitions",
 					node: component,
 					data: {
 						name: name$9,
-						suggestion: "Move it to the top level."
+						suggestion: "Move it to the top level or pass it as a prop."
 					}
 				});
+				continue;
 			}
-			for (const { name: name$9 = "unknown", node: component } of cComponents) {
-				if (AST.findParentNode(component, (n) => isClassComponent$1(n) || isFunctionComponent(n)) == null) continue;
+			const parentComponent = AST.findParentNode(component, isFunctionComponent);
+			if (parentComponent != null && !isDirectValueOfRenderPropertyLoose(parentComponent)) {
 				context.report({
 					messageId: "noNestedComponentDefinitions",
 					node: component,
@@ -1789,9 +1750,29 @@ function create$28(context) {
 						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."
 					}
 				});
+				continue;
 			}
+			if (isInsideRenderMethod(component)) context.report({
+				messageId: "noNestedComponentDefinitions",
+				node: component,
+				data: {
+					name: name$9,
+					suggestion: "Move it to the top level."
+				}
+			});
 		}
-	};
+		for (const { name: name$9 = "unknown", node: component } of cComponents) {
+			if (AST.findParentNode(component, (n) => isClassComponent$1(n) || isFunctionComponent(n)) == null) continue;
+			context.report({
+				messageId: "noNestedComponentDefinitions",
+				node: component,
+				data: {
+					name: name$9,
+					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."
+				}
+			});
+		}
+	} });
 }
 /**
 * Determines whether the node is inside JSX attribute value
@@ -1843,9 +1824,7 @@ function create$27(context) {
 	const collector = useComponentCollector(context, { hint });
 	const collectorLegacy = useComponentCollectorLegacy(context);
 	const lazyComponentDeclarations = /* @__PURE__ */ new Set();
-	return {
-		...collector.listeners,
-		...collectorLegacy.listeners,
+	return defineRuleListener(collector.visitors, collectorLegacy.visitors, {
 		ImportExpression(node) {
 			const lazyCall = AST.findParentNode(node, (n) => isLazyCall(context, n));
 			if (lazyCall != null) lazyComponentDeclarations.add(lazyCall);
@@ -1864,7 +1843,7 @@ function create$27(context) {
 				node: lazy
 			});
 		}
-	};
+	});
 }
 
 //#endregion
@@ -1926,21 +1905,18 @@ var no_redundant_should_component_update_default = createRule({
 });
 function create$25(context) {
 	if (!context.sourceCode.text.includes("shouldComponentUpdate")) return {};
-	const { ctx, listeners } = useComponentCollectorLegacy(context);
-	return {
-		...listeners,
-		"Program:exit"(program) {
-			for (const { name: name$9 = "PureComponent", node: component, flag } of ctx.getAllComponents(program)) {
-				if ((flag & ComponentFlag.PureComponent) === 0n) continue;
-				const { body } = component.body;
-				for (const member of body) if (isShouldComponentUpdate(member)) context.report({
-					messageId: "noRedundantShouldComponentUpdate",
-					node: member,
-					data: { componentName: name$9 }
-				});
-			}
+	const { ctx, visitors } = useComponentCollectorLegacy(context);
+	return defineRuleListener(visitors, { "Program:exit"(program) {
+		for (const { name: name$9 = "PureComponent", node: component, flag } of ctx.getAllComponents(program)) {
+			if ((flag & ComponentFlag.PureComponent) === 0n) continue;
+			const { body } = component.body;
+			for (const member of body) if (isShouldComponentUpdate(member)) context.report({
+				messageId: "noRedundantShouldComponentUpdate",
+				node: member,
+				data: { componentName: name$9 }
+			});
 		}
-	};
+	} });
 }
 
 //#endregion
@@ -2111,10 +2087,9 @@ function create$20(context) {
 		...getJsxConfigFromContext(context),
 		...getJsxConfigFromAnnotation(context)
 	};
-	const { ctx, listeners } = useComponentCollector(context);
+	const { ctx, visitors } = useComponentCollector(context);
 	const constantKeys = /* @__PURE__ */ new Set();
-	return {
-		...listeners,
+	return defineRuleListener(visitors, {
 		JSXAttribute(node) {
 			if (node.name.name !== "key") return;
 			const jsxElement = node.parent.parent;
@@ -2149,7 +2124,7 @@ function create$20(context) {
 				});
 			}
 		}
-	};
+	});
 }
 function getArrayMethodCallbackPosition(methodName) {
 	switch (methodName) {
@@ -2348,24 +2323,21 @@ var no_unnecessary_use_prefix_default = createRule({
 	defaultOptions: []
 });
 function create$17(context) {
-	const { ctx, listeners } = useHookCollector(context);
-	return {
-		...listeners,
-		"Program:exit"(program) {
-			for (const { id, name: name$9, node, hookCalls } of ctx.getAllHooks(program)) {
-				if (hookCalls.length > 0) continue;
-				if (AST.isFunctionEmpty(node)) continue;
-				if (WELL_KNOWN_HOOKS.includes(name$9)) continue;
-				if (containsUseComments(context, node)) continue;
-				if (AST.findParentNode(node, AST.isViMockCallback) != null) continue;
-				context.report({
-					messageId: "noUnnecessaryUsePrefix",
-					node: id ?? node,
-					data: { name: name$9 }
-				});
-			}
+	const { ctx, visitors } = useHookCollector(context);
+	return defineRuleListener(visitors, { "Program:exit"(program) {
+		for (const { id, name: name$9, node, hookCalls } of ctx.getAllHooks(program)) {
+			if (hookCalls.length > 0) continue;
+			if (AST.isFunctionEmpty(node)) continue;
+			if (WELL_KNOWN_HOOKS.includes(name$9)) continue;
+			if (containsUseComments(context, node)) continue;
+			if (AST.findParentNode(node, AST.isViMockCallback) != null) continue;
+			context.report({
+				messageId: "noUnnecessaryUsePrefix",
+				node: id ?? node,
+				data: { name: name$9 }
+			});
 		}
-	};
+	} });
 }
 
 //#endregion
@@ -2422,19 +2394,16 @@ var no_unsafe_component_will_mount_default = createRule({
 });
 function create$15(context) {
 	if (!context.sourceCode.text.includes("UNSAFE_componentWillMount")) return {};
-	const { ctx, listeners } = useComponentCollectorLegacy(context);
-	return {
-		...listeners,
-		"Program:exit"(program) {
-			for (const { node: component } of ctx.getAllComponents(program)) {
-				const { body } = component.body;
-				for (const member of body) if (isUnsafeComponentWillMount(member)) context.report({
-					messageId: "noUnsafeComponentWillMount",
-					node: member
-				});
-			}
+	const { ctx, visitors } = useComponentCollectorLegacy(context);
+	return defineRuleListener(visitors, { "Program:exit"(program) {
+		for (const { node: component } of ctx.getAllComponents(program)) {
+			const { body } = component.body;
+			for (const member of body) if (isUnsafeComponentWillMount(member)) context.report({
+				messageId: "noUnsafeComponentWillMount",
+				node: member
+			});
 		}
-	};
+	} });
 }
 
 //#endregion
@@ -2453,19 +2422,16 @@ var no_unsafe_component_will_receive_props_default = createRule({
 });
 function create$14(context) {
 	if (!context.sourceCode.text.includes("UNSAFE_componentWillReceiveProps")) return {};
-	const { ctx, listeners } = useComponentCollectorLegacy(context);
-	return {
-		...listeners,
-		"Program:exit"(program) {
-			for (const { node: component } of ctx.getAllComponents(program)) {
-				const { body } = component.body;
-				for (const member of body) if (isUnsafeComponentWillReceiveProps(member)) context.report({
-					messageId: "noUnsafeComponentWillReceiveProps",
-					node: member
-				});
-			}
+	const { ctx, visitors } = useComponentCollectorLegacy(context);
+	return defineRuleListener(visitors, { "Program:exit"(program) {
+		for (const { node: component } of ctx.getAllComponents(program)) {
+			const { body } = component.body;
+			for (const member of body) if (isUnsafeComponentWillReceiveProps(member)) context.report({
+				messageId: "noUnsafeComponentWillReceiveProps",
+				node: member
+			});
 		}
-	};
+	} });
 }
 
 //#endregion
@@ -2484,19 +2450,16 @@ var no_unsafe_component_will_update_default = createRule({
 });
 function create$13(context) {
 	if (!context.sourceCode.text.includes("UNSAFE_componentWillUpdate")) return {};
-	const { ctx, listeners } = useComponentCollectorLegacy(context);
-	return {
-		...listeners,
-		"Program:exit"(program) {
-			for (const { node: component } of ctx.getAllComponents(program)) {
-				const { body } = component.body;
-				for (const member of body) if (isUnsafeComponentWillUpdate(member)) context.report({
-					messageId: "noUnsafeComponentWillUpdate",
-					node: member
-				});
-			}
+	const { ctx, visitors } = useComponentCollectorLegacy(context);
+	return defineRuleListener(visitors, { "Program:exit"(program) {
+		for (const { node: component } of ctx.getAllComponents(program)) {
+			const { body } = component.body;
+			for (const member of body) if (isUnsafeComponentWillUpdate(member)) context.report({
+				messageId: "noUnsafeComponentWillUpdate",
+				node: member
+			});
 		}
-	};
+	} });
 }
 
 //#endregion
@@ -2516,10 +2479,9 @@ var no_unstable_context_value_default = createRule({
 function create$12(context) {
 	const { version: version$1 } = getSettingsFromContext(context);
 	const isReact18OrBelow = compare(version$1, "19.0.0", "<");
-	const { ctx, listeners } = useComponentCollector(context);
+	const { ctx, visitors } = useComponentCollector(context);
 	const constructions = /* @__PURE__ */ new WeakMap();
-	return {
-		...listeners,
+	return defineRuleListener(visitors, {
 		JSXOpeningElement(node) {
 			const selfName = getJsxElementType(context, node.parent).split(".").at(-1);
 			if (selfName == null) return;
@@ -2550,7 +2512,7 @@ function create$12(context) {
 				});
 			}
 		}
-	};
+	});
 }
 function isContextName(name$9, isReact18OrBelow) {
 	if (name$9 === "Provider") return true;
@@ -2590,12 +2552,11 @@ function extractIdentifier(node) {
 	return null;
 }
 function create$11(context, [options]) {
-	const { ctx, listeners } = useComponentCollector(context);
+	const { ctx, visitors } = useComponentCollector(context);
 	const declarators = /* @__PURE__ */ new WeakMap();
 	const { safeDefaultProps = [] } = options;
 	const safePatterns = safeDefaultProps.map((s) => toRegExp(s));
-	return {
-		...listeners,
+	return defineRuleListener(visitors, {
 		[AST.SEL_OBJECT_DESTRUCTURING_VARIABLE_DECLARATOR](node) {
 			const functionEntry = ctx.getCurrentEntry();
 			if (functionEntry == null) return;
@@ -2629,7 +2590,7 @@ function create$11(context, [options]) {
 				}
 			}
 		}
-	};
+	});
 }
 
 //#endregion
@@ -2764,29 +2725,26 @@ var no_unused_props_default = createRule({
 });
 function create$9(context) {
 	const services = ESLintUtils.getParserServices(context, false);
-	const { ctx, listeners } = useComponentCollector(context);
-	return {
-		...listeners,
-		"Program:exit"(program) {
-			const checker = services.program.getTypeChecker();
-			const totalDeclaredProps = /* @__PURE__ */ new Set();
-			const totalUsedProps = /* @__PURE__ */ new Set();
-			for (const component of ctx.getAllComponents(program)) {
-				const [props] = component.node.params;
-				if (props == null) continue;
-				const usedPropKeys = /* @__PURE__ */ new Set();
-				if (!collectUsedPropKeysOfParameter(context, usedPropKeys, props)) continue;
-				const tsNode = services.esTreeNodeToTSNodeMap.get(props);
-				const declaredProps = checker.getTypeAtLocation(tsNode).getProperties();
-				for (const declaredProp of declaredProps) {
-					totalDeclaredProps.add(declaredProp);
-					if (usedPropKeys.has(declaredProp.name)) totalUsedProps.add(declaredProp);
-				}
+	const { ctx, visitors } = useComponentCollector(context);
+	return defineRuleListener(visitors, { "Program:exit"(program) {
+		const checker = services.program.getTypeChecker();
+		const totalDeclaredProps = /* @__PURE__ */ new Set();
+		const totalUsedProps = /* @__PURE__ */ new Set();
+		for (const component of ctx.getAllComponents(program)) {
+			const [props] = component.node.params;
+			if (props == null) continue;
+			const usedPropKeys = /* @__PURE__ */ new Set();
+			if (!collectUsedPropKeysOfParameter(context, usedPropKeys, props)) continue;
+			const tsNode = services.esTreeNodeToTSNodeMap.get(props);
+			const declaredProps = checker.getTypeAtLocation(tsNode).getProperties();
+			for (const declaredProp of declaredProps) {
+				totalDeclaredProps.add(declaredProp);
+				if (usedPropKeys.has(declaredProp.name)) totalUsedProps.add(declaredProp);
 			}
-			const unusedProps = [...totalDeclaredProps].filter((x) => !totalUsedProps.has(x));
-			for (const unusedProp of unusedProps) reportUnusedProp(context, services, unusedProp);
 		}
-	};
+		const unusedProps = [...totalDeclaredProps].filter((x) => !totalUsedProps.has(x));
+		for (const unusedProp of unusedProps) reportUnusedProp(context, services, unusedProp);
+	} });
 }
 function collectUsedPropKeysOfParameter(context, usedPropKeys, parameter) {
 	switch (parameter.type) {
@@ -3251,10 +3209,9 @@ var prefer_destructuring_assignment_default = createRule({
 	defaultOptions: []
 });
 function create$4(context) {
-	const { ctx, listeners } = useComponentCollector(context);
+	const { ctx, visitors } = useComponentCollector(context);
 	const exprs = [];
-	return {
-		...listeners,
+	return defineRuleListener(visitors, {
 		MemberExpression(node) {
 			if (!isInsideComponentOrHook(node)) return;
 			if (!isMemberExpressionWithObjectName(node)) return;
@@ -3288,7 +3245,7 @@ function create$4(context) {
 				});
 			}
 		}
-	};
+	});
 }
 
 //#endregion
@@ -3345,25 +3302,22 @@ var prefer_read_only_props_default = createRule({
 function create$2(context) {
 	const services = ESLintUtils.getParserServices(context, false);
 	const checker = services.program.getTypeChecker();
-	const { ctx, listeners } = useComponentCollector(context);
-	return {
-		...listeners,
-		"Program:exit"(program) {
-			for (const component of ctx.getAllComponents(program)) {
-				const [props] = component.node.params;
-				if (component.id == null || component.name == null) continue;
-				if (props == null) continue;
-				const propsType = getConstrainedTypeAtLocation(services, props);
-				if (isTypeReadonly(services.program, propsType)) continue;
-				if (isTypeReadonlyLoose(services, propsType)) continue;
-				if (propsType.isClassOrInterface() && isClassOrInterfaceReadonlyLoose(checker, propsType)) continue;
-				context.report({
-					messageId: "preferReadOnlyProps",
-					node: props
-				});
-			}
+	const { ctx, visitors } = useComponentCollector(context);
+	return defineRuleListener(visitors, { "Program:exit"(program) {
+		for (const component of ctx.getAllComponents(program)) {
+			const [props] = component.node.params;
+			if (component.id == null || component.name == null) continue;
+			if (props == null) continue;
+			const propsType = getConstrainedTypeAtLocation(services, props);
+			if (isTypeReadonly(services.program, propsType)) continue;
+			if (isTypeReadonlyLoose(services, propsType)) continue;
+			if (propsType.isClassOrInterface() && isClassOrInterfaceReadonlyLoose(checker, propsType)) continue;
+			context.report({
+				messageId: "preferReadOnlyProps",
+				node: props
+			});
 		}
-	};
+	} });
 }
 function isTypeReadonlyLoose(services, type) {
 	try {

package/package.json

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