@eslint-react/core 3.0.0-next.12 → 3.0.0-next.14

package/dist/index.d.ts

@@ -108,6 +108,68 @@ declare const ComponentDetectionHint: {
  * Default component detection hint
  */
 declare const DEFAULT_COMPONENT_DETECTION_HINT: bigint;
+/**
+ * Check whether given node is a render method of a class component
+ * @example
+ * ```tsx
+ * class Component extends React.Component {
+ *   renderHeader = () => <div />;
+ *   renderFooter = () => <div />;
+ * }
+ * ```
+ * @param node The AST node to check
+ * @returns `true` if node is a render function, `false` if not
+ */
+declare function isRenderMethodLike(node: TSESTree.Node): node is ast.TSESTreeMethodOrProperty;
+/**
+ * Unsafe check whether given node is a render function
+ * ```tsx
+ * const renderRow = () => <div />
+ * `                 ^^^^^^^^^^^^`
+ * _ = <Component renderRow={() => <div />} />
+ * `                         ^^^^^^^^^^^^^   `
+ * ```
+ * @param context The rule context
+ * @param node The AST node to check
+ * @returns `true` if node is a render function, `false` if not
+ */
+declare function isRenderFunctionLoose(context: RuleContext, node: TSESTree.Node): node is ast.TSESTreeFunction;
+/**
+ * Unsafe check whether given JSXAttribute is a render prop
+ * ```tsx
+ * _ = <Component renderRow={() => <div />} />
+ * `              ^^^^^^^^^^^^^^^^^^^^^^^^^  `
+ * ```
+ * @param context The rule context
+ * @param node The AST node to check
+ * @returns `true` if node is a render prop, `false` if not
+ */
+declare function isRenderPropLoose(context: RuleContext, node: TSESTree.JSXAttribute): boolean;
+/**
+ * Unsafe check whether given node is declared directly inside a render property
+ * ```tsx
+ * const rows = { render: () => <div /> }
+ * `                      ^^^^^^^^^^^^^ `
+ * _ = <Component rows={ [{ render: () => <div /> }] } />
+ * `                                ^^^^^^^^^^^^^       `
+ *  ```
+ * @internal
+ * @param node The AST node to check
+ * @returns `true` if component is declared inside a render property, `false` if not
+ */
+declare function isDirectValueOfRenderPropertyLoose(node: TSESTree.Node): boolean;
+/**
+ * Unsafe check whether given node is declared inside a render prop
+ * ```tsx
+ * _ = <Component renderRow={"node"} />
+ * `                         ^^^^^^   `
+ * _ = <Component rows={ [{ render: "node" }] } />
+ * `                                ^^^^^^       `
+ * ```
+ * @param node The AST node to check
+ * @returns `true` if component is declared inside a render prop, `false` if not
+ */
+declare function isDeclaredInRenderPropLoose(node: TSESTree.Node): boolean;
 /**
  * Determine if a function node represents a valid React component definition
  *
@@ -404,72 +466,6 @@ declare function isComponentNameLoose(name: string): boolean;
  */
 declare function isFunctionWithLooseComponentName(context: RuleContext, fn: ast.TSESTreeFunction, allowNone?: boolean): boolean;
 //#endregion
-//#region src/component/component-render-method.d.ts
-/**
- * Check whether given node is a render method of a class component
- * @example
- * ```tsx
- * class Component extends React.Component {
- *   renderHeader = () => <div />;
- *   renderFooter = () => <div />;
- * }
- * ```
- * @param node The AST node to check
- * @returns `true` if node is a render function, `false` if not
- */
-declare function isRenderMethodLike(node: TSESTree.Node): node is ast.TSESTreeMethodOrProperty;
-//#endregion
-//#region src/component/component-render-prop.d.ts
-/**
- * Unsafe check whether given node is a render function
- * ```tsx
- * const renderRow = () => <div />
- * `                 ^^^^^^^^^^^^`
- * _ = <Component renderRow={() => <div />} />
- * `                         ^^^^^^^^^^^^^   `
- * ```
- * @param context The rule context
- * @param node The AST node to check
- * @returns `true` if node is a render function, `false` if not
- */
-declare function isRenderFunctionLoose(context: RuleContext, node: TSESTree.Node): node is ast.TSESTreeFunction;
-/**
- * Unsafe check whether given JSXAttribute is a render prop
- * ```tsx
- * _ = <Component renderRow={() => <div />} />
- * `              ^^^^^^^^^^^^^^^^^^^^^^^^^  `
- * ```
- * @param context The rule context
- * @param node The AST node to check
- * @returns `true` if node is a render prop, `false` if not
- */
-declare function isRenderPropLoose(context: RuleContext, node: TSESTree.JSXAttribute): boolean;
-/**
- * Unsafe check whether given node is declared directly inside a render property
- * ```tsx
- * const rows = { render: () => <div /> }
- * `                      ^^^^^^^^^^^^^ `
- * _ = <Component rows={ [{ render: () => <div /> }] } />
- * `                                ^^^^^^^^^^^^^       `
- *  ```
- * @internal
- * @param node The AST node to check
- * @returns `true` if component is declared inside a render property, `false` if not
- */
-declare function isDirectValueOfRenderPropertyLoose(node: TSESTree.Node): boolean;
-/**
- * Unsafe check whether given node is declared inside a render prop
- * ```tsx
- * _ = <Component renderRow={"node"} />
- * `                         ^^^^^^   `
- * _ = <Component rows={ [{ render: "node" }] } />
- * `                                ^^^^^^       `
- * ```
- * @param node The AST node to check
- * @returns `true` if component is declared inside a render prop, `false` if not
- */
-declare function isDeclaredInRenderPropLoose(node: TSESTree.Node): boolean;
-//#endregion
 //#region src/component/component-wrapper.d.ts
 /**
  * Check if the node is a call expression for a component wrapper

package/dist/index.js

@@ -1,4 +1,4 @@
-import { findImportSource, findProperty, findVariable, getVariableDefinitionNode } from "@eslint-react/var";
+import { findImportSource, findProperty, findVariable, getVariableInitializer } from "@eslint-react/var";
 import * as ast from "@eslint-react/ast";
 import { constFalse, constTrue, dual, flip, getOrElseUpdate, identity, unit } from "@eslint-react/eff";
 import { AST_NODE_TYPES } from "@typescript-eslint/types";
@@ -369,7 +369,7 @@ function getJsxAttribute(context, node, initialScope) {
 			if (attr.type === AST_NODE_TYPES.JSXAttribute) return getJsxAttributeName(context, attr) === name;
 			switch (attr.argument.type) {
 				case AST_NODE_TYPES.Identifier: {
-					const variableNode = getVariableDefinitionNode(findVariable(attr.argument.name, scope), 0);
+					const variableNode = getVariableInitializer(findVariable(attr.argument.name, scope), 0);
 					if (variableNode?.type === AST_NODE_TYPES.ObjectExpression) return findProperty(name, variableNode.properties, scope) != null;
 					return false;
 				}
@@ -597,7 +597,7 @@ function isJsxLike(code, node, hint = DEFAULT_JSX_DETECTION_HINT) {
 			const { name } = node;
 			if (name === "undefined") return !(hint & JsxDetectionHint.DoNotIncludeJsxWithUndefinedValue);
 			if (ast.isJSXTagNameExpression(node)) return true;
-			return isJsxLike(code, getVariableDefinitionNode(findVariable(name, code.getScope(node)), 0), hint);
+			return isJsxLike(code, getVariableInitializer(findVariable(name, code.getScope(node)), 0), hint);
 		}
 	}
 	return false;
@@ -793,24 +793,6 @@ function isFunctionWithLooseComponentName(context, fn, allowNone = false) {
 	return false;
 }
 
-//#endregion
-//#region src/component/component-render-method.ts
-/**
-* Check whether given node is a render method of a class component
-* @example
-* ```tsx
-* class Component extends React.Component {
-*   renderHeader = () => <div />;
-*   renderFooter = () => <div />;
-* }
-* ```
-* @param node The AST node to check
-* @returns `true` if node is a render function, `false` if not
-*/
-function isRenderMethodLike(node) {
-	return ast.isMethodOrProperty(node) && node.key.type === AST_NODE_TYPES.Identifier && node.key.name.startsWith("render") && node.parent.parent.type === AST_NODE_TYPES.ClassDeclaration;
-}
-
 //#endregion
 //#region src/component/component-detection.ts
 /**
@@ -831,6 +813,21 @@ const ComponentDetectionHint = {
 */
 const DEFAULT_COMPONENT_DETECTION_HINT = 0n | ComponentDetectionHint.DoNotIncludeJsxWithBigIntValue | ComponentDetectionHint.DoNotIncludeJsxWithBooleanValue | ComponentDetectionHint.DoNotIncludeJsxWithNumberValue | ComponentDetectionHint.DoNotIncludeJsxWithStringValue | ComponentDetectionHint.DoNotIncludeJsxWithUndefinedValue | ComponentDetectionHint.DoNotIncludeFunctionDefinedAsArrayFlatMapCallback | ComponentDetectionHint.DoNotIncludeFunctionDefinedAsArrayMapCallback | ComponentDetectionHint.DoNotIncludeFunctionDefinedInArrayExpression | ComponentDetectionHint.DoNotIncludeFunctionDefinedInArrayPattern | ComponentDetectionHint.RequireAllArrayElementsToBeJsx | ComponentDetectionHint.RequireBothBranchesOfConditionalExpressionToBeJsx | ComponentDetectionHint.RequireBothSidesOfLogicalExpressionToBeJsx;
 /**
+* Check whether given node is a render method of a class component
+* @example
+* ```tsx
+* class Component extends React.Component {
+*   renderHeader = () => <div />;
+*   renderFooter = () => <div />;
+* }
+* ```
+* @param node The AST node to check
+* @returns `true` if node is a render function, `false` if not
+*/
+function isRenderMethodLike(node) {
+	return ast.isMethodOrProperty(node) && node.key.type === AST_NODE_TYPES.Identifier && node.key.name.startsWith("render") && node.parent.parent.type === AST_NODE_TYPES.ClassDeclaration;
+}
+/**
 * Check if the given node is a function within a render method of a class component
 *
 * @param node The AST node to check
@@ -849,6 +846,77 @@ function isRenderMethodCallback(node) {
 	return greatGrandparent != null && isRenderMethodLike(parent) && isClassComponent(greatGrandparent);
 }
 /**
+* Unsafe check whether given node is a render function
+* ```tsx
+* const renderRow = () => <div />
+* `                 ^^^^^^^^^^^^`
+* _ = <Component renderRow={() => <div />} />
+* `                         ^^^^^^^^^^^^^   `
+* ```
+* @param context The rule context
+* @param node The AST node to check
+* @returns `true` if node is a render function, `false` if not
+*/
+function isRenderFunctionLoose(context, node) {
+	if (!ast.isFunction(node)) return false;
+	const id = ast.getFunctionId(node);
+	switch (true) {
+		case id?.type === AST_NODE_TYPES.Identifier: return id.name.startsWith("render");
+		case id?.type === AST_NODE_TYPES.MemberExpression && id.property.type === AST_NODE_TYPES.Identifier: return id.property.name.startsWith("render");
+		case node.parent.type === AST_NODE_TYPES.JSXExpressionContainer && node.parent.parent.type === AST_NODE_TYPES.JSXAttribute && node.parent.parent.name.type === AST_NODE_TYPES.JSXIdentifier: return node.parent.parent.name.name.startsWith("render");
+	}
+	return false;
+}
+/**
+* Unsafe check whether given JSXAttribute is a render prop
+* ```tsx
+* _ = <Component renderRow={() => <div />} />
+* `              ^^^^^^^^^^^^^^^^^^^^^^^^^  `
+* ```
+* @param context The rule context
+* @param node The AST node to check
+* @returns `true` if node is a render prop, `false` if not
+*/
+function isRenderPropLoose(context, node) {
+	if (node.name.type !== AST_NODE_TYPES.JSXIdentifier) return false;
+	return node.name.name.startsWith("render") && node.value?.type === AST_NODE_TYPES.JSXExpressionContainer && isRenderFunctionLoose(context, node.value.expression);
+}
+/**
+* Unsafe check whether given node is declared directly inside a render property
+* ```tsx
+* const rows = { render: () => <div /> }
+* `                      ^^^^^^^^^^^^^ `
+* _ = <Component rows={ [{ render: () => <div /> }] } />
+* `                                ^^^^^^^^^^^^^       `
+*  ```
+* @internal
+* @param node The AST node to check
+* @returns `true` if component is declared inside a render property, `false` if not
+*/
+function isDirectValueOfRenderPropertyLoose(node) {
+	const matching = (node) => {
+		return node.type === AST_NODE_TYPES.Property && node.key.type === AST_NODE_TYPES.Identifier && node.key.name.startsWith("render");
+	};
+	return matching(node) || node.parent != null && matching(node.parent);
+}
+/**
+* Unsafe check whether given node is declared inside a render prop
+* ```tsx
+* _ = <Component renderRow={"node"} />
+* `                         ^^^^^^   `
+* _ = <Component rows={ [{ render: "node" }] } />
+* `                                ^^^^^^       `
+* ```
+* @param node The AST node to check
+* @returns `true` if component is declared inside a render prop, `false` if not
+*/
+function isDeclaredInRenderPropLoose(node) {
+	if (isDirectValueOfRenderPropertyLoose(node)) return true;
+	const parent = ast.findParentNode(node, ast.is(AST_NODE_TYPES.JSXExpressionContainer))?.parent;
+	if (parent?.type !== AST_NODE_TYPES.JSXAttribute) return false;
+	return parent.name.type === AST_NODE_TYPES.JSXIdentifier && parent.name.name.startsWith("render");
+}
+/**
 * Check if a function node should be excluded based on provided detection hints
 *
 * @param node The function node to check
@@ -1132,80 +1200,6 @@ function isComponentWillUnmountCallback(node) {
 	return ast.isFunction(node) && isComponentWillUnmount(node.parent) && node.parent.value === node;
 }
 
-//#endregion
-//#region src/component/component-render-prop.ts
-/**
-* Unsafe check whether given node is a render function
-* ```tsx
-* const renderRow = () => <div />
-* `                 ^^^^^^^^^^^^`
-* _ = <Component renderRow={() => <div />} />
-* `                         ^^^^^^^^^^^^^   `
-* ```
-* @param context The rule context
-* @param node The AST node to check
-* @returns `true` if node is a render function, `false` if not
-*/
-function isRenderFunctionLoose(context, node) {
-	if (!ast.isFunction(node)) return false;
-	const id = ast.getFunctionId(node);
-	switch (true) {
-		case id?.type === AST_NODE_TYPES.Identifier: return id.name.startsWith("render");
-		case id?.type === AST_NODE_TYPES.MemberExpression && id.property.type === AST_NODE_TYPES.Identifier: return id.property.name.startsWith("render");
-		case node.parent.type === AST_NODE_TYPES.JSXExpressionContainer && node.parent.parent.type === AST_NODE_TYPES.JSXAttribute && node.parent.parent.name.type === AST_NODE_TYPES.JSXIdentifier: return node.parent.parent.name.name.startsWith("render");
-	}
-	return false;
-}
-/**
-* Unsafe check whether given JSXAttribute is a render prop
-* ```tsx
-* _ = <Component renderRow={() => <div />} />
-* `              ^^^^^^^^^^^^^^^^^^^^^^^^^  `
-* ```
-* @param context The rule context
-* @param node The AST node to check
-* @returns `true` if node is a render prop, `false` if not
-*/
-function isRenderPropLoose(context, node) {
-	if (node.name.type !== AST_NODE_TYPES.JSXIdentifier) return false;
-	return node.name.name.startsWith("render") && node.value?.type === AST_NODE_TYPES.JSXExpressionContainer && isRenderFunctionLoose(context, node.value.expression);
-}
-/**
-* Unsafe check whether given node is declared directly inside a render property
-* ```tsx
-* const rows = { render: () => <div /> }
-* `                      ^^^^^^^^^^^^^ `
-* _ = <Component rows={ [{ render: () => <div /> }] } />
-* `                                ^^^^^^^^^^^^^       `
-*  ```
-* @internal
-* @param node The AST node to check
-* @returns `true` if component is declared inside a render property, `false` if not
-*/
-function isDirectValueOfRenderPropertyLoose(node) {
-	const matching = (node) => {
-		return node.type === AST_NODE_TYPES.Property && node.key.type === AST_NODE_TYPES.Identifier && node.key.name.startsWith("render");
-	};
-	return matching(node) || node.parent != null && matching(node.parent);
-}
-/**
-* Unsafe check whether given node is declared inside a render prop
-* ```tsx
-* _ = <Component renderRow={"node"} />
-* `                         ^^^^^^   `
-* _ = <Component rows={ [{ render: "node" }] } />
-* `                                ^^^^^^       `
-* ```
-* @param node The AST node to check
-* @returns `true` if component is declared inside a render prop, `false` if not
-*/
-function isDeclaredInRenderPropLoose(node) {
-	if (isDirectValueOfRenderPropertyLoose(node)) return true;
-	const parent = ast.findParentNode(node, ast.is(AST_NODE_TYPES.JSXExpressionContainer))?.parent;
-	if (parent?.type !== AST_NODE_TYPES.JSXAttribute) return false;
-	return parent.name.type === AST_NODE_TYPES.JSXIdentifier && parent.name.name.startsWith("render");
-}
-
 //#endregion
 //#region src/hierarchy/find-enclosing-component-or-hook.ts
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eslint-react/core",
-  "version": "3.0.0-next.12",
+  "version": "3.0.0-next.14",
   "description": "ESLint React's ESLint utility module for static analysis of React core APIs and patterns.",
   "homepage": "https://github.com/Rel1cx/eslint-react",
   "bugs": {
@@ -34,10 +34,10 @@
     "@typescript-eslint/types": "canary",
     "@typescript-eslint/utils": "canary",
     "ts-pattern": "^5.9.0",
-    "@eslint-react/ast": "3.0.0-next.12",
-    "@eslint-react/eff": "3.0.0-next.12",
-    "@eslint-react/var": "3.0.0-next.12",
-    "@eslint-react/shared": "3.0.0-next.12"
+    "@eslint-react/ast": "3.0.0-next.14",
+    "@eslint-react/eff": "3.0.0-next.14",
+    "@eslint-react/shared": "3.0.0-next.14",
+    "@eslint-react/var": "3.0.0-next.14"
   },
   "devDependencies": {
     "tsdown": "^0.20.3",