@eslint-react/ast 4.0.2-beta.3 → 4.0.2-beta.5

package/dist/index.d.ts

@@ -32,6 +32,9 @@ type TSESTreeArrayTupleType = TSESTree$1.TSArrayType | TSESTree$1.TSTupleType;
 type TSESTreeProperty = TSESTree$1.PropertyDefinition | TSESTree$1.TSIndexSignature | TSESTree$1.TSParameterProperty | TSESTree$1.TSPropertySignature;
 /**
  * Represents all JSX-related nodes in TSESTree
+ * Note: This type is for type-level operations. The isJSX() runtime guard in node-is.ts
+ * has a slightly different set of nodes as it includes JSXExpressionContainer which is
+ * commonly needed for runtime checks but not included in this type union.
  */
 type TSESTreeJSX = TSESTree$1.JSXAttribute | TSESTree$1.JSXChild | TSESTree$1.JSXClosingElement | TSESTree$1.JSXClosingFragment | TSESTree$1.JSXEmptyExpression | TSESTree$1.JSXIdentifierToken | TSESTree$1.JSXOpeningElement | TSESTree$1.JSXOpeningFragment | TSESTree$1.JSXSpreadAttribute | TSESTree$1.JSXTagNameExpression | TSESTree$1.JSXTextToken;
 type TSESTreeJSXElementLike = TSESTree$1.JSXElement | TSESTree$1.JSXFragment;
@@ -105,9 +108,12 @@ declare function isDirectiveKind(kind: string): kind is DirectiveKind;
 //#endregion
 //#region src/directive-name.d.ts
 /**
- * Check if a string is a directive name
+ * Check if a string is a directive name (heuristic check)
+ * Note: This is a broad check that matches any string starting with "use ".
+ * For strict validation against known directives, use isDirectiveKind.
  * @param name The string to check
- * @returns True if the string is a directive name, false otherwise
+ * @returns True if the string looks like a directive name, false otherwise
+ * @see isDirectiveKind
  */
 declare function isDirectiveName(name: string): boolean;
 //#endregion
@@ -164,8 +170,8 @@ declare const getNestedCallExpressions: (node: TSESTree.Node) => TSESTree.CallEx
  */
 declare function getFileDirectives(node: TSESTree.Program): TSESTreeDirective[];
 /**
- * Check if a directive with the given name exists in the file or function directives
- * @param node The program or function AST node
+ * Check if a directive with the given name exists in the file directives
+ * @param node The program AST node
  * @param name The directive name to check (ex: "use strict", "use memo", "use no memo")
  * @returns True if the directive exists, false otherwise
  */
@@ -205,7 +211,7 @@ type FunctionID = ReturnType<typeof getFunctionId>;
  * Represents various AST paths for React component function declarations
  * Each tuple type represents a specific component definition pattern
  */
