eslint-plugin-react-x 2.1.1-next.3 → 2.1.1

package/dist/index.d.ts

@@ -3,19 +3,55 @@ import * as _eslint_react_kit0 from "@eslint-react/kit";
 //#region src/index.d.ts
 declare const _default: {
   configs: {
+    /**
+     * Enforce rules that are recommended by ESLint React for general purpose React + React DOM projects
+     */
     recommended: {
       plugins: {};
       name?: string;
       rules?: Record<string, _eslint_react_kit0.RuleConfig>;
       settings?: _eslint_react_kit0.SettingsConfig;
     };
+    /**
+     * Same as the `recommended` preset but disables rules that can be enforced by TypeScript
+     */
+    "recommended-typescript": {
+      plugins: {};
+      name?: string;
+      rules?: Record<string, _eslint_react_kit0.RuleConfig>;
+      settings?: _eslint_react_kit0.SettingsConfig;
+    };
+    /**
+     * Same as the `recommended-typescript` preset but enables additional rules that require type information
+     */
     "recommended-type-checked": {
       plugins: {};
       name?: string;
       rules?: Record<string, _eslint_react_kit0.RuleConfig>;
       settings?: _eslint_react_kit0.SettingsConfig;
     };
-    "recommended-typescript": {
+    /**
+     * More strict version of the `recommended` preset
+     */
+    strict: {
+      plugins: {};
+      name?: string;
+      rules?: Record<string, _eslint_react_kit0.RuleConfig>;
+      settings?: _eslint_react_kit0.SettingsConfig;
+    };
+    /**
+     * Same as the `strict` preset but enables additional rules that require type information
+     */
+    "strict-typescript": {
+      plugins: {};
+      name?: string;
+      rules?: Record<string, _eslint_react_kit0.RuleConfig>;
+      settings?: _eslint_react_kit0.SettingsConfig;
+    };
+    /**
+     * Same as the `strict-typescript` preset but enables additional rules that require type information
+     */
+    "strict-type-checked": {
       plugins: {};
       name?: string;
       rules?: Record<string, _eslint_react_kit0.RuleConfig>;

package/dist/index.js

@@ -19,8 +19,8 @@ import { getTypeImmutability, isImmutable, isReadonlyDeep, isReadonlyShallow, is
 var __defProp = Object.defineProperty;
 var __export = (all) => {
 	let target = {};
-	for (var name$4 in all) __defProp(target, name$4, {
-		get: all[name$4],
+	for (var name$7 in all) __defProp(target, name$7, {
+		get: all[name$7],
 		enumerable: true
 	});
 	return target;
@@ -28,8 +28,8 @@ var __export = (all) => {
 
 //#endregion
 //#region package.json
-var name$3 = "eslint-plugin-react-x";
-var version = "2.1.1-next.3";
+var name$6 = "eslint-plugin-react-x";
+var version = "2.1.1";
 
 //#endregion
 //#region src/utils/create-rule.ts
@@ -209,10 +209,10 @@ function create$59(context) {
 		const props = [];
 		for (const attr of node.attributes) {
 			if (attr.type === AST_NODE_TYPES.JSXSpreadAttribute) continue;
-			const name$4 = attr.name.name;
-			if (typeof name$4 !== "string") continue;
-			if (!props.includes(name$4)) {
-				props.push(name$4);
+			const name$7 = attr.name.name;
+			if (typeof name$7 !== "string") continue;
+			if (!props.includes(name$7)) {
+				props.push(name$7);
 				continue;
 			}
 			context.report({
@@ -278,17 +278,17 @@ var jsx_no_undef_default = createRule({
 });
 function create$57(context) {
 	return { JSXOpeningElement(node) {
-		const name$4 = match(node.name).with({ type: AST_NODE_TYPES.JSXIdentifier }, (n) => n.name).with({
+		const name$7 = match(node.name).with({ type: AST_NODE_TYPES.JSXIdentifier }, (n) => n.name).with({
 			type: AST_NODE_TYPES.JSXMemberExpression,
 			object: { type: AST_NODE_TYPES.JSXIdentifier }
 		}, (n) => n.object.name).otherwise(() => null);
-		if (name$4 == null) return;
-		if (name$4 === "this") return;
-		if (/^[a-z]/u.test(name$4)) return;
-		if (findVariable(name$4, context.sourceCode.getScope(node)) == null) context.report({
+		if (name$7 == null) return;
+		if (name$7 === "this") return;
+		if (/^[a-z]/u.test(name$7)) return;
+		if (findVariable(name$7, context.sourceCode.getScope(node)) == null) context.report({
 			messageId: "jsxNoUndef",
 			node,
-			data: { name: name$4 }
+			data: { name: name$7 }
 		});
 	} };
 }
@@ -437,12 +437,12 @@ function create$54(context) {
 		JSXOpeningFragment: handleJsxElement
 	};
 }
-function debugReport(context, node, name$4) {
+function debugReport(context, node, name$7) {
 	if (process.env["ESLINT_REACT_DEBUG"] !== "1") return;
 	context.report({
 		messageId: "jsxUsesReact",
 		node,
-		data: { name: name$4 }
+		data: { name: name$7 }
 	});
 }
 
@@ -578,8 +578,8 @@ function create$52(context) {
 const RULE_NAME$51 = "no-array-index-key";
 const RULE_FEATURES$49 = [];
 const REACT_CHILDREN_METHOD = ["forEach", "map"];
-function isReactChildrenMethod(name$4) {
-	return REACT_CHILDREN_METHOD.includes(name$4);
+function isReactChildrenMethod(name$7) {
+	return REACT_CHILDREN_METHOD.includes(name$7);
 }
 function isUsingReactChildren(context, node) {
 	const { importSource = "react" } = coerceSettings(context.settings);
@@ -595,8 +595,8 @@ function getMapIndexParamName(context, node) {
 	const { callee } = node;
 	if (callee.type !== AST_NODE_TYPES.MemberExpression) return unit;
 	if (callee.property.type !== AST_NODE_TYPES.Identifier) return unit;
-	const { name: name$4 } = callee.property;
-	const indexPosition = AST.getArrayMethodCallbackIndexParamPosition(name$4);
+	const { name: name$7 } = callee.property;
+	const indexPosition = AST.getArrayMethodCallbackIndexParamPosition(name$7);
 	if (indexPosition === -1) return unit;
 	const callbackArg = node.arguments[isUsingReactChildren(context, node) ? 1 : 0];
 	if (callbackArg == null) return unit;
@@ -628,7 +628,7 @@ var no_array_index_key_default = createRule({
 function create$51(context) {
 	const indexParamNames = [];
 	function isArrayIndex(node) {
-		return node.type === AST_NODE_TYPES.Identifier && indexParamNames.some((name$4) => name$4 != null && name$4 === node.name);
+		return node.type === AST_NODE_TYPES.Identifier && indexParamNames.some((name$7) => name$7 != null && name$7 === node.name);
 	}
 	function isCreateOrCloneElementCall(node) {
 		return isCreateElementCall(context, node) || isCloneElementCall(context, node);
@@ -636,7 +636,7 @@ function create$51(context) {
 	function getReportDescriptors(node) {
 		switch (node.type) {
 			case AST_NODE_TYPES.Identifier:
-				if (indexParamNames.some((name$4) => name$4 != null && name$4 === node.name)) return [{
+				if (indexParamNames.some((name$7) => name$7 != null && name$7 === node.name)) return [{
 					messageId: "noArrayIndexKey",
 					node
 				}];
@@ -877,12 +877,12 @@ function create$44(context) {
 		...listeners,
 		"Program:exit"(program) {
 			const components = ctx.getAllComponents(program);
-			for (const { name: name$4 = "anonymous", node: component } of components.values()) {
+			for (const { name: name$7 = "anonymous", node: component } of components.values()) {
 				if (component.body.body.some((m) => isComponentDidCatch(m) || isGetDerivedStateFromError(m))) continue;
 				context.report({
 					messageId: "noClassComponent",
 					node: component,
-					data: { name: name$4 }
+					data: { name: name$7 }
 				});
 			}
 		}
@@ -1310,17 +1310,17 @@ function create$34(context, [option]) {
 		else if (node.name.type === AST_NODE_TYPES.JSXNamespacedName) nodeName = node.name.name.name;
 		for (const attr of node.attributes) {
 			if (attr.type === AST_NODE_TYPES.JSXSpreadAttribute) continue;
-			const name$4 = attr.name.name;
-			if (typeof name$4 !== "string") continue;
+			const name$7 = attr.name.name;
+			if (typeof name$7 !== "string") continue;
 			for (const forbiddenPropItem of forbid) {
 				if (typeof forbiddenPropItem !== "string" && nodeName != null) {
 					if ("excludedNodes" in forbiddenPropItem && forbiddenPropItem.excludedNodes.includes(nodeName)) continue;
 					if ("includedNodes" in forbiddenPropItem && !forbiddenPropItem.includedNodes.includes(nodeName)) continue;
 				}
-				if (toRegExp(typeof forbiddenPropItem === "string" ? forbiddenPropItem : forbiddenPropItem.prop).test(name$4)) context.report({
+				if (toRegExp(typeof forbiddenPropItem === "string" ? forbiddenPropItem : forbiddenPropItem.prop).test(name$7)) context.report({
 					messageId,
 					node: attr,
-					data: { name: name$4 }
+					data: { name: name$7 }
 				});
 			}
 		}
@@ -1836,15 +1836,15 @@ function create$26(context) {
 			const isClassComponent$1 = (node) => {
 				return AST.isClass(node) && classComponents.some((component) => component.node === node);
 			};
-			for (const { name: name$4, node: component } of functionComponents) {
-				if (name$4 == null) continue;
+			for (const { name: name$7, node: component } of functionComponents) {
+				if (name$7 == null) continue;
 				if (isDirectValueOfRenderPropertyLoose(component)) continue;
 				if (isInsideJSXAttributeValue(component)) {
 					if (!isDeclaredInRenderPropLoose(component)) context.report({
 						messageId: "noNestedComponentDefinitions",
 						node: component,
 						data: {
-							name: name$4,
+							name: name$7,
 							suggestion: "Move it to the top level or pass it as a prop."
 						}
 					});
@@ -1855,7 +1855,7 @@ function create$26(context) {
 						messageId: "noNestedComponentDefinitions",
 						node: component,
 						data: {
-							name: name$4,
+							name: name$7,
 							suggestion: "Move it to the top level or pass it as a prop."
 						}
 					});
@@ -1867,7 +1867,7 @@ function create$26(context) {
 						messageId: "noNestedComponentDefinitions",
 						node: component,
 						data: {
-							name: name$4,
+							name: name$7,
 							suggestion: component.parent.type === AST_NODE_TYPES.Property ? "Move it to the top level or pass it as a prop." : "Move it to the top level."
 						}
 					});
@@ -1877,18 +1877,18 @@ function create$26(context) {
 					messageId: "noNestedComponentDefinitions",
 					node: component,
 					data: {
-						name: name$4,
+						name: name$7,
 						suggestion: "Move it to the top level."
 					}
 				});
 			}
-			for (const { name: name$4 = "unknown", node: component } of classComponents) {
+			for (const { name: name$7 = "unknown", node: component } of classComponents) {
 				if (AST.findParentNode(component, (n) => isClassComponent$1(n) || isFunctionComponent(n)) == null) continue;
 				context.report({
 					messageId: "noNestedComponentDefinitions",
 					node: component,
 					data: {
-						name: name$4,
+						name: name$7,
 						suggestion: component.parent.type === AST_NODE_TYPES.Property ? "Move it to the top level or pass it as a prop." : "Move it to the top level."
 					}
 				});
@@ -2046,13 +2046,13 @@ function create$23(context) {
 		...listeners,
 		"Program:exit"(program) {
 			const components = ctx.getAllComponents(program);
-			for (const { name: name$4 = "PureComponent", node: component, flag } of components.values()) {
+			for (const { name: name$7 = "PureComponent", node: component, flag } of components.values()) {
 				if ((flag & ComponentFlag.PureComponent) === 0n) continue;
 				const { body } = component.body;
 				for (const member of body) if (isShouldComponentUpdate(member)) context.report({
 					messageId: "noRedundantShouldComponentUpdate",
 					node: member,
-					data: { componentName: name$4 }
+					data: { componentName: name$7 }
 				});
 			}
 		}
@@ -2396,16 +2396,16 @@ function create$15(context) {
 		...listeners,
 		"Program:exit"(program) {
 			const allHooks = ctx.getAllHooks(program);
-			for (const { id, name: name$4, node, hookCalls } of allHooks.values()) {
+			for (const { id, name: name$7, node, hookCalls } of allHooks.values()) {
 				if (hookCalls.length > 0) continue;
 				if (AST.isFunctionEmpty(node)) continue;
-				if (WELL_KNOWN_HOOKS.includes(name$4)) continue;
+				if (WELL_KNOWN_HOOKS.includes(name$7)) continue;
 				if (containsUseComments(context, node)) continue;
 				if (AST.findParentNode(node, AST.isViMockCallback) != null) continue;
 				context.report({
 					messageId: "noUnnecessaryUsePrefix",
 					node: id ?? node,
-					data: { name: name$4 }
+					data: { name: name$7 }
 				});
 			}
 		}
@@ -2578,9 +2578,9 @@ function create$11(context) {
 		}
 	};
 }
-function isContextName(name$4, isReact18OrBelow) {
-	if (name$4 === "Provider") return true;
-	if (!isReact18OrBelow) return name$4.endsWith("Context") || name$4.endsWith("CONTEXT");
+function isContextName(name$7, isReact18OrBelow) {
+	if (name$7 === "Provider") return true;
+	if (!isReact18OrBelow) return name$7.endsWith("Context") || name$7.endsWith("CONTEXT");
 	return false;
 }
 
@@ -2618,8 +2618,8 @@ function create$10(context) {
 				const { params } = component;
 				const [props] = params;
 				if (props == null) continue;
-				const properties = match(props).with({ type: AST_NODE_TYPES.ObjectPattern }, ({ properties: properties$1 }) => properties$1).with({ type: AST_NODE_TYPES.Identifier }, ({ name: name$4 }) => {
-					return declarators.get(component)?.filter((d) => d.init.name === name$4).flatMap((d) => d.id.properties) ?? [];
+				const properties = match(props).with({ type: AST_NODE_TYPES.ObjectPattern }, ({ properties: properties$1 }) => properties$1).with({ type: AST_NODE_TYPES.Identifier }, ({ name: name$7 }) => {
+					return declarators.get(component)?.filter((d) => d.init.name === name$7).flatMap((d) => d.id.properties) ?? [];
 				}).otherwise(() => []);
 				for (const prop of properties) {
 					if (prop.type !== AST_NODE_TYPES.Property || prop.value.type !== AST_NODE_TYPES.AssignmentPattern) continue;
@@ -3030,8 +3030,8 @@ var no_use_context_default = createRule({
 });
 function create$6(context) {
 	if (!context.sourceCode.text.includes("useContext")) return {};
-	const settings$3 = getSettingsFromContext(context);
-	if (compare(settings$3.version, "19.0.0", "<")) return {};
+	const settings$6 = getSettingsFromContext(context);
+	if (compare(settings$6.version, "19.0.0", "<")) return {};
 	const hookCalls = /* @__PURE__ */ new Set();
 	return {
 		CallExpression(node) {
@@ -3039,7 +3039,7 @@ function create$6(context) {
 			hookCalls.add(node);
 		},
 		ImportDeclaration(node) {
-			if (node.source.value !== settings$3.importSource) return;
+			if (node.source.value !== settings$6.importSource) return;
 			const isUseImported = node.specifiers.some(isMatching({ local: {
 				type: AST_NODE_TYPES.Identifier,
 				name: "use"
@@ -3457,7 +3457,7 @@ function create(context) {
 //#region src/plugin.ts
 const plugin = {
 	meta: {
-		name: name$3,
+		name: name$6,
 		version
 	},
 	rules: {
@@ -3529,13 +3529,13 @@ const plugin = {
 //#endregion
 //#region src/configs/recommended.ts
 var recommended_exports = /* @__PURE__ */ __export({
-	name: () => name$2,
-	plugins: () => plugins$2,
-	rules: () => rules$3,
-	settings: () => settings$2
+	name: () => name$5,
+	plugins: () => plugins$5,
+	rules: () => rules$6,
+	settings: () => settings$5
 });
-const name$2 = "react-x/recommended";
-const rules$3 = {
+const name$5 = "react-x/recommended";
+const rules$6 = {
 	"react-x/jsx-no-comment-textnodes": "warn",
 	"react-x/jsx-no-duplicate-props": "warn",
 	"react-x/jsx-uses-react": "warn",
@@ -3575,15 +3575,15 @@ const rules$3 = {
 	"react-x/no-useless-forward-ref": "warn",
 	"react-x/prefer-use-state-lazy-initialization": "warn"
 };
-const plugins$2 = { "react-x": plugin };
-const settings$2 = { "react-x": DEFAULT_ESLINT_REACT_SETTINGS };
+const plugins$5 = { "react-x": plugin };
+const settings$5 = { "react-x": DEFAULT_ESLINT_REACT_SETTINGS };
 
 //#endregion
 //#region src/configs/_ts.ts
 /**
 * Disables rules that are already handled by TypeScript
 */
-const rules$2 = {
+const rules$5 = {
 	"react-x/jsx-no-duplicate-props": "off",
 	"react-x/jsx-no-undef": "off",
 	"react-x/jsx-uses-react": "off",
@@ -3593,31 +3593,92 @@ const rules$2 = {
 //#endregion
 //#region src/configs/recommended-typescript.ts
 var recommended_typescript_exports = /* @__PURE__ */ __export({
+	name: () => name$4,
+	plugins: () => plugins$4,
+	rules: () => rules$4,
+	settings: () => settings$4
+});
+const name$4 = "react-x/recommended-typescript";
+const rules$4 = {
+	...rules$6,
+	...rules$5
+};
+const plugins$4 = { ...plugins$5 };
+const settings$4 = { ...settings$5 };
+
+//#endregion
+//#region src/configs/recommended-type-checked.ts
+var recommended_type_checked_exports = /* @__PURE__ */ __export({
+	name: () => name$3,
+	plugins: () => plugins$3,
+	rules: () => rules$3,
+	settings: () => settings$3
+});
+const name$3 = "react-x/recommended-type-checked";
+const rules$3 = {
+	...rules$4,
+	"react-x/no-leaked-conditional-rendering": "error"
+};
+const plugins$3 = { ...plugins$4 };
+const settings$3 = { ...settings$4 };
+
+//#endregion
+//#region src/configs/strict.ts
+var strict_exports = /* @__PURE__ */ __export({
+	name: () => name$2,
+	plugins: () => plugins$2,
+	rules: () => rules$2,
+	settings: () => settings$2
+});
+const name$2 = "react-x/strict";
+const rules$2 = {
+	...rules$6,
+	"react-x/jsx-no-iife": "error",
+	"react-x/no-children-prop": "warn",
+	"react-x/no-class-component": "error",
+	"react-x/no-misused-capture-owner-stack": "error",
+	"react-x/no-unnecessary-key": "warn",
+	"react-x/no-unnecessary-use-callback": "warn",
+	"react-x/no-unnecessary-use-memo": "warn",
+	"react-x/no-unstable-context-value": "warn",
+	"react-x/no-unstable-default-props": "warn",
+	"react-x/no-unused-class-component-members": "warn",
+	"react-x/no-unused-state": "warn",
+	"react-x/no-useless-fragment": "warn",
+	"react-x/prefer-destructuring-assignment": "warn"
+};
+const plugins$2 = { ...plugins$5 };
+const settings$2 = { ...settings$5 };
+
+//#endregion
+//#region src/configs/strict-typescript.ts
+var strict_typescript_exports = /* @__PURE__ */ __export({
 	name: () => name$1,
 	plugins: () => plugins$1,
 	rules: () => rules$1,
 	settings: () => settings$1
 });
-const name$1 = "react-x/recommended-typescript";
+const name$1 = "react-x/strict-typescript";
 const rules$1 = {
-	...rules$3,
-	...rules$2
+	...rules$2,
+	...rules$5
 };
 const plugins$1 = { ...plugins$2 };
 const settings$1 = { ...settings$2 };
 
 //#endregion
-//#region src/configs/recommended-type-checked.ts
-var recommended_type_checked_exports = /* @__PURE__ */ __export({
+//#region src/configs/strict-type-checked.ts
+var strict_type_checked_exports = /* @__PURE__ */ __export({
 	name: () => name,
 	plugins: () => plugins,
 	rules: () => rules,
 	settings: () => settings
 });
-const name = "react-x/recommended-type-checked";
+const name = "react-x/strict-type-checked";
 const rules = {
 	...rules$1,
-	"react-x/no-leaked-conditional-rendering": "error"
+	"react-x/no-leaked-conditional-rendering": "error",
+	"react-x/no-unused-props": "warn"
 };
 const plugins = { ...plugins$1 };
 const settings = { ...settings$1 };
@@ -3629,8 +3690,11 @@ var src_default = {
 	...plugin,
 	configs: {
 		["recommended"]: toFlatConfig(recommended_exports),
+		["recommended-typescript"]: toFlatConfig(recommended_typescript_exports),
 		["recommended-type-checked"]: toFlatConfig(recommended_type_checked_exports),
-		["recommended-typescript"]: toFlatConfig(recommended_typescript_exports)
+		["strict"]: toFlatConfig(strict_exports),
+		["strict-typescript"]: toFlatConfig(strict_typescript_exports),
+		["strict-type-checked"]: toFlatConfig(strict_type_checked_exports)
 	}
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "2.1.1-next.3",
+  "version": "2.1.1",
   "description": "A set of composable ESLint rules for for libraries and frameworks that use React as a UI runtime.",
   "keywords": [
     "react",
@@ -43,12 +43,12 @@
     "string-ts": "^2.2.1",
     "ts-api-utils": "^2.1.0",
     "ts-pattern": "^5.8.0",
-    "@eslint-react/ast": "2.1.1-next.3",
-    "@eslint-react/eff": "2.1.1-next.3",
-    "@eslint-react/kit": "2.1.1-next.3",
-    "@eslint-react/core": "2.1.1-next.3",
-    "@eslint-react/shared": "2.1.1-next.3",
-    "@eslint-react/var": "2.1.1-next.3"
+    "@eslint-react/ast": "2.1.1",
+    "@eslint-react/core": "2.1.1",
+    "@eslint-react/kit": "2.1.1",
+    "@eslint-react/shared": "2.1.1",
+    "@eslint-react/eff": "2.1.1",
+    "@eslint-react/var": "2.1.1"
   },
   "devDependencies": {
     "@types/react": "^19.2.2",