eslint-plugin-react-x 5.2.3-next.1 → 5.2.4-beta.0

package/dist/index.js

@@ -143,7 +143,7 @@ const rules$7 = {
 //#endregion
 //#region package.json
 var name$6 = "eslint-plugin-react-x";
-var version = "5.2.3-next.1";
+var version = "5.2.4-beta.0";
 
 //#endregion
 //#region src/rules/component-hook-factories/lib.ts
@@ -1325,7 +1325,7 @@ function create$46(context) {
 			const { object, property } = node.callee;
 			if (property.type !== AST_NODE_TYPES.Identifier) return;
 			if (!MUTATING_ARRAY_METHODS.has(property.name)) return;
-			const rootId = Extract.rootIdentifier(object);
+			const rootId = Extract.getRootIdentifier(object);
 			if (rootId == null) return;
 			if (rootId.name === "draft") return;
 			const enclosingFn = Traverse.findParent(node, Check.isFunction);
@@ -1346,7 +1346,7 @@ function create$46(context) {
 		},
 		AssignmentExpression(node) {
 			if (node.left.type !== AST_NODE_TYPES.MemberExpression) return;
-			const rootId = Extract.rootIdentifier(node.left);
+			const rootId = Extract.getRootIdentifier(node.left);
 			if (rootId == null) return;
 			if (rootId.name === "draft") return;
 			const enclosingFn = Traverse.findParent(node, Check.isFunction);
@@ -1613,14 +1613,14 @@ function create$45(context) {
 			classStack.pop();
 		},
 		MemberExpression(node) {
-			if (!Check.thisExpression(node.object)) return;
+			if (node.object.type !== AST_NODE_TYPES.ThisExpression) return;
 			const [currClass, isComponent = false] = classStack.at(-1) ?? [];
 			if (currClass == null || !isComponent) return;
 			const [currMethod, isStatic = false] = methodStack.at(-1) ?? [];
 			if (currMethod == null || isStatic) return;
 			const [setState, hasThisState = false] = setStateStack.at(-1) ?? [];
 			if (setState == null || hasThisState) return;
-			if (Extract.propertyName(node.property) !== "state") return;
+			if (Extract.getPropertyName(node.property) !== "state") return;
 			context.report({
 				messageId: "default",
 				node
@@ -1645,8 +1645,8 @@ function create$45(context) {
 			if (currMethod == null || isStatic) return;
 			const [setState, hasThisState = false] = setStateStack.at(-1) ?? [];
 			if (setState == null || hasThisState) return;
-			if (node.init == null || !Check.thisExpression(node.init) || node.id.type !== AST_NODE_TYPES.ObjectPattern) return;
-			if (!node.id.properties.some((prop) => prop.type === AST_NODE_TYPES.Property && isKeyLiteral$1(prop, prop.key) && Extract.propertyName(prop.key) === "state")) 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 && isKeyLiteral$1(prop, prop.key) && Extract.getPropertyName(prop.key) === "state")) return;
 			context.report({
 				messageId: "default",
 				node
@@ -2170,7 +2170,7 @@ function create$30(context) {
 		const aValue = a.value;
 		const bValue = b.value;
 		if (aValue == null || bValue == null) return false;
-		return Compare.areEqual(aValue, bValue);
+		return Compare.isEqual(aValue, bValue);
 	}
 	return merge({
 		"JSXAttribute[name.name='key']"(node) {
@@ -2194,7 +2194,7 @@ function create$30(context) {
 					if (!Check.isFunction(iter)) return;
 					const arg0 = call?.arguments[0];
 					if (call == null || arg0 == null) return;
-					if (Extract.unwrapped(arg0) !== iter) return;
+					if (Extract.unwrap(arg0) !== iter) return;
 					keyedEntries.set(call, {
 						hasDuplicate: node.value?.type === AST_NODE_TYPES.Literal,
 						keys: [node],
@@ -2508,7 +2508,7 @@ function create$25(context) {
 		}, ({ left, right }) => {
 			if (left.type === AST_NODE_TYPES.UnaryExpression && left.operator === "!") return getReportDescriptor(right);
 			const initialScope = context.sourceCode.getScope(left);
-			if (Check.identifier(left, "NaN") || getStaticValue(left, initialScope)?.value === "NaN") return {
+			if (Check.isIdentifier("NaN")(left) || getStaticValue(left, initialScope)?.value === "NaN") return {
 				data: { value: context.sourceCode.getText(left) },
 				messageId: "default",
 				node: left
@@ -2765,8 +2765,8 @@ function isProcessEnvNodeEnvCompare(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.literal(node.right, "string")) return node.right.value === value;
-	if (Check.literal(node.left, "string") && isProcessEnvNodeEnv(node.right)) return node.left.value === value;
+	if (isProcessEnvNodeEnv(node.left) && Check.isLiteral("string")(node.right)) return node.right.value === value;
+	if (Check.isLiteral("string")(node.left) && isProcessEnvNodeEnv(node.right)) return node.left.value === value;
 	return false;
 }
 
@@ -3207,7 +3207,7 @@ var no_unstable_context_value_default = createRule({
 function create$11(context) {
 	const { compilationMode, version } = getSettingsFromContext(context);
 	if (compilationMode === "infer" || compilationMode === "all") return {};
-	if (compilationMode === "annotation" && context.sourceCode.ast.body.some((stmt) => Check.directive(stmt) && stmt.directive === "use memo")) return {};
+	if (compilationMode === "annotation" && context.sourceCode.ast.body.some(Check.isDirective("use memo"))) return {};
 	const isReact18OrBelow = compare(version, "19.0.0", "<");
 	const { api, visitor } = core.getFunctionComponentCollector(context);
 	const constructions = /* @__PURE__ */ new WeakMap();
@@ -3237,7 +3237,7 @@ function create$11(context) {
 					const suggestion = kind === "function" ? "Consider wrapping it in a useCallback hook." : "Consider wrapping it in a useMemo hook.";
 					context.report({
 						data: {
-							kind: Extract.humanReadableKind(constructionNode),
+							kind: Extract.getHumanReadableKind(constructionNode),
 							suggestion
 						},
 						messageId: "unstableContextValue",
@@ -3254,6 +3254,14 @@ function isContextName(name, isReact18OrBelow) {
 	return false;
 }
 
+//#endregion
+//#region src/rules/no-unstable-default-props/lib.ts
+const SEL_OBJECT_DESTRUCTURING_VARIABLE_DECLARATOR = [
+	"VariableDeclarator",
+	"[id.type='ObjectPattern']",
+	"[init.type='Identifier']"
+].join("");
+
 //#endregion
 //#region src/rules/no-unstable-default-props/no-unstable-default-props.ts
 const RULE_NAME$10 = "no-unstable-default-props";
@@ -3288,13 +3296,8 @@ function extractIdentifier(node) {
 function create$10(context, [options]) {
 	const { compilationMode } = getSettingsFromContext(context);
 	if (compilationMode === "infer" || compilationMode === "all") return {};
-	if (compilationMode === "annotation" && context.sourceCode.ast.body.some((stmt) => Check.directive(stmt) && stmt.directive === "use memo")) return {};
+	if (compilationMode === "annotation" && context.sourceCode.ast.body.some(Check.isDirective("use memo"))) return {};
 	const { api, visitor } = core.getFunctionComponentCollector(context);
-	const SEL_OBJECT_DESTRUCTURING_VARIABLE_DECLARATOR = [
-		"VariableDeclarator",
-		"[id.type='ObjectPattern']",
-		"[init.type='Identifier']"
-	].join("");
 	const declarators = /* @__PURE__ */ new WeakMap();
 	const { safeDefaultProps = [] } = options;
 	const safePatterns = safeDefaultProps.map((s) => toRegExp(s));
@@ -3319,7 +3322,7 @@ function create$10(context, [options]) {
 						if (identifier != null && safePatterns.some((pattern) => pattern.test(identifier))) continue;
 					}
 					context.report({
-						data: { kind: Extract.humanReadableKind(right) },
+						data: { kind: Extract.getHumanReadableKind(right) },
 						messageId: "default",
 						node: right
 					});
@@ -3390,7 +3393,7 @@ function create$9(context) {
 		const usages = propertyUsages.get(currentClass);
 		if (defs == null) return;
 		for (const def of defs) {
-			const methodName = Extract.propertyName(def);
+			const methodName = Extract.getPropertyName(def);
 			if (methodName == null) continue;
 			if ((usages?.has(methodName) ?? false) || LIFECYCLE_METHODS.has(methodName)) continue;
 			context.report({
@@ -3423,12 +3426,12 @@ function create$9(context) {
 			const currentMethod = methodStack.at(-1);
 			if (currentClass == null || currentMethod == null) return;
 			if (!core.isClassComponent(currentClass) || currentMethod.static) return;
-			if (!Check.thisExpression(node.object) || !isKeyLiteral(node, node.property)) return;
+			if (node.object.type !== AST_NODE_TYPES.ThisExpression || !isKeyLiteral(node, node.property)) return;
 			if (node.parent.type === AST_NODE_TYPES.AssignmentExpression && node.parent.left === node) {
 				propertyDefs.get(currentClass)?.add(node.property);
 				return;
 			}
-			const propertyName = Extract.propertyName(node.property);
+			const propertyName = Extract.getPropertyName(node.property);
 			if (propertyName != null) propertyUsages.get(currentClass)?.add(propertyName);
 		},
 		MethodDefinition: methodEnter,
@@ -3440,9 +3443,9 @@ function create$9(context) {
 			const currentMethod = methodStack.at(-1);
 			if (currentClass == null || currentMethod == null) return;
 			if (!core.isClassComponent(currentClass) || currentMethod.static) return;
-			if (node.init != null && Check.thisExpression(node.init) && node.id.type === AST_NODE_TYPES.ObjectPattern) {
+			if (node.init != null && node.init.type === AST_NODE_TYPES.ThisExpression && node.id.type === AST_NODE_TYPES.ObjectPattern) {
 				for (const prop of node.id.properties) if (prop.type === AST_NODE_TYPES.Property && isKeyLiteral(prop, prop.key)) {
-					const keyName = Extract.propertyName(prop.key);
+					const keyName = Extract.getPropertyName(prop.key);
 					if (keyName != null) propertyUsages.get(currentClass)?.add(keyName);
 				}
 			}
@@ -3910,7 +3913,7 @@ function create$6(context) {
 	const nEntries = [];
 	return merge(hCollector.visitor, cCollector.visitor, {
 		CallExpression(node) {
-			const expr = Extract.unwrapped(node.callee);
+			const expr = Extract.unwrap(node.callee);
 			switch (true) {
 				case expr.type === AST_NODE_TYPES.Identifier: {
 					if (!IMPURE_FUNCS.get("globalThis")?.has(expr.name)) return;
@@ -3937,7 +3940,7 @@ function create$6(context) {
 			}
 		},
 		NewExpression(node) {
-			const expr = Extract.unwrapped(node.callee);
+			const expr = Extract.unwrap(node.callee);
 			if (expr.type !== AST_NODE_TYPES.Identifier) return;
 			if (!IMPURE_CTORS.has(expr.name)) return;
 			if (expr.name === "Date" && node.arguments.length > 0) return;
@@ -3998,7 +4001,7 @@ function create$5(context) {
 			type: AST_NODE_TYPES.BinaryExpression,
 			operator: P.union("===", "==", "!==", "!=")
 		}, parent)) return false;
-		const otherSide = parent.left === node || Extract.unwrapped(parent.left) === node ? parent.right : parent.left;
+		const otherSide = parent.left === node || Extract.unwrap(parent.left) === node ? parent.right : parent.left;
 		if (otherSide.type !== AST_NODE_TYPES.Literal || otherSide.value != null) return false;
 		return parent.parent.type === AST_NODE_TYPES.IfStatement && parent.parent.test === parent;
 	}
@@ -4014,8 +4017,8 @@ function create$5(context) {
 		if (op !== "===" && op !== "==" && op !== "!==" && op !== "!=") return false;
 		const { left, right } = test;
 		const checkSides = (a, b) => {
-			a = Check.isTypeExpression(a) ? Extract.unwrapped(a) : a;
-			return a.type === AST_NODE_TYPES.MemberExpression && a.object.type === AST_NODE_TYPES.Identifier && a.object.name === refName && b.type === AST_NODE_TYPES.Literal && b.value == null && Extract.propertyName(a.property) === "current";
+			a = Check.isTypeExpression(a) ? Extract.unwrap(a) : a;
+			return a.type === AST_NODE_TYPES.MemberExpression && a.object.type === AST_NODE_TYPES.Identifier && a.object.name === refName && b.type === AST_NODE_TYPES.Literal && b.value == null && Extract.getPropertyName(a.property) === "current";
 		};
 		return checkSides(left, right) || checkSides(right, left);
 	}
@@ -4040,12 +4043,12 @@ function create$5(context) {
 			}
 		},
 		MemberExpression(node) {
-			if (!Check.identifier(node.property, "current")) return;
+			if (!Check.isIdentifier("current")(node.property)) return;
 			refAccesses.push({
 				isWrite: (() => {
 					let parent = node.parent;
 					while (Check.isTypeExpression(parent)) parent = parent.parent;
-					return match(parent).with({ type: AST_NODE_TYPES.AssignmentExpression }, (p) => p.left === node || Extract.unwrapped(p.left) === node).with({ type: AST_NODE_TYPES.UpdateExpression }, (p) => p.argument === node || Extract.unwrapped(p.argument) === node).otherwise(() => false);
+					return match(parent).with({ type: AST_NODE_TYPES.AssignmentExpression }, (p) => p.left === node || Extract.unwrap(p.left) === node).with({ type: AST_NODE_TYPES.UpdateExpression }, (p) => p.argument === node || Extract.unwrap(p.argument) === node).otherwise(() => false);
 				})(),
 				node
 			});
@@ -6390,8 +6393,8 @@ function create$4(context) {
 		return node.parent?.type === AST_NODE_TYPES.CallExpression && node.parent.callee !== node && isUseEffectCall(node.parent);
 	}
 	function getCallName(node) {
-		if (node.type === AST_NODE_TYPES.CallExpression) return Extract.fullyQualifiedName(node.callee, getText);
-		return Extract.fullyQualifiedName(node, getText);
+		if (node.type === AST_NODE_TYPES.CallExpression) return Extract.getFullyQualifiedName(node.callee, getText);
+		return Extract.getFullyQualifiedName(node, getText);
 	}
 	function getCallKind(node) {
 		return match(node).when(isUseStateCall, () => "useState").when(isUseEffectCall, () => "useEffect").when(isSetStateCall, () => "setState").when(isThenCall, () => "then").otherwise(() => "other");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "5.2.3-next.1",
+  "version": "5.2.4-beta.0",
   "description": "A set of composable ESLint rules for libraries and frameworks that use React as a UI runtime.",
   "keywords": [
     "react",
@@ -46,12 +46,12 @@
     "string-ts": "^2.3.1",
     "ts-api-utils": "^2.5.0",
     "ts-pattern": "^5.9.0",
-    "@eslint-react/ast": "5.2.3-next.1",
-    "@eslint-react/core": "5.2.3-next.1",
-    "@eslint-react/eslint": "5.2.3-next.1",
-    "@eslint-react/jsx": "5.2.3-next.1",
-    "@eslint-react/var": "5.2.3-next.1",
-    "@eslint-react/shared": "5.2.3-next.1"
+    "@eslint-react/ast": "5.2.4-beta.0",
+    "@eslint-react/core": "5.2.4-beta.0",
+    "@eslint-react/jsx": "5.2.4-beta.0",
+    "@eslint-react/shared": "5.2.4-beta.0",
+    "@eslint-react/eslint": "5.2.4-beta.0",
+    "@eslint-react/var": "5.2.4-beta.0"
   },
   "devDependencies": {
     "@types/react": "^19.2.14",
@@ -59,8 +59,8 @@
     "eslint": "^10.2.0",
     "tsdown": "^0.21.8",
     "tsl-dx": "^0.10.3",
-    "@local/configs": "0.0.0",
-    "@local/eff": "3.0.0-beta.72"
+    "@local/eff": "3.0.0-beta.72",
+    "@local/configs": "0.0.0"
   },
   "peerDependencies": {
     "eslint": "^10.2.0",