eslint-plugin-react-x 2.0.0-next.47 → 2.0.0-next.48

package/dist/index.d.ts

@@ -69,12 +69,18 @@ declare const _default: {
                         readonly "no-unused-state": _typescript_eslint_utils_ts_eslint.RuleModule<"noUnusedState", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "no-use-context": _typescript_eslint_utils_ts_eslint.RuleModule<"noUseContext", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "no-useless-forward-ref": _typescript_eslint_utils_ts_eslint.RuleModule<"noUselessForwardRef", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+                        readonly "no-useless-fragment": _typescript_eslint_utils_ts_eslint.RuleModule<"uselessFragment", readonly [{
+                            allowExpressions: boolean;
+                        }], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "prefer-destructuring-assignment": _typescript_eslint_utils_ts_eslint.RuleModule<"preferDestructuringAssignment", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "prefer-namespace-import": _typescript_eslint_utils_ts_eslint.RuleModule<"preferNamespaceImport", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "prefer-read-only-props": _typescript_eslint_utils_ts_eslint.RuleModule<"preferReadOnlyProps", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "prefer-use-state-lazy-initialization": _typescript_eslint_utils_ts_eslint.RuleModule<"preferUseStateLazyInitialization", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "avoid-shorthand-boolean": _typescript_eslint_utils_ts_eslint.RuleModule<"avoidShorthandBoolean", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "avoid-shorthand-fragment": _typescript_eslint_utils_ts_eslint.RuleModule<"avoidShorthandFragment", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+                        readonly "no-comment-textnodes": _typescript_eslint_utils_ts_eslint.RuleModule<"noCommentTextnodes", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+                        readonly "no-complex-conditional-rendering": _typescript_eslint_utils_ts_eslint.RuleModule<"noComplexConditionalRendering", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+                        readonly "prefer-react-namespace-import": _typescript_eslint_utils_ts_eslint.RuleModule<"preferReactNamespaceImport", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "prefer-shorthand-boolean": _typescript_eslint_utils_ts_eslint.RuleModule<"preferShorthandBoolean", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "prefer-shorthand-fragment": _typescript_eslint_utils_ts_eslint.RuleModule<"preferShorthandFragment", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                     };
@@ -149,12 +155,18 @@ declare const _default: {
                         readonly "no-unused-state": _typescript_eslint_utils_ts_eslint.RuleModule<"noUnusedState", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "no-use-context": _typescript_eslint_utils_ts_eslint.RuleModule<"noUseContext", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "no-useless-forward-ref": _typescript_eslint_utils_ts_eslint.RuleModule<"noUselessForwardRef", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+                        readonly "no-useless-fragment": _typescript_eslint_utils_ts_eslint.RuleModule<"uselessFragment", readonly [{
+                            allowExpressions: boolean;
+                        }], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "prefer-destructuring-assignment": _typescript_eslint_utils_ts_eslint.RuleModule<"preferDestructuringAssignment", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "prefer-namespace-import": _typescript_eslint_utils_ts_eslint.RuleModule<"preferNamespaceImport", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "prefer-read-only-props": _typescript_eslint_utils_ts_eslint.RuleModule<"preferReadOnlyProps", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "prefer-use-state-lazy-initialization": _typescript_eslint_utils_ts_eslint.RuleModule<"preferUseStateLazyInitialization", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "avoid-shorthand-boolean": _typescript_eslint_utils_ts_eslint.RuleModule<"avoidShorthandBoolean", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "avoid-shorthand-fragment": _typescript_eslint_utils_ts_eslint.RuleModule<"avoidShorthandFragment", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+                        readonly "no-comment-textnodes": _typescript_eslint_utils_ts_eslint.RuleModule<"noCommentTextnodes", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+                        readonly "no-complex-conditional-rendering": _typescript_eslint_utils_ts_eslint.RuleModule<"noComplexConditionalRendering", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+                        readonly "prefer-react-namespace-import": _typescript_eslint_utils_ts_eslint.RuleModule<"preferReactNamespaceImport", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "prefer-shorthand-boolean": _typescript_eslint_utils_ts_eslint.RuleModule<"preferShorthandBoolean", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "prefer-shorthand-fragment": _typescript_eslint_utils_ts_eslint.RuleModule<"preferShorthandFragment", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                     };
@@ -229,12 +241,18 @@ declare const _default: {
                         readonly "no-unused-state": _typescript_eslint_utils_ts_eslint.RuleModule<"noUnusedState", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "no-use-context": _typescript_eslint_utils_ts_eslint.RuleModule<"noUseContext", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "no-useless-forward-ref": _typescript_eslint_utils_ts_eslint.RuleModule<"noUselessForwardRef", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+                        readonly "no-useless-fragment": _typescript_eslint_utils_ts_eslint.RuleModule<"uselessFragment", readonly [{
+                            allowExpressions: boolean;
+                        }], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "prefer-destructuring-assignment": _typescript_eslint_utils_ts_eslint.RuleModule<"preferDestructuringAssignment", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "prefer-namespace-import": _typescript_eslint_utils_ts_eslint.RuleModule<"preferNamespaceImport", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "prefer-read-only-props": _typescript_eslint_utils_ts_eslint.RuleModule<"preferReadOnlyProps", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "prefer-use-state-lazy-initialization": _typescript_eslint_utils_ts_eslint.RuleModule<"preferUseStateLazyInitialization", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "avoid-shorthand-boolean": _typescript_eslint_utils_ts_eslint.RuleModule<"avoidShorthandBoolean", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "avoid-shorthand-fragment": _typescript_eslint_utils_ts_eslint.RuleModule<"avoidShorthandFragment", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+                        readonly "no-comment-textnodes": _typescript_eslint_utils_ts_eslint.RuleModule<"noCommentTextnodes", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+                        readonly "no-complex-conditional-rendering": _typescript_eslint_utils_ts_eslint.RuleModule<"noComplexConditionalRendering", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+                        readonly "prefer-react-namespace-import": _typescript_eslint_utils_ts_eslint.RuleModule<"preferReactNamespaceImport", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "prefer-shorthand-boolean": _typescript_eslint_utils_ts_eslint.RuleModule<"preferShorthandBoolean", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "prefer-shorthand-fragment": _typescript_eslint_utils_ts_eslint.RuleModule<"preferShorthandFragment", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                     };
@@ -307,12 +325,18 @@ declare const _default: {
         readonly "no-unused-state": _typescript_eslint_utils_ts_eslint.RuleModule<"noUnusedState", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         readonly "no-use-context": _typescript_eslint_utils_ts_eslint.RuleModule<"noUseContext", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         readonly "no-useless-forward-ref": _typescript_eslint_utils_ts_eslint.RuleModule<"noUselessForwardRef", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+        readonly "no-useless-fragment": _typescript_eslint_utils_ts_eslint.RuleModule<"uselessFragment", readonly [{
+            allowExpressions: boolean;
+        }], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         readonly "prefer-destructuring-assignment": _typescript_eslint_utils_ts_eslint.RuleModule<"preferDestructuringAssignment", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         readonly "prefer-namespace-import": _typescript_eslint_utils_ts_eslint.RuleModule<"preferNamespaceImport", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         readonly "prefer-read-only-props": _typescript_eslint_utils_ts_eslint.RuleModule<"preferReadOnlyProps", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         readonly "prefer-use-state-lazy-initialization": _typescript_eslint_utils_ts_eslint.RuleModule<"preferUseStateLazyInitialization", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         readonly "avoid-shorthand-boolean": _typescript_eslint_utils_ts_eslint.RuleModule<"avoidShorthandBoolean", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         readonly "avoid-shorthand-fragment": _typescript_eslint_utils_ts_eslint.RuleModule<"avoidShorthandFragment", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+        readonly "no-comment-textnodes": _typescript_eslint_utils_ts_eslint.RuleModule<"noCommentTextnodes", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+        readonly "no-complex-conditional-rendering": _typescript_eslint_utils_ts_eslint.RuleModule<"noComplexConditionalRendering", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+        readonly "prefer-react-namespace-import": _typescript_eslint_utils_ts_eslint.RuleModule<"preferReactNamespaceImport", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         readonly "prefer-shorthand-boolean": _typescript_eslint_utils_ts_eslint.RuleModule<"preferShorthandBoolean", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         readonly "prefer-shorthand-fragment": _typescript_eslint_utils_ts_eslint.RuleModule<"preferShorthandFragment", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
     };

