eslint-plugin-react-x 1.24.0-next.6 → 1.24.0-next.9

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.24.0-next.6";
+var version = "1.24.0-next.9";
 var createRule = shared.createRuleForPlugin("x");
 
 // src/rules/avoid-shorthand-boolean.ts
@@ -72,7 +72,7 @@ var avoid_shorthand_boolean_default = createRule({
             messageId: "avoidShorthandBoolean",
             node,
             data: {
-              propName: JSX7__namespace.getPropName(node)
+              propName: JSX7__namespace.getAttributeName(node)
             },
             fix: (fixer) => fixer.insertTextAfter(node.name, `={true}`)
           });
@@ -610,15 +610,15 @@ var no_children_prop_default = createRule({
   create(context) {
     return {
       JSXElement(node) {
-        const prop = JSX7__namespace.getProp(
+        const attribute = JSX7__namespace.getAttributeNode(
           "children",
           context.sourceCode.getScope(node),
           node.openingElement.attributes
         );
-        if (prop != null) {
+        if (attribute != null) {
           context.report({
             messageId: "noChildrenProp",
-            node: prop
+            node: attribute
           });
         }
       }
@@ -661,12 +661,6 @@ var RULE_NAME12 = "no-class-component";
 var RULE_FEATURES12 = [
   "CHK"
 ];
-function isComponentDidCatch(node) {
-  return (node.type === types.AST_NODE_TYPES.MethodDefinition || node.type === types.AST_NODE_TYPES.PropertyDefinition) && !node.static && node.key.type === types.AST_NODE_TYPES.Identifier && node.key.name === "componentDidCatch";
-}
-function isGetDerivedStateFromError(node) {
-  return (node.type === types.AST_NODE_TYPES.MethodDefinition || node.type === types.AST_NODE_TYPES.PropertyDefinition) && node.static && node.key.type === types.AST_NODE_TYPES.Identifier && node.key.name === "getDerivedStateFromError";
-}
 var no_class_component_default = createRule({
   meta: {
     type: "problem",
@@ -690,7 +684,7 @@ var no_class_component_default = createRule({
       "Program:exit"(node) {
         const components = ctx.getAllComponents(node);
         for (const { name: name2 = "anonymous", node: component } of components.values()) {
-          if (component.body.body.some((m) => isComponentDidCatch(m) || isGetDerivedStateFromError(m))) {
+          if (component.body.body.some((m) => core.isComponentDidCatch(m) || core.isGetDerivedStateFromError(m))) {
             continue;
           }
           context.report({
@@ -1444,7 +1438,7 @@ var no_implicit_key_default = createRule({
     return {
       JSXOpeningElement(node) {
         const initialScope = context.sourceCode.getScope(node);
-        const keyPropFound = JSX7__namespace.findPropInAttributes("key", initialScope, node.attributes);
+        const keyPropFound = JSX7__namespace.getAttributeNode("key", initialScope, node.attributes);
         const keyPropOnElement = node.attributes.some(
           (n) => n.type === types.AST_NODE_TYPES.JSXAttribute && n.name.type === types.AST_NODE_TYPES.JSXIdentifier && n.name.name === "key"
         );
@@ -1708,7 +1702,7 @@ var no_missing_key_default = createRule({
       switch (node.type) {
         case types.AST_NODE_TYPES.JSXElement: {
           const initialScope = context.sourceCode.getScope(node);
-          if (!JSX7__namespace.hasProp("key", initialScope, node.openingElement.attributes)) {
+          if (!JSX7__namespace.hasAttribute("key", initialScope, node.openingElement.attributes)) {
             return {
               messageId: "noMissingKey",
               node
@@ -1769,7 +1763,7 @@ var no_missing_key_default = createRule({
         }
         const initialScope = context.sourceCode.getScope(node);
         for (const element of elements) {
-          if (!JSX7__namespace.hasProp("key", initialScope, element.openingElement.attributes)) {
+          if (!JSX7__namespace.hasAttribute("key", initialScope, element.openingElement.attributes)) {
             context.report({
               messageId: "noMissingKey",
               node: element
@@ -1871,7 +1865,7 @@ var no_nested_components_default = createRule({
             continue;
           }
           const isInsideProperty = component.parent.type === types.AST_NODE_TYPES.Property;
-          const isInsideJSXPropValue = component.parent.type === types.AST_NODE_TYPES.JSXAttribute || JSX7__namespace.findParentProp(node, (n) => n.value?.type === types.AST_NODE_TYPES.JSXExpressionContainer) != null;
+          const isInsideJSXPropValue = component.parent.type === types.AST_NODE_TYPES.JSXAttribute || JSX7__namespace.findParentAttributeNode(node, (n) => n.value?.type === types.AST_NODE_TYPES.JSXExpressionContainer) != null;
           if (isInsideJSXPropValue) {
             if (!core.isDeclaredInRenderPropLoose(component)) {
               context.report({
@@ -2680,15 +2674,6 @@ function isAssignmentToThisState2(node) {
   const { left } = node;
   return left.type === types.AST_NODE_TYPES.MemberExpression && AST11__namespace.isThisExpression(left.object) && getName4(left.property) === "state";
 }
-var isGetDerivedStateFromProps = tsPattern.isMatching({
-  key: {
-    name: "getDerivedStateFromProps"
-  },
-  static: true,
-  value: {
-    params: tsPattern.P.array()
-  }
-});
 var no_unused_state_default = createRule({
   meta: {
     type: "problem",
@@ -2735,7 +2720,7 @@ var no_unused_state_default = createRule({
         return;
       }
       if (node.static) {
-        if (isGetDerivedStateFromProps(node) && node.value.params.length > 1) {
+        if (core.isGetDerivedStateFromProps(node) && tsPattern.isMatching({ params: [tsPattern.P.nonNullable, ...tsPattern.P.array()] })(node.value)) {
           const defNode = stateDefs.get(currentClass)?.node;
           stateDefs.set(currentClass, { node: defNode, isUsed: true });
         }
@@ -3128,7 +3113,7 @@ var prefer_shorthand_boolean_default = createRule({
     return {
       JSXAttribute(node) {
         const { value } = node;
-        const propName = JSX7__namespace.getPropName(node);
+        const propName = JSX7__namespace.getAttributeName(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;

package/dist/index.mjs

@@ -1,7 +1,7 @@
 import * as JSX7 from '@eslint-react/jsx';
 import { createRuleForPlugin, getSettingsFromContext, unsafeDecodeSettings } from '@eslint-react/shared';
 import * as AST11 from '@eslint-react/ast';
-import { isForwardRefCall, isThisSetState, isClassComponent, isChildrenCount, isChildrenForEach, isChildrenMap, isChildrenOnly, isChildrenToArray, useComponentCollectorLegacy, isCloneElementCall, isCreateRefCall, isComponentName, useComponentCollector, DEFAULT_COMPONENT_HINT, ERFunctionComponentFlag, isChildrenToArrayCall, ERComponentHint, isDirectValueOfRenderPropertyLoose, isDeclaredInRenderPropLoose, isInsideCreateElementProps, isInsideRenderMethod, ERClassComponentFlag, isReactHookCall, isCreateElementCall, isInitializedFromReact } from '@eslint-react/core';
+import { isForwardRefCall, isThisSetState, isClassComponent, isChildrenCount, isChildrenForEach, isChildrenMap, isChildrenOnly, isChildrenToArray, useComponentCollectorLegacy, isComponentDidCatch, isGetDerivedStateFromError, isCloneElementCall, isCreateRefCall, isComponentName, useComponentCollector, DEFAULT_COMPONENT_HINT, ERFunctionComponentFlag, isChildrenToArrayCall, ERComponentHint, isDirectValueOfRenderPropertyLoose, isDeclaredInRenderPropLoose, isInsideCreateElementProps, isInsideRenderMethod, ERClassComponentFlag, isReactHookCall, isCreateElementCall, isGetDerivedStateFromProps, isInitializedFromReact } from '@eslint-react/core';
 import { _, getOrUpdate } from '@eslint-react/eff';
 import { AST_NODE_TYPES } from '@typescript-eslint/types';
 import { isMatching, match, P } from 'ts-pattern';
@@ -15,7 +15,7 @@ import { getTypeImmutability, isUnknown, isImmutable, isReadonlyShallow, isReado
 
 // package.json
 var name = "eslint-plugin-react-x";
-var version = "1.24.0-next.6";
+var version = "1.24.0-next.9";
 var createRule = createRuleForPlugin("x");
 
 // src/rules/avoid-shorthand-boolean.ts
@@ -45,7 +45,7 @@ var avoid_shorthand_boolean_default = createRule({
             messageId: "avoidShorthandBoolean",
             node,
             data: {
-              propName: JSX7.getPropName(node)
+              propName: JSX7.getAttributeName(node)
             },
             fix: (fixer) => fixer.insertTextAfter(node.name, `={true}`)
           });
@@ -583,15 +583,15 @@ var no_children_prop_default = createRule({
   create(context) {
     return {
       JSXElement(node) {
-        const prop = JSX7.getProp(
+        const attribute = JSX7.getAttributeNode(
           "children",
           context.sourceCode.getScope(node),
           node.openingElement.attributes
         );
-        if (prop != null) {
+        if (attribute != null) {
           context.report({
             messageId: "noChildrenProp",
-            node: prop
+            node: attribute
           });
         }
       }
@@ -634,12 +634,6 @@ var RULE_NAME12 = "no-class-component";
 var RULE_FEATURES12 = [
   "CHK"
 ];
-function isComponentDidCatch(node) {
-  return (node.type === AST_NODE_TYPES.MethodDefinition || node.type === AST_NODE_TYPES.PropertyDefinition) && !node.static && node.key.type === AST_NODE_TYPES.Identifier && node.key.name === "componentDidCatch";
-}
-function isGetDerivedStateFromError(node) {
-  return (node.type === AST_NODE_TYPES.MethodDefinition || node.type === AST_NODE_TYPES.PropertyDefinition) && node.static && node.key.type === AST_NODE_TYPES.Identifier && node.key.name === "getDerivedStateFromError";
-}
 var no_class_component_default = createRule({
   meta: {
     type: "problem",
@@ -1417,7 +1411,7 @@ var no_implicit_key_default = createRule({
     return {
       JSXOpeningElement(node) {
         const initialScope = context.sourceCode.getScope(node);
-        const keyPropFound = JSX7.findPropInAttributes("key", initialScope, node.attributes);
+        const keyPropFound = JSX7.getAttributeNode("key", initialScope, node.attributes);
         const keyPropOnElement = node.attributes.some(
           (n) => n.type === AST_NODE_TYPES.JSXAttribute && n.name.type === AST_NODE_TYPES.JSXIdentifier && n.name.name === "key"
         );
@@ -1681,7 +1675,7 @@ var no_missing_key_default = createRule({
       switch (node.type) {
         case AST_NODE_TYPES.JSXElement: {
           const initialScope = context.sourceCode.getScope(node);
-          if (!JSX7.hasProp("key", initialScope, node.openingElement.attributes)) {
+          if (!JSX7.hasAttribute("key", initialScope, node.openingElement.attributes)) {
             return {
               messageId: "noMissingKey",
               node
@@ -1742,7 +1736,7 @@ var no_missing_key_default = createRule({
         }
         const initialScope = context.sourceCode.getScope(node);
         for (const element of elements) {
-          if (!JSX7.hasProp("key", initialScope, element.openingElement.attributes)) {
+          if (!JSX7.hasAttribute("key", initialScope, element.openingElement.attributes)) {
             context.report({
               messageId: "noMissingKey",
               node: element
@@ -1844,7 +1838,7 @@ var no_nested_components_default = createRule({
             continue;
           }
           const isInsideProperty = component.parent.type === AST_NODE_TYPES.Property;
-          const isInsideJSXPropValue = component.parent.type === AST_NODE_TYPES.JSXAttribute || JSX7.findParentProp(node, (n) => n.value?.type === AST_NODE_TYPES.JSXExpressionContainer) != null;
+          const isInsideJSXPropValue = component.parent.type === AST_NODE_TYPES.JSXAttribute || JSX7.findParentAttributeNode(node, (n) => n.value?.type === AST_NODE_TYPES.JSXExpressionContainer) != null;
           if (isInsideJSXPropValue) {
             if (!isDeclaredInRenderPropLoose(component)) {
               context.report({
@@ -2653,15 +2647,6 @@ function isAssignmentToThisState2(node) {
   const { left } = node;
   return left.type === AST_NODE_TYPES.MemberExpression && AST11.isThisExpression(left.object) && getName4(left.property) === "state";
 }
-var isGetDerivedStateFromProps = isMatching({
-  key: {
-    name: "getDerivedStateFromProps"
-  },
-  static: true,
-  value: {
-    params: P.array()
-  }
-});
 var no_unused_state_default = createRule({
   meta: {
     type: "problem",
@@ -2708,7 +2693,7 @@ var no_unused_state_default = createRule({
         return;
       }
       if (node.static) {
-        if (isGetDerivedStateFromProps(node) && node.value.params.length > 1) {
+        if (isGetDerivedStateFromProps(node) && isMatching({ params: [P.nonNullable, ...P.array()] })(node.value)) {
           const defNode = stateDefs.get(currentClass)?.node;
           stateDefs.set(currentClass, { node: defNode, isUsed: true });
         }
@@ -3101,7 +3086,7 @@ var prefer_shorthand_boolean_default = createRule({
     return {
       JSXAttribute(node) {
         const { value } = node;
-        const propName = JSX7.getPropName(node);
+        const propName = JSX7.getAttributeName(node);
         const hasValueTrue = value?.type === AST_NODE_TYPES.JSXExpressionContainer && value.expression.type === AST_NODE_TYPES.Literal && value.expression.value === true;
         if (!hasValueTrue) {
           return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "1.24.0-next.6",
+  "version": "1.24.0-next.9",
   "description": "A set of composable ESLint rules for libraries and frameworks that use React as a UI runtime.",
   "keywords": [
     "react",
@@ -40,23 +40,23 @@
     "./package.json"
   ],
   "dependencies": {
-    "@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",
+    "@typescript-eslint/scope-manager": "^8.20.0",
+    "@typescript-eslint/type-utils": "^8.20.0",
+    "@typescript-eslint/types": "^8.20.0",
+    "@typescript-eslint/utils": "^8.20.0",
     "compare-versions": "^6.1.1",
     "is-immutable-type": "^5.0.1",
     "string-ts": "^2.2.0",
     "ts-pattern": "^5.6.0",
-    "@eslint-react/ast": "1.24.0-next.6",
-    "@eslint-react/core": "1.24.0-next.6",
-    "@eslint-react/eff": "1.24.0-next.6",
-    "@eslint-react/jsx": "1.24.0-next.6",
-    "@eslint-react/shared": "1.24.0-next.6",
-    "@eslint-react/var": "1.24.0-next.6"
+    "@eslint-react/ast": "1.24.0-next.9",
+    "@eslint-react/core": "1.24.0-next.9",
+    "@eslint-react/eff": "1.24.0-next.9",
+    "@eslint-react/shared": "1.24.0-next.9",
+    "@eslint-react/jsx": "1.24.0-next.9",
+    "@eslint-react/var": "1.24.0-next.9"
   },
   "devDependencies": {
-    "@types/react": "^19.0.6",
+    "@types/react": "^19.0.7",
     "@types/react-dom": "^19.0.3",
     "ts-api-utils": "^2.0.0",
     "tsup": "^8.3.5",