eslint-plugin-react-x 2.0.5-next.3 → 2.0.5-next.5

package/dist/index.js

@@ -115,7 +115,7 @@ const settings = { ...settings$1 };
 //#endregion
 //#region package.json
 var name = "eslint-plugin-react-x";
-var version = "2.0.5-next.3";
+var version = "2.0.5-next.5";
 
 //#endregion
 //#region src/utils/create-rule.ts
@@ -208,10 +208,10 @@ var jsx_key_before_spread_default = createRule({
 	meta: {
 		type: "problem",
 		docs: {
-			description: "Enforces that the 'key' attribute is placed before the spread attribute in JSX elements.",
+			description: "Enforces that the 'key' prop is placed before the spread prop in JSX elements.",
 			[Symbol.for("rule_features")]: RULE_FEATURES$59
 		},
-		messages: { jsxKeyBeforeSpread: "The 'key' attribute must be placed before the spread attribute." },
+		messages: { jsxKeyBeforeSpread: "The 'key' prop must be placed before any spread props." },
 		schema: []
 	},
 	name: RULE_NAME$61,
@@ -220,16 +220,16 @@ var jsx_key_before_spread_default = createRule({
 });
 function create$61(context) {
 	return { JSXOpeningElement(node) {
-		let firstSpreadAttributeIndex = null;
-		for (const [index, attr] of node.attributes.entries()) {
-			if (attr.type === AST_NODE_TYPES.JSXSpreadAttribute) {
-				firstSpreadAttributeIndex ??= index;
+		let firstSpreadPropIndex = null;
+		for (const [index, prop] of node.attributes.entries()) {
+			if (prop.type === AST_NODE_TYPES.JSXSpreadAttribute) {
+				firstSpreadPropIndex ??= index;
 				continue;
 			}
-			if (firstSpreadAttributeIndex == null) continue;
-			if (attr.name.name === "key" && index > firstSpreadAttributeIndex) context.report({
+			if (firstSpreadPropIndex == null) continue;
+			if (prop.name.name === "key" && index > firstSpreadPropIndex) context.report({
 				messageId: "jsxKeyBeforeSpread",
-				node: attr
+				node: prop
 			});
 		}
 	} };
@@ -262,7 +262,6 @@ function create$60(context) {
 	const visitorFunction = (node) => {
 		if (!AST.isOneOf([AST_NODE_TYPES.JSXElement, AST_NODE_TYPES.JSXFragment])(node.parent)) return;
 		if (!hasCommentLike(node)) return;
-		if (!node.parent.type.includes("JSX")) return;
 		context.report({
 			messageId: "jsxNoCommentTextnodes",
 			node
@@ -1287,7 +1286,7 @@ var no_duplicate_key_default = createRule({
 	defaultOptions: []
 });
 function create$35(context) {
-	if (!context.sourceCode.getText().includes("key=")) return {};
+	if (!context.sourceCode.text.includes("key=")) return {};
 	const keyedEntries = /* @__PURE__ */ new Map();
 	function isKeyValueEqual(a, b) {
 		const aValue = a.value;
@@ -1452,11 +1451,11 @@ function create$33(context) {
 }
 /**
 * Determines whether the given CallExpression can be safely auto-fixed by replacing
-* the usage of `forwardRef` with passing `ref` as a prop.
+* the usage of `forwardRef` with passing `ref` as a prop
 *
-* @param context - The rule context object.
-* @param node - The CallExpression node to check.
-* @returns True if the call can be auto-fixed, false otherwise.
+* @param context - The rule context object
+* @param node - The CallExpression node to check
+* @returns True if the call can be auto-fixed, false otherwise
 */
 function canFix$1(context, node) {
 	const { importSource } = getSettingsFromContext(context);
@@ -1467,6 +1466,12 @@ function canFix$1(context, node) {
 		default: return false;
 	}
 }
+/**
+* Generates the fix for the `forwardRef` call
+* @param context - The rule context
+* @param node - The `forwardRef` call expression
+* @returns A fixer function that applies the changes
+*/
 function getFix$1(context, node) {
 	return (fixer) => {
 		const [componentNode] = node.arguments;
@@ -1478,6 +1483,14 @@ function getFix$1(context, node) {
 		];
 	};
 }
+/**
+* Generates fixes for the component's props and ref arguments
+* @param context - The rule context
+* @param fixer - The rule fixer
+* @param node - The function component node
+* @param typeArguments - The type arguments from the `forwardRef` call
+* @returns An array of fixes for the component's signature
+*/
 function getComponentPropsFixes(context, fixer, node, typeArguments) {
 	const getText = (node$1) => context.sourceCode.getText(node$1);
 	const [arg0, arg1] = node.params;
@@ -1577,6 +1590,11 @@ function create$31(context) {
 		...compare(version$1, "18.0.0", "<") ? [] : ["string", "falsy string"]
 	];
 	const services = ESLintUtils.getParserServices(context, false);
+	/**
+	* Recursively inspects a node to find potential leaked conditional rendering
+	* @param node The AST node to inspect
+	* @returns A report descriptor if a problem is found, otherwise `unit`
+	*/
 	function getReportDescriptor(node) {
 		if (node == null) return unit;
 		if (AST.is(AST_NODE_TYPES.JSXExpressionContainer)(node)) return getReportDescriptor(node.expression);
@@ -1835,8 +1853,8 @@ var no_misused_capture_owner_stack_default = createRule({
 		},
 		fixable: "code",
 		messages: {
-			useNamespaceImport: "Don't use named imports of 'captureOwnerStack' in files that are bundled for development and production. Use a namespace import instead.",
-			missingDevelopmentOnlyCheck: `Don't call 'captureOwnerStack' directly. Use 'if (process.env.NODE_ENV !== "production") {...}' to conditionally access it.`
+			missingDevelopmentOnlyCheck: `Don't call 'captureOwnerStack' directly. Use 'if (process.env.NODE_ENV !== "production") {...}' to conditionally access it.`,
+			useNamespaceImport: "Don't use named imports of 'captureOwnerStack' in files that are bundled for development and production. Use a namespace import instead."
 		},
 		schema: []
 	},
@@ -1977,28 +1995,19 @@ function isInsideJSXAttributeValue(node) {
 	return node.parent.type === AST_NODE_TYPES.JSXAttribute || findParentJsxAttribute(node, (n) => n.value?.type === AST_NODE_TYPES.JSXExpressionContainer) != null;
 }
 /**
-* Check whether given node is declared inside class component's render block
-* ```tsx
-* class Component extends React.Component {
-*   render() {
-*     class NestedClassComponent extends React.Component {
-*      render() { return <div />; }
-*     }
-*     const nestedFunctionComponent = () => <div />;
-*  }
-* }
-* ```
+* Checks whether a given node is declared inside a class component's render block
+* Ex: class C extends React.Component { render() { const Nested = () => <div />; } }
 * @param node The AST node being checked
-* @returns `true` if node is inside class component's render block, `false` if not
+* @returns `true` if the node is inside a class component's render block
 */
 function isInsideRenderMethod(node) {
 	return AST.findParentNode(node, (n) => isRenderMethodLike(n) && isClassComponent(n.parent.parent)) != null;
 }
 /**
-* Determines whether inside `createElement`'s props.
+* Determines whether the node is inside `createElement`'s props argument
 * @param context The rule context
 * @param node The AST node to check
-* @returns `true` if the node is inside createElement's props
+* @returns `true` if the node is inside `createElement`'s props
 */
 function isInsideCreateElementProps(context, node) {
 	const call = AST.findParentNode(node, isCreateElementCall(context));
@@ -2163,12 +2172,12 @@ function create$22(context) {
 	if (!context.sourceCode.text.includes("componentDidMount")) return {};
 	return { CallExpression(node) {
 		if (!isThisSetState(node)) return;
-		const clazz = AST.findParentNode(node, isClassComponent);
-		const method = AST.findParentNode(node, (n) => n === clazz || isComponentDidMount(n));
-		if (clazz == null || method == null || method === clazz) return;
-		const methodScope = context.sourceCode.getScope(method);
-		const upperScope = context.sourceCode.getScope(node).upper;
-		if (method.parent === clazz.body && upperScope === methodScope) context.report({
+		const enclosingClassNode = AST.findParentNode(node, isClassComponent);
+		const enclosingMethodNode = AST.findParentNode(node, (n) => n === enclosingClassNode || isComponentDidMount(n));
+		if (enclosingClassNode == null || enclosingMethodNode == null || enclosingMethodNode === enclosingClassNode) return;
+		const enclosingMethodScope = context.sourceCode.getScope(enclosingMethodNode);
+		const setStateCallParentScope = context.sourceCode.getScope(node).upper;
+		if (enclosingMethodNode.parent === enclosingClassNode.body && setStateCallParentScope === enclosingMethodScope) context.report({
 			messageId: "noSetStateInComponentDidMount",
 			node
 		});
@@ -2197,12 +2206,12 @@ function create$21(context) {
 	if (!context.sourceCode.text.includes("componentDidUpdate")) return {};
 	return { CallExpression(node) {
 		if (!isThisSetState(node)) return;
-		const clazz = AST.findParentNode(node, isClassComponent);
-		const method = AST.findParentNode(node, (n) => n === clazz || isComponentDidUpdate(n));
-		if (clazz == null || method == null || method === clazz) return;
-		const methodScope = context.sourceCode.getScope(method);
-		const upperScope = context.sourceCode.getScope(node).upper;
-		if (method.parent === clazz.body && upperScope === methodScope) context.report({
+		const enclosingClassNode = AST.findParentNode(node, isClassComponent);
+		const enclosingMethodNode = AST.findParentNode(node, (n) => n === enclosingClassNode || isComponentDidUpdate(n));
+		if (enclosingClassNode == null || enclosingMethodNode == null || enclosingMethodNode === enclosingClassNode) return;
+		const enclosingMethodScope = context.sourceCode.getScope(enclosingMethodNode);
+		const setStateCallParentScope = context.sourceCode.getScope(node).upper;
+		if (enclosingMethodNode.parent === enclosingClassNode.body && setStateCallParentScope === enclosingMethodScope) context.report({
 			messageId: "noSetStateInComponentDidUpdate",
 			node
 		});
@@ -2217,7 +2226,7 @@ var no_set_state_in_component_will_update_default = createRule({
 	meta: {
 		type: "problem",
 		docs: {
-			description: "Disallows calling `this.setState` in `componentWillUpdate` outside of functions, such as callbacks.",
+			description: "Disallow calling `this.setState` in `componentWillUpdate` outside of functions, such as callbacks.",
 			[Symbol.for("rule_features")]: RULE_FEATURES$19
 		},
 		messages: { noSetStateInComponentWillUpdate: "Do not call `this.setState` in `componentWillUpdate` outside of functions, such as callbacks." },
@@ -2231,12 +2240,12 @@ function create$20(context) {
 	if (!context.sourceCode.text.includes("componentWillUpdate")) return {};
 	return { CallExpression(node) {
 		if (!isThisSetState(node)) return;
-		const clazz = AST.findParentNode(node, isClassComponent);
-		const method = AST.findParentNode(node, (n) => n === clazz || isComponentWillUpdate(n));
-		if (clazz == null || method == null || method === clazz) return;
-		const methodScope = context.sourceCode.getScope(method);
-		const upperScope = context.sourceCode.getScope(node).upper;
-		if (method.parent === clazz.body && upperScope === methodScope) context.report({
+		const enclosingClassNode = AST.findParentNode(node, isClassComponent);
+		const enclosingMethodNode = AST.findParentNode(node, (n) => n === enclosingClassNode || isComponentWillUpdate(n));
+		if (enclosingClassNode == null || enclosingMethodNode == null || enclosingMethodNode === enclosingClassNode) return;
+		const enclosingMethodScope = context.sourceCode.getScope(enclosingMethodNode);
+		const setStateCallParentScope = context.sourceCode.getScope(node).upper;
+		if (enclosingMethodNode.parent === enclosingClassNode.body && setStateCallParentScope === enclosingMethodScope) context.report({
 			messageId: "noSetStateInComponentWillUpdate",
 			node
 		});
@@ -2275,21 +2284,27 @@ function create$19(context) {
 		"ClassBody:exit": onClassBodyExit,
 		JSXAttribute(node) {
 			if (node.name.name !== "ref") return;
-			const refNameText = getAttributeValueText(context, node.value);
-			if (refNameText == null) return;
+			const refName = getJsxAttributeValueText(context, node.value);
+			if (refName == null) return;
 			context.report({
 				messageId: "noStringRefs",
 				node,
 				fix(fixer) {
 					if (node.value == null) return null;
 					if (!state.isWithinClassComponent) return null;
-					return fixer.replaceText(node.value, `{(ref) => { this.refs[${refNameText}] = ref; }}`);
+					return fixer.replaceText(node.value, `{(ref) => { this.refs[${refName}] = ref; }}`);
 				}
 			});
 		}
 	};
 }
-function getAttributeValueText(context, node) {
+/**
+* Extracts the text content from a JSX attribute's value
+* @param context - The rule context
+* @param node - The JSX attribute value node
+* @returns The text of the attribute value, or null if not a string-like value
+*/
+function getJsxAttributeValueText(context, node) {
 	if (node == null) return null;
 	switch (true) {
 		case node.type === AST_NODE_TYPES.Literal && typeof node.value === "string": return context.sourceCode.getText(node);
@@ -2336,6 +2351,11 @@ function create$18(context) {
 		});
 	} };
 }
+/**
+* Checks if a node is a callback function passed to an array's `.map()` method
+* @param node The node to check
+* @returns `true` if the node is a map callback, `false` otherwise
+*/
 function isMapCallback(node) {
 	if (node.parent == null) return false;
 	if (!AST.isArrayMapCall(node.parent)) return false;
@@ -2781,7 +2801,7 @@ function create$9(context) {
 		for (const def of defs) {
 			const methodName = AST.getPropertyName(def);
 			if (methodName == null) continue;
-			if (usages?.has(methodName) || LIFECYCLE_METHODS.has(methodName)) continue;
+			if ((usages?.has(methodName) ?? false) || LIFECYCLE_METHODS.has(methodName)) continue;
 			context.report({
 				messageId: "noUnusedClassComponentMembers",
 				node: def,
@@ -3308,9 +3328,6 @@ function checkNode(context, node, allowExpressions) {
 		fix: getFix(context, node)
 	});
 }
-/**
-* Generate fix for removing useless fragment
-*/
 function getFix(context, node) {
 	if (!canFix(context, node)) return null;
 	return (fixer) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "2.0.5-next.3",
+  "version": "2.0.5-next.5",
   "description": "A set of composable ESLint rules for for libraries and frameworks that use React as a UI runtime.",
   "keywords": [
     "react",
@@ -43,16 +43,16 @@
     "string-ts": "^2.2.1",
     "ts-api-utils": "^2.1.0",
     "ts-pattern": "^5.8.0",
-    "@eslint-react/ast": "2.0.5-next.3",
-    "@eslint-react/eff": "2.0.5-next.3",
-    "@eslint-react/kit": "2.0.5-next.3",
-    "@eslint-react/shared": "2.0.5-next.3",
-    "@eslint-react/core": "2.0.5-next.3",
-    "@eslint-react/var": "2.0.5-next.3"
+    "@eslint-react/ast": "2.0.5-next.5",
+    "@eslint-react/eff": "2.0.5-next.5",
+    "@eslint-react/kit": "2.0.5-next.5",
+    "@eslint-react/var": "2.0.5-next.5",
+    "@eslint-react/shared": "2.0.5-next.5",
+    "@eslint-react/core": "2.0.5-next.5"
   },
   "devDependencies": {
-    "@types/react": "^19.1.16",
-    "@types/react-dom": "^19.1.9",
+    "@types/react": "^19.2.0",
+    "@types/react-dom": "^19.2.0",
     "tsdown": "^0.15.6",
     "@local/configs": "0.0.0"
   },