eslint-plugin-react-x 2.0.0-beta.55 → 2.0.0-beta.57

package/dist/index.js

@@ -115,7 +115,7 @@ const settings = { ...settings$1 };
 //#endregion
 //#region package.json
 var name = "eslint-plugin-react-x";
-var version = "2.0.0-beta.55";
+var version = "2.0.0-beta.57";
 
 //#endregion
 //#region src/utils/create-rule.ts
@@ -591,9 +591,9 @@ function create$59(context) {
 //#region src/rules/no-array-index-key.ts
 const RULE_NAME$58 = "no-array-index-key";
 const RULE_FEATURES$56 = [];
-const reactChildrenMethod = ["forEach", "map"];
+const REACT_CHILDREN_METHOD = ["forEach", "map"];
 function isReactChildrenMethod(name$4) {
-	return reactChildrenMethod.some((method) => method === name$4);
+	return REACT_CHILDREN_METHOD.includes(name$4);
 }
 function isUsingReactChildren(context, node) {
 	const { importSource = "react" } = coerceSettings(context.settings);
@@ -3212,20 +3212,19 @@ function create$11(context, [option]) {
 }
 /**
 * Check if a Literal or JSXText node is whitespace
-* @param node The AST node to check
-* @returns boolean `true` if the node is whitespace
 */
 function isWhiteSpace(node) {
 	return typeof node.value === "string" && node.raw.trim() === "";
 }
 /**
-* Check if a Literal or JSXText node is padding spaces
-* @param node The AST node to check
-* @returns boolean
+* Check if a node is padding spaces (whitespace with line breaks)
 */
 function isPaddingSpaces(node) {
 	return ER.isJsxText(node) && isWhiteSpace(node) && node.raw.includes("\n");
 }
+/**
+* Trim whitespace like React would in JSX
+*/
 function trimLikeReact(text) {
 	const leadingSpaces = /^\s*/.exec(text)?.[0] ?? "";
 	const trailingSpaces = /\s*$/.exec(text)?.[0] ?? "";
@@ -3233,6 +3232,9 @@ function trimLikeReact(text) {
 	const end = trailingSpaces.includes("\n") ? text.length - trailingSpaces.length : text.length;
 	return text.slice(start, end);
 }
+/**
+* Check if a fragment node is useless and should be reported
+*/
 function checkNode(context, node, allowExpressions) {
 	const initialScope = context.sourceCode.getScope(node);
 	if (node.type === AST_NODE_TYPES.JSXElement && ER.hasAttribute(context, "key", node.openingElement.attributes, initialScope)) return;
@@ -3273,18 +3275,16 @@ function checkNode(context, node, allowExpressions) {
 	}
 	const nonPaddingChildren = node.children.filter((child) => !isPaddingSpaces(child));
 	const firstNonPaddingChild = nonPaddingChildren.at(0);
-	switch (true) {
-		case nonPaddingChildren.length === 0:
-		case nonPaddingChildren.length === 1 && firstNonPaddingChild?.type !== AST_NODE_TYPES.JSXExpressionContainer:
-			context.report({
-				messageId: "uselessFragment",
-				node,
-				data: { reason: "contains less than two children" },
-				fix: getFix(context, node)
-			});
-			return;
-	}
+	if (nonPaddingChildren.length === 0 || nonPaddingChildren.length === 1 && firstNonPaddingChild?.type !== AST_NODE_TYPES.JSXExpressionContainer) context.report({
+		messageId: "uselessFragment",
+		node,
+		data: { reason: "contains less than two children" },
+		fix: getFix(context, node)
+	});
 }
+/**
+* Generate fix for removing useless fragment
+*/
 function getFix(context, node) {
 	if (!canFix(context, node)) return null;
 	return (fixer) => {
@@ -3294,11 +3294,13 @@ function getFix(context, node) {
 		return fixer.replaceText(node, trimLikeReact(childrenText));
 	};
 }
+/**
+* Check if it's safe to automatically fix the fragment
+*/
 function canFix(context, node) {
 	if (node.parent.type === AST_NODE_TYPES.JSXElement || node.parent.type === AST_NODE_TYPES.JSXFragment) return ER.isHostElement(context, node.parent);
 	if (node.children.length === 0) return false;
-	if (node.children.some((child) => ER.isJsxText(child) && !isWhiteSpace(child) || AST.is(AST_NODE_TYPES.JSXExpressionContainer)(child))) return false;
-	return true;
+	return !node.children.some((child) => ER.isJsxText(child) && !isWhiteSpace(child) || AST.is(AST_NODE_TYPES.JSXExpressionContainer)(child));
 }
 
 //#endregion

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "2.0.0-beta.55",
+  "version": "2.0.0-beta.57",
   "description": "A set of composable ESLint rules for for libraries and frameworks that use React as a UI runtime.",
   "keywords": [
     "react",
@@ -42,12 +42,12 @@
     "is-immutable-type": "^5.0.1",
     "string-ts": "^2.2.1",
     "ts-pattern": "^5.8.0",
-    "@eslint-react/ast": "2.0.0-beta.55",
-    "@eslint-react/eff": "2.0.0-beta.55",
-    "@eslint-react/kit": "2.0.0-beta.55",
-    "@eslint-react/shared": "2.0.0-beta.55",
-    "@eslint-react/var": "2.0.0-beta.55",
-    "@eslint-react/core": "2.0.0-beta.55"
+    "@eslint-react/ast": "2.0.0-beta.57",
+    "@eslint-react/core": "2.0.0-beta.57",
+    "@eslint-react/kit": "2.0.0-beta.57",
+    "@eslint-react/eff": "2.0.0-beta.57",
+    "@eslint-react/shared": "2.0.0-beta.57",
+    "@eslint-react/var": "2.0.0-beta.57"
   },
   "devDependencies": {
     "@types/react": "^19.1.12",