eslint-plugin-react-x 2.0.0-next.131 → 2.0.0-next.133

package/dist/index.js

@@ -133,7 +133,7 @@ var settings3 = {
 
 // package.json
 var name4 = "eslint-plugin-react-x";
-var version = "2.0.0-next.131";
+var version = "2.0.0-next.133";
 var createRule = ESLintUtils.RuleCreator(getDocsUrl("x"));
 
 // src/rules/jsx-key-before-spread.ts
@@ -4182,24 +4182,21 @@ function create63(context) {
     }
   };
 }
-var RULE_NAME64 = "prefer-shorthand-boolean";
-var RULE_FEATURES62 = [
-  "FIX"
-];
-var prefer_shorthand_boolean_default = createRule({
+var RULE_NAME64 = "no-comment-textnodes";
+var RULE_FEATURES62 = [];
+var no_comment_textnodes_default = createRule({
   meta: {
     type: "problem",
     deprecated: true,
     docs: {
-      description: "Enforces shorthand syntax for boolean attributes.",
+      description: "Prevents comments from being inserted as text nodes.",
       [Symbol.for("rule_features")]: RULE_FEATURES62
     },
-    fixable: "code",
     messages: {
-      preferShorthandBoolean: "Use shorthand boolean attribute '{{propName}}'."
+      noCommentTextnodes: "Possible misused comment in text node. Comments inside children section of tag should be placed inside braces."
     },
     replacedBy: [
-      "react-x/jsx-shorthand-boolean"
+      "react-x/jsx-no-comment-textnodes"
     ],
     schema: []
   },
@@ -4208,44 +4205,48 @@ var prefer_shorthand_boolean_default = createRule({
   defaultOptions: []
 });
 function create64(context) {
-  return {
-    JSXAttribute(node) {
-      const { value } = node;
-      const propName = ER27.getAttributeName(context, node);
-      const hasValueTrue = value?.type === AST_NODE_TYPES.JSXExpressionContainer && value.expression.type === AST_NODE_TYPES.Literal && value.expression.value === true;
-      if (!hasValueTrue) {
-        return;
-      }
-      context.report({
-        messageId: "preferShorthandBoolean",
-        node: node.value ?? node,
-        data: {
-          propName
-        },
-        fix: (fixer) => fixer.removeRange([node.name.range[1], value.range[1]])
-      });
+  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_NAME65 = "prefer-shorthand-fragment";
+var RULE_NAME65 = "no-complex-conditional-rendering";
 var RULE_FEATURES63 = [
-  "FIX"
+  "EXP"
 ];
-var prefer_shorthand_fragment_default = createRule({
+var no_complex_conditional_rendering_default = createRule({
   meta: {
     type: "problem",
     deprecated: true,
     docs: {
-      description: "Enforces shorthand syntax for fragments.",
+      description: "Disallow complex conditional rendering in JSX expressions.",
       [Symbol.for("rule_features")]: RULE_FEATURES63
     },
-    fixable: "code",
     messages: {
-      preferShorthandFragment: "Use fragment shorthand syntax instead of 'Fragment' component."
+      noComplexConditionalRendering: "Avoid complex conditional rendering. Extract the logic into separate elements or components."
     },
-    replacedBy: [
-      "react-x/jsx-shorthand-fragment"
-    ],
     schema: []
   },
   name: RULE_NAME65,
@@ -4253,28 +4254,28 @@ var prefer_shorthand_fragment_default = createRule({
   defaultOptions: []
 });
 function create65(context) {
-  return {
-    JSXElement(node) {
-      if (!ER27.isFragmentElement(context, node)) return;
-      const hasAttributes = node.openingElement.attributes.length > 0;
-      if (hasAttributes) {
-        return;
-      }
-      context.report({
-        messageId: "preferShorthandFragment",
-        node,
-        fix: (fixer) => {
-          const { closingElement, openingElement } = node;
-          if (closingElement == null) {
-            return [];
-          }
-          return [
-            fixer.replaceTextRange([openingElement.range[0], openingElement.range[1]], "<>"),
-            fixer.replaceTextRange([closingElement.range[0], closingElement.range[1]], "</>")
-          ];
-        }
-      });
+  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
   };
 }
 var RULE_NAME66 = "prefer-react-namespace-import";
@@ -4337,21 +4338,24 @@ function create66(context) {
     }
   };
 }
-var RULE_NAME67 = "no-comment-textnodes";
-var RULE_FEATURES65 = [];
-var no_comment_textnodes_default = createRule({
+var RULE_NAME67 = "prefer-shorthand-boolean";
+var RULE_FEATURES65 = [
+  "FIX"
+];
+var prefer_shorthand_boolean_default = createRule({
   meta: {
     type: "problem",
     deprecated: true,
     docs: {
-      description: "Prevents comments from being inserted as text nodes.",
+      description: "Enforces shorthand syntax for boolean attributes.",
       [Symbol.for("rule_features")]: RULE_FEATURES65
     },
+    fixable: "code",
     messages: {
-      noCommentTextnodes: "Possible misused comment in text node. Comments inside children section of tag should be placed inside braces."
+      preferShorthandBoolean: "Use shorthand boolean attribute '{{propName}}'."
     },
     replacedBy: [
-      "react-x/jsx-no-comment-textnodes"
+      "react-x/jsx-shorthand-boolean"
     ],
     schema: []
   },
@@ -4360,48 +4364,44 @@ var no_comment_textnodes_default = createRule({
   defaultOptions: []
 });
 function create67(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
+    JSXAttribute(node) {
+      const { value } = node;
+      const propName = ER27.getAttributeName(context, node);
+      const hasValueTrue = value?.type === AST_NODE_TYPES.JSXExpressionContainer && value.expression.type === AST_NODE_TYPES.Literal && value.expression.value === true;
+      if (!hasValueTrue) {
+        return;
+      }
+      context.report({
+        messageId: "preferShorthandBoolean",
+        node: node.value ?? node,
+        data: {
+          propName
+        },
+        fix: (fixer) => fixer.removeRange([node.name.range[1], value.range[1]])
+      });
+    }
   };
 }
-var RULE_NAME68 = "no-complex-conditional-rendering";
+var RULE_NAME68 = "prefer-shorthand-fragment";
 var RULE_FEATURES66 = [
-  "EXP"
+  "FIX"
 ];
-var no_complex_conditional_rendering_default = createRule({
+var prefer_shorthand_fragment_default = createRule({
   meta: {
     type: "problem",
     deprecated: true,
     docs: {
-      description: "Disallow complex conditional rendering in JSX expressions.",
+      description: "Enforces shorthand syntax for fragments.",
       [Symbol.for("rule_features")]: RULE_FEATURES66
     },
+    fixable: "code",
     messages: {
-      noComplexConditionalRendering: "Avoid complex conditional rendering. Extract the logic into separate elements or components."
+      preferShorthandFragment: "Use fragment shorthand syntax instead of 'Fragment' component."
     },
+    replacedBy: [
+      "react-x/jsx-shorthand-fragment"
+    ],
     schema: []
   },
   name: RULE_NAME68,
@@ -4409,28 +4409,28 @@ var no_complex_conditional_rendering_default = createRule({
   defaultOptions: []
 });
 function create68(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
+    JSXElement(node) {
+      if (!ER27.isFragmentElement(context, node)) return;
+      const hasAttributes = node.openingElement.attributes.length > 0;
+      if (hasAttributes) {
+        return;
+      }
+      context.report({
+        messageId: "preferShorthandFragment",
+        node,
+        fix: (fixer) => {
+          const { closingElement, openingElement } = node;
+          if (closingElement == null) {
+            return [];
+          }
+          return [
+            fixer.replaceTextRange([openingElement.range[0], openingElement.range[1]], "<>"),
+            fixer.replaceTextRange([closingElement.range[0], closingElement.range[1]], "</>")
+          ];
+        }
+      });
+    }
   };
 }
 

package/package.json

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