release: 2.3.13-beta.3

Author: Rel1cx

VERSION

@@ -1 +1 @@
-2.3.13-next.3
+2.3.13-beta.3

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eslint-react/monorepo",
-  "version": "2.3.13-next.3",
+  "version": "2.3.13-beta.3",
   "private": true,
   "description": "Monorepo for eslint-plugin-react-[x, dom, web-api, hooks-extra, naming-convention].",
   "keywords": [

packages/core/docs/README.md

@@ -120,10 +120,10 @@
 | [getComponentFlagFromInitPath](functions/getComponentFlagFromInitPath.md) | - |
 | [getComponentNameFromId](functions/getComponentNameFromId.md) | Get component name from an identifier or identifier sequence (e.g., MemberExpression) |
 | [getFunctionComponentId](functions/getFunctionComponentId.md) | - |
-| [getJsxAttribute](functions/getJsxAttribute.md) | Get a function to find JSX attributes by name, considering direct attributes and spread attributes. |
+| [getJsxAttribute](functions/getJsxAttribute.md) | Creates a helper function to find a specific JSX attribute by name Handles direct attributes and spread attributes (variables or object literals) |
 | [getJsxAttributeName](functions/getJsxAttributeName.md) | Get the stringified name of a JSX attribute |
-| [getJsxConfigFromAnnotation](functions/getJsxConfigFromAnnotation.md) | Get JsxConfig from pragma comments (annotations) in the source code. |
-| [getJsxConfigFromContext](functions/getJsxConfigFromContext.md) | Get JsxConfig from the rule context by reading compiler options. |
+| [getJsxConfigFromAnnotation](functions/getJsxConfigFromAnnotation.md) | Get JsxConfig from pragma comments (annotations) in the source code |
+| [getJsxConfigFromContext](functions/getJsxConfigFromContext.md) | Get JsxConfig from the rule context by reading compiler options |
 | [getJsxElementType](functions/getJsxElementType.md) | Extracts the element type name from a JSX element or fragment For JSX elements, returns the stringified name (e.g., "div", "Button", "React.Fragment") For JSX fragments, returns an empty string |
 | [getPhaseKindOfFunction](functions/getPhaseKindOfFunction.md) | - |
 | [hasNoneOrLooseComponentName](functions/hasNoneOrLooseComponentName.md) | Check if the function has no name or a loose component name |
@@ -156,7 +156,7 @@
 | [isRenderMethodLike](functions/isRenderMethodLike.md) | Check whether given node is a render method of a class component |
 | [isRenderPropLoose](functions/isRenderPropLoose.md) | Unsafe check whether given JSXAttribute is a render prop `_ = <Component renderRow={() => <div />} /> ` ^^^^^^^^^^^^^^^^^^^^^^^^^ `` |
 | [isUseEffectLikeCall](functions/isUseEffectLikeCall.md) | Detects useEffect calls and variations (useLayoutEffect, etc.) using regex pattern. |
-| [resolveJsxAttributeValue](functions/resolveJsxAttributeValue.md) | - |
+| [resolveJsxAttributeValue](functions/resolveJsxAttributeValue.md) | Resolves the static value of a JSX attribute or spread attribute |
 | [stringifyJsx](functions/stringifyJsx.md) | Incomplete but sufficient stringification of JSX nodes for common use cases |
 | [useComponentCollector](functions/useComponentCollector.md) | Get a ctx and listeners for the rule to collect function components |
 | [useComponentCollectorLegacy](functions/useComponentCollectorLegacy.md) | Get a ctx and listeners object for the rule to collect class components |

packages/core/docs/functions/getJsxAttribute.md

@@ -9,20 +9,19 @@ function getJsxAttribute(
    initialScope?: Scope): (name: string) => JSXAttribute | JSXSpreadAttribute | undefined;
 ```
 
-Get a function to find JSX attributes by name, considering direct attributes and spread attributes.
+Creates a helper function to find a specific JSX attribute by name
+Handles direct attributes and spread attributes (variables or object literals)
 
 ## Parameters
 
 | Parameter | Type | Description |
 | ------ | ------ | ------ |
 | `context` | `RuleContext` | The ESLint rule context |
 | `node` | `JSXElement` | The JSX element node |
-| `initialScope?` | `Scope` | Optional initial scope for variable resolution |
+| `initialScope?` | `Scope` | (Optional) The initial scope to use for variable resolution |
 
 ## Returns
 
-A function that takes an attribute name and returns the corresponding JSX attribute node or undefined
-
 ```ts
 (name: string): JSXAttribute | JSXSpreadAttribute | undefined;
 ```

packages/core/docs/functions/getJsxConfigFromAnnotation.md

@@ -6,16 +6,16 @@
 function getJsxConfigFromAnnotation(context: RuleContext): JsxConfig;
 ```
 
-Get JsxConfig from pragma comments (annotations) in the source code.
+Get JsxConfig from pragma comments (annotations) in the source code
 
 ## Parameters
 
 | Parameter | Type | Description |
 | ------ | ------ | ------ |
-| `context` | `RuleContext` | The RuleContext. |
+| `context` | `RuleContext` | The RuleContext |
 
 ## Returns
 
 [`JsxConfig`](../interfaces/JsxConfig.md)
 
-JsxConfig derived from pragma comments.
+JsxConfig derived from pragma comments

packages/core/docs/functions/getJsxConfigFromContext.md

@@ -11,13 +11,13 @@ function getJsxConfigFromContext(context: RuleContext): {
 };
 ```
 
-Get JsxConfig from the rule context by reading compiler options.
+Get JsxConfig from the rule context by reading compiler options
 
 ## Parameters
 
 | Parameter | Type | Description |
 | ------ | ------ | ------ |
-| `context` | `RuleContext` | The RuleContext. |
+| `context` | `RuleContext` | The RuleContext |
 
 ## Returns
 
@@ -30,7 +30,7 @@ Get JsxConfig from the rule context by reading compiler options.
 }
 ```
 
-JsxConfig derived from compiler options.
+JsxConfig derived from compiler options
 
 | Name | Type |
 | ------ | ------ |

packages/core/docs/functions/resolveJsxAttributeValue.md

@@ -46,12 +46,14 @@ function resolveJsxAttributeValue(context: RuleContext, attribute: TSESTreeJSXAt
 };
 ```
 
+Resolves the static value of a JSX attribute or spread attribute
+
 ## Parameters
 
-| Parameter | Type |
-| ------ | ------ |
-| `context` | `RuleContext` |
-| `attribute` | `TSESTreeJSXAttributeLike` |
+| Parameter | Type | Description |
+| ------ | ------ | ------ |
+| `context` | `RuleContext` | The ESLint rule context |
+| `attribute` | `TSESTreeJSXAttributeLike` | The JSX attribute node to resolve |
 
 ## Returns
 
@@ -95,3 +97,5 @@ function resolveJsxAttributeValue(context: RuleContext, attribute: TSESTreeJSXAt
   `node`: `Expression`;
   `toStatic`: `unknown`;
 \}
+
+An object containing the value kind, the node (if applicable), and a `toStatic` helper

packages/core/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eslint-react/core",
-  "version": "2.3.13-next.3",
+  "version": "2.3.13-beta.3",
   "description": "ESLint React's ESLint utility module for static analysis of React core APIs and patterns.",
   "homepage": "https://github.com/Rel1cx/eslint-react",
   "bugs": {

packages/core/src/jsx/jsx-attribute-value.ts

@@ -18,8 +18,20 @@ export type JsxAttributeValue =
   | { kind: "spreadProps"; node: TSESTree.JSXSpreadAttribute["argument"]; toStatic(name?: string): unknown } // Spread props (e.g., {...props})
   | { kind: "spreadChild"; node: TSESTree.JSXSpreadChild["expression"]; toStatic(): unknown }; // Spread children (e.g., {...["Hello", " ", "spread", " ", "children"]})
 
+/**
+ * Resolves the static value of a JSX attribute or spread attribute
+ *
+ * @param context - The ESLint rule context
+ * @param attribute - The JSX attribute node to resolve
+ * @returns An object containing the value kind, the node (if applicable), and a `toStatic` helper
+ */
 export function resolveJsxAttributeValue(context: RuleContext, attribute: AST.TSESTreeJSXAttributeLike) {
   const initialScope = context.sourceCode.getScope(attribute);
+
+  /**
+   * Handles standard JSX attributes (e.g., prop="value", prop={value}, prop)
+   * @param node The JSX attribute node
+   */
   function handleJsxAttribute(node: TSESTree.JSXAttribute) {
     // Case 1: Boolean attribute with no value (e.g., disabled)
     if (node.value == null) {
@@ -74,6 +86,10 @@ export function resolveJsxAttributeValue(context: RuleContext, attribute: AST.TS
     }
   }
 
+  /**
+   * Handles JSX spread attributes (e.g., {...props})
+   * @param node The JSX spread attribute node
+   */
   function handleJsxSpreadAttribute(node: TSESTree.JSXSpreadAttribute) {
     // For spread attributes (e.g., {...props}), try to extract static value
     return {
@@ -88,6 +104,7 @@ export function resolveJsxAttributeValue(context: RuleContext, attribute: AST.TS
       },
     } as const satisfies JsxAttributeValue;
   }
+
   switch (attribute.type) {
     case T.JSXAttribute:
       return handleJsxAttribute(attribute);

packages/core/src/jsx/jsx-attribute.ts

@@ -7,37 +7,41 @@ import type { Scope } from "@typescript-eslint/scope-manager";
 import { getJsxAttributeName } from "./jsx-attribute-name";
 
 /**
- * Get a function to find JSX attributes by name, considering direct attributes and spread attributes.
+ * Creates a helper function to find a specific JSX attribute by name
+ * Handles direct attributes and spread attributes (variables or object literals)
  * @param context The ESLint rule context
  * @param node The JSX element node
- * @param initialScope Optional initial scope for variable resolution
- * @returns A function that takes an attribute name and returns the corresponding JSX attribute node or undefined
+ * @param initialScope (Optional) The initial scope to use for variable resolution
  */
 export function getJsxAttribute(context: RuleContext, node: TSESTree.JSXElement, initialScope?: Scope) {
   const scope = initialScope ?? context.sourceCode.getScope(node);
   const attributes = node.openingElement.attributes;
+
   /**
-   * Find a JSX attribute by name, considering both direct attributes and spread attributes.
-   * @param name The name of the attribute to find
-   * @returns The JSX attribute node if found, otherwise undefined
+   * Finds the last occurrence of a specific attribute
+   * @param name The attribute name to search for
    */
   return (name: string) => {
     return attributes.findLast((attr) => {
-      // Case 1: Direct JSX attribute (e.g., className="value")
+      // 1. Direct attribute: className="value"
       if (attr.type === T.JSXAttribute) {
         return getJsxAttributeName(context, attr) === name;
       }
+
       switch (attr.argument.type) {
-        // Case 2: Spread from variable (e.g., {...props})
+        // 2. Spread variable: {...props}
         case T.Identifier: {
           const variable = findVariable(attr.argument.name, scope);
           const variableNode = getVariableDefinitionNode(variable, 0);
+
+          // Check if the variable resolves to an object with the target property
           if (variableNode?.type === T.ObjectExpression) {
             return findProperty(name, variableNode.properties, scope) != null;
           }
           return false;
         }
-        // Case 3: Spread from object literal (e.g., {{...{prop: value}}})
+
+        // 3. Spread literal: {{...{prop: value}}}
         case T.ObjectExpression:
           return findProperty(name, attr.argument.properties, scope) != null;
       }