package/dist/index.js

@@ -125,7 +125,7 @@ var settings3 = {
 
 // package.json
 var name4 = "eslint-plugin-react-x";
-var version = "2.0.0-next.47";
+var version = "2.0.0-next.48";
 var createRule = ESLintUtils.RuleCreator(getDocsUrl("x"));
 
 // src/rules/jsx-key-before-spread.ts
@@ -3527,7 +3527,157 @@ function create54(context) {
     }
   };
 }
-var RULE_NAME55 = "prefer-destructuring-assignment";
+var RULE_NAME55 = "no-useless-fragment";
+var defaultOptions3 = [{
+  allowExpressions: true
+}];
+var no_useless_fragment_default = createRule({
+  meta: {
+    type: "problem",
+    defaultOptions: [...defaultOptions3],
+    docs: {
+      description: "Disallow useless fragment elements."
+    },
+    fixable: "code",
+    messages: {
+      uselessFragment: "A fragment {{reason}} is useless."
+    },
+    schema: [{
+      type: "object",
+      additionalProperties: false,
+      properties: {
+        allowExpressions: {
+          type: "boolean",
+          description: "Allow fragments with a single expression child"
+        }
+      }
+    }]
+  },
+  name: RULE_NAME55,
+  create: create55,
+  defaultOptions: defaultOptions3
+});
+function create55(context, [option]) {
+  const { allowExpressions = true } = option;
+  return {
+    JSXElement(node) {
+      if (!ER26.isFragmentElement(context, node)) return;
+      checkNode(context, node, allowExpressions);
+    },
+    JSXFragment(node) {
+      checkNode(context, node, allowExpressions);
+    }
+  };
+}
+function isWhiteSpace(node) {
+  return typeof node.value === "string" && node.raw.trim() === "";
+}
+function isPaddingSpaces(node) {
+  return ER26.isJsxText(node) && isWhiteSpace(node) && node.raw.includes("\n");
+}
+function trimLikeReact(text) {
+  const leadingSpaces = /^\s*/.exec(text)?.[0] ?? "";
+  const trailingSpaces = /\s*$/.exec(text)?.[0] ?? "";
+  const start = leadingSpaces.includes("\n") ? leadingSpaces.length : 0;
+  const end = trailingSpaces.includes("\n") ? text.length - trailingSpaces.length : text.length;
+  return text.slice(start, end);
+}
+function checkNode(context, node, allowExpressions) {
+  const initialScope = context.sourceCode.getScope(node);
+  if (ER26.isKeyedElement(context, node, initialScope)) {
+    return;
+  }
+  if (ER26.isHostElement(context, node.parent)) {
+    context.report({
+      messageId: "uselessFragment",
+      node,
+      data: {
+        reason: "placed inside a host component"
+      },
+      fix: getFix2(context, node)
+    });
+  }
+  if (node.children.length === 0) {
+    context.report({
+      messageId: "uselessFragment",
+      node,
+      data: {
+        reason: "contains less than two children"
+      },
+      fix: getFix2(context, node)
+    });
+    return;
+  }
+  const isChildElement = AST13.isOneOf([AST_NODE_TYPES.JSXElement, AST_NODE_TYPES.JSXFragment])(node.parent);
+  switch (true) {
+    // <Foo content={<>ee eeee eeee ...</>} />
+    case (allowExpressions && !isChildElement && node.children.length === 1 && ER26.isJsxText(node.children.at(0))): {
+      return;
+    }
+    // <Foo><>hello, world</></Foo>
+    case (!allowExpressions && isChildElement): {
+      context.report({
+        messageId: "uselessFragment",
+        node,
+        data: {
+          reason: "contains less than two children"
+        },
+        fix: getFix2(context, node)
+      });
+      return;
+    }
+    case (!allowExpressions && !isChildElement && node.children.length === 1): {
+      context.report({
+        messageId: "uselessFragment",
+        node,
+        data: {
+          reason: "contains less than two children"
+        },
+        fix: getFix2(context, node)
+      });
+      return;
+    }
+  }
+  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: getFix2(context, node)
+      });
+      return;
+    }
+  }
+  return;
+}
+function getFix2(context, node) {
+  if (!canFix(context, node)) return null;
+  return (fixer) => {
+    const opener = node.type === AST_NODE_TYPES.JSXFragment ? node.openingFragment : node.openingElement;
+    const closer = node.type === AST_NODE_TYPES.JSXFragment ? node.closingFragment : node.closingElement;
+    const childrenText = opener.type === AST_NODE_TYPES.JSXOpeningElement && opener.selfClosing ? "" : context.sourceCode.getText().slice(opener.range[1], closer?.range[0]);
+    return fixer.replaceText(node, trimLikeReact(childrenText));
+  };
+}
+function canFix(context, node) {
+  if (node.parent.type === AST_NODE_TYPES.JSXElement || node.parent.type === AST_NODE_TYPES.JSXFragment) {
+    return ER26.isHostElement(context, node.parent);
+  }
+  if (node.children.length === 0) {
+    return false;
+  }
+  if (node.children.some((child) => ER26.isJsxText(child) && !isWhiteSpace(child) || AST13.is(AST_NODE_TYPES.JSXExpressionContainer)(child))) {
+    return false;
+  }
+  return true;
+}
+var RULE_NAME56 = "prefer-destructuring-assignment";
 var RULE_FEATURES55 = [];
 function isMemberExpressionWithObjectName(node) {
   return node.object.type === AST_NODE_TYPES.Identifier && "name" in node.object;
@@ -3544,11 +3694,11 @@ var prefer_destructuring_assignment_default = createRule({
     },
     schema: []
   },
-  name: RULE_NAME55,
-  create: create55,
+  name: RULE_NAME56,
+  create: create56,
   defaultOptions: []
 });