-type FunctionInitPath = readonly [TSESTree.FunctionDeclaration] | readonly [TSESTree.VariableDeclaration, TSESTree.VariableDeclarator, TSESTreeFunction] | readonly [TSESTree.VariableDeclaration, TSESTree.VariableDeclarator, TSESTree.CallExpression, TSESTreeFunction] | readonly [TSESTree.VariableDeclaration, TSESTree.VariableDeclarator, TSESTree.CallExpression, TSESTree.CallExpression, TSESTreeFunction] | readonly [TSESTree.VariableDeclaration, TSESTree.VariableDeclarator, TSESTree.ObjectExpression, TSESTree.Property, TSESTreeFunction] | readonly [TSESTree.VariableDeclaration, TSESTree.VariableDeclarator, TSESTree.ObjectExpression, TSESTree.Property, TSESTree.CallExpression, TSESTreeFunction] | readonly [TSESTree.VariableDeclaration, TSESTree.VariableDeclarator, TSESTree.ObjectExpression, TSESTree.Property, TSESTree.CallExpression, TSESTree.CallExpression, TSESTreeFunction] | readonly [TSESTree.ClassDeclaration, TSESTree.ClassBody, TSESTree.MethodDefinition, TSESTreeFunction] | readonly [TSESTree.ClassDeclaration, TSESTree.ClassBody, TSESTree.PropertyDefinition, TSESTreeFunction];
+type FunctionInitPath = readonly [TSESTree.FunctionDeclaration] | readonly [TSESTree.VariableDeclaration, TSESTree.VariableDeclarator, TSESTreeFunction] | readonly [TSESTree.VariableDeclaration, TSESTree.VariableDeclarator, TSESTree.CallExpression, TSESTreeFunction] | readonly [TSESTree.VariableDeclaration, TSESTree.VariableDeclarator, TSESTree.CallExpression, TSESTree.CallExpression, TSESTreeFunction] | readonly [TSESTree.VariableDeclaration, TSESTree.VariableDeclarator, TSESTree.ObjectExpression, TSESTree.Property, TSESTreeFunction] | readonly [TSESTree.VariableDeclaration, TSESTree.VariableDeclarator, TSESTree.ObjectExpression, TSESTree.Property, TSESTree.CallExpression, TSESTreeFunction] | readonly [TSESTree.VariableDeclaration, TSESTree.VariableDeclarator, TSESTree.ObjectExpression, TSESTree.Property, TSESTree.CallExpression, TSESTree.CallExpression, TSESTreeFunction] | readonly [TSESTree.ClassDeclaration | TSESTree.ClassExpression, TSESTree.ClassBody, TSESTree.MethodDefinition, TSESTreeFunction] | readonly [TSESTree.ClassDeclaration | TSESTree.ClassExpression, TSESTree.ClassBody, TSESTree.PropertyDefinition, TSESTreeFunction];
 /**
  * Identifies the initialization path of a function node in the ast.
  * Determine what kind of component declaration pattern the function belongs to.
@@ -250,8 +256,10 @@ declare function isIdentifier(node: TSESTree.Node | null, name?: string): node i
 //#region src/identifier-name.d.ts
 /**
  * Check if a string is a valid JavaScript identifier name
+ * Note: This only checks for ASCII identifiers. Unicode identifiers (e.g., `const 变量 = 1`)
+ * are not supported by this simple regex check.
  * @param name The string to check
- * @returns True if the string is a valid identifier name
+ * @returns True if the string is a valid ASCII identifier name
  */
 declare function isIdentifierName(name: string): boolean;
 //#endregion
