@eslint-react/jsx 5.27.2 → 5.27.4

package/dist/index.d.ts

@@ -128,7 +128,7 @@ type AttributeValue = {
   toStatic(): unknown;
 } | {
   readonly kind: "element";
-  readonly node: TSESTree.JSXElement;
+  readonly node: TSESTreeJSXElementLike;
   toStatic(): undefined;
 } | {
   readonly kind: "missing";
@@ -194,7 +194,8 @@ export declare function getAttributeStaticValue(context: RuleContext, element: T
 export declare function getChildren(element: TSESTreeJSXElementLike): TSESTree.JSXChild[];
 /**
  * Check if the element has at least one meaningful child, that is, a child that
- * is not purely whitespace text or an empty string expression (`{""}`).
+ * is not purely whitespace text, an empty expression container (`{}`), or an
+ * empty string expression (`{""}`).
  *
  * Unlike {@link getChildren} (which only filters whitespace containing a newline),
  * this check treats any whitespace-only text as non-meaningful, so `hasChildren(node)`
@@ -282,8 +283,8 @@ export declare function getElementSelfType(node: TSESTreeJSXElementLike): string
  * Mirrors Babel's `cleanJSXElementLiteralChild` algorithm:
  * 1. Split the raw text into lines.
  * 2. Find the last non-empty line.
- * 3. Trim leading spaces on non-first lines and trailing spaces on non-last lines.
- * 4. Collapse tabs into spaces.
+ * 3. Collapse tabs into spaces.
+ * 4. Trim leading spaces on non-first lines and trailing spaces on non-last lines.
  * 5. Append a single space after each non-last non-empty line.
  * @param text The raw JSX text string to collapse.
  * @returns The collapsed string, or `null` when the text contains only whitespace.
@@ -296,6 +297,10 @@ export declare function collapseMultilineText(text: string): string | null;
  * {@link collapseMultilineText}) and contains a newline.
  *
  * For the looser "any whitespace-only text" check, see {@link isWhitespaceText}.
+ *
+ * Reads the entity-decoded `node.value`; `value` and `raw` differ only when the
+ * text contains HTML entities (ex: ` `), which is never whitespace-only,
+ * so reading either field yields the same result here.
  * @param node The JSX child node to check.
  * @returns `true` if the node is purely formatting whitespace.
  */
@@ -305,6 +310,10 @@ export declare function isPaddingWhitespace(node: TSESTree.JSXChild): boolean;
  *
  * Looser variant of {@link isPaddingWhitespace}; matches every `JSXText` node
  * whose raw content is empty after trimming, regardless of newlines.
+ *
+ * Reads `node.raw` rather than the entity-decoded `value`; for whitespace-only
+ * text the two are identical (an entity is never whitespace), matching React's
+ * behavior of rendering entity text like ` ` as a real text node.
  * @param node The JSX child node to check.
  * @returns `true` if the node is a whitespace-only `JSXText`.
  */

package/dist/index.js

@@ -213,6 +213,11 @@ function resolveJsxAttribute(context, node) {
 			node: node.value,
 			toStatic() {}
 		};
+		case AST_NODE_TYPES.JSXFragment: return {
+			kind: "element",
+			node: node.value,
+			toStatic() {}
+		};
 		case AST_NODE_TYPES.JSXSpreadChild: return {
 			kind: "spreadChild",
 			node: node.value,
@@ -266,6 +271,10 @@ function isWhitespaceText(node) {
 	if (node.type !== AST_NODE_TYPES.JSXText) return false;
 	return node.raw.trim() === "";
 }
+function isEmptyExpressionContainer(node) {
+	if (node.type !== AST_NODE_TYPES.JSXExpressionContainer) return false;
+	return node.expression.type === AST_NODE_TYPES.JSXEmptyExpression;
+}
 function isEmptyStringExpression(node) {
 	if (node.type !== AST_NODE_TYPES.JSXExpressionContainer) return false;
 	const expr = node.expression;
@@ -278,17 +287,15 @@ function getChildren(element) {
 	const children = [];
 	for (const child of element.children) {
 		if (isPaddingWhitespace(child)) continue;
-		if (child.type === AST_NODE_TYPES.JSXExpressionContainer) {
-			if (child.expression.type === AST_NODE_TYPES.JSXEmptyExpression) continue;
-			if (isEmptyStringExpression(child)) continue;
-		}
+		if (isEmptyExpressionContainer(child)) continue;
+		if (isEmptyStringExpression(child)) continue;
 		children.push(child);
 	}
 	return children;
 }
 function hasChildren(element) {
 	if (element.children.length === 0) return false;
-	return !element.children.every(or(isWhitespaceText, isEmptyStringExpression));
+	return !element.children.every(or(or(isWhitespaceText, isEmptyExpressionContainer), isEmptyStringExpression));
 }
 //#endregion
 //#region src/element-type.ts

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eslint-react/jsx",
-  "version": "5.27.2",
+  "version": "5.27.4",
   "description": "ESLint React's TSESTree JSX utility module for static analysis of JSX patterns.",
   "homepage": "https://github.com/Rel1cx/eslint-react",
   "bugs": {
@@ -29,9 +29,9 @@
     "dist"
   ],
   "dependencies": {
-    "@eslint-react/ast": "5.27.2",
-    "@eslint-react/eslint": "5.27.2",
-    "@eslint-react/var": "5.27.2",
+    "@eslint-react/ast": "5.27.4",
+    "@eslint-react/eslint": "5.27.4",
+    "@eslint-react/var": "5.27.4",
     "@typescript-eslint/types": "^8.71.2",
     "@typescript-eslint/utils": "^8.71.2",
     "ts-pattern": "^5.9.2"