-function create55(context) {
+function create56(context) {
   const { ctx, listeners } = ER26.useComponentCollector(context);
   const memberExpressionWithNames = [];
   return {
@@ -3608,7 +3758,7 @@ function create55(context) {
     }
   };
 }
-var RULE_NAME56 = "prefer-namespace-import";
+var RULE_NAME57 = "prefer-namespace-import";
 var RULE_FEATURES56 = [
   "FIX"
 ];
@@ -3625,11 +3775,11 @@ var prefer_namespace_import_default = createRule({
     },
     schema: []
   },
-  name: RULE_NAME56,
-  create: create56,
+  name: RULE_NAME57,
+  create: create57,
   defaultOptions: []
 });
-function create56(context) {
+function create57(context) {
   const { importSource } = getSettingsFromContext(context);
   return {
     [`ImportDeclaration[source.value="${importSource}"] ImportDefaultSpecifier`](node) {
@@ -3664,7 +3814,7 @@ function create56(context) {
     }
   };
 }
-var RULE_NAME57 = "prefer-read-only-props";
+var RULE_NAME58 = "prefer-read-only-props";
 var RULE_FEATURES57 = [
   "TSC",
   "EXP"
@@ -3681,11 +3831,11 @@ var prefer_read_only_props_default = createRule({
     },
     schema: []
   },
-  name: RULE_NAME57,
-  create: create57,
+  name: RULE_NAME58,
+  create: create58,
   defaultOptions: []
 });
-function create57(context) {
+function create58(context) {
   const services = ESLintUtils.getParserServices(context, false);
   const { ctx, listeners } = ER26.useComponentCollector(context);
   return {
@@ -3717,7 +3867,7 @@ function isTypeReadonlyLoose(services, type) {
     return true;
   }
 }
-var RULE_NAME58 = "prefer-use-state-lazy-initialization";
+var RULE_NAME59 = "prefer-use-state-lazy-initialization";
 var RULE_FEATURES58 = [
   "EXP"
 ];
@@ -3738,11 +3888,11 @@ var prefer_use_state_lazy_initialization_default = createRule({
     },
     schema: []
   },
-  name: RULE_NAME58,
-  create: create58,
+  name: RULE_NAME59,
+  create: create59,
   defaultOptions: []
 });
-function create58(context) {
+function create59(context) {
   const alias = getSettingsFromContext(context).additionalHooks.useState ?? [];
   const isUseStateCall = ER26.isReactHookCallWithNameAlias(context, "useState", alias);
   return {
@@ -3779,11 +3929,12 @@ function create58(context) {
     }
   };
 }
-var RULE_NAME59 = "avoid-shorthand-boolean";
+var RULE_NAME60 = "avoid-shorthand-boolean";
 var RULE_FEATURES59 = [];
 var avoid_shorthand_boolean_default = createRule({
   meta: {
     type: "problem",
+    deprecated: true,
     docs: {
       description: "Enforces explicit boolean values for boolean attributes.",
       [Symbol.for("rule_features")]: RULE_FEATURES59
@@ -3792,13 +3943,16 @@ var avoid_shorthand_boolean_default = createRule({
     messages: {
       avoidShorthandBoolean: "Avoid using shorthand boolean attribute '{{propName}}'. Use '{{propName}}={true}' instead."
     },
+    replacedBy: [
+      "react-x/jsx-shorthand-boolean"
+    ],
     schema: []
   },
-  name: RULE_NAME59,
-  create: create59,
+  name: RULE_NAME60,
+  create: create60,
   defaultOptions: []
 });
-function create59(context) {
+function create60(context) {
   return {
     JSXAttribute(node) {
       if (node.value == null) {
@@ -3814,11 +3968,12 @@ function create59(context) {
     }
   };
 }
-var RULE_NAME60 = "avoid-shorthand-fragment";
+var RULE_NAME61 = "avoid-shorthand-fragment";
 var RULE_FEATURES60 = [];
 var avoid_shorthand_fragment_default = createRule({
   meta: {
     type: "problem",
+    deprecated: true,
     docs: {
       description: "Enforces explicit `<Fragment>` components instead of the shorthand `<>` or `</>` syntax.",
       [Symbol.for("rule_features")]: RULE_FEATURES60
@@ -3826,13 +3981,16 @@ var avoid_shorthand_fragment_default = createRule({
     messages: {
       avoidShorthandFragment: "Avoid using shorthand fragment syntax. Use '{{jsxFragmentFactory}}' component instead."
     },
+    replacedBy: [
+      "react-x/jsx-shorthand-fragment"
+    ],
     schema: []
   },
-  name: RULE_NAME60,
-  create: create60,
+  name: RULE_NAME61,
+  create: create61,
   defaultOptions: []
 });
-function create60(context) {
+function create61(context) {
   const jsxConfigFromContext = JsxConfig.getFromContext(context);
   const jsxConfigFromAnnotation = JsxConfig.getFromAnnotation(context);
   const jsxConfig = {
@@ -3851,13 +4009,14 @@ function create60(context) {
     }
   };
 }
-var RULE_NAME61 = "prefer-shorthand-boolean";
+var RULE_NAME62 = "prefer-shorthand-boolean";
 var RULE_FEATURES61 = [
   "FIX"
 ];
 var prefer_shorthand_boolean_default = createRule({
   meta: {
     type: "problem",
+    deprecated: true,
     docs: {
       description: "Enforces shorthand syntax for boolean attributes.",
       [Symbol.for("rule_features")]: RULE_FEATURES61
@@ -3866,13 +4025,16 @@ var prefer_shorthand_boolean_default = createRule({
     messages: {
       preferShorthandBoolean: "Use shorthand boolean attribute '{{propName}}'."
     },
+    replacedBy: [
+      "react-x/jsx-shorthand-boolean"
+    ],
     schema: []
   },
-  name: RULE_NAME61,
-  create: create61,
+  name: RULE_NAME62,
+  create: create62,
   defaultOptions: []
 });
-function create61(context) {
+function create62(context) {
   return {
     JSXAttribute(node) {
       const { value } = node;
@@ -3892,13 +4054,14 @@ function create61(context) {
     }
   };
 }
-var RULE_NAME62 = "prefer-shorthand-fragment";
+var RULE_NAME63 = "prefer-shorthand-fragment";
 var RULE_FEATURES62 = [
   "FIX"
 ];
 var prefer_shorthand_fragment_default = createRule({
   meta: {
     type: "problem",
+    deprecated: true,
     docs: {
       description: "Enforces shorthand syntax for fragments.",
       [Symbol.for("rule_features")]: RULE_FEATURES62
@@ -3907,13 +4070,16 @@ var prefer_shorthand_fragment_default = createRule({
     messages: {
       preferShorthandFragment: "Use fragment shorthand syntax instead of 'Fragment' component."
     },
+    replacedBy: [
+      "react-x/jsx-shorthand-fragment"
+    ],
     schema: []
   },
-  name: RULE_NAME62,
-  create: create62,
+  name: RULE_NAME63,
+  create: create63,
   defaultOptions: []
 });
-function create62(context) {
+function create63(context) {
   return {
     JSXElement(node) {
       if (!ER26.isFragmentElement(context, node)) return;
@@ -3938,6 +4104,162 @@ function create62(context) {
     }
   };
 }
+var RULE_NAME64 = "prefer-react-namespace-import";
+var RULE_FEATURES63 = [
+  "FIX"
+];
+var prefer_react_namespace_import_default = createRule({
+  meta: {
+    type: "problem",
+    deprecated: true,
+    docs: {
+      description: "Enforces React is imported via a namespace import.",
+      [Symbol.for("rule_features")]: RULE_FEATURES63
+    },
+    fixable: "code",
+    messages: {
+      preferReactNamespaceImport: `Prefer importing React as 'import * as React from "{{importSource}}"';`
+    },
+    replacedBy: [
+      "react-x/prefer-namespace-import"
+    ],
+    schema: []
+  },
+  name: RULE_NAME64,
+  create: create64,
+  defaultOptions: []
+});
+function create64(context) {
+  const { importSource } = getSettingsFromContext(context);
+  return {
+    [`ImportDeclaration[source.value="${importSource}"] ImportDefaultSpecifier`](node) {
+      const hasOtherSpecifiers = node.parent.specifiers.length > 1;
+      context.report({
+        messageId: "preferReactNamespaceImport",
+        node: hasOtherSpecifiers ? node : node.parent,
+        data: { importSource },
+        fix(fixer) {
+          const importDeclarationText = context.sourceCode.getText(node.parent);
+          const semi = importDeclarationText.endsWith(";") ? ";" : "";
+          const quote = node.parent.source.raw.at(0) ?? "'";
+          const isTypeImport = node.parent.importKind === "type";
+          const importStringPrefix = `import${isTypeImport ? " type" : ""}`;
+          const importSourceQuoted = `${quote}${importSource}${quote}`;
+          if (!hasOtherSpecifiers) {
+            return fixer.replaceText(
+              node.parent,
+              `${importStringPrefix} * as ${node.local.name} from ${importSourceQuoted}${semi}`
+            );
+          }
+          const specifiers = importDeclarationText.slice(importDeclarationText.indexOf("{"), importDeclarationText.indexOf("}") + 1);
+          return fixer.replaceText(
+            node.parent,
+            [
+              `${importStringPrefix} * as ${node.local.name} from ${importSourceQuoted}${semi}`,
+              `${importStringPrefix} ${specifiers} from ${importSourceQuoted}${semi}`
+            ].join("\n")
+          );
+        }
+      });
+    }
+  };
+}
+var RULE_NAME65 = "no-comment-textnodes";
+var RULE_FEATURES64 = [];
+var no_comment_textnodes_default = createRule({
+  meta: {
+    type: "problem",
+    deprecated: true,
+    docs: {
+      description: "Prevents comments from being inserted as text nodes.",
+      [Symbol.for("rule_features")]: RULE_FEATURES64
+    },
+    messages: {
+      noCommentTextnodes: "Possible misused comment in text node. Comments inside children section of tag should be placed inside braces."
+    },
+    replacedBy: [
+      "react-x/jsx-no-comment-textnodes"
+    ],
+    schema: []
+  },
+  name: RULE_NAME65,
+  create: create65,
+  defaultOptions: []
+});
+function create65(context) {
+  function hasCommentLike(node) {
+    if (AST13.isOneOf([AST_NODE_TYPES.JSXAttribute, AST_NODE_TYPES.JSXExpressionContainer])(node.parent)) {
+      return false;
+    }
+    const rawValue = context.sourceCode.getText(node);
+    return /^\s*\/(?:\/|\*)/mu.test(rawValue);
+  }
+  const visitorFunction = (node) => {
+    if (!AST13.isOneOf([AST_NODE_TYPES.JSXElement, AST_NODE_TYPES.JSXFragment])(node.parent)) {
+      return;
+    }
+    if (!hasCommentLike(node)) {
+      return;
+    }
+    if (!node.parent.type.includes("JSX")) {
+      return;
+    }
+    context.report({
+      messageId: "noCommentTextnodes",
+      node
+    });
+  };
+  return {
+    JSXText: visitorFunction,
+    Literal: visitorFunction
+  };
+}
+var RULE_NAME66 = "no-complex-conditional-rendering";
+var RULE_FEATURES65 = [
+  "EXP"
+];
+var no_complex_conditional_rendering_default = createRule({
+  meta: {
+    type: "problem",
+    deprecated: true,
+    docs: {
+      description: "Disallow complex conditional rendering in JSX expressions.",
+      [Symbol.for("rule_features")]: RULE_FEATURES65
+    },
+    messages: {
+      noComplexConditionalRendering: "Avoid complex conditional rendering. Extract the logic into separate elements or components."
+    },
+    schema: []
+  },
+  name: RULE_NAME66,
+  create: create66,
+  defaultOptions: []
+});
+function create66(context) {
+  const visitorFunction = (node) => {
+    const jsxExpContainer = node.parent?.parent;
+    if (!AST13.is(AST_NODE_TYPES.JSXExpressionContainer)(jsxExpContainer)) {
+      return;
+    }
+    if (!AST13.isOneOf([AST_NODE_TYPES.JSXElement, AST_NODE_TYPES.JSXFragment])(jsxExpContainer.parent)) {
+      return;
+    }
+    if (!jsxExpContainer.parent.children.includes(jsxExpContainer)) {
+      return;
+    }
+    context.report({
+      messageId: "noComplexConditionalRendering",
+      node: jsxExpContainer
+    });
+  };
+  return {
+    "JSXExpressionContainer > ConditionalExpression > ConditionalExpression": visitorFunction,
+    "JSXExpressionContainer > ConditionalExpression > LogicalExpression": visitorFunction,
+    "JSXExpressionContainer > LogicalExpression > ConditionalExpression": visitorFunction,
+    "JSXExpressionContainer > LogicalExpression[operator='&&'] > LogicalExpression[operator='||']": visitorFunction,
+    "JSXExpressionContainer > LogicalExpression[operator='||'] > LogicalExpression[operator='&&']": visitorFunction
+  };
+}
 
 // src/plugin.ts
 var plugin = {
@@ -4000,6 +4322,7 @@ var plugin = {
     "no-unused-state": no_unused_state_default,
     "no-use-context": no_use_context_default,
     "no-useless-forward-ref": no_useless_forward_ref_default,
+    "no-useless-fragment": no_useless_fragment_default,
     "prefer-destructuring-assignment": prefer_destructuring_assignment_default,
     "prefer-namespace-import": prefer_namespace_import_default,
     "prefer-read-only-props": prefer_read_only_props_default,
@@ -4007,6 +4330,9 @@ var plugin = {
     // Removed rules
     "avoid-shorthand-boolean": avoid_shorthand_boolean_default,
     "avoid-shorthand-fragment": avoid_shorthand_fragment_default,
+    "no-comment-textnodes": no_comment_textnodes_default,
+    "no-complex-conditional-rendering": no_complex_conditional_rendering_default,
+    "prefer-react-namespace-import": prefer_react_namespace_import_default,
     "prefer-shorthand-boolean": prefer_shorthand_boolean_default,
     "prefer-shorthand-fragment": prefer_shorthand_fragment_default
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "2.0.0-next.47",
+  "version": "2.0.0-next.48",
   "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.7.1",
-    "@eslint-react/ast": "2.0.0-next.47",
-    "@eslint-react/core": "2.0.0-next.47",
-    "@eslint-react/eff": "2.0.0-next.47",
-    "@eslint-react/kit": "2.0.0-next.47",
-    "@eslint-react/shared": "2.0.0-next.47",
-    "@eslint-react/var": "2.0.0-next.47"
+    "@eslint-react/eff": "2.0.0-next.48",
+    "@eslint-react/ast": "2.0.0-next.48",
+    "@eslint-react/core": "2.0.0-next.48",
+    "@eslint-react/var": "2.0.0-next.48",
+    "@eslint-react/shared": "2.0.0-next.48",
+    "@eslint-react/kit": "2.0.0-next.48"
   },
   "devDependencies": {
     "@types/react": "^19.1.8",