eslint-plugin-react-jsx 5.16.0 → 5.17.0

package/dist/index.js

@@ -1,10 +1,10 @@
 import { DEFAULT_ESLINT_REACT_SETTINGS } from "@eslint-react/shared";
 import { ESLintUtils } from "@typescript-eslint/utils";
+import { Check, Extract } from "@eslint-react/ast";
 import * as core from "@eslint-react/core";
-import "@eslint-react/eslint";
-import { collapseMultilineText, findAttribute, getChildren, getElementFullType, hasAnyAttribute, hasChildren, isFragmentElement, isHostElement, isWhitespaceText } from "@eslint-react/jsx";
 import { AST_NODE_TYPES } from "@typescript-eslint/types";
-import { Check, Extract } from "@eslint-react/ast";
+import "@eslint-react/eslint";
+import { collapseMultilineText, findAttribute, getAttributeName, getChildren, getElementFullType, hasAnyAttribute, hasChildren, isEmptyStringExpression, isFragmentElement, isHostElement, isWhitespaceText } from "@eslint-react/jsx";
 import ts from "typescript";
 
 //#region \0rolldown/runtime.js
@@ -26,7 +26,7 @@ var __exportAll = (all, no_symbols) => {
 //#endregion
 //#region package.json
 var name$2 = "eslint-plugin-react-jsx";
-var version = "5.16.0";
+var version = "5.17.0";
 
 //#endregion
 //#region src/utils/create-rule.ts
@@ -36,46 +36,36 @@ function getDocsUrl(ruleName) {
 const createRule = ESLintUtils.RuleCreator(getDocsUrl);
 
 //#endregion
-//#region src/rules/no-children-prop-with-children/lib.ts
+//#region src/utils/find-create-element-children-prop.ts
 /**
-* Find a `children` property inside an ObjectExpression.
-* @param node The object expression.
-* @returns The Property node whose key is "children", or null.
+* Finds the statically named `children` property in the props object of a `createElement` call.
+* @param context The rule context
+* @param node The `CallExpression` node to inspect
+* @returns The `children` property node, or `null` when the call has no static `children` prop
 */
-function findChildrenProperty$1(node) {
-	for (const prop of node.properties) {
-		if (prop.type !== AST_NODE_TYPES.Property) continue;
-		const key = Extract.unwrap(prop.key);
-		if (key.type === AST_NODE_TYPES.Identifier && key.name === "children") return prop;
-		if (key.type === AST_NODE_TYPES.Literal && key.value === "children") return prop;
-	}
+function findCreateElementChildrenProp(context, node) {
+	if (!core.isCreateElementCall(context, node)) return null;
+	const propsArg = node.arguments[1];
+	if (propsArg == null) return null;
+	const propsObject = Extract.unwrap(propsArg);
+	if (propsObject.type !== AST_NODE_TYPES.ObjectExpression) return null;
+	for (const prop of propsObject.properties) if (prop.type === AST_NODE_TYPES.Property && Extract.getStaticPropertyName(prop) === "children") return prop;
 	return null;
 }
+
+//#endregion
+//#region src/utils/remove-jsx-attribute.ts
 /**
-* Compute the removal range for a JSX attribute, consuming any leading
-* whitespace (spaces, tabs, newlines) so the resulting markup stays clean.
-* @param context The rule context.
-* @param prop The JSX attribute.
-*/
-function getPropRemovalRange$1(context, prop) {
-	const { sourceCode } = context;
-	let start = prop.range[0];
-	const end = prop.range[1];
-	while (start > 0 && /\s/.test(sourceCode.text[start - 1])) start--;
-	return [start, end];
-}
-/**
-* Compute the range covering **all** children content of a JSX element or
-* fragment (from the start of the first child to the end of the last child).
-*
-* Returns `null` when there are no children at all.
-* @param node The JSX element or fragment.
+* Builds a fix that removes a JSX attribute along with the whitespace before it.
+* @param context The rule context
+* @param fixer The rule fixer
+* @param attribute The `JSXAttribute` node to remove
+* @returns A fix that removes the attribute and its preceding whitespace
 */
-function getChildrenContentRange(node) {
-	if (node.children.length === 0) return null;
-	const first = node.children[0];
-	const last = node.children[node.children.length - 1];
-	return [first.range[0], last.range[1]];
+function removeJsxAttribute(context, fixer, attribute) {
+	let start = attribute.range[0];
+	while (start > 0 && /\s/.test(context.sourceCode.text[start - 1] ?? "")) start--;
+	return fixer.removeRange([start, attribute.range[1]]);
 }
 
 //#endregion
@@ -101,12 +91,9 @@ var no_children_prop_with_children_default = createRule({
 function create$7(context) {
 	return {
 		CallExpression(node) {
-			if (!core.isCreateElementCall(context, node)) return;
-			const [, propsArg, firstExtra] = node.arguments;
-			if (propsArg == null || propsArg.type !== AST_NODE_TYPES.ObjectExpression) return;
-			const childrenProp = findChildrenProperty$1(propsArg);
+			const childrenProp = findCreateElementChildrenProp(context, node);
 			if (childrenProp == null) return;
-			if (firstExtra == null) return;
+			if (node.arguments[2] == null) return;
 			context.report({
 				messageId: "default",
 				node: childrenProp
@@ -114,8 +101,7 @@ function create$7(context) {
 		},
 		JSXElement(node) {
 			const childrenProp = findAttribute(context, node, "children");
-			if (childrenProp == null) return;
-			if (!hasChildren(node)) return;
+			if (childrenProp == null || !hasChildren(node)) return;
 			if (childrenProp.type !== AST_NODE_TYPES.JSXAttribute) {
 				context.report({
 					messageId: "default",
@@ -127,16 +113,14 @@ function create$7(context) {
 				messageId: "default",
 				node: childrenProp,
 				suggest: [{
-					fix(fixer) {
-						const [start, end] = getPropRemovalRange$1(context, childrenProp);
-						return fixer.removeRange([start, end]);
-					},
+					fix: (fixer) => removeJsxAttribute(context, fixer, childrenProp),
 					messageId: "removeChildrenProp"
 				}, {
 					fix(fixer) {
-						const range = getChildrenContentRange(node);
-						if (range == null) return [];
-						return fixer.removeRange(range);
+						const first = node.children.at(0);
+						const last = node.children.at(-1);
+						if (first == null || last == null) return [];
+						return fixer.removeRange([first.range[0], last.range[1]]);
 					},
 					messageId: "removeChildrenContent"
 				}]
@@ -145,61 +129,6 @@ function create$7(context) {
 	};
 }
 
-//#endregion
-//#region src/rules/no-children-prop/lib.ts
-/**
-* Find a `children` property inside an ObjectExpression.
-* @param node The object expression.
-* @returns The Property node whose key is "children", or null.
-*/
-function findChildrenProperty(node) {
-	for (const prop of node.properties) {
-		if (prop.type !== AST_NODE_TYPES.Property) continue;
-		const key = Extract.unwrap(prop.key);
-		if (key.type === AST_NODE_TYPES.Identifier && key.name === "children") return prop;
-		if (key.type === AST_NODE_TYPES.Literal && key.value === "children") return prop;
-	}
-	return null;
-}
-/**
-* Compute the removal range for a JSX attribute, consuming any leading
-* whitespace (spaces, tabs, newlines) so the resulting markup stays clean.
-* @param context The rule context.
-* @param prop The JSX attribute.
-*/
-function getPropRemovalRange(context, prop) {
-	const { sourceCode } = context;
-	let start = prop.range[0];
-	const end = prop.range[1];
-	while (start > 0 && /\s/.test(sourceCode.text[start - 1])) start--;
-	return [start, end];
-}
-/**
-* Extract the text to use as JSX children content from a `children` prop.
-*
-* - `children="text"`        -> `text`  (raw string, no quotes)
-* - `children={<div />}`     -> `<div />`  (JSX element, no braces)
-* - `children={<>…</>}`      -> `<>…</>`  (JSX fragment, no braces)
-* - `children={expression}`  -> `{expression}`  (wrapped in braces)
-* - `children`               -> `null`  (boolean shorthand, cannot extract)
-* @param context The rule context.
-* @param prop The JSX attribute.
-*/
-function getChildrenPropText(context, prop) {
-	const { sourceCode } = context;
-	const { value } = prop;
-	if (value == null) return null;
-	if (value.type === AST_NODE_TYPES.Literal) return String(value.value);
-	if (value.type === AST_NODE_TYPES.JSXExpressionContainer) {
-		const { expression } = value;
-		if (expression.type === AST_NODE_TYPES.JSXEmptyExpression) return null;
-		const exprText = sourceCode.getText(expression);
-		if (Check.isJSXElementOrFragment(expression)) return exprText;
-		return `{${exprText}}`;
-	}
-	return null;
-}
-
 //#endregion
 //#region src/rules/no-children-prop/no-children-prop.ts
 const RULE_NAME$6 = "no-children-prop";
@@ -222,10 +151,7 @@ var no_children_prop_default = createRule({
 function create$6(context) {
 	return {
 		CallExpression(node) {
-			if (!core.isCreateElementCall(context, node)) return;
-			const [, propsArg] = node.arguments;
-			if (propsArg == null || propsArg.type !== AST_NODE_TYPES.ObjectExpression) return;
-			const childrenProp = findChildrenProperty(propsArg);
+			const childrenProp = findCreateElementChildrenProp(context, node);
 			if (childrenProp == null) return;
 			context.report({
 				messageId: "default",
@@ -242,7 +168,7 @@ function create$6(context) {
 				});
 				return;
 			}
-			const childrenText = getChildrenPropText(context, childrenProp);
+			const childrenText = getChildrenText(context, childrenProp);
 			if (childrenText == null) {
 				context.report({
 					messageId: "default",
@@ -254,27 +180,60 @@ function create$6(context) {
 				messageId: "default",
 				node: childrenProp,
 				suggest: [{
-					fix(fixer) {
-						const sourceCode = context.sourceCode;
-						const { openingElement } = node;
-						const [removeStart, removeEnd] = getPropRemovalRange(context, childrenProp);
-						if (openingElement.selfClosing) {
-							const tagName = sourceCode.getText(openingElement.name);
-							const selfCloseOffset = sourceCode.getText(openingElement).lastIndexOf("/>");
-							let wsStart = openingElement.range[0] + selfCloseOffset;
-							while (wsStart > removeEnd && /\s/.test(sourceCode.text[wsStart - 1])) wsStart--;
-							return [fixer.removeRange([removeStart, removeEnd]), fixer.replaceTextRange([wsStart, openingElement.range[1]], `>${childrenText}</${tagName}>`)];
-						}
-						const fixes = [fixer.removeRange([removeStart, removeEnd])];
-						if (node.closingElement != null) fixes.push(fixer.insertTextBefore(node.closingElement, childrenText));
-						return fixes;
-					},
+					fix: buildFix$1(context, node, childrenProp, childrenText),
 					messageId: "moveChildrenToContent"
 				}]
 			});
 		}
 	};
 }
+/**
+* Turns the value of a 'children' JSXAttribute into text usable as element content
+* @param context The rule context
+* @param attribute The 'children' JSXAttribute node
+* @returns The text to insert as element content, or `null` when no usable value exists
+*/
+function getChildrenText(context, attribute) {
+	const { value } = attribute;
+	if (value?.type === AST_NODE_TYPES.Literal) return escapeJsxText(String(value.value));
+	if (value?.type === AST_NODE_TYPES.JSXExpressionContainer && value.expression.type !== AST_NODE_TYPES.JSXEmptyExpression) {
+		const exprText = context.sourceCode.getText(value.expression);
+		return Check.isJSXElementOrFragment(value.expression) ? exprText : `{${exprText}}`;
+	}
+	return null;
+}
+/**
+* Escapes characters that would be parsed as markup in JSX text
+* @param text The raw string value
+* @returns The escaped text safe for insertion as JSX content
+*/
+function escapeJsxText(text) {
+	return text.replaceAll("&", "&amp;").replaceAll("<", "&lt;").replaceAll(">", "&gt;").replaceAll("{", "&#123;").replaceAll("}", "&#125;");
+}
+/**
+* Builds the fix that moves the 'children' prop value into the element's content
+* @param context The rule context object
+* @param node The JSXElement node being reported
+* @param prop The 'children' JSXAttribute to remove
+* @param childrenText The text to insert as element content
+* @returns A fixer function that applies the changes
+*/
+function buildFix$1(context, node, prop, childrenText) {
+	return (fixer) => {
+		const sourceCode = context.sourceCode;
+		const { openingElement } = node;
+		const removePropFix = removeJsxAttribute(context, fixer, prop);
+		if (openingElement.selfClosing) {
+			const tagName = sourceCode.getText(openingElement.name);
+			const selfCloseOffset = sourceCode.getText(openingElement).lastIndexOf("/>");
+			let wsStart = openingElement.range[0] + selfCloseOffset;
+			while (wsStart > prop.range[1] && /\s/.test(sourceCode.text[wsStart - 1])) wsStart--;
+			return [removePropFix, fixer.replaceTextRange([wsStart, openingElement.range[1]], `>${childrenText}</${tagName}>`)];
+		}
+		if (node.closingElement == null) return [removePropFix];
+		return [removePropFix, fixer.insertTextBefore(node.closingElement, childrenText)];
+	};
+}
 
 //#endregion
 //#region src/rules/no-comment-textnodes/no-comment-textnodes.ts
@@ -291,22 +250,15 @@ var no_comment_textnodes_default = createRule({
 	defaultOptions: []
 });
 function create$5(context) {
-	function hasCommentLike(node) {
-		if (Check.isOneOf([AST_NODE_TYPES.JSXAttribute, AST_NODE_TYPES.JSXExpressionContainer])(node.parent)) return false;
-		return /^\s*\/(?:\/|\*)/mu.test(context.sourceCode.getText(node));
-	}
-	const visit = (node) => {
+	function visit(node) {
 		if (!Check.isJSXElementOrFragment(node.parent)) return;
-		if (!hasCommentLike(node)) return;
+		if (!/^\s*\/(?:\/|\*)/mu.test(context.sourceCode.getText(node))) return;
 		context.report({
 			messageId: "default",
 			node
 		});
-	};
-	return {
-		JSXText: visit,
-		Literal: visit
-	};
+	}
+	return { JSXText: visit };
 }
 
 //#endregion
@@ -316,7 +268,7 @@ var no_key_after_spread_default = createRule({
 	meta: {
 		type: "problem",
 		docs: { description: "Prevent patterns that cause deoptimization when using the automatic JSX runtime." },
-		messages: { noKeyAfterSpread: "Placing 'key' after spread props causes deoptimization when using the automatic JSX runtime. Put 'key' before any spread props." },
+		messages: { default: "Placing 'key' after spread props causes deoptimization when using the automatic JSX runtime. Put 'key' before any spread props." },
 		schema: []
 	},
 	name: RULE_NAME$4,
@@ -325,17 +277,16 @@ var no_key_after_spread_default = createRule({
 });
 function create$4(context) {
 	const { jsx } = core.getJsxConfig(context);
-	if (!(jsx === ts.JsxEmit.ReactJSX || jsx === ts.JsxEmit.ReactJSXDev)) return {};
+	if (jsx !== ts.JsxEmit.ReactJSX && jsx !== ts.JsxEmit.ReactJSXDev) return {};
 	return { JSXOpeningElement(node) {
-		let firstSpreadPropIndex = null;
-		for (const [index, prop] of node.attributes.entries()) {
+		let hasSpreadBefore = false;
+		for (const prop of node.attributes) {
 			if (prop.type === AST_NODE_TYPES.JSXSpreadAttribute) {
-				firstSpreadPropIndex ??= index;
+				hasSpreadBefore = true;
 				continue;
 			}
-			if (firstSpreadPropIndex == null) continue;
-			if (prop.name.type === AST_NODE_TYPES.JSXIdentifier && prop.name.name === "key" && index > firstSpreadPropIndex) context.report({
-				messageId: "noKeyAfterSpread",
+			if (hasSpreadBefore && getAttributeName(prop) === "key") context.report({
+				messageId: "default",
 				node: prop
 			});
 		}
@@ -362,43 +313,45 @@ var no_leaked_dollar_default = createRule({
 	defaultOptions: []
 });
 function create$3(context) {
-	/**
-	* Visitor function for JSXElement and JSXFragment nodes
-	* @param node The JSXElement or JSXFragment node to be checked
-	*/
-	const visit = (node) => {
+	if (!context.sourceCode.text.includes("$")) return {};
+	function visit(node) {
 		for (const [index, child] of node.children.entries()) {
 			if (child.type !== AST_NODE_TYPES.JSXText || !child.value.endsWith("$")) continue;
 			if (node.children[index + 1]?.type !== AST_NODE_TYPES.JSXExpressionContainer) continue;
-			if (child.value === "$" && node.children.length === 2) continue;
-			const pos = child.loc.end;
+			if (child.value.trim() === "$" && node.children.every((sibling, siblingIndex) => siblingIndex === index || siblingIndex === index + 1 || isNonSubstantiveChild(sibling))) continue;
+			if (!context.sourceCode.getText(child).endsWith("$")) continue;
+			const dollarStart = child.range[1] - 1;
+			const dollarEnd = child.range[1];
 			context.report({
 				loc: {
-					end: {
-						column: pos.column,
-						line: pos.line
-					},
-					start: {
-						column: pos.column - 1,
-						line: pos.line
-					}
+					end: context.sourceCode.getLocFromIndex(dollarEnd),
+					start: context.sourceCode.getLocFromIndex(dollarStart)
 				},
 				messageId: "default",
 				node: child,
 				suggest: [{
 					fix(fixer) {
-						return fixer.removeRange([child.range[1] - 1, child.range[1]]);
+						return fixer.removeRange([dollarStart, dollarEnd]);
 					},
 					messageId: "removeDollarSign"
 				}]
 			});
 		}
-	};
+	}
 	return {
 		JSXElement: visit,
 		JSXFragment: visit
 	};
 }
+/**
+* Whether a JSX child carries no renderable content: padding whitespace, an
+* empty expression (ex: `{}` or a comment), or an empty string expression.
+* @param child The JSX child node to check.
+*/
+function isNonSubstantiveChild(child) {
+	if (isWhitespaceText(child) || isEmptyStringExpression(child)) return true;
+	return child.type === AST_NODE_TYPES.JSXExpressionContainer && child.expression.type === AST_NODE_TYPES.JSXEmptyExpression;
+}
 
 //#endregion
 //#region src/rules/no-leaked-semicolon/no-leaked-semicolon.ts
@@ -407,6 +360,7 @@ var no_leaked_semicolon_default = createRule({
 	meta: {
 		type: "problem",
 		docs: { description: "Catches `;` at the start of JSX text nodes — typically from accidentally placing a statement-ending `;` inside JSX. The `;` \"leaks\" into the rendered output." },
+		fixable: "code",
 		hasSuggestions: true,
 		messages: {
 			default: "Leaked ';' in JSX. This ';' will be rendered as text nodes.",
@@ -418,13 +372,10 @@ var no_leaked_semicolon_default = createRule({
 	create: create$2,
 	defaultOptions: []
 });
-function hasLeakedSemicolon(text) {
-	return text.startsWith(";\n") || text.startsWith(";\r");
-}
 function create$2(context) {
-	const visit = (node) => {
+	function visit(node) {
 		if (!Check.isJSXElementOrFragment(node.parent)) return;
-		if (!hasLeakedSemicolon(context.sourceCode.getText(node))) return;
+		if (!/^;+[ \t]*(?:\r\n|\r|\n)/u.test(context.sourceCode.getText(node))) return;
 		context.report({
 			messageId: "default",
 			node,
@@ -435,11 +386,8 @@ function create$2(context) {
 				messageId: "removeSemicolon"
 			}]
 		});
-	};
-	return {
-		JSXText: visit,
-		Literal: visit
-	};
+	}
+	return { JSXText: visit };
 }
 
 //#endregion
@@ -459,7 +407,7 @@ var no_namespace_default = createRule({
 function create$1(context) {
 	return { JSXElement(node) {
 		const name = getElementFullType(node);
-		if (typeof name !== "string" || !name.includes(":")) return;
+		if (!name.includes(":")) return;
 		context.report({
 			data: { name },
 			messageId: "noNamespace",
@@ -492,7 +440,6 @@ const schema = [{
 var no_useless_fragment_default = createRule({
 	meta: {
 		type: "suggestion",
-		defaultOptions: [...defaultOptions],
 		docs: { description: "Disallows useless fragment elements." },
 		fixable: "code",
 		messages: { default: "A fragment {{reason}} is useless." },
@@ -503,89 +450,96 @@ var no_useless_fragment_default = createRule({
 	defaultOptions
 });
 function create(context, [option]) {
-	const { allowEmptyFragment = false, allowExpressions = true } = option;
+	const options = {
+		allowEmptyFragment: option.allowEmptyFragment ?? false,
+		allowExpressions: option.allowExpressions ?? true
+	};
 	const jsxConfig = core.getJsxConfig(context);
-	/**
-	* Whether the fragment has too few meaningful children to justify its
-	* existence (the "contains less than two children" reason).
-	* @param node The fragment node to check.
-	*/
-	function isContentUseless(node) {
-		if (node.children.length === 0) return !allowEmptyFragment;
-		const insideJsx = Check.isJSXElementOrFragment(node.parent);
-		if (!allowExpressions) {
-			if (insideJsx) return true;
-			if (node.children.length === 1) return true;
-		}
-		if (allowExpressions && !insideJsx && node.children.length === 1) {
-			const child = node.children[0];
-			if (child != null && child.type === AST_NODE_TYPES.JSXText) return false;
-		}
-		const meaningful = getChildren(node);
-		if (meaningful.length === 0) return true;
-		if (meaningful.length === 1 && meaningful[0].type !== AST_NODE_TYPES.JSXExpressionContainer) return true;
-		return false;
-	}
-	/**
-	* Whether it is safe to auto-fix the fragment by unwrapping it.
-	* @param node The fragment node to check.
-	*/
-	function isSafeToFix(node) {
-		if (Check.isJSXElementOrFragment(node.parent)) return isHostElement(node.parent);
-		if (node.children.length === 0) return false;
-		return !node.children.some((child) => {
-			if (child.type === AST_NODE_TYPES.JSXExpressionContainer) return true;
-			if (child.type === AST_NODE_TYPES.JSXText) return !isWhitespaceText(child);
-			return false;
-		});
-	}
-	/**
-	* Build an autofix that unwraps the fragment, replacing it with its
-	* meaningful children content.  Returns `null` when the fix is unsafe.
-	* @param node The fragment node to fix.
-	*/
-	function buildFix(node) {
-		if (!isSafeToFix(node)) return null;
-		return (fixer) => {
-			let text = "";
-			for (const child of node.children) if (child.type === AST_NODE_TYPES.JSXText) {
-				const cleaned = collapseMultilineText(child.value);
-				if (cleaned != null) text += cleaned;
-			} else text += context.sourceCode.getText(child);
-			return fixer.replaceText(node, text);
-		};
-	}
-	/**
-	* Inspect a fragment node and report if it is useless.
-	*
-	* A fragment may be reported for **two independent reasons** on the same
-	* node (e.g. `<p><>foo</></p>` is both "placed inside a host component"
-	* and* "contains less than two children").
-	* @param node The fragment node to inspect.
-	*/
-	function visit(node) {
-		if (isHostElement(node.parent)) context.report({
-			data: { reason: "placed inside a host component" },
-			fix: buildFix(node),
-			messageId: "default",
-			node
-		});
-		if (isContentUseless(node)) context.report({
-			data: { reason: "contains less than two children" },
-			fix: buildFix(node),
-			messageId: "default",
-			node
-		});
-	}
 	return {
 		JSXElement(node) {
 			if (!isFragmentElement(node, jsxConfig.jsxFragmentFactory)) return;
 			if (hasAnyAttribute(context, node, ["key", "ref"])) return;
-			visit(node);
+			visitFragment(context, node, options);
 		},
 		JSXFragment(node) {
-			visit(node);
+			visitFragment(context, node, options);
+		}
+	};
+}
+/**
+* Inspect a fragment node and report if it is useless.
+*
+* A fragment may be reported for **two independent reasons** on the same
+* node (e.g. `<p><>foo</></p>` is both "placed inside a host component"
+* and "contains less than two children").
+* @param context The rule context.
+* @param node The fragment node to inspect.
+* @param options The resolved rule options.
+*/
+function visitFragment(context, node, options) {
+	const reasons = [];
+	if (isHostElement(node.parent)) reasons.push("placed inside a host component");
+	if (isContentUseless(node, options)) reasons.push("contains less than two children");
+	if (reasons.length === 0) return;
+	const fix = buildFix(context, node);
+	for (const reason of reasons) context.report({
+		data: { reason },
+		fix,
+		messageId: "default",
+		node
+	});
+}
+/**
+* Whether the fragment has too few meaningful children to justify its
+* existence (the "contains less than two children" reason).
+* @param node The fragment node to check.
+* @param options The resolved rule options.
+*/
+function isContentUseless(node, options) {
+	const { allowEmptyFragment, allowExpressions } = options;
+	if (node.children.length === 0) return !allowEmptyFragment;
+	const insideJsx = Check.isJSXElementOrFragment(node.parent);
+	if (allowExpressions && !insideJsx && node.children.length === 1 && node.children[0]?.type === AST_NODE_TYPES.JSXText) return false;
+	const meaningful = getChildren(node);
+	if (meaningful.length === 0) return true;
+	if (meaningful.length > 1) return false;
+	if (meaningful[0]?.type === AST_NODE_TYPES.JSXExpressionContainer) return !allowExpressions;
+	return true;
+}
+/**
+* Whether it is safe to auto-fix the fragment by unwrapping it.
+* @param node The fragment node to check.
+*/
+function isSafeToFix(node) {
+	if (node.type === AST_NODE_TYPES.JSXElement && node.openingElement.attributes.some((attr) => attr.type === AST_NODE_TYPES.JSXSpreadAttribute)) return false;
+	if (Check.isJSXElementOrFragment(node.parent)) return isHostElement(node.parent);
+	if (node.children.length === 0) return false;
+	return node.children.every((child) => {
+		if (Check.isJSXElementOrFragment(child)) return true;
+		return child.type === AST_NODE_TYPES.JSXText && isWhitespaceText(child);
+	});
+}
+/**
+* Build an autofix that unwraps the fragment, replacing it with its
+* meaningful children content.  Returns `null` when the fix is unsafe.
+* @param context The rule context.
+* @param node The fragment node to fix.
+*/
+function buildFix(context, node) {
+	if (!isSafeToFix(node)) return null;
+	return (fixer) => {
+		const sourceCode = context.sourceCode;
+		let text = "";
+		for (const child of node.children) if (child.type === AST_NODE_TYPES.JSXText) {
+			const cleaned = collapseMultilineText(child.value);
+			if (cleaned != null) text += cleaned;
+		} else text += sourceCode.getText(child);
+		let start = node.range[0];
+		if (text === "" && node.children.length > 0) {
+			const lineStart = Math.max(sourceCode.text.lastIndexOf("\n", start - 1), sourceCode.text.lastIndexOf("\r", start - 1)) + 1;
+			if (/^[ \t]*$/u.test(sourceCode.text.slice(lineStart, start))) start = lineStart;
 		}
+		return fixer.replaceTextRange([start, node.range[1]], text);
 	};
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-jsx",
-  "version": "5.16.0",
+  "version": "5.17.0",
   "description": "ESLint React's ESLint plugin for React Flavored JSX rules.",
   "keywords": [
     "react",
@@ -39,11 +39,11 @@
   "dependencies": {
     "@typescript-eslint/types": "^8.64.0",
     "@typescript-eslint/utils": "^8.64.0",
-    "@eslint-react/core": "5.16.0",
-    "@eslint-react/jsx": "5.16.0",
-    "@eslint-react/ast": "5.16.0",
-    "@eslint-react/shared": "5.16.0",
-    "@eslint-react/eslint": "5.16.0"
+    "@eslint-react/ast": "5.17.0",
+    "@eslint-react/core": "5.17.0",
+    "@eslint-react/eslint": "5.17.0",
+    "@eslint-react/jsx": "5.17.0",
+    "@eslint-react/shared": "5.17.0"
   },
   "devDependencies": {
     "@types/react": "^19.2.17",