eslint-plugin-react-x 1.22.2-next.6 → 1.22.2-next.8

package/dist/index.js

@@ -1,7 +1,7 @@
 'use strict';
 
 var eff = require('@eslint-react/eff');
-var JSX8 = require('@eslint-react/jsx');
+var JSX7 = require('@eslint-react/jsx');
 var shared = require('@eslint-react/shared');
 var AST11 = require('@eslint-react/ast');
 var core = require('@eslint-react/core');
@@ -35,14 +35,14 @@ function _interopNamespace(e) {
   return Object.freeze(n);
 }
 
-var JSX8__namespace = /*#__PURE__*/_interopNamespace(JSX8);
+var JSX7__namespace = /*#__PURE__*/_interopNamespace(JSX7);
 var AST11__namespace = /*#__PURE__*/_interopNamespace(AST11);
 var VAR__namespace = /*#__PURE__*/_interopNamespace(VAR);
 var ts__default = /*#__PURE__*/_interopDefault(ts);
 
 // package.json
 var name = "eslint-plugin-react-x";
-var version = "1.22.2-next.6";
+var version = "1.22.2-next.8";
 var createRule = shared.createRuleForPlugin("x");
 
 // src/rules/avoid-shorthand-boolean.ts
@@ -70,7 +70,7 @@ var avoid_shorthand_boolean_default = createRule({
         messageId: "avoidShorthandBoolean",
         node,
         data: {
-          propName: JSX8__namespace.getPropName(node)
+          propName: JSX7__namespace.getPropName(node)
         },
         fix: (fixer) => fixer.insertTextAfter(node.name, `={true}`)
       });
