eslint-plugin-react-x 3.0.0-next.40 → 3.0.0-next.42

package/dist/index.js

@@ -68,7 +68,7 @@ const rules$7 = {
 //#endregion
 //#region package.json
 var name$6 = "eslint-plugin-react-x";
-var version = "3.0.0-next.40";
+var version = "3.0.0-next.42";
 
 //#endregion
 //#region src/utils/create-rule.ts
@@ -2246,10 +2246,14 @@ function create$39(context) {
 const RULE_NAME$38 = "no-context-provider";
 var no_context_provider_default = createRule({
 	meta: {
-		type: "problem",
+		type: "suggestion",
 		docs: { description: "Replaces usage of '<Context.Provider>' with '<Context>'." },
 		fixable: "code",
-		messages: { default: "In React 19, you can render '<Context>' as a provider instead of '<Context.Provider>'." },
+		hasSuggestions: true,
+		messages: {
+			default: "In React 19, you can render '<Context>' as a provider instead of '<Context.Provider>'.",
+			replace: "Replace '<Context.Provider>' with '<Context>'."
+		},
 		schema: []
 	},
 	name: RULE_NAME$38,
@@ -2270,13 +2274,16 @@ function create$38(context) {
 		context.report({
 			messageId: "default",
 			node,
-			fix(fixer) {
-				if (!core.isComponentNameLoose(contextSelfName)) return null;
-				const openingElement = node.openingElement;
-				const closingElement = node.closingElement;
-				if (closingElement == null) return fixer.replaceText(openingElement.name, contextFullName);
-				return [fixer.replaceText(openingElement.name, contextFullName), fixer.replaceText(closingElement.name, contextFullName)];
-			}
+			suggest: [{
+				messageId: "replace",
+				fix(fixer) {
+					if (!core.isComponentNameLoose(contextSelfName)) return null;
+					const openingElement = node.openingElement;
+					const closingElement = node.closingElement;
+					if (closingElement == null) return fixer.replaceText(openingElement.name, contextFullName);
+					return [fixer.replaceText(openingElement.name, contextFullName), fixer.replaceText(closingElement.name, contextFullName)];
+				}
+			}]
 		});
 	} });
 }
@@ -2405,10 +2412,14 @@ function create$35(context) {
 const RULE_NAME$34 = "no-forward-ref";
 var no_forward_ref_default = createRule({
 	meta: {
-		type: "problem",
+		type: "suggestion",
 		docs: { description: "Replaces usage of 'forwardRef' with passing 'ref' as a prop." },
 		fixable: "code",
-		messages: { default: "In React 19, 'forwardRef' is no longer necessary. Pass 'ref' as a prop instead." },
+		hasSuggestions: true,
+		messages: {
+			default: "In React 19, 'forwardRef' is no longer necessary. Pass 'ref' as a prop instead.",
+			replace: "Replace 'forwardRef' with passing 'ref' as a prop."
+		},
 		schema: []
 	},
 	name: RULE_NAME$34,
@@ -2422,11 +2433,14 @@ function create$34(context) {
 	return defineRuleListener({ CallExpression(node) {
 		if (!core.isForwardRefCall(context, node)) return;
 		const id = ast.getFunctionId(node);
-		const fix = canFix(context, node) ? getFix(context, node) : null;
+		const suggest = canFix(context, node) ? [{
+			messageId: "replace",
+			fix: getFix(context, node)
+		}] : [];
 		context.report({
 			messageId: "default",
 			node: id ?? node,
-			fix
+			suggest
 		});
 	} });
 }
@@ -3897,10 +3911,14 @@ function create$11(context) {
 const RULE_NAME$10 = "no-use-context";
 var no_use_context_default = createRule({
 	meta: {
-		type: "problem",
+		type: "suggestion",
 		docs: { description: "Replaces usage of 'useContext' with 'use'." },
 		fixable: "code",
-		messages: { default: "In React 19, 'use' is preferred over 'useContext' because it is more flexible." },
+		hasSuggestions: true,
+		messages: {
+			default: "In React 19, 'use' is preferred over 'useContext' because it is more flexible.",
+			replace: "Replace 'useContext' with 'use'."
+		},
 		schema: []
 	},
 	name: RULE_NAME$10,
@@ -3909,65 +3927,25 @@ var no_use_context_default = createRule({
 });
 function create$10(context) {
 	if (!context.sourceCode.text.includes("useContext")) return {};
-	const settings = getSettingsFromContext(context);
-	if (compare(settings.version, "19.0.0", "<")) return {};
-	const hookCalls = /* @__PURE__ */ new Set();
-	return defineRuleListener({
-		CallExpression(node) {
-			if (!core.isHookCall(node)) return;
-			hookCalls.add(node);
-		},
-		ImportDeclaration(node) {
-			if (node.source.value !== settings.importSource) return;
-			const isUseImported = node.specifiers.some(isMatching({ local: {
-				type: AST_NODE_TYPES.Identifier,
-				name: "use"
-			} }));
-			for (const specifier of node.specifiers) {
-				if (specifier.type !== AST_NODE_TYPES.ImportSpecifier) continue;
-				if (specifier.imported.type !== AST_NODE_TYPES.Identifier) continue;
-				if (specifier.imported.name === "useContext") context.report({
-					messageId: "default",
-					node: specifier,
-					fix(fixer) {
-						if (isUseImported) {
-							const tokenBefore = context.sourceCode.getTokenBefore(specifier);
-							return [
-								fixer.remove(specifier),
-								...tokenBefore?.value === "," ? [fixer.replaceTextRange([tokenBefore.range[1], specifier.range[0]], "")] : [],
-								...getCorrelativeTokens(context, specifier).map((token) => fixer.remove(token))
-							];
-						}
-						return fixer.replaceText(specifier.imported, "use");
-					}
-				});
-			}
-		},
-		"Program:exit"() {
-			for (const node of hookCalls) {
-				if (!core.isUseContextCall(node)) continue;
-				context.report({
-					messageId: "default",
-					node: node.callee,
-					fix(fixer) {
-						switch (node.callee.type) {
-							case AST_NODE_TYPES.Identifier: return fixer.replaceText(node.callee, "use");
-							case AST_NODE_TYPES.MemberExpression: return fixer.replaceText(node.callee.property, "use");
-						}
-						return null;
+	const { version } = getSettingsFromContext(context);
+	if (compare(version, "19.0.0", "<")) return {};
+	return defineRuleListener({ CallExpression(node) {
+		if (!core.isUseContextCall(node)) return;
+		context.report({
+			messageId: "default",
+			node: node.callee,
+			suggest: [{
+				messageId: "replace",
+				fix(fixer) {
+					switch (node.callee.type) {
+						case AST_NODE_TYPES.Identifier: return fixer.replaceText(node.callee, "use");
+						case AST_NODE_TYPES.MemberExpression: return fixer.replaceText(node.callee.property, "use");
 					}
-				});
-			}
-		}
-	});
-}
-function getCorrelativeTokens(context, node) {
-	const tokenBefore = context.sourceCode.getTokenBefore(node);
-	const tokenAfter = context.sourceCode.getTokenAfter(node);
-	const tokens = [];
-	if (tokenAfter?.value !== "," && tokenBefore?.value === ",") tokens.push(tokenBefore);
-	if (tokenAfter?.value === ",") tokens.push(tokenAfter);
-	return tokens;
+					return null;
+				}
+			}]
+		});
+	} });
 }
 
 //#endregion
@@ -4115,7 +4093,7 @@ function trimLikeReact(text) {
 const RULE_NAME$8 = "prefer-destructuring-assignment";
 var prefer_destructuring_assignment_default = createRule({
 	meta: {
-		type: "problem",
+		type: "suggestion",
 		docs: { description: "Enforces destructuring assignment for component props and context." },
 		messages: { default: "Use destructuring assignment for component props." },
 		schema: []

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "3.0.0-next.40",
+  "version": "3.0.0-next.42",
   "description": "A set of composable ESLint rules for libraries and frameworks that use React as a UI runtime.",
   "keywords": [
     "react",
@@ -45,11 +45,11 @@
     "string-ts": "^2.3.1",
     "ts-api-utils": "^2.4.0",
     "ts-pattern": "^5.9.0",
-    "@eslint-react/ast": "3.0.0-next.40",
-    "@eslint-react/core": "3.0.0-next.40",
-    "@eslint-react/eff": "3.0.0-next.40",
-    "@eslint-react/var": "3.0.0-next.40",
-    "@eslint-react/shared": "3.0.0-next.40"
+    "@eslint-react/ast": "3.0.0-next.42",
+    "@eslint-react/core": "3.0.0-next.42",
+    "@eslint-react/eff": "3.0.0-next.42",
+    "@eslint-react/shared": "3.0.0-next.42",
+    "@eslint-react/var": "3.0.0-next.42"
   },
   "devDependencies": {
     "@types/react": "^19.2.14",