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

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.6";
 var createRule = shared.createRuleForPlugin("x");
 
 // src/rules/avoid-shorthand-boolean.ts
@@ -281,7 +281,7 @@ function isReactChildrenMethod(name2) {
   return reactChildrenMethod.some((method) => method === name2);
 }
 function isUsingReactChildren(node, context) {
-  const settings = shared.unsafeReadSettings(context.settings);
+  const settings = shared.unsafeDecodeSettings(context.settings);
   const { callee } = node;
   if (!("property" in callee) || !("object" in callee) || !("name" in callee.property)) {
     return false;
@@ -290,7 +290,7 @@ function isUsingReactChildren(node, context) {
   const initialScope = context.sourceCode.getScope(node);
   if (callee.object.type === types.AST_NODE_TYPES.Identifier && callee.object.name === "Children") return true;
   if (callee.object.type === types.AST_NODE_TYPES.MemberExpression && "name" in callee.object.object) {
-    return core.isInitializedFromReact(callee.object.object.name, initialScope, { ...settings, strictImportCheck: true });
+    return core.isInitializedFromReact(callee.object.object.name, initialScope, settings.importSource);
   }
   return false;
 }
@@ -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) {
@@ -952,7 +952,7 @@ var no_create_ref_default = createRule({
     return {
       CallExpression(node) {
         if (!core.isCreateRefCall(node, context)) return;
-        if (eff.O.isSome(AST11__namespace.traverseUp(node, core.isClassComponent))) return;
+        if (eff.O.isSome(AST11__namespace.findParentNode(node, core.isClassComponent))) return;
         context.report({ messageId: "noCreateRef", node });
       }
     };
@@ -1044,19 +1044,24 @@ var no_direct_mutation_state_default = createRule({
   create(context) {
     function getReportDescriptor(node) {
       if (!isAssignmentToThisState(node)) return eff.O.none();
-      const maybeParentClass = AST11__namespace.traverseUpGuard(
-        node,
-        AST11__namespace.isOneOf([types.AST_NODE_TYPES.ClassDeclaration, types.AST_NODE_TYPES.ClassExpression])
+      return eff.F.pipe(
+        eff.O.Do,
+        eff.O.bind("parentClass", () => AST11__namespace.findParentNodeGuard(
+          node,
+          AST11__namespace.isOneOf([
+            types.AST_NODE_TYPES.ClassDeclaration,
+            types.AST_NODE_TYPES.ClassExpression
+          ])
+        )),
+        eff.O.bind("parentConstructor", () => AST11__namespace.findParentNodeGuard(node, isConstructorFunction)),
+        eff.O.filter(
+          ({ parentClass, parentConstructor }) => core.isClassComponent(parentClass) && context.sourceCode.getScope(node).block !== parentConstructor
+        ),
+        eff.O.map(() => ({
+          messageId: "noDirectMutationState",
+          node
+        }))
       );
-      if (eff.O.isNone(maybeParentClass)) return eff.O.none();
-      const parentClass = maybeParentClass.value;
-      if (!core.isClassComponent(parentClass)) return eff.O.none();
-      const maybeParentConstructor = AST11__namespace.traverseUpGuard(node, isConstructorFunction);
-      if (eff.O.exists(maybeParentConstructor, (n) => context.sourceCode.getScope(node).block === n)) return eff.O.none();
-      return eff.O.some({
-        messageId: "noDirectMutationState",
-        node
-      });
     }
     return {
       AssignmentExpression: eff.F.flow(getReportDescriptor, eff.O.map(context.report))
@@ -1146,20 +1151,20 @@ var no_duplicate_key_default = createRule({
             break;
           }
           default: {
-            const entry = eff.F.pipe(
+            const mbEntry = eff.F.pipe(
               eff.O.Do,
-              eff.O.bind("call", () => AST11__namespace.traverseUpGuard(jsxElement, AST11__namespace.isMapCallLoose)),
-              eff.O.bind("iter", ({ call }) => AST11__namespace.traverseUpStop(jsxElement, call, AST11__namespace.isFunction)),
-              eff.O.bind("arg0", ({ call }) => eff.O.fromNullable(call.arguments[0]))
+              eff.O.bind("call", () => AST11__namespace.findParentNodeGuard(jsxElement, AST11__namespace.isMapCallLoose)),
+              eff.O.bind("iter", ({ call: call2 }) => AST11__namespace.findParentNodeStop(jsxElement, call2, AST11__namespace.isFunction)),
+              eff.O.bind("arg0", ({ call: call2 }) => eff.O.fromNullable(call2.arguments[0]))
             );
-            for (const { arg0, call, iter } of eff.O.toArray(entry)) {
-              if (AST11__namespace.unwrapTypeExpression(arg0) !== iter) continue;
-              keyedEntries.set(call, {
-                hasDuplicate: node.value?.type === types.AST_NODE_TYPES.Literal,
-                keys: [node],
-                root: call
-              });
-            }
+            if (eff.O.isNone(mbEntry)) return;
+            const { arg0, call, iter } = mbEntry.value;
+            if (AST11__namespace.unwrapTypeExpression(arg0) !== iter) return;
+            keyedEntries.set(call, {
+              hasDuplicate: node.value?.type === types.AST_NODE_TYPES.Literal,
+              keys: [node],
+              root: call
+            });
           }
         }
       },
@@ -1202,7 +1207,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) {
@@ -1299,9 +1304,7 @@ var no_implicit_key_default = createRule({
     function getReportDescriptor(node) {
       const initialScope = context.sourceCode.getScope(node);
       const keyPropFound = JSX7__namespace.findPropInAttributes(node.attributes, initialScope)("key");
-      const keyPropOnElement = node.attributes.some(
-        (n) => AST11__namespace.is(types.AST_NODE_TYPES.JSXAttribute)(n) && n.name.name === "key"
-      );
+      const keyPropOnElement = node.attributes.some((n) => AST11__namespace.is(types.AST_NODE_TYPES.JSXAttribute)(n) && n.name.name === "key");
       if (eff.O.isSome(keyPropFound) && !keyPropOnElement) {
         return eff.O.some({ messageId: "noImplicitKey", node: keyPropFound.value });
       }
@@ -1425,7 +1428,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",
@@ -1579,10 +1582,8 @@ var no_missing_key_default = createRule({
     function checkBlockStatement(node) {
       return AST11__namespace.getNestedReturnStatements(node).reduce((acc, statement) => {
         if (!statement.argument) return acc;
-        const maybeDescriptor = checkIteratorElement(statement.argument);
-        if (eff.O.isNone(maybeDescriptor)) return acc;
-        const descriptor = maybeDescriptor.value;
-        return [...acc, descriptor];
+        const mbDescriptor = checkIteratorElement(statement.argument);
+        return eff.O.isNone(mbDescriptor) ? acc : [...acc, mbDescriptor.value];
       }, []);
     }
     return {
@@ -1680,7 +1681,7 @@ var no_nested_components_default = createRule({
           if (eff.O.isNone(componentName)) continue;
           const name2 = componentName.value;
           const isInsideProperty = component.parent.type === types.AST_NODE_TYPES.Property;
-          const isInsideJSXPropValue = component.parent.type === types.AST_NODE_TYPES.JSXAttribute || eff.O.isSome(JSX7__namespace.traverseUpProp(node, (n) => n.value?.type === types.AST_NODE_TYPES.JSXExpressionContainer));
+          const isInsideJSXPropValue = component.parent.type === types.AST_NODE_TYPES.JSXAttribute || eff.O.isSome(JSX7__namespace.findParentProp(node, (n) => n.value?.type === types.AST_NODE_TYPES.JSXExpressionContainer));
           if (isInsideJSXPropValue) {
             if (!core.isDeclaredInRenderPropLoose(component)) {
               context.report({
@@ -1703,8 +1704,14 @@ var no_nested_components_default = createRule({
             });
             continue;
           }
-          const maybeParentComponent = AST11__namespace.traverseUpGuard(component, isFunctionComponent);
-          if (eff.O.isSome(maybeParentComponent) && !core.isDirectValueOfRenderPropertyLoose(maybeParentComponent.value)) {
+          const isParnetComponentNotDirectValueOfRenderProperty = eff.O.exists(
+            AST11__namespace.findParentNodeGuard(
+              component,
+              isFunctionComponent
+            ),
+            eff.not(core.isDirectValueOfRenderPropertyLoose)
+          );
+          if (isParnetComponentNotDirectValueOfRenderProperty) {
             context.report({
               messageId: isInsideProperty ? "nestedComponentInProps" : "nestedComponent",
               node: component,
@@ -1714,8 +1721,7 @@ var no_nested_components_default = createRule({
             });
             continue;
           }
-          const isInsideClassComponentRenderMethod = core.isInsideRenderMethod(component);
-          if (isInsideClassComponentRenderMethod) {
+          if (core.isInsideRenderMethod(component)) {
             context.report({
               messageId: "nestedComponent",
               node: component,
@@ -1726,7 +1732,7 @@ var no_nested_components_default = createRule({
           }
         }
         for (const { name: name2, node: component } of classComponents) {
-          if (eff.O.isNone(AST11__namespace.traverseUp(component, eff.or(isClassComponent11, isFunctionComponent)))) {
+          if (eff.O.isNone(AST11__namespace.findParentNode(component, eff.or(isClassComponent11, isFunctionComponent)))) {
             continue;
           }
           context.report({
@@ -1858,8 +1864,8 @@ var no_set_state_in_component_did_mount_default = createRule({
       if (!core.isThisSetState(node)) return eff.O.none();
       return eff.F.pipe(
         eff.O.Do,
-        eff.O.bind("clazz", () => AST11__namespace.traverseUpGuard(node, core.isClassComponent)),
-        eff.O.bind("method", ({ clazz }) => AST11__namespace.traverseUpStop(node, clazz, isComponentDidMount)),
+        eff.O.bind("clazz", () => AST11__namespace.findParentNodeGuard(node, core.isClassComponent)),
+        eff.O.bind("method", ({ clazz }) => AST11__namespace.findParentNodeStop(node, clazz, isComponentDidMount)),
         eff.O.bind("methodScope", ({ method }) => eff.O.some(context.sourceCode.getScope(method))),
         eff.O.bind("upperScope", () => eff.O.fromNullable(context.sourceCode.getScope(node).upper)),
         eff.O.filter(
@@ -1908,11 +1914,17 @@ var no_set_state_in_component_did_update_default = createRule({
       if (!core.isThisSetState(node)) return eff.O.none();
       return eff.F.pipe(
         eff.O.Do,
-        eff.O.bind("clazz", () => AST11__namespace.traverseUpGuard(node, core.isClassComponent)),
-        eff.O.bind("method", ({ clazz }) => AST11__namespace.traverseUpStop(node, clazz, isComponentDidUpdate)),
+        eff.O.bind("clazz", () => AST11__namespace.findParentNodeGuard(node, core.isClassComponent)),
+        eff.O.bind("method", ({ clazz }) => AST11__namespace.findParentNodeStop(node, clazz, isComponentDidUpdate)),
+        eff.O.bind("methodScope", ({ method }) => eff.O.some(context.sourceCode.getScope(method))),
         eff.O.bind("upperScope", () => eff.O.fromNullable(context.sourceCode.getScope(node).upper)),
         eff.O.filter(
-          ({ clazz, method, upperScope }) => method.parent === clazz.body && upperScope === context.sourceCode.getScope(method)
+          ({
+            clazz,
+            method,
+            methodScope,
+            upperScope
+          }) => method.parent === clazz.body && upperScope === methodScope
         ),
         eff.O.map(() => ({
           messageId: "noSetStateInComponentDidUpdate",
@@ -1952,11 +1964,17 @@ var no_set_state_in_component_will_update_default = createRule({
       if (!core.isThisSetState(node)) return eff.O.none();
       return eff.F.pipe(
         eff.O.Do,
-        eff.O.bind("clazz", () => AST11__namespace.traverseUpGuard(node, core.isClassComponent)),
-        eff.O.bind("method", ({ clazz }) => AST11__namespace.traverseUpStop(node, clazz, isComponentWillUpdate3)),
+        eff.O.bind("clazz", () => AST11__namespace.findParentNodeGuard(node, core.isClassComponent)),
+        eff.O.bind("method", ({ clazz }) => AST11__namespace.findParentNodeStop(node, clazz, isComponentWillUpdate3)),
+        eff.O.bind("methodScope", ({ method }) => eff.O.some(context.sourceCode.getScope(method))),
         eff.O.bind("upperScope", () => eff.O.fromNullable(context.sourceCode.getScope(node).upper)),
         eff.O.filter(
-          ({ clazz, method, upperScope }) => method.parent === clazz.body && upperScope === context.sourceCode.getScope(method)
+          ({
+            clazz,
+            method,
+            methodScope,
+            upperScope
+          }) => method.parent === clazz.body && upperScope === methodScope
         ),
         eff.O.map(() => ({
           messageId: "noSetStateInComponentWillUpdate",
@@ -2167,7 +2185,7 @@ var no_unstable_context_value_default = createRule({
         const openingElementName = node.name;
         if (openingElementName.type !== types.AST_NODE_TYPES.JSXMemberExpression) return;
         if (openingElementName.property.name !== "Provider") return;
-        const constructionEntry = eff.F.pipe(
+        eff.F.pipe(
           eff.O.Do,
           eff.O.bind("function", ctx.getCurrentFunction),
           eff.O.bind("attribute", () => eff.O.fromNullable(
@@ -2180,16 +2198,14 @@ var no_unstable_context_value_default = createRule({
           eff.O.bind("construction", ({ valueExpression }) => {
             const initialScope = context.sourceCode.getScope(valueExpression);
             return eff.O.some(VAR__namespace.getValueConstruction(valueExpression, initialScope));
+          }),
+          eff.O.map((vc) => {
+            if (vc.construction.kind === "None") return;
+            if (core.isReactHookCall(vc.construction.node)) return;
+            const prevs = constructions.get(vc.function.node) ?? [];
+            constructions.set(vc.function.node, [...prevs, vc.construction]);
           })
         );
-        for (const { construction, function: [_, fNode] } of eff.O.toArray(constructionEntry)) {
-          if (construction.kind === "None") continue;
-          if (core.isReactHookCall(construction.node)) continue;
-          constructions.set(fNode, [
-            ...constructions.get(fNode) ?? [],
-            construction
-          ]);
-        }
       },
       "Program:exit"(node) {
         const components = ctx.getAllComponents(node).values();
@@ -2267,10 +2283,11 @@ var no_unstable_default_props_default = createRule({
         }
       },
       "VariableDeclarator[id.type='ObjectPattern'][init.type='Identifier']"(node) {
-        for (const [_, currentFn] of eff.O.toArray(ctx.getCurrentFunction())) {
-          const prevDeclarators = declarators.get(currentFn) ?? [];
-          declarators.set(currentFn, [...prevDeclarators, node]);
-        }
+        const mbEntry = ctx.getCurrentFunction();
+        if (eff.O.isNone(mbEntry)) return;
+        const entry = mbEntry.value;
+        const prevs = declarators.get(entry.node) ?? [];
+        declarators.set(entry.node, [...prevs, node]);
       }
     };
   },
@@ -2398,7 +2415,9 @@ var no_unused_class_component_members_default = createRule({
         if (node.init && AST11__namespace.isThisExpression(node.init) && node.id.type === types.AST_NODE_TYPES.ObjectPattern) {
           for (const prop of node.id.properties) {
             if (prop.type === types.AST_NODE_TYPES.Property && AST11__namespace.isKeyLiteralLike(prop, prop.key)) {
-              for (const name2 of eff.O.toArray(getName3(prop.key))) {
+              const mbName = getName3(prop.key);
+              if (eff.O.isSome(mbName)) {
+                const name2 = mbName.value;
                 propertyUsages.get(currentClass)?.add(name2);
               }
             }
@@ -2562,7 +2581,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 +2593,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 +2606,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;
@@ -2659,8 +2684,10 @@ var prefer_destructuring_assignment_default = createRule({
         const components = Array.from(ctx.getAllComponents(node).values());
         function isFunctionComponent(block) {
           if (!AST11__namespace.isFunction(block)) return false;
-          const maybeId = AST11__namespace.getFunctionIdentifier(block);
-          return eff.O.isSome(maybeId) && core.isComponentName(maybeId.value.name) && components.some((component) => component.node === block);
+          return eff.O.exists(
+            AST11__namespace.getFunctionIdentifier(block),
+            (id) => core.isComponentName(id.name) && components.some((component) => component.node === block)
+          );
         }
         for (const [initialScope, memberExpression] of memberExpressionWithNames) {
           let scope = initialScope;
@@ -2718,7 +2745,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 +2866,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 +2876,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 +2890,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 { F, O, or, isString, not, isNullable } from '@eslint-react/eff';
 import * as JSX7 from '@eslint-react/jsx';
-import { createRuleForPlugin, normalizeSettings, decodeSettings, unsafeReadSettings } from '@eslint-react/shared';
+import { createRuleForPlugin, getSettingsFromContext, unsafeDecodeSettings } 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.6";
 var createRule = createRuleForPlugin("x");
 
 // src/rules/avoid-shorthand-boolean.ts
@@ -254,7 +254,7 @@ function isReactChildrenMethod(name2) {
   return reactChildrenMethod.some((method) => method === name2);
 }
 function isUsingReactChildren(node, context) {
-  const settings = unsafeReadSettings(context.settings);
+  const settings = unsafeDecodeSettings(context.settings);
   const { callee } = node;
   if (!("property" in callee) || !("object" in callee) || !("name" in callee.property)) {
     return false;
@@ -263,7 +263,7 @@ function isUsingReactChildren(node, context) {
   const initialScope = context.sourceCode.getScope(node);
   if (callee.object.type === AST_NODE_TYPES.Identifier && callee.object.name === "Children") return true;
   if (callee.object.type === AST_NODE_TYPES.MemberExpression && "name" in callee.object.object) {
-    return isInitializedFromReact(callee.object.object.name, initialScope, { ...settings, strictImportCheck: true });
+    return isInitializedFromReact(callee.object.object.name, initialScope, settings.importSource);
   }
   return false;
 }
@@ -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) {
@@ -925,7 +925,7 @@ var no_create_ref_default = createRule({
     return {
       CallExpression(node) {
         if (!isCreateRefCall(node, context)) return;
-        if (O.isSome(AST11.traverseUp(node, isClassComponent))) return;
+        if (O.isSome(AST11.findParentNode(node, isClassComponent))) return;
         context.report({ messageId: "noCreateRef", node });
       }
     };
@@ -1017,19 +1017,24 @@ var no_direct_mutation_state_default = createRule({
   create(context) {
     function getReportDescriptor(node) {
       if (!isAssignmentToThisState(node)) return O.none();
-      const maybeParentClass = AST11.traverseUpGuard(
-        node,
-        AST11.isOneOf([AST_NODE_TYPES.ClassDeclaration, AST_NODE_TYPES.ClassExpression])
+      return F.pipe(
+        O.Do,
+        O.bind("parentClass", () => AST11.findParentNodeGuard(
+          node,
+          AST11.isOneOf([
+            AST_NODE_TYPES.ClassDeclaration,
+            AST_NODE_TYPES.ClassExpression
+          ])
+        )),
+        O.bind("parentConstructor", () => AST11.findParentNodeGuard(node, isConstructorFunction)),
+        O.filter(
+          ({ parentClass, parentConstructor }) => isClassComponent(parentClass) && context.sourceCode.getScope(node).block !== parentConstructor
+        ),
+        O.map(() => ({
+          messageId: "noDirectMutationState",
+          node
+        }))
       );
-      if (O.isNone(maybeParentClass)) return O.none();
-      const parentClass = maybeParentClass.value;
-      if (!isClassComponent(parentClass)) return O.none();
-      const maybeParentConstructor = AST11.traverseUpGuard(node, isConstructorFunction);
-      if (O.exists(maybeParentConstructor, (n) => context.sourceCode.getScope(node).block === n)) return O.none();
-      return O.some({
-        messageId: "noDirectMutationState",
-        node
-      });
     }
     return {
       AssignmentExpression: F.flow(getReportDescriptor, O.map(context.report))
@@ -1119,20 +1124,20 @@ var no_duplicate_key_default = createRule({
             break;
           }
           default: {
-            const entry = F.pipe(
+            const mbEntry = F.pipe(
               O.Do,
-              O.bind("call", () => AST11.traverseUpGuard(jsxElement, AST11.isMapCallLoose)),
-              O.bind("iter", ({ call }) => AST11.traverseUpStop(jsxElement, call, AST11.isFunction)),
-              O.bind("arg0", ({ call }) => O.fromNullable(call.arguments[0]))
+              O.bind("call", () => AST11.findParentNodeGuard(jsxElement, AST11.isMapCallLoose)),
+              O.bind("iter", ({ call: call2 }) => AST11.findParentNodeStop(jsxElement, call2, AST11.isFunction)),
+              O.bind("arg0", ({ call: call2 }) => O.fromNullable(call2.arguments[0]))
             );
-            for (const { arg0, call, iter } of O.toArray(entry)) {
-              if (AST11.unwrapTypeExpression(arg0) !== iter) continue;
-              keyedEntries.set(call, {
-                hasDuplicate: node.value?.type === AST_NODE_TYPES.Literal,
-                keys: [node],
-                root: call
-              });
-            }
+            if (O.isNone(mbEntry)) return;
+            const { arg0, call, iter } = mbEntry.value;
+            if (AST11.unwrapTypeExpression(arg0) !== iter) return;
+            keyedEntries.set(call, {
+              hasDuplicate: node.value?.type === AST_NODE_TYPES.Literal,
+              keys: [node],
+              root: call
+            });
           }
         }
       },
@@ -1175,7 +1180,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) {
@@ -1272,9 +1277,7 @@ var no_implicit_key_default = createRule({
     function getReportDescriptor(node) {
       const initialScope = context.sourceCode.getScope(node);
       const keyPropFound = JSX7.findPropInAttributes(node.attributes, initialScope)("key");
-      const keyPropOnElement = node.attributes.some(
-        (n) => AST11.is(AST_NODE_TYPES.JSXAttribute)(n) && n.name.name === "key"
-      );
+      const keyPropOnElement = node.attributes.some((n) => AST11.is(AST_NODE_TYPES.JSXAttribute)(n) && n.name.name === "key");
       if (O.isSome(keyPropFound) && !keyPropOnElement) {
         return O.some({ messageId: "noImplicitKey", node: keyPropFound.value });
       }
@@ -1398,7 +1401,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",
@@ -1552,10 +1555,8 @@ var no_missing_key_default = createRule({
     function checkBlockStatement(node) {
       return AST11.getNestedReturnStatements(node).reduce((acc, statement) => {
         if (!statement.argument) return acc;
-        const maybeDescriptor = checkIteratorElement(statement.argument);
-        if (O.isNone(maybeDescriptor)) return acc;
-        const descriptor = maybeDescriptor.value;
-        return [...acc, descriptor];
+        const mbDescriptor = checkIteratorElement(statement.argument);
+        return O.isNone(mbDescriptor) ? acc : [...acc, mbDescriptor.value];
       }, []);
     }
     return {
@@ -1653,7 +1654,7 @@ var no_nested_components_default = createRule({
           if (O.isNone(componentName)) continue;
           const name2 = componentName.value;
           const isInsideProperty = component.parent.type === AST_NODE_TYPES.Property;
-          const isInsideJSXPropValue = component.parent.type === AST_NODE_TYPES.JSXAttribute || O.isSome(JSX7.traverseUpProp(node, (n) => n.value?.type === AST_NODE_TYPES.JSXExpressionContainer));
+          const isInsideJSXPropValue = component.parent.type === AST_NODE_TYPES.JSXAttribute || O.isSome(JSX7.findParentProp(node, (n) => n.value?.type === AST_NODE_TYPES.JSXExpressionContainer));
           if (isInsideJSXPropValue) {
             if (!isDeclaredInRenderPropLoose(component)) {
               context.report({
@@ -1676,8 +1677,14 @@ var no_nested_components_default = createRule({
             });
             continue;
           }
-          const maybeParentComponent = AST11.traverseUpGuard(component, isFunctionComponent);
-          if (O.isSome(maybeParentComponent) && !isDirectValueOfRenderPropertyLoose(maybeParentComponent.value)) {
+          const isParnetComponentNotDirectValueOfRenderProperty = O.exists(
+            AST11.findParentNodeGuard(
+              component,
+              isFunctionComponent
+            ),
+            not(isDirectValueOfRenderPropertyLoose)
+          );
+          if (isParnetComponentNotDirectValueOfRenderProperty) {
             context.report({
               messageId: isInsideProperty ? "nestedComponentInProps" : "nestedComponent",
               node: component,
@@ -1687,8 +1694,7 @@ var no_nested_components_default = createRule({
             });
             continue;
           }
-          const isInsideClassComponentRenderMethod = isInsideRenderMethod(component);
-          if (isInsideClassComponentRenderMethod) {
+          if (isInsideRenderMethod(component)) {
             context.report({
               messageId: "nestedComponent",
               node: component,
@@ -1699,7 +1705,7 @@ var no_nested_components_default = createRule({
           }
         }
         for (const { name: name2, node: component } of classComponents) {
-          if (O.isNone(AST11.traverseUp(component, or(isClassComponent11, isFunctionComponent)))) {
+          if (O.isNone(AST11.findParentNode(component, or(isClassComponent11, isFunctionComponent)))) {
             continue;
           }
           context.report({
@@ -1831,8 +1837,8 @@ var no_set_state_in_component_did_mount_default = createRule({
       if (!isThisSetState(node)) return O.none();
       return F.pipe(
         O.Do,
-        O.bind("clazz", () => AST11.traverseUpGuard(node, isClassComponent)),
-        O.bind("method", ({ clazz }) => AST11.traverseUpStop(node, clazz, isComponentDidMount)),
+        O.bind("clazz", () => AST11.findParentNodeGuard(node, isClassComponent)),
+        O.bind("method", ({ clazz }) => AST11.findParentNodeStop(node, clazz, isComponentDidMount)),
         O.bind("methodScope", ({ method }) => O.some(context.sourceCode.getScope(method))),
         O.bind("upperScope", () => O.fromNullable(context.sourceCode.getScope(node).upper)),
         O.filter(
@@ -1881,11 +1887,17 @@ var no_set_state_in_component_did_update_default = createRule({
       if (!isThisSetState(node)) return O.none();
       return F.pipe(
         O.Do,
-        O.bind("clazz", () => AST11.traverseUpGuard(node, isClassComponent)),
-        O.bind("method", ({ clazz }) => AST11.traverseUpStop(node, clazz, isComponentDidUpdate)),
+        O.bind("clazz", () => AST11.findParentNodeGuard(node, isClassComponent)),
+        O.bind("method", ({ clazz }) => AST11.findParentNodeStop(node, clazz, isComponentDidUpdate)),
+        O.bind("methodScope", ({ method }) => O.some(context.sourceCode.getScope(method))),
         O.bind("upperScope", () => O.fromNullable(context.sourceCode.getScope(node).upper)),
         O.filter(
-          ({ clazz, method, upperScope }) => method.parent === clazz.body && upperScope === context.sourceCode.getScope(method)
+          ({
+            clazz,
+            method,
+            methodScope,
+            upperScope
+          }) => method.parent === clazz.body && upperScope === methodScope
         ),
         O.map(() => ({
           messageId: "noSetStateInComponentDidUpdate",
@@ -1925,11 +1937,17 @@ var no_set_state_in_component_will_update_default = createRule({
       if (!isThisSetState(node)) return O.none();
       return F.pipe(
         O.Do,
-        O.bind("clazz", () => AST11.traverseUpGuard(node, isClassComponent)),
-        O.bind("method", ({ clazz }) => AST11.traverseUpStop(node, clazz, isComponentWillUpdate3)),
+        O.bind("clazz", () => AST11.findParentNodeGuard(node, isClassComponent)),
+        O.bind("method", ({ clazz }) => AST11.findParentNodeStop(node, clazz, isComponentWillUpdate3)),
+        O.bind("methodScope", ({ method }) => O.some(context.sourceCode.getScope(method))),
         O.bind("upperScope", () => O.fromNullable(context.sourceCode.getScope(node).upper)),
         O.filter(
-          ({ clazz, method, upperScope }) => method.parent === clazz.body && upperScope === context.sourceCode.getScope(method)
+          ({
+            clazz,
+            method,
+            methodScope,
+            upperScope
+          }) => method.parent === clazz.body && upperScope === methodScope
         ),
         O.map(() => ({
           messageId: "noSetStateInComponentWillUpdate",
@@ -2140,7 +2158,7 @@ var no_unstable_context_value_default = createRule({
         const openingElementName = node.name;
         if (openingElementName.type !== AST_NODE_TYPES.JSXMemberExpression) return;
         if (openingElementName.property.name !== "Provider") return;
-        const constructionEntry = F.pipe(
+        F.pipe(
           O.Do,
           O.bind("function", ctx.getCurrentFunction),
           O.bind("attribute", () => O.fromNullable(
@@ -2153,16 +2171,14 @@ var no_unstable_context_value_default = createRule({
           O.bind("construction", ({ valueExpression }) => {
             const initialScope = context.sourceCode.getScope(valueExpression);
             return O.some(VAR.getValueConstruction(valueExpression, initialScope));
+          }),
+          O.map((vc) => {
+            if (vc.construction.kind === "None") return;
+            if (isReactHookCall(vc.construction.node)) return;
+            const prevs = constructions.get(vc.function.node) ?? [];
+            constructions.set(vc.function.node, [...prevs, vc.construction]);
           })
         );
-        for (const { construction, function: [_, fNode] } of O.toArray(constructionEntry)) {
-          if (construction.kind === "None") continue;
-          if (isReactHookCall(construction.node)) continue;
-          constructions.set(fNode, [
-            ...constructions.get(fNode) ?? [],
-            construction
-          ]);
-        }
       },
       "Program:exit"(node) {
         const components = ctx.getAllComponents(node).values();
@@ -2240,10 +2256,11 @@ var no_unstable_default_props_default = createRule({
         }
       },
       "VariableDeclarator[id.type='ObjectPattern'][init.type='Identifier']"(node) {
-        for (const [_, currentFn] of O.toArray(ctx.getCurrentFunction())) {
-          const prevDeclarators = declarators.get(currentFn) ?? [];
-          declarators.set(currentFn, [...prevDeclarators, node]);
-        }
+        const mbEntry = ctx.getCurrentFunction();
+        if (O.isNone(mbEntry)) return;
+        const entry = mbEntry.value;
+        const prevs = declarators.get(entry.node) ?? [];
+        declarators.set(entry.node, [...prevs, node]);
       }
     };
   },
@@ -2371,7 +2388,9 @@ var no_unused_class_component_members_default = createRule({
         if (node.init && AST11.isThisExpression(node.init) && node.id.type === AST_NODE_TYPES.ObjectPattern) {
           for (const prop of node.id.properties) {
             if (prop.type === AST_NODE_TYPES.Property && AST11.isKeyLiteralLike(prop, prop.key)) {
-              for (const name2 of O.toArray(getName3(prop.key))) {
+              const mbName = getName3(prop.key);
+              if (O.isSome(mbName)) {
+                const name2 = mbName.value;
                 propertyUsages.get(currentClass)?.add(name2);
               }
             }
@@ -2535,7 +2554,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 +2566,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 +2579,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;
@@ -2632,8 +2657,10 @@ var prefer_destructuring_assignment_default = createRule({
         const components = Array.from(ctx.getAllComponents(node).values());
         function isFunctionComponent(block) {
           if (!AST11.isFunction(block)) return false;
-          const maybeId = AST11.getFunctionIdentifier(block);
-          return O.isSome(maybeId) && isComponentName(maybeId.value.name) && components.some((component) => component.node === block);
+          return O.exists(
+            AST11.getFunctionIdentifier(block),
+            (id) => isComponentName(id.name) && components.some((component) => component.node === block)
+          );
         }
         for (const [initialScope, memberExpression] of memberExpressionWithNames) {
           let scope = initialScope;
@@ -2691,7 +2718,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 +2839,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 +2849,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 +2863,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.6",
   "description": "A set of composable ESLint rules for libraries and frameworks that use React as a UI runtime.",
   "keywords": [
     "react",
@@ -40,25 +40,25 @@
     "./package.json"
   ],
   "dependencies": {
-    "@typescript-eslint/scope-manager": "^8.19.0",
-    "@typescript-eslint/type-utils": "^8.19.0",
-    "@typescript-eslint/types": "^8.19.0",
-    "@typescript-eslint/utils": "^8.19.0",
+    "@typescript-eslint/scope-manager": "^8.19.1",
+    "@typescript-eslint/type-utils": "^8.19.1",
+    "@typescript-eslint/types": "^8.19.1",
+    "@typescript-eslint/utils": "^8.19.1",
     "compare-versions": "^6.1.1",
     "is-immutable-type": "^5.0.1",
     "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.6",
+    "@eslint-react/core": "1.23.2-beta.6",
+    "@eslint-react/eff": "1.23.2-beta.6",
+    "@eslint-react/jsx": "1.23.2-beta.6",
+    "@eslint-react/types": "1.23.2-beta.6",
+    "@eslint-react/var": "1.23.2-beta.6",
+    "@eslint-react/shared": "1.23.2-beta.6"
   },
   "devDependencies": {
-    "@types/react": "^19.0.2",
+    "@types/react": "^19.0.3",
     "@types/react-dom": "^19.0.2",
     "tsup": "^8.3.5",
     "@workspace/configs": "0.0.0"