eslint-plugin-react-x 2.6.3-beta.0 → 2.6.4-beta.0

package/dist/index.d.ts

@@ -4,18 +4,27 @@ import * as _eslint_react_shared0 from "@eslint-react/shared";
 declare const _default: {
   configs: {
     /**
-             * Enforce rules that are recommended by ESLint React for general purpose React + React DOM projects
+             * Disable experimental rules that might be subject to change in the future
              */
-    recommended: {
+    "disable-experimental": {
       plugins: {};
       name?: string;
       rules?: Record<string, _eslint_react_shared0.RuleConfig>;
       settings?: _eslint_react_shared0.SettingsConfig;
     };
     /**
-             * Same as the `recommended` preset but disables rules that can be enforced by TypeScript
+             * Disable rules that can be enforced by TypeScript
              */
-    "recommended-typescript": {
+    "disable-type-checked": {
+      plugins: {};
+      name?: string;
+      rules?: Record<string, _eslint_react_shared0.RuleConfig>;
+      settings?: _eslint_react_shared0.SettingsConfig;
+    };
+    /**
+             * Enforce rules that are recommended by ESLint React for general purpose React + React DOM projects
+             */
+    recommended: {
       plugins: {};
       name?: string;
       rules?: Record<string, _eslint_react_shared0.RuleConfig>;
@@ -31,18 +40,18 @@ declare const _default: {
       settings?: _eslint_react_shared0.SettingsConfig;
     };
     /**
-             * More strict version of the `recommended` preset
+             * Same as the `recommended` preset but disables rules that can be enforced by TypeScript
              */
-    strict: {
+    "recommended-typescript": {
       plugins: {};
       name?: string;
       rules?: Record<string, _eslint_react_shared0.RuleConfig>;
       settings?: _eslint_react_shared0.SettingsConfig;
     };
     /**
-             * Same as the `strict` preset but enables additional rules that require type information
+             * More strict version of the `recommended` preset
              */
-    "strict-typescript": {
+    strict: {
       plugins: {};
       name?: string;
       rules?: Record<string, _eslint_react_shared0.RuleConfig>;
@@ -57,6 +66,15 @@ declare const _default: {
       rules?: Record<string, _eslint_react_shared0.RuleConfig>;
       settings?: _eslint_react_shared0.SettingsConfig;
     };
+    /**
+             * Same as the `strict` preset but disables rules that can be enforced by TypeScript
+             */
+    "strict-typescript": {
+      plugins: {};
+      name?: string;
+      rules?: Record<string, _eslint_react_shared0.RuleConfig>;
+      settings?: _eslint_react_shared0.SettingsConfig;
+    };
   };
   meta: {
     name: string;

package/dist/index.js

@@ -19,9 +19,9 @@ import { camelCase } from "string-ts";
 var __defProp = Object.defineProperty;
 var __exportAll = (all, symbols) => {
 	let target = {};
-	for (var name$7 in all) {
-		__defProp(target, name$7, {
-			get: all[name$7],
+	for (var name$9 in all) {
+		__defProp(target, name$9, {
+			get: all[name$9],
 			enumerable: true
 		});
 	}
@@ -31,10 +31,44 @@ var __exportAll = (all, symbols) => {
 	return target;
 };
 
+//#endregion
+//#region src/configs/disable-experimental.ts
+var disable_experimental_exports = /* @__PURE__ */ __exportAll({
+	name: () => name$8,
+	rules: () => rules$8
+});
+const name$8 = "react-x/disable-experimental";
+const rules$8 = {
+	"react-x/jsx-key-before-spread": "off",
+	"react-x/jsx-no-iife": "off",
+	"react-x/no-duplicate-key": "off",
+	"react-x/no-implicit-key": "off",
+	"react-x/no-misused-capture-owner-stack": "off",
+	"react-x/no-unnecessary-key": "off",
+	"react-x/no-unnecessary-use-callback": "off",
+	"react-x/no-unnecessary-use-memo": "off",
+	"react-x/no-unnecessary-use-ref": "off",
+	"react-x/no-unused-props": "off",
+	"react-x/prefer-read-only-props": "off"
+};
+
+//#endregion
+//#region src/configs/disable-type-checked.ts
+var disable_type_checked_exports = /* @__PURE__ */ __exportAll({
+	name: () => name$7,
+	rules: () => rules$7
+});
+const name$7 = "react-x/disable-type-checked";
+const rules$7 = {
+	"react-x/no-leaked-conditional-rendering": "off",
+	"react-x/no-unused-props": "off",
+	"react-x/prefer-read-only-props": "off"
+};
+
 //#endregion
 //#region package.json
 var name$6 = "eslint-plugin-react-x";
-var version = "2.6.3-beta.0";
+var version = "2.6.4-beta.0";
 
 //#endregion
 //#region src/utils/create-rule.ts
@@ -282,10 +316,10 @@ function create$60(context) {
 		const props = [];
 		for (const attr of node.attributes) {
 			if (attr.type === AST_NODE_TYPES.JSXSpreadAttribute) continue;
-			const name$7 = attr.name.name;
-			if (typeof name$7 !== "string") continue;
-			if (!props.includes(name$7)) {
-				props.push(name$7);
+			const name$9 = attr.name.name;
+			if (typeof name$9 !== "string") continue;
+			if (!props.includes(name$9)) {
+				props.push(name$9);
 				continue;
 			}
 			context.report({
@@ -343,17 +377,17 @@ var jsx_no_undef_default = createRule({
 });
 function create$58(context) {
 	return { JSXOpeningElement(node) {
-		const name$7 = match(node.name).with({ type: AST_NODE_TYPES.JSXIdentifier }, (n) => n.name).with({
+		const name$9 = 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$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({
+		if (name$9 == null) return;
+		if (name$9 === "this") return;
+		if (/^[a-z]/u.test(name$9)) return;
+		if (findVariable(name$9, context.sourceCode.getScope(node)) == null) context.report({
 			messageId: "jsxNoUndef",
 			node,
-			data: { name: name$7 }
+			data: { name: name$9 }
 		});
 	} };
 }
@@ -491,12 +525,12 @@ function create$55(context) {
 		JSXOpeningFragment: handleJsxElement
 	};
 }
-function debugReport(context, node, name$7) {
+function debugReport(context, node, name$9) {
 	if (process.env["ESLINT_REACT_DEBUG"] !== "1") return;
 	context.report({
 		messageId: "jsxUsesReact",
 		node,
-		data: { name: name$7 }
+		data: { name: name$9 }
 	});
 }
 
@@ -640,8 +674,8 @@ const arrayIndexParamPosition = new Map([
 function getArrayIndexParamPosition(methodName) {
 	return arrayIndexParamPosition.get(methodName) ?? -1;
 }
-function isReactChildrenMethod(name$7) {
-	return REACT_CHILDREN_METHOD.includes(name$7);
+function isReactChildrenMethod(name$9) {
+	return REACT_CHILDREN_METHOD.includes(name$9);
 }
 function isUsingReactChildren(context, node) {
 	const { importSource = "react" } = coerceSettings(context.settings);
@@ -657,8 +691,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$7 } = callee.property;
-	const indexPosition = getArrayIndexParamPosition(name$7);
+	const { name: name$9 } = callee.property;
+	const indexPosition = getArrayIndexParamPosition(name$9);
 	if (indexPosition === -1) return unit;
 	const callbackArg = node.arguments[isUsingReactChildren(context, node) ? 1 : 0];
 	if (callbackArg == null) return unit;
@@ -687,7 +721,7 @@ var no_array_index_key_default = createRule({
 function create$52(context) {
 	const indexParamNames = [];
 	function isArrayIndex(node) {
-		return node.type === AST_NODE_TYPES.Identifier && indexParamNames.some((name$7) => name$7 != null && name$7 === node.name);
+		return node.type === AST_NODE_TYPES.Identifier && indexParamNames.some((name$9) => name$9 != null && name$9 === node.name);
 	}
 	function isCreateOrCloneElementCall(node) {
 		return isCreateElementCall(context, node) || isCloneElementCall(context, node);
@@ -695,7 +729,7 @@ function create$52(context) {
 	function getReportDescriptors(node) {
 		switch (node.type) {
 			case AST_NODE_TYPES.Identifier:
-				if (indexParamNames.some((name$7) => name$7 != null && name$7 === node.name)) return [{
+				if (indexParamNames.some((name$9) => name$9 != null && name$9 === node.name)) return [{
 					messageId: "noArrayIndexKey",
 					node
 				}];
@@ -907,12 +941,12 @@ function create$45(context) {
 	return {
 		...listeners,
 		"Program:exit"(program) {
-			for (const { name: name$7 = "anonymous", node: component } of ctx.getAllComponents(program)) {
+			for (const { name: name$9 = "anonymous", node: component } of ctx.getAllComponents(program)) {
 				if (component.body.body.some((m) => isComponentDidCatch(m) || isGetDerivedStateFromError(m))) continue;
 				context.report({
 					messageId: "noClassComponent",
 					node: component,
-					data: { name: name$7 }
+					data: { name: name$9 }
 				});
 			}
 		}
@@ -1713,15 +1747,15 @@ function create$28(context) {
 			const isClassComponent$1 = (node) => {
 				return AST.isClass(node) && cComponents.some((component) => component.node === node);
 			};
-			for (const { name: name$7, node: component } of fComponents) {
-				if (name$7 == null) continue;
+			for (const { name: name$9, node: component } of fComponents) {
+				if (name$9 == null) continue;
 				if (isDirectValueOfRenderPropertyLoose(component)) continue;
 				if (isInsideJSXAttributeValue(component)) {
 					if (!isDeclaredInRenderPropLoose(component)) context.report({
 						messageId: "noNestedComponentDefinitions",
 						node: component,
 						data: {
-							name: name$7,
+							name: name$9,
 							suggestion: "Move it to the top level or pass it as a prop."
 						}
 					});
@@ -1732,7 +1766,7 @@ function create$28(context) {
 						messageId: "noNestedComponentDefinitions",
 						node: component,
 						data: {
-							name: name$7,
+							name: name$9,
 							suggestion: "Move it to the top level or pass it as a prop."
 						}
 					});
@@ -1744,7 +1778,7 @@ function create$28(context) {
 						messageId: "noNestedComponentDefinitions",
 						node: component,
 						data: {
-							name: name$7,
+							name: name$9,
 							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."
 						}
 					});
@@ -1754,18 +1788,18 @@ function create$28(context) {
 					messageId: "noNestedComponentDefinitions",
 					node: component,
 					data: {
-						name: name$7,
+						name: name$9,
 						suggestion: "Move it to the top level."
 					}
 				});
 			}
-			for (const { name: name$7 = "unknown", node: component } of cComponents) {
+			for (const { name: name$9 = "unknown", node: component } of cComponents) {
 				if (AST.findParentNode(component, (n) => isClassComponent$1(n) || isFunctionComponent(n)) == null) continue;
 				context.report({
 					messageId: "noNestedComponentDefinitions",
 					node: component,
 					data: {
-						name: name$7,
+						name: name$9,
 						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."
 					}
 				});
@@ -1910,13 +1944,13 @@ function create$25(context) {
 	return {
 		...listeners,
 		"Program:exit"(program) {
-			for (const { name: name$7 = "PureComponent", node: component, flag } of ctx.getAllComponents(program)) {
+			for (const { name: name$9 = "PureComponent", node: component, flag } of ctx.getAllComponents(program)) {
 				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$7 }
+					data: { componentName: name$9 }
 				});
 			}
 		}
@@ -2299,16 +2333,16 @@ function create$17(context) {
 	return {
 		...listeners,
 		"Program:exit"(program) {
-			for (const { id, name: name$7, node, hookCalls } of ctx.getAllHooks(program)) {
+			for (const { id, name: name$9, node, hookCalls } of ctx.getAllHooks(program)) {
 				if (hookCalls.length > 0) continue;
 				if (AST.isFunctionEmpty(node)) continue;
-				if (WELL_KNOWN_HOOKS.includes(name$7)) continue;
+				if (WELL_KNOWN_HOOKS.includes(name$9)) continue;
 				if (containsUseComments(context, node)) continue;
 				if (AST.findParentNode(node, AST.isViMockCallback) != null) continue;
 				context.report({
 					messageId: "noUnnecessaryUsePrefix",
 					node: id ?? node,
-					data: { name: name$7 }
+					data: { name: name$9 }
 				});
 			}
 		}
@@ -2498,9 +2532,9 @@ function create$12(context) {
 		}
 	};
 }
-function isContextName(name$7, isReact18OrBelow) {
-	if (name$7 === "Provider") return true;
-	if (!isReact18OrBelow) return name$7.endsWith("Context") || name$7.endsWith("CONTEXT");
+function isContextName(name$9, isReact18OrBelow) {
+	if (name$9 === "Provider") return true;
+	if (!isReact18OrBelow) return name$9.endsWith("Context") || name$9.endsWith("CONTEXT");
 	return false;
 }
 
@@ -2552,8 +2586,8 @@ function create$11(context, [options]) {
 				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$7 }) => {
-					return declarators.get(component)?.filter((d) => d.init.name === name$7).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$9 }) => {
+					return declarators.get(component)?.filter((d) => d.init.name === name$9).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;
@@ -3443,17 +3477,17 @@ function create(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$7 = attr.name.name;
-			if (typeof name$7 !== "string") continue;
+			const name$9 = attr.name.name;
+			if (typeof name$9 !== "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$7)) context.report({
+				if (toRegExp(typeof forbiddenPropItem === "string" ? forbiddenPropItem : forbiddenPropItem.prop).test(name$9)) context.report({
 					messageId,
 					node: attr,
-					data: { name: name$7 }
+					data: { name: name$9 }
 				});
 			}
 		}
@@ -3698,12 +3732,14 @@ const { toFlatConfig } = getConfigAdapters("react-x", plugin);
 var src_default = {
 	...plugin,
 	configs: {
+		["disable-experimental"]: toFlatConfig(disable_experimental_exports),
+		["disable-type-checked"]: toFlatConfig(disable_type_checked_exports),
 		["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)
+		["strict-type-checked"]: toFlatConfig(strict_type_checked_exports),
+		["strict-typescript"]: toFlatConfig(strict_typescript_exports)
 	}
 };
 

package/package.json

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