@@ -151,7 +151,7 @@ var RULE_FEATURES4 = [
   "CHK"
 ];
 function getName(node) {
-  if (node.type === types.AST_NODE_TYPES.TSAsExpression) {
+  if (AST11__namespace.isTypeExpression(node)) {
     return getName(node.expression);
   }
   if (node.type === types.AST_NODE_TYPES.Identifier || node.type === types.AST_NODE_TYPES.PrivateIdentifier) {
@@ -537,7 +537,7 @@ var no_children_prop_default = createRule({
     return {
       JSXElement(node) {
         const initialScope = context.sourceCode.getScope(node);
-        const prop = JSX8__namespace.getProp(node.openingElement.attributes, "children", initialScope);
+        const prop = JSX7__namespace.getProp(node.openingElement.attributes, "children", initialScope);
         const reportDescriptor = eff.O.map(prop, (prop2) => ({ messageId: "noChildrenProp", node: prop2 }));
         eff.O.map(reportDescriptor, context.report);
       }
@@ -912,7 +912,7 @@ var no_context_provider_default = createRule({
     if (compareVersions.compare(version2, "19.0.0", "<")) return {};
     return {
       JSXElement(node) {
-        const elementName = JSX8__namespace.getElementName(node.openingElement);
+        const elementName = JSX7__namespace.getElementName(node.openingElement);
         if (!elementName.endsWith(".Provider")) return;
         context.report({
           messageId: "noContextProvider",
@@ -1009,7 +1009,7 @@ var RULE_FEATURES22 = [
   "CHK"
 ];
 function getName2(node) {
-  if (node.type === types.AST_NODE_TYPES.TSAsExpression) {
+  if (AST11__namespace.isTypeExpression(node)) {
     return getName2(node.expression);
   }
   if (node.type === types.AST_NODE_TYPES.Identifier || node.type === types.AST_NODE_TYPES.PrivateIdentifier) {
@@ -1123,117 +1123,61 @@ var no_duplicate_key_default = createRule({
   },
   name: RULE_NAME24,
   create(context) {
-    const state = { isWithinChildrenToArray: false };
-    function isKeyEqual(a, b) {
-      return VAR__namespace.isNodeValueEqual(a, b, [context.sourceCode.getScope(a), context.sourceCode.getScope(b)]);
-    }
-    function checkIteratorElement(node) {
-      if (node.type !== types.AST_NODE_TYPES.JSXElement) return eff.O.none();
-      const initialScope = context.sourceCode.getScope(node);
-      return eff.F.pipe(
-        JSX8__namespace.findPropInAttributes(node.openingElement.attributes, initialScope)("key"),
-        eff.O.flatMap((k) => "value" in k ? eff.O.fromNullable(k.value) : eff.O.none()),
-        eff.O.flatMap((v) => {
-          return isKeyEqual(v, v) ? eff.O.some({
-            messageId: "noDuplicateKey",
-            node: v,
-            data: {
-              value: context.sourceCode.getText(v)
-            }
-          }) : eff.O.none();
-        })
-      );
-    }
-    function checkExpression(node) {
-      switch (node.type) {
-        case types.AST_NODE_TYPES.ConditionalExpression:
-          if (!("consequent" in node)) return eff.O.none();
-          return eff.F.pipe(
-            checkIteratorElement(node.consequent),
-            eff.O.orElse(() => checkIteratorElement(node.alternate))
-          );
-        case types.AST_NODE_TYPES.JSXElement:
-        case types.AST_NODE_TYPES.JSXFragment:
-          return checkIteratorElement(node);
-        case types.AST_NODE_TYPES.LogicalExpression:
-          if (!("left" in node)) return eff.O.none();
-          return eff.F.pipe(
-            checkIteratorElement(node.left),
-            eff.O.orElse(() => checkIteratorElement(node.right))
-          );
-        default:
-          return eff.O.none();
-      }
+    if (!context.sourceCode.getText().includes("key=")) return {};
+    const keyedEntries = /* @__PURE__ */ new Map();
+    function isKeyValueEqual(a, b) {
+      const aValue = a.value;
+      const bValue = b.value;
+      if (aValue === null || bValue === null) return false;
+      return AST11__namespace.isNodeEqual(aValue, bValue);
     }
-    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 seen = /* @__PURE__ */ new WeakSet();
     return {
-      "ArrayExpression, JSXElement > JSXElement"(node) {
-        if (state.isWithinChildrenToArray) return;
-        const elements = tsPattern.match(node).with({ type: types.AST_NODE_TYPES.ArrayExpression }, ({ elements: elements2 }) => elements2).with({ type: types.AST_NODE_TYPES.JSXElement }, ({ parent }) => "children" in parent ? parent.children : []).otherwise(() => []).filter(AST11__namespace.is(types.AST_NODE_TYPES.JSXElement)).filter((element) => !seen.has(element));
-        const keys = elements.reduce(
-          (acc, element) => {
-            const attr = element.openingElement.attributes.findLast((attr2) => {
-              if (attr2.type !== types.AST_NODE_TYPES.JSXAttribute) return false;
-              return attr2.name.name === "key";
+      "JSXAttribute[name.name='key']"(node) {
+        const jsxElement = node.parent.parent;
+        switch (jsxElement.parent.type) {
+          case types.AST_NODE_TYPES.ArrayExpression:
+          case types.AST_NODE_TYPES.JSXElement:
+          case types.AST_NODE_TYPES.JSXFragment: {
+            const root = jsxElement.parent;
+            const prevKeys = keyedEntries.get(root)?.keys ?? [];
+            keyedEntries.set(root, {
+              hasDuplicate: prevKeys.some((prevKey) => isKeyValueEqual(prevKey, node)),
+              keys: [...prevKeys, node],
+              root: jsxElement.parent
             });
-            if (!attr || !("value" in attr) || attr.value === null) return acc;
-            const { value } = attr;
-            if (acc.length === 0) return [[element, attr, value]];
-            if (acc.some(([_, _1, v]) => isKeyEqual(v, value))) {
-              return [...acc, [element, attr, value]];
-            }
-            return acc;
-          },
-          []
-        );
-        if (keys.length < 2) return;
-        for (const [element, attr, value] of keys) {
-          seen.add(element);
-          context.report({
-            messageId: "noDuplicateKey",
-            node: attr,
-            data: {
-              value: context.sourceCode.getText(value)
+            break;
+          }
+          default: {
+            const entry = 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]))
+            );
+            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
+              });
             }
-          });
+          }
         }
       },
-      CallExpression(node) {
-        state.isWithinChildrenToArray ||= core.isChildrenToArrayCall(node, context);
-        if (state.isWithinChildrenToArray) return;
-        const isMapCall = AST11__namespace.isMapCallLoose(node);
-        const isArrayFromCall = tsPattern.isMatching({
-          type: types.AST_NODE_TYPES.CallExpression,
-          callee: {
-            type: types.AST_NODE_TYPES.MemberExpression,
-            property: {
-              name: "from"
-            }
-          }
-        }, node);
-        if (!isMapCall && !isArrayFromCall) return;
-        const fn = node.arguments[isMapCall ? 0 : 1];
-        if (!AST11__namespace.isOneOf([types.AST_NODE_TYPES.ArrowFunctionExpression, types.AST_NODE_TYPES.FunctionExpression])(fn)) return;
-        if (fn.body.type === types.AST_NODE_TYPES.BlockStatement) {
-          for (const descriptor of checkBlockStatement(fn.body)) {
-            context.report(descriptor);
+      "Program:exit"() {
+        for (const { hasDuplicate, keys } of keyedEntries.values()) {
+          if (!hasDuplicate) continue;
+          for (const key of keys) {
+            context.report({
+              messageId: "noDuplicateKey",
+              node: key,
+              data: {
+                value: context.sourceCode.getText(key)
+              }
+            });
           }
-          return;
         }
-        eff.O.map(checkExpression(fn.body), context.report);
-      },
-      "CallExpression:exit"(node) {
-        if (!core.isChildrenToArrayCall(node, context)) return;
-        state.isWithinChildrenToArray = false;
       }
     };
   },
@@ -1356,7 +1300,7 @@ var no_implicit_key_default = createRule({
   create(context) {
     function getReportDescriptor(node) {
       const initialScope = context.sourceCode.getScope(node);
-      const keyPropFound = JSX8__namespace.findPropInAttributes(node.attributes, initialScope)("key");
+      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"
       );
@@ -1601,7 +1545,7 @@ var no_missing_key_default = createRule({
       switch (node.type) {
         case types.AST_NODE_TYPES.JSXElement: {
           const initialScope = context.sourceCode.getScope(node);
-          if (!JSX8__namespace.hasProp(node.openingElement.attributes, "key", initialScope)) {
+          if (!JSX7__namespace.hasProp(node.openingElement.attributes, "key", initialScope)) {
             return eff.O.some({
               messageId: "noMissingKey",
               node
@@ -1650,7 +1594,7 @@ var no_missing_key_default = createRule({
         if (elements.length === 0) return;
         const initialScope = context.sourceCode.getScope(node);
         for (const element of elements) {
-          if (!JSX8__namespace.hasProp(element.openingElement.attributes, "key", initialScope)) {
+          if (!JSX7__namespace.hasProp(element.openingElement.attributes, "key", initialScope)) {
             context.report({
               messageId: "noMissingKey",
               node: element
@@ -1738,7 +1682,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(JSX8__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.traverseUpProp(node, (n) => n.value?.type === types.AST_NODE_TYPES.JSXExpressionContainer));
           if (isInsideJSXPropValue) {
             if (!core.isDeclaredInRenderPropLoose(component)) {
               context.report({
@@ -2362,7 +2306,7 @@ var LIFECYCLE_METHODS = /* @__PURE__ */ new Set([
   "UNSAFE_componentWillUpdate"
 ]);
 function getName3(node) {
-  if (node.type === types.AST_NODE_TYPES.TSAsExpression) {
+  if (AST11__namespace.isTypeExpression(node)) {
     return getName3(node.expression);
   }
   if (node.type === types.AST_NODE_TYPES.Identifier || node.type === types.AST_NODE_TYPES.PrivateIdentifier) {
@@ -2476,7 +2420,7 @@ var RULE_FEATURES43 = [
   "CHK"
 ];
 function getName4(node) {
-  if (node.type === types.AST_NODE_TYPES.TSAsExpression) {
+  if (AST11__namespace.isTypeExpression(node)) {
     return getName4(node.expression);
   }
   if (node.type === types.AST_NODE_TYPES.Identifier || node.type === types.AST_NODE_TYPES.PrivateIdentifier) {
@@ -2619,18 +2563,18 @@ var no_unused_state_default = createRule({
 var RULE_NAME44 = "no-useless-fragment";
 function check(node, context, allowExpressions) {
   const initialScope = context.sourceCode.getScope(node);
-  if (JSX8__namespace.isKeyedElement(node, initialScope)) return;
-  if (JSX8__namespace.isBuiltInElement(node.parent)) context.report({ messageId: "noUselessFragmentInBuiltIn", 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 });
   const isChildren = AST11__namespace.isOneOf([types.AST_NODE_TYPES.JSXElement, types.AST_NODE_TYPES.JSXFragment])(node.parent);
   const [firstChildren] = node.children;
-  if (allowExpressions && node.children.length === 1 && JSX8__namespace.isLiteral(firstChildren) && !isChildren) return;
+  if (allowExpressions && node.children.length === 1 && JSX7__namespace.isLiteral(firstChildren) && !isChildren) return;
   if (!allowExpressions && isChildren) {
     return context.report({ messageId: "noUselessFragment", node });
   } else if (!allowExpressions && !isChildren && node.children.length === 1) {
     return context.report({ messageId: "noUselessFragment", node });
   }
-  const nonPaddingChildren = node.children.filter((child) => !JSX8__namespace.isPaddingSpaces(child));
+  const nonPaddingChildren = node.children.filter((child) => !JSX7__namespace.isPaddingSpaces(child));
   if (nonPaddingChildren.length > 1) return;
   if (nonPaddingChildren.length === 0) return context.report({ messageId: "noUselessFragment", node });
   const [first] = nonPaddingChildren;
@@ -2872,7 +2816,7 @@ var prefer_shorthand_boolean_default = createRule({
   create(context) {
     function getReportDescriptor(node) {
       const { value } = node;
-      const propName = JSX8__namespace.getPropName(node);
+      const propName = JSX7__namespace.getPropName(node);
       const hasValueTrue = value?.type === types.AST_NODE_TYPES.JSXExpressionContainer && value.expression.type === types.AST_NODE_TYPES.Literal && value.expression.value === true;
       if (!hasValueTrue) return eff.O.none();
       return eff.O.some({

package/dist/index.mjs

@@ -1,8 +1,8 @@
 import { F, O, isString, isNullable } from '@eslint-react/eff';
-import * as JSX8 from '@eslint-react/jsx';
+import * as JSX7 from '@eslint-react/jsx';
 import { createRuleForPlugin, normalizeSettings, decodeSettings, unsafeReadSettings } from '@eslint-react/shared';
 import * as AST11 from '@eslint-react/ast';
-import { isForwardRefCall, isThisSetState, isClassComponent, isChildrenCount, isChildrenForEach, isChildrenMap, isChildrenOnly, isChildrenToArray, useComponentCollectorLegacy, isCloneElementCall, isCreateRefCall, isComponentName, isChildrenToArrayCall, useComponentCollector, ERFunctionComponentFlag, ERComponentHint, isDirectValueOfRenderPropertyLoose, isDeclaredInRenderPropLoose, isInsideCreateElementProps, isInsideRenderMethod, ERClassComponentFlag, isFragmentElement, isCreateElementCall, isInitializedFromReact } from '@eslint-react/core';
+import { isForwardRefCall, isThisSetState, isClassComponent, isChildrenCount, isChildrenForEach, isChildrenMap, isChildrenOnly, isChildrenToArray, useComponentCollectorLegacy, isCloneElementCall, isCreateRefCall, isComponentName, useComponentCollector, ERFunctionComponentFlag, isChildrenToArrayCall, ERComponentHint, isDirectValueOfRenderPropertyLoose, isDeclaredInRenderPropLoose, isInsideCreateElementProps, isInsideRenderMethod, ERClassComponentFlag, isFragmentElement, isCreateElementCall, isInitializedFromReact } from '@eslint-react/core';
 import { AST_NODE_TYPES } from '@typescript-eslint/types';
 import { isMatching, P, match } from 'ts-pattern';
 import { compare } from 'compare-versions';
@@ -15,7 +15,7 @@ import { getTypeImmutability, isUnknown, isImmutable, isReadonlyShallow, isReado
 
 // package.json
 var name = "eslint-plugin-react-x";
-var version = "1.22.2-next.6";
+var version = "1.22.2-next.8";
 var createRule = createRuleForPlugin("x");
 
 // src/rules/avoid-shorthand-boolean.ts
@@ -43,7 +43,7 @@ var avoid_shorthand_boolean_default = createRule({
         messageId: "avoidShorthandBoolean",
         node,
         data: {
-          propName: JSX8.getPropName(node)
+          propName: JSX7.getPropName(node)
         },
         fix: (fixer) => fixer.insertTextAfter(node.name, `={true}`)
       });
@@ -124,7 +124,7 @@ var RULE_FEATURES4 = [
   "CHK"
 ];
 function getName(node) {
-  if (node.type === AST_NODE_TYPES.TSAsExpression) {
+  if (AST11.isTypeExpression(node)) {
     return getName(node.expression);
   }
   if (node.type === AST_NODE_TYPES.Identifier || node.type === AST_NODE_TYPES.PrivateIdentifier) {
@@ -510,7 +510,7 @@ var no_children_prop_default = createRule({
     return {
       JSXElement(node) {
         const initialScope = context.sourceCode.getScope(node);
-        const prop = JSX8.getProp(node.openingElement.attributes, "children", initialScope);
+        const prop = JSX7.getProp(node.openingElement.attributes, "children", initialScope);
         const reportDescriptor = O.map(prop, (prop2) => ({ messageId: "noChildrenProp", node: prop2 }));
         O.map(reportDescriptor, context.report);
       }
@@ -885,7 +885,7 @@ var no_context_provider_default = createRule({
     if (compare(version2, "19.0.0", "<")) return {};
     return {
       JSXElement(node) {
-        const elementName = JSX8.getElementName(node.openingElement);
+        const elementName = JSX7.getElementName(node.openingElement);
         if (!elementName.endsWith(".Provider")) return;
         context.report({
           messageId: "noContextProvider",
@@ -982,7 +982,7 @@ var RULE_FEATURES22 = [
   "CHK"
 ];
 function getName2(node) {
-  if (node.type === AST_NODE_TYPES.TSAsExpression) {
+  if (AST11.isTypeExpression(node)) {
     return getName2(node.expression);
   }
   if (node.type === AST_NODE_TYPES.Identifier || node.type === AST_NODE_TYPES.PrivateIdentifier) {
@@ -1096,117 +1096,61 @@ var no_duplicate_key_default = createRule({
   },
   name: RULE_NAME24,
   create(context) {
-    const state = { isWithinChildrenToArray: false };
-    function isKeyEqual(a, b) {
-      return VAR.isNodeValueEqual(a, b, [context.sourceCode.getScope(a), context.sourceCode.getScope(b)]);
-    }
-    function checkIteratorElement(node) {
-      if (node.type !== AST_NODE_TYPES.JSXElement) return O.none();
-      const initialScope = context.sourceCode.getScope(node);
-      return F.pipe(
-        JSX8.findPropInAttributes(node.openingElement.attributes, initialScope)("key"),
-        O.flatMap((k) => "value" in k ? O.fromNullable(k.value) : O.none()),
-        O.flatMap((v) => {
-          return isKeyEqual(v, v) ? O.some({
-            messageId: "noDuplicateKey",
-            node: v,
-            data: {
-              value: context.sourceCode.getText(v)
-            }
-          }) : O.none();
-        })
-      );
-    }
-    function checkExpression(node) {
-      switch (node.type) {
-        case AST_NODE_TYPES.ConditionalExpression:
-          if (!("consequent" in node)) return O.none();
-          return F.pipe(
-            checkIteratorElement(node.consequent),
-            O.orElse(() => checkIteratorElement(node.alternate))
-          );
-        case AST_NODE_TYPES.JSXElement:
-        case AST_NODE_TYPES.JSXFragment:
-          return checkIteratorElement(node);
-        case AST_NODE_TYPES.LogicalExpression:
-          if (!("left" in node)) return O.none();
-          return F.pipe(
-            checkIteratorElement(node.left),
-            O.orElse(() => checkIteratorElement(node.right))
-          );
-        default:
-          return O.none();
-      }
+    if (!context.sourceCode.getText().includes("key=")) return {};
+    const keyedEntries = /* @__PURE__ */ new Map();
+    function isKeyValueEqual(a, b) {
+      const aValue = a.value;
+      const bValue = b.value;
+      if (aValue === null || bValue === null) return false;
+      return AST11.isNodeEqual(aValue, bValue);
     }
-    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 seen = /* @__PURE__ */ new WeakSet();
     return {
-      "ArrayExpression, JSXElement > JSXElement"(node) {
-        if (state.isWithinChildrenToArray) return;
-        const elements = match(node).with({ type: AST_NODE_TYPES.ArrayExpression }, ({ elements: elements2 }) => elements2).with({ type: AST_NODE_TYPES.JSXElement }, ({ parent }) => "children" in parent ? parent.children : []).otherwise(() => []).filter(AST11.is(AST_NODE_TYPES.JSXElement)).filter((element) => !seen.has(element));
-        const keys = elements.reduce(
-          (acc, element) => {
-            const attr = element.openingElement.attributes.findLast((attr2) => {
-              if (attr2.type !== AST_NODE_TYPES.JSXAttribute) return false;
-              return attr2.name.name === "key";
+      "JSXAttribute[name.name='key']"(node) {
+        const jsxElement = node.parent.parent;
+        switch (jsxElement.parent.type) {
+          case AST_NODE_TYPES.ArrayExpression:
+          case AST_NODE_TYPES.JSXElement:
+          case AST_NODE_TYPES.JSXFragment: {
+            const root = jsxElement.parent;
+            const prevKeys = keyedEntries.get(root)?.keys ?? [];
+            keyedEntries.set(root, {
+              hasDuplicate: prevKeys.some((prevKey) => isKeyValueEqual(prevKey, node)),
+              keys: [...prevKeys, node],
+              root: jsxElement.parent
             });
-            if (!attr || !("value" in attr) || attr.value === null) return acc;
-            const { value } = attr;
-            if (acc.length === 0) return [[element, attr, value]];
-            if (acc.some(([_, _1, v]) => isKeyEqual(v, value))) {
-              return [...acc, [element, attr, value]];
-            }
-            return acc;
-          },
-          []
-        );
-        if (keys.length < 2) return;
-        for (const [element, attr, value] of keys) {
-          seen.add(element);
-          context.report({
-            messageId: "noDuplicateKey",
-            node: attr,
-            data: {
-              value: context.sourceCode.getText(value)
+            break;
+          }
+          default: {
+            const entry = 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]))
+            );
+            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
+              });
             }
-          });
+          }
         }
       },
-      CallExpression(node) {
-        state.isWithinChildrenToArray ||= isChildrenToArrayCall(node, context);
-        if (state.isWithinChildrenToArray) return;
-        const isMapCall = AST11.isMapCallLoose(node);
-        const isArrayFromCall = isMatching({
-          type: AST_NODE_TYPES.CallExpression,
-          callee: {
-            type: AST_NODE_TYPES.MemberExpression,
-            property: {
-              name: "from"
-            }
-          }
-        }, node);
-        if (!isMapCall && !isArrayFromCall) return;
-        const fn = node.arguments[isMapCall ? 0 : 1];
-        if (!AST11.isOneOf([AST_NODE_TYPES.ArrowFunctionExpression, AST_NODE_TYPES.FunctionExpression])(fn)) return;
-        if (fn.body.type === AST_NODE_TYPES.BlockStatement) {
-          for (const descriptor of checkBlockStatement(fn.body)) {
-            context.report(descriptor);
+      "Program:exit"() {
+        for (const { hasDuplicate, keys } of keyedEntries.values()) {
+          if (!hasDuplicate) continue;
+          for (const key of keys) {
+            context.report({
+              messageId: "noDuplicateKey",
+              node: key,
+              data: {
+                value: context.sourceCode.getText(key)
+              }
+            });
           }
-          return;
         }
-        O.map(checkExpression(fn.body), context.report);
-      },
-      "CallExpression:exit"(node) {
-        if (!isChildrenToArrayCall(node, context)) return;
-        state.isWithinChildrenToArray = false;
       }
     };
   },
@@ -1329,7 +1273,7 @@ var no_implicit_key_default = createRule({
   create(context) {
     function getReportDescriptor(node) {
       const initialScope = context.sourceCode.getScope(node);
-      const keyPropFound = JSX8.findPropInAttributes(node.attributes, initialScope)("key");
+      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"
       );
@@ -1574,7 +1518,7 @@ var no_missing_key_default = createRule({
       switch (node.type) {
         case AST_NODE_TYPES.JSXElement: {
           const initialScope = context.sourceCode.getScope(node);
-          if (!JSX8.hasProp(node.openingElement.attributes, "key", initialScope)) {
+          if (!JSX7.hasProp(node.openingElement.attributes, "key", initialScope)) {
             return O.some({
               messageId: "noMissingKey",
               node
@@ -1623,7 +1567,7 @@ var no_missing_key_default = createRule({
         if (elements.length === 0) return;
         const initialScope = context.sourceCode.getScope(node);
         for (const element of elements) {
-          if (!JSX8.hasProp(element.openingElement.attributes, "key", initialScope)) {
+          if (!JSX7.hasProp(element.openingElement.attributes, "key", initialScope)) {
             context.report({
               messageId: "noMissingKey",
               node: element
@@ -1711,7 +1655,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(JSX8.traverseUpProp(node, (n) => n.value?.type === AST_NODE_TYPES.JSXExpressionContainer));
+          const isInsideJSXPropValue = component.parent.type === AST_NODE_TYPES.JSXAttribute || O.isSome(JSX7.traverseUpProp(node, (n) => n.value?.type === AST_NODE_TYPES.JSXExpressionContainer));
           if (isInsideJSXPropValue) {
             if (!isDeclaredInRenderPropLoose(component)) {
               context.report({
@@ -2335,7 +2279,7 @@ var LIFECYCLE_METHODS = /* @__PURE__ */ new Set([
   "UNSAFE_componentWillUpdate"
 ]);
 function getName3(node) {
-  if (node.type === AST_NODE_TYPES.TSAsExpression) {
+  if (AST11.isTypeExpression(node)) {
     return getName3(node.expression);
   }
   if (node.type === AST_NODE_TYPES.Identifier || node.type === AST_NODE_TYPES.PrivateIdentifier) {
@@ -2449,7 +2393,7 @@ var RULE_FEATURES43 = [
   "CHK"
 ];
 function getName4(node) {
-  if (node.type === AST_NODE_TYPES.TSAsExpression) {
+  if (AST11.isTypeExpression(node)) {
     return getName4(node.expression);
   }
   if (node.type === AST_NODE_TYPES.Identifier || node.type === AST_NODE_TYPES.PrivateIdentifier) {
@@ -2592,18 +2536,18 @@ var no_unused_state_default = createRule({
 var RULE_NAME44 = "no-useless-fragment";
 function check(node, context, allowExpressions) {
   const initialScope = context.sourceCode.getScope(node);
-  if (JSX8.isKeyedElement(node, initialScope)) return;
-  if (JSX8.isBuiltInElement(node.parent)) context.report({ messageId: "noUselessFragmentInBuiltIn", 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 });
   const isChildren = AST11.isOneOf([AST_NODE_TYPES.JSXElement, AST_NODE_TYPES.JSXFragment])(node.parent);
   const [firstChildren] = node.children;
-  if (allowExpressions && node.children.length === 1 && JSX8.isLiteral(firstChildren) && !isChildren) return;
+  if (allowExpressions && node.children.length === 1 && JSX7.isLiteral(firstChildren) && !isChildren) return;
   if (!allowExpressions && isChildren) {
     return context.report({ messageId: "noUselessFragment", node });
   } else if (!allowExpressions && !isChildren && node.children.length === 1) {
     return context.report({ messageId: "noUselessFragment", node });
   }
-  const nonPaddingChildren = node.children.filter((child) => !JSX8.isPaddingSpaces(child));
+  const nonPaddingChildren = node.children.filter((child) => !JSX7.isPaddingSpaces(child));
   if (nonPaddingChildren.length > 1) return;
   if (nonPaddingChildren.length === 0) return context.report({ messageId: "noUselessFragment", node });
   const [first] = nonPaddingChildren;
@@ -2845,7 +2789,7 @@ var prefer_shorthand_boolean_default = createRule({
   create(context) {
     function getReportDescriptor(node) {
       const { value } = node;
-      const propName = JSX8.getPropName(node);
+      const propName = JSX7.getPropName(node);
       const hasValueTrue = value?.type === AST_NODE_TYPES.JSXExpressionContainer && value.expression.type === AST_NODE_TYPES.Literal && value.expression.value === true;
       if (!hasValueTrue) return O.none();
       return O.some({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "1.22.2-next.6",
+  "version": "1.22.2-next.8",
   "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.22.2-next.6",
-    "@eslint-react/core": "1.22.2-next.6",
-    "@eslint-react/eff": "1.22.2-next.6",
-    "@eslint-react/jsx": "1.22.2-next.6",
-    "@eslint-react/shared": "1.22.2-next.6",
-    "@eslint-react/types": "1.22.2-next.6",
-    "@eslint-react/var": "1.22.2-next.6"
+    "@eslint-react/ast": "1.22.2-next.8",
+    "@eslint-react/core": "1.22.2-next.8",
+    "@eslint-react/jsx": "1.22.2-next.8",
+    "@eslint-react/eff": "1.22.2-next.8",
+    "@eslint-react/shared": "1.22.2-next.8",
+    "@eslint-react/types": "1.22.2-next.8",
+    "@eslint-react/var": "1.22.2-next.8"
   },
   "devDependencies": {
     "@types/react": "^19.0.2",