@@ -361,8 +369,11 @@ declare const isJSXElementLike: (data: TSESTree.Node | null | undefined) => data
 declare const isJSXTagNameExpression: (node: TSESTree.Node | null | undefined) => node is TSESTree.JSXIdentifier | TSESTree.JSXMemberExpression | TSESTree.JSXNamespacedName;
 /**
  * Check if a node is a JSX-related node
+ * Note: This runtime guard includes JSXExpressionContainer which is commonly needed
+ * for AST traversals but not included in the TSESTreeJSX type union.
  * @param node The node to check
  * @returns True if the node is a JSX node
+ * @see TSESTreeJSX
  */
 declare const isJSX: (node: TSESTree.Node | null | undefined) => node is TSESTree.JSXAttribute | TSESTree.JSXClosingElement | TSESTree.JSXClosingFragment | TSESTree.JSXElement | TSESTree.JSXEmptyExpression | TSESTree.JSXExpressionContainer | TSESTree.JSXFragment | TSESTree.JSXIdentifier | TSESTree.JSXMemberExpression | TSESTree.JSXNamespacedName | TSESTree.JSXOpeningElement | TSESTree.JSXOpeningFragment | TSESTree.JSXSpreadAttribute | TSESTree.JSXSpreadChild | TSESTree.JSXText;
 /**
@@ -515,6 +526,8 @@ declare function getPropertyName(node: TSESTree.Node): string | null;
 //#region src/property-traverse.d.ts
 /**
  * Recursively traverses an object expression's properties to find a property with the specified name
+ * Note: This only handles inline object literals in spread elements. Properties from variables
+ * (e.g., `{...props}` where props is a variable) are not resolved.
  * @param properties The properties of the object expression to traverse
  * @param name The name of the property to find
  * @returns The matching property node, or null if not found

package/dist/index.js

@@ -18,9 +18,12 @@ function getClassId(node) {
 //#endregion
 //#region src/directive-name.ts
 /**
-* Check if a string is a directive name
+* Check if a string is a directive name (heuristic check)
+* Note: This is a broad check that matches any string starting with "use ".
+* For strict validation against known directives, use isDirectiveKind.
 * @param name The string to check
-* @returns True if the string is a directive name, false otherwise
+* @returns True if the string looks like a directive name, false otherwise
+* @see isDirectiveKind
 */
 function isDirectiveName(name) {
 	return name.startsWith("use ") && name.length > 4;
@@ -282,8 +285,11 @@ const isJSXTagNameExpression = isOneOf([
 ]);
 /**
 * Check if a node is a JSX-related node
+* Note: This runtime guard includes JSXExpressionContainer which is commonly needed
+* for AST traversals but not included in the TSESTreeJSX type union.
 * @param node The node to check
 * @returns True if the node is a JSX node
+* @see TSESTreeJSX
 */
 const isJSX = isOneOf([
 	AST_NODE_TYPES.JSXAttribute,
@@ -596,8 +602,8 @@ function getFileDirectives(node) {
 	return directives;
 }
 /**
-* Check if a directive with the given name exists in the file or function directives
-* @param node The program or function AST node
+* Check if a directive with the given name exists in the file directives
+* @param node The program AST node
 * @param name The directive name to check (ex: "use strict", "use memo", "use no memo")
 * @returns True if the directive exists, false otherwise
 */
@@ -693,13 +699,13 @@ function getFunctionInitPath(node) {
 			parent,
 			node
 		];
-		case parent.type === AST_NODE_TYPES.MethodDefinition && parent.parent.parent.type === AST_NODE_TYPES.ClassDeclaration: return [
+		case parent.type === AST_NODE_TYPES.MethodDefinition: return [
 			parent.parent.parent,
 			parent.parent,
 			parent,
 			node
 		];
-		case parent.type === AST_NODE_TYPES.PropertyDefinition && parent.parent.parent.type === AST_NODE_TYPES.ClassDeclaration: return [
+		case parent.type === AST_NODE_TYPES.PropertyDefinition: return [
 			parent.parent.parent,
 			parent.parent,
 			parent,
@@ -761,8 +767,10 @@ function isIdentifier(node, name) {
 //#region src/identifier-name.ts
 /**
 * Check if a string is a valid JavaScript identifier name
+* Note: This only checks for ASCII identifiers. Unicode identifiers (e.g., `const 变量 = 1`)
+* are not supported by this simple regex check.
 * @param name The string to check
-* @returns True if the string is a valid identifier name
+* @returns True if the string is a valid ASCII identifier name
 */
 function isIdentifierName(name) {
 	return /^[A-Z$_][\w$]*$/i.test(name);
@@ -818,7 +826,7 @@ const isNodeEqual = dual(2, (a, b) => {
 		case a.type === AST_NODE_TYPES.PrivateIdentifier && b.type === AST_NODE_TYPES.PrivateIdentifier: return a.name === b.name;
 		case a.type === AST_NODE_TYPES.MemberExpression && b.type === AST_NODE_TYPES.MemberExpression: return isNodeEqual(a.property, b.property) && isNodeEqual(a.object, b.object);
 		case a.type === AST_NODE_TYPES.JSXAttribute && b.type === AST_NODE_TYPES.JSXAttribute:
-			if (a.name.name !== b.name.name) return false;
+			if (!isNodeEqual(a.name, b.name)) return false;
 			if (a.value == null || b.value == null) return a.value === b.value;
 			return isNodeEqual(a.value, b.value);
 		case a.type === AST_NODE_TYPES.ThisExpression && b.type === AST_NODE_TYPES.ThisExpression: return true;
@@ -954,6 +962,8 @@ function getPropertyName(node) {
 //#region src/property-traverse.ts
 /**
 * Recursively traverses an object expression's properties to find a property with the specified name
+* Note: This only handles inline object literals in spread elements. Properties from variables
+* (e.g., `{...props}` where props is a variable) are not resolved.
 * @param properties The properties of the object expression to traverse
 * @param name The name of the property to find
 * @returns The matching property node, or null if not found

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eslint-react/ast",
-  "version": "4.0.2-beta.3",
+  "version": "4.0.2-beta.5",
   "description": "ESLint React's TSESTree AST utility module.",
   "homepage": "https://github.com/Rel1cx/eslint-react",
   "bugs": {