eslint-plugin-react-x 4.2.3 → 4.2.5-beta.0

package/README.md

@@ -36,4 +36,4 @@ export default defineConfig(
 
 ## Rules
 
-<https://eslint-react.xyz/docs/rules/overview#x-rules>
+<https://eslint-react.xyz/docs/rules#x-rules>

package/dist/index.js

@@ -145,7 +145,7 @@ const rules$7 = {
 //#endregion
 //#region package.json
 var name$6 = "eslint-plugin-react-x";
-var version = "4.2.3";
+var version = "4.2.5-beta.0";
 
 //#endregion
 //#region src/utils/create-rule.ts
@@ -429,22 +429,31 @@ var error_boundaries_default = createRule({
 function create$53(context) {
 	if (!context.sourceCode.text.includes("try")) return {};
 	const hint = JsxDetectionHint.DoNotIncludeJsxWithNullValue | JsxDetectionHint.DoNotIncludeJsxWithNumberValue | JsxDetectionHint.DoNotIncludeJsxWithBigIntValue | JsxDetectionHint.DoNotIncludeJsxWithStringValue | JsxDetectionHint.DoNotIncludeJsxWithBooleanValue | JsxDetectionHint.DoNotIncludeJsxWithUndefinedValue | JsxDetectionHint.DoNotIncludeJsxWithEmptyArrayValue;
-	const { api, visitor } = core.getComponentCollector(context);
+	const fCollector = core.getComponentCollector(context);
+	const hCollector = core.getHookCollector(context);
 	const reported = /* @__PURE__ */ new Set();
-	return defineRuleListener(visitor, {
+	const useCalls = /* @__PURE__ */ new Set();
+	return defineRuleListener(fCollector.visitor, hCollector.visitor, {
 		CallExpression(node) {
 			if (!core.isUseCall(node)) return;
-			const stmt = ast.findParent(node, ast.is(AST_NODE_TYPES.TryStatement));
-			if (stmt != null && !reported.has(stmt)) {
-				context.report({
-					messageId: "tryCatchWithUse",
-					node
-				});
-				reported.add(stmt);
-			}
+			useCalls.add(node);
 		},
 		"Program:exit"(node) {
-			for (const { rets } of api.getAllComponents(node)) for (const ret of rets) {
+			const comps = fCollector.api.getAllComponents(node);
+			const hooks = hCollector.api.getAllHooks(node);
+			const funcs = [...comps, ...hooks];
+			for (const call of useCalls) {
+				const stmt = ast.findParent(call, ast.is(AST_NODE_TYPES.TryStatement));
+				const func = ast.findParent(stmt, (n) => funcs.some((f) => f.node === n));
+				if (stmt != null && func != null && !reported.has(stmt)) {
+					context.report({
+						messageId: "tryCatchWithUse",
+						node: stmt
+					});
+					reported.add(stmt);
+				}
+			}
+			for (const { rets } of funcs) for (const ret of rets) {
 				if (ret == null) continue;
 				if (!isJsxLike(context, ret, hint)) continue;
 				const stmt = ast.findParent(ret, ast.is(AST_NODE_TYPES.TryStatement));
@@ -3340,6 +3349,7 @@ function create$18(context) {
 	const { api, visitor } = core.getHookCollector(context);
 	return defineRuleListener(visitor, { "Program:exit"(program) {
 		for (const { id, name, hookCalls, node } of api.getAllHooks(program)) {
+			if (name == null) continue;
 			if (hookCalls.length > 0) continue;
 			if (ast.isFunctionEmpty(node)) continue;
 			if (WELL_KNOWN_HOOKS.includes(name)) continue;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "4.2.3",
+  "version": "4.2.5-beta.0",
   "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.5.0",
     "ts-pattern": "^5.9.0",
-    "@eslint-react/ast": "4.2.3",
-    "@eslint-react/core": "4.2.3",
-    "@eslint-react/jsx": "4.2.3",
-    "@eslint-react/shared": "4.2.3",
-    "@eslint-react/var": "4.2.3"
+    "@eslint-react/ast": "4.2.5-beta.0",
+    "@eslint-react/core": "4.2.5-beta.0",
+    "@eslint-react/shared": "4.2.5-beta.0",
+    "@eslint-react/var": "4.2.5-beta.0",
+    "@eslint-react/jsx": "4.2.5-beta.0"
   },
   "devDependencies": {
     "@types/react": "^19.2.14",