eslint-plugin-react-x 5.8.7-beta.4 → 5.8.7-beta.5

package/dist/index.js

@@ -143,7 +143,7 @@ const rules$6 = {
 //#endregion
 //#region package.json
 var name$6 = "eslint-plugin-react-x";
-var version = "5.8.7-beta.4";
+var version = "5.8.7-beta.5";
 
 //#endregion
 //#region src/utils/create-rule.ts
@@ -2236,7 +2236,7 @@ const RULE_NAME$34 = "no-create-ref";
 var no_create_ref_default = createRule({
 	meta: {
 		type: "suggestion",
-		docs: { description: "Disallows 'createRef' in function components." },
+		docs: { description: "Disallows 'createRef' in function components and Hooks." },
 		messages: { default: "[Deprecated] Use 'useRef' instead." },
 		schema: []
 	},
@@ -2245,12 +2245,28 @@ var no_create_ref_default = createRule({
 	defaultOptions: []
 });
 function create$34(context) {
-	return merge({ CallExpression(node) {
-		if (core.isCreateRefCall(context, node) && Traverse.findParent(node, core.isClassComponent) == null) context.report({
-			messageId: "default",
-			node
-		});
-	} });
+	if (!context.sourceCode.text.includes("createRef")) return {};
+	const fc = core.getFunctionComponentCollector(context);
+	const hc = core.getHookCollector(context);
+	const createRefCalls = /* @__PURE__ */ new Set();
+	return merge(fc.visitor, hc.visitor, {
+		CallExpression(node) {
+			if (!core.isCreateRefCall(context, node)) return;
+			createRefCalls.add(node);
+		},
+		"Program:exit"(node) {
+			const comps = fc.api.getAllComponents(node);
+			const hooks = hc.api.getAllHooks(node);
+			const funcs = [...comps, ...hooks];
+			for (const call of createRefCalls) {
+				if (Traverse.findParent(call, (n) => funcs.some((f) => f.node === n)) == null) continue;
+				context.report({
+					messageId: "default",
+					node: call
+				});
+			}
+		}
+	});
 }
 
 //#endregion
@@ -2891,30 +2907,32 @@ function create$24(context) {
 //#region src/rules/no-misused-capture-owner-stack/lib.ts
 /**
 * Check if the given node is a member expression that accesses `process.env.NODE_ENV`
+* @param context The rule context
 * @param node The AST node
 * @returns True if the node is a member expression that accesses `process.env.NODE_ENV`, false otherwise
 */
-function isProcessEnvNodeEnv(node) {
-	return node != null && node.type === AST_NODE_TYPES.MemberExpression && node.object.type === AST_NODE_TYPES.MemberExpression && node.object.object.type === AST_NODE_TYPES.Identifier && node.object.object.name === "process" && node.object.property.type === AST_NODE_TYPES.Identifier && node.object.property.name === "env" && node.property.type === AST_NODE_TYPES.Identifier && node.property.name === "NODE_ENV";
+function isProcessEnvNodeEnv(context, node) {
+	return node != null && core.isAPI("process.env.NODE_ENV")(context, node);
 }
 /**
 * Check if the given node is a binary expression that compares `process.env.NODE_ENV` with a string literal.
+* @param context The rule context
 * @param node The AST node
 * @param operator The operator used in the comparison
 * @param value The string literal value to compare against
 * @returns True if the node is a binary expression that compares `process.env.NODE_ENV` with the specified value, false otherwise
 */
-function isProcessEnvNodeEnvCompare(node, operator, value) {
+function isProcessEnvNodeEnvCompare(context, node, operator, value) {
 	if (node == null) return false;
 	if (node.type !== AST_NODE_TYPES.BinaryExpression) return false;
 	if (node.operator !== operator) return false;
-	if (isProcessEnvNodeEnv(node.left) && Check.isStringLiteral(node.right)) return node.right.value === value;
-	if (Check.isStringLiteral(node.left) && isProcessEnvNodeEnv(node.right)) return node.left.value === value;
+	if (isProcessEnvNodeEnv(context, node.left) && Check.isStringLiteral(node.right)) return node.right.value === value;
+	if (Check.isStringLiteral(node.left) && isProcessEnvNodeEnv(context, node.right)) return node.left.value === value;
 	return false;
 }
-function isDevelopmentOnlyCheck(node) {
+function isDevelopmentOnlyCheck(context, node) {
 	if (node.type !== AST_NODE_TYPES.IfStatement) return false;
-	return isProcessEnvNodeEnvCompare(node.test, "!==", "production");
+	return isProcessEnvNodeEnvCompare(context, node.test, "!==", "production");
 }
 
 //#endregion
@@ -2940,7 +2958,7 @@ function create$23(context) {
 	return merge({
 		CallExpression(node) {
 			if (!core.isCaptureOwnerStackCall(context, node)) return;
-			if (Traverse.findParent(node, isDevelopmentOnlyCheck) == null) context.report({
+			if (Traverse.findParent(node, (n) => isDevelopmentOnlyCheck(context, n)) == null) context.report({
 				messageId: "missingDevelopmentOnlyCheck",
 				node
 			});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "5.8.7-beta.4",
+  "version": "5.8.7-beta.5",
   "description": "A set of composable ESLint rules for libraries and frameworks that use React as a UI runtime.",
   "keywords": [
     "react",
@@ -45,12 +45,12 @@
     "string-ts": "^2.3.1",
     "ts-api-utils": "^2.5.0",
     "ts-pattern": "^5.9.0",
-    "@eslint-react/ast": "5.8.7-beta.4",
-    "@eslint-react/core": "5.8.7-beta.4",
-    "@eslint-react/eslint": "5.8.7-beta.4",
-    "@eslint-react/jsx": "5.8.7-beta.4",
-    "@eslint-react/shared": "5.8.7-beta.4",
-    "@eslint-react/var": "5.8.7-beta.4"
+    "@eslint-react/core": "5.8.7-beta.5",
+    "@eslint-react/eslint": "5.8.7-beta.5",
+    "@eslint-react/ast": "5.8.7-beta.5",
+    "@eslint-react/jsx": "5.8.7-beta.5",
+    "@eslint-react/shared": "5.8.7-beta.5",
+    "@eslint-react/var": "5.8.7-beta.5"
   },
   "devDependencies": {
     "@types/react": "^19.2.15",