eslint-plugin-react-x 1.23.1 → 1.23.2-beta.2

package/dist/index.js

@@ -42,7 +42,7 @@ var ts__default = /*#__PURE__*/_interopDefault(ts);
 
 // package.json
 var name = "eslint-plugin-react-x";
-var version = "1.23.1";
+var version = "1.23.2-beta.2";
 var createRule = shared.createRuleForPlugin("x");
 
 // src/rules/avoid-shorthand-boolean.ts
@@ -906,7 +906,7 @@ var no_context_provider_default = createRule({
   name: RULE_NAME19,
   create(context) {
     if (!context.sourceCode.text.includes(".Provider")) return {};
-    const { version: version2 } = shared.normalizeSettings(shared.decodeSettings(context.settings));
+    const { version: version2 } = shared.getSettingsFromContext(context);
     if (compareVersions.compare(version2, "19.0.0", "<")) return {};
     return {
       JSXElement(node) {
@@ -1202,7 +1202,7 @@ var no_forward_ref_default = createRule({
   name: RULE_NAME25,
   create(context) {
     if (!context.sourceCode.text.includes("forwardRef")) return {};
-    const { version: version2 } = shared.normalizeSettings(shared.decodeSettings(context.settings));
+    const { version: version2 } = shared.getSettingsFromContext(context);
     if (compareVersions.compare(version2, "19.0.0", "<")) return {};
     return {
       CallExpression(node) {
@@ -1425,7 +1425,7 @@ var no_leaked_conditional_rendering_default = createRule({
   name: RULE_NAME27,
   create(context) {
     if (!context.sourceCode.text.includes("&&") && !context.sourceCode.text.includes("?")) return {};
-    const { version: version2 } = shared.normalizeSettings(shared.decodeSettings(context.settings));
+    const { version: version2 } = shared.getSettingsFromContext(context);
     const allowedVariants = [
       "any",
       "boolean",
@@ -2562,7 +2562,10 @@ function checkAndReport(node, context, allowExpressions) {
   const initialScope = context.sourceCode.getScope(node);
   if (JSX7__namespace.isKeyedElement(node, initialScope)) return;
   if (JSX7__namespace.isBuiltInElement(node.parent)) context.report({ messageId: "noUselessFragmentInBuiltIn", node });
-  if (node.children.length === 0) return context.report({ messageId: "noUselessFragment", node });
+  if (node.children.length === 0) {
+    context.report({ messageId: "noUselessFragment", node });
+    return;
+  }
   const isChildElement = AST11__namespace.isOneOf([types.AST_NODE_TYPES.JSXElement, types.AST_NODE_TYPES.JSXFragment])(node.parent);
   switch (true) {
     // <Foo content={<>ee eeee eeee ...</>} />
@@ -2571,10 +2574,12 @@ function checkAndReport(node, context, allowExpressions) {
     }
     // <Foo><>hello, world</></Foo>
     case (!allowExpressions && isChildElement): {
-      return context.report({ messageId: "noUselessFragment", node });
+      context.report({ messageId: "noUselessFragment", node });
+      return;
     }
     case (!allowExpressions && !isChildElement && node.children.length === 1): {
-      return context.report({ messageId: "noUselessFragment", node });
+      context.report({ messageId: "noUselessFragment", node });
+      return;
     }
   }
   const nonPaddingChildren = node.children.filter((child) => !JSX7__namespace.isPaddingSpaces(child));
@@ -2582,7 +2587,8 @@ function checkAndReport(node, context, allowExpressions) {
   switch (true) {
     case nonPaddingChildren.length === 0:
     case (nonPaddingChildren.length === 1 && firstNonPaddingChild?.type !== types.AST_NODE_TYPES.JSXExpressionContainer): {
-      return context.report({ messageId: "noUselessFragment", node });
+      context.report({ messageId: "noUselessFragment", node });
+      return;
     }
   }
   return;
@@ -2718,7 +2724,7 @@ var prefer_react_namespace_import_default = createRule({
   },
   name: RULE_NAME46,
   create(context) {
-    const importSource = shared.decodeSettings(context.settings).importSource ?? "react";
+    const importSource = shared.getSettingsFromContext(context).importSource ?? "react";
     return {
       [`ImportDeclaration[source.value="${importSource}"] ImportDefaultSpecifier`](node) {
         const hasOtherSpecifiers = node.parent.specifiers.length > 1;
@@ -2839,7 +2845,8 @@ var prefer_shorthand_boolean_default = createRule({
 });
 var RULE_NAME49 = "prefer-shorthand-fragment";
 var RULE_FEATURES48 = [
-  "CHK"
+  "CHK",
+  "FIX"
 ];
 var prefer_shorthand_fragment_default = createRule({
   meta: {
@@ -2848,6 +2855,7 @@ var prefer_shorthand_fragment_default = createRule({
       description: "enforce using fragment syntax instead of 'Fragment' component",
       [Symbol.for("rule_features")]: RULE_FEATURES48
     },
+    fixable: "code",
     messages: {
       preferShorthandFragment: "Use fragment shorthand syntax instead of 'Fragment' component."
     },
@@ -2861,7 +2869,15 @@ var prefer_shorthand_fragment_default = createRule({
       if (hasAttributes) return eff.O.none();
       return eff.O.some({
         messageId: "preferShorthandFragment",
-        node
+        node,
+        fix: (fixer) => {
+          const { closingElement, openingElement } = node;
+          if (!closingElement) return [];
+          return [
+            fixer.replaceTextRange([openingElement.range[0], openingElement.range[1]], "<>"),
+            fixer.replaceTextRange([closingElement.range[0], closingElement.range[1]], "</>")
+          ];
+        }
       });
     }
     return {

package/dist/index.mjs

@@ -1,6 +1,6 @@
 import { F, O, or, isString, isNullable } from '@eslint-react/eff';
 import * as JSX7 from '@eslint-react/jsx';
-import { createRuleForPlugin, normalizeSettings, decodeSettings, unsafeReadSettings } from '@eslint-react/shared';
+import { createRuleForPlugin, getSettingsFromContext, unsafeReadSettings } from '@eslint-react/shared';
 import * as AST11 from '@eslint-react/ast';
 import { isForwardRefCall, isThisSetState, isClassComponent, isCreateElementCall, isCloneElementCall, isChildrenCount, isChildrenForEach, isChildrenMap, isChildrenOnly, isChildrenToArray, useComponentCollectorLegacy, isCreateRefCall, isComponentName, useComponentCollector, ERFunctionComponentFlag, isChildrenToArrayCall, ERComponentHint, isDirectValueOfRenderPropertyLoose, isDeclaredInRenderPropLoose, isInsideCreateElementProps, isInsideRenderMethod, ERClassComponentFlag, isReactHookCall, isInitializedFromReact } from '@eslint-react/core';
 import { AST_NODE_TYPES } from '@typescript-eslint/types';
@@ -15,7 +15,7 @@ import { getTypeImmutability, isUnknown, isImmutable, isReadonlyShallow, isReado
 
 // package.json
 var name = "eslint-plugin-react-x";
-var version = "1.23.1";
+var version = "1.23.2-beta.2";
 var createRule = createRuleForPlugin("x");
 
 // src/rules/avoid-shorthand-boolean.ts
@@ -879,7 +879,7 @@ var no_context_provider_default = createRule({
   name: RULE_NAME19,
   create(context) {
     if (!context.sourceCode.text.includes(".Provider")) return {};
-    const { version: version2 } = normalizeSettings(decodeSettings(context.settings));
+    const { version: version2 } = getSettingsFromContext(context);
     if (compare(version2, "19.0.0", "<")) return {};
     return {
       JSXElement(node) {
@@ -1175,7 +1175,7 @@ var no_forward_ref_default = createRule({
   name: RULE_NAME25,
   create(context) {
     if (!context.sourceCode.text.includes("forwardRef")) return {};
-    const { version: version2 } = normalizeSettings(decodeSettings(context.settings));
+    const { version: version2 } = getSettingsFromContext(context);
     if (compare(version2, "19.0.0", "<")) return {};
     return {
       CallExpression(node) {
@@ -1398,7 +1398,7 @@ var no_leaked_conditional_rendering_default = createRule({
   name: RULE_NAME27,
   create(context) {
     if (!context.sourceCode.text.includes("&&") && !context.sourceCode.text.includes("?")) return {};
-    const { version: version2 } = normalizeSettings(decodeSettings(context.settings));
+    const { version: version2 } = getSettingsFromContext(context);
     const allowedVariants = [
       "any",
       "boolean",
@@ -2535,7 +2535,10 @@ function checkAndReport(node, context, allowExpressions) {
   const initialScope = context.sourceCode.getScope(node);
   if (JSX7.isKeyedElement(node, initialScope)) return;
   if (JSX7.isBuiltInElement(node.parent)) context.report({ messageId: "noUselessFragmentInBuiltIn", node });
-  if (node.children.length === 0) return context.report({ messageId: "noUselessFragment", node });
+  if (node.children.length === 0) {
+    context.report({ messageId: "noUselessFragment", node });
+    return;
+  }
   const isChildElement = AST11.isOneOf([AST_NODE_TYPES.JSXElement, AST_NODE_TYPES.JSXFragment])(node.parent);
   switch (true) {
     // <Foo content={<>ee eeee eeee ...</>} />
@@ -2544,10 +2547,12 @@ function checkAndReport(node, context, allowExpressions) {
     }
     // <Foo><>hello, world</></Foo>
     case (!allowExpressions && isChildElement): {
-      return context.report({ messageId: "noUselessFragment", node });
+      context.report({ messageId: "noUselessFragment", node });
+      return;
     }
     case (!allowExpressions && !isChildElement && node.children.length === 1): {
-      return context.report({ messageId: "noUselessFragment", node });
+      context.report({ messageId: "noUselessFragment", node });
+      return;
     }
   }
   const nonPaddingChildren = node.children.filter((child) => !JSX7.isPaddingSpaces(child));
@@ -2555,7 +2560,8 @@ function checkAndReport(node, context, allowExpressions) {
   switch (true) {
     case nonPaddingChildren.length === 0:
     case (nonPaddingChildren.length === 1 && firstNonPaddingChild?.type !== AST_NODE_TYPES.JSXExpressionContainer): {
-      return context.report({ messageId: "noUselessFragment", node });
+      context.report({ messageId: "noUselessFragment", node });
+      return;
     }
   }
   return;
@@ -2691,7 +2697,7 @@ var prefer_react_namespace_import_default = createRule({
   },
   name: RULE_NAME46,
   create(context) {
-    const importSource = decodeSettings(context.settings).importSource ?? "react";
+    const importSource = getSettingsFromContext(context).importSource ?? "react";
     return {
       [`ImportDeclaration[source.value="${importSource}"] ImportDefaultSpecifier`](node) {
         const hasOtherSpecifiers = node.parent.specifiers.length > 1;
@@ -2812,7 +2818,8 @@ var prefer_shorthand_boolean_default = createRule({
 });
 var RULE_NAME49 = "prefer-shorthand-fragment";
 var RULE_FEATURES48 = [
-  "CHK"
+  "CHK",
+  "FIX"
 ];
 var prefer_shorthand_fragment_default = createRule({
   meta: {
@@ -2821,6 +2828,7 @@ var prefer_shorthand_fragment_default = createRule({
       description: "enforce using fragment syntax instead of 'Fragment' component",
       [Symbol.for("rule_features")]: RULE_FEATURES48
     },
+    fixable: "code",
     messages: {
       preferShorthandFragment: "Use fragment shorthand syntax instead of 'Fragment' component."
     },
@@ -2834,7 +2842,15 @@ var prefer_shorthand_fragment_default = createRule({
       if (hasAttributes) return O.none();
       return O.some({
         messageId: "preferShorthandFragment",
-        node
+        node,
+        fix: (fixer) => {
+          const { closingElement, openingElement } = node;
+          if (!closingElement) return [];
+          return [
+            fixer.replaceTextRange([openingElement.range[0], openingElement.range[1]], "<>"),
+            fixer.replaceTextRange([closingElement.range[0], closingElement.range[1]], "</>")
+          ];
+        }
       });
     }
     return {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "1.23.1",
+  "version": "1.23.2-beta.2",
   "description": "A set of composable ESLint rules for libraries and frameworks that use React as a UI runtime.",
   "keywords": [
     "react",
@@ -49,13 +49,13 @@
     "string-ts": "^2.2.0",
     "ts-api-utils": "^2.0.0",
     "ts-pattern": "^5.6.0",
-    "@eslint-react/ast": "1.23.1",
-    "@eslint-react/core": "1.23.1",
-    "@eslint-react/eff": "1.23.1",
-    "@eslint-react/shared": "1.23.1",
-    "@eslint-react/jsx": "1.23.1",
-    "@eslint-react/types": "1.23.1",
-    "@eslint-react/var": "1.23.1"
+    "@eslint-react/ast": "1.23.2-beta.2",
+    "@eslint-react/core": "1.23.2-beta.2",
+    "@eslint-react/jsx": "1.23.2-beta.2",
+    "@eslint-react/eff": "1.23.2-beta.2",
+    "@eslint-react/types": "1.23.2-beta.2",
+    "@eslint-react/shared": "1.23.2-beta.2",
+    "@eslint-react/var": "1.23.2-beta.2"
   },
   "devDependencies": {
     "@types/react": "^19.0.2",