eslint-plugin-react-x 3.0.0-next.71 → 3.0.0-next.76

package/dist/index.js

@@ -9,7 +9,6 @@ import { AST_NODE_TYPES } from "@typescript-eslint/types";
 import { computeObjectType, findEnclosingAssignmentTarget, isAssignmentTargetEqual, resolve } from "@eslint-react/var";
 import { DefinitionType } from "@typescript-eslint/scope-manager";
 import { findVariable, getStaticValue, isIdentifier, isVariableDeclarator } from "@typescript-eslint/utils/ast-utils";
-import { constFalse, constTrue, constVoid, flow, getOrElseUpdate, identity, not } from "@eslint-react/eff";
 import { compare } from "compare-versions";
 import { getConstrainedTypeAtLocation } from "@typescript-eslint/type-utils";
 import { unionConstituents } from "ts-api-utils";
@@ -51,14 +50,11 @@ const conflictingRules$1 = [
 	"react/jsx-key",
 	"react/jsx-no-comment-textnodes",
 	"react/jsx-no-constructed-context-values",
-	"react/jsx-no-duplicate-props",
 	"react/jsx-no-leaked-render",
 	"react/jsx-no-script-url",
 	"react/jsx-no-target-blank",
 	"react/jsx-no-useless-fragment",
 	"react/jsx-pascal-case",
-	"react/jsx-uses-react",
-	"react/jsx-uses-vars",
 	"react/no-access-state-in-setstate",
 	"react/no-array-index-key",
 	"react/no-children-prop",
@@ -117,17 +113,17 @@ var disable_experimental_exports = /* @__PURE__ */ __exportAll({
 });
 const name$8 = "react-x/disable-experimental";
 const rules$8 = {
-	"react-x/exhaustive-deps": "off",
 	"react-x/immutability": "off",
 	"react-x/jsx-key-before-spread": "off",
 	"react-x/no-duplicate-key": "off",
+	"react-x/no-implicit-children": "off",
 	"react-x/no-implicit-key": "off",
+	"react-x/no-implicit-ref": "off",
 	"react-x/no-misused-capture-owner-stack": "off",
 	"react-x/no-unnecessary-use-callback": "off",
 	"react-x/no-unnecessary-use-memo": "off",
 	"react-x/no-unused-props": "off",
 	"react-x/refs": "off",
-	"react-x/rules-of-hooks": "off",
 	"react-x/set-state-in-render": "off",
 	"react-x/unstable-rules-of-props": "off"
 };
@@ -140,7 +136,9 @@ var disable_type_checked_exports = /* @__PURE__ */ __exportAll({
 });
 const name$7 = "react-x/disable-type-checked";
 const rules$7 = {
+	"react-x/no-implicit-children": "off",
 	"react-x/no-implicit-key": "off",
+	"react-x/no-implicit-ref": "off",
 	"react-x/no-leaked-conditional-rendering": "off",
 	"react-x/no-unused-props": "off"
 };
@@ -148,7 +146,7 @@ const rules$7 = {
 //#endregion
 //#region package.json
 var name$6 = "eslint-plugin-react-x";
-var version = "3.0.0-next.71";
+var version = "3.0.0-next.76";
 
 //#endregion
 //#region src/utils/create-rule.ts
@@ -305,7 +303,7 @@ function getTypeVariants(types) {
 
 //#endregion
 //#region src/rules/component-hook-factories/component-hook-factories.ts
-const RULE_NAME$63 = "component-hook-factories";
+const RULE_NAME$62 = "component-hook-factories";
 var component_hook_factories_default = createRule({
 	meta: {
 		type: "problem",
@@ -316,11 +314,11 @@ var component_hook_factories_default = createRule({
 		},
 		schema: []
 	},
-	name: RULE_NAME$63,
-	create: create$63,
+	name: RULE_NAME$62,
+	create: create$62,
 	defaultOptions: []
 });
-function create$63(context) {
+function create$62(context) {
 	const hint = core.ComponentDetectionHint.DoNotIncludeJsxWithNumberValue | core.ComponentDetectionHint.DoNotIncludeJsxWithBooleanValue | core.ComponentDetectionHint.DoNotIncludeJsxWithNullValue | core.ComponentDetectionHint.DoNotIncludeJsxWithStringValue | core.ComponentDetectionHint.DoNotIncludeJsxWithUndefinedValue | core.ComponentDetectionHint.RequireBothSidesOfLogicalExpressionToBeJsx | core.ComponentDetectionHint.RequireBothBranchesOfConditionalExpressionToBeJsx | core.ComponentDetectionHint.DoNotIncludeFunctionDefinedInArrayPattern | core.ComponentDetectionHint.DoNotIncludeFunctionDefinedInArrayExpression | core.ComponentDetectionHint.DoNotIncludeFunctionDefinedAsArrayMapCallback;
 	const fCollector = core.useComponentCollector(context, { hint });
 	const cCollector = core.useComponentCollectorLegacy(context);
@@ -364,7 +362,7 @@ function create$63(context) {
 
 //#endregion
 //#region src/rules/error-boundaries/error-boundaries.ts
-const RULE_NAME$62 = "error-boundaries";
+const RULE_NAME$61 = "error-boundaries";
 var error_boundaries_default = createRule({
 	meta: {
 		type: "problem",
@@ -375,11 +373,11 @@ var error_boundaries_default = createRule({
 		},
 		schema: []
 	},
-	name: RULE_NAME$62,
-	create: create$62,
+	name: RULE_NAME$61,
+	create: create$61,
 	defaultOptions: []
 });
-function create$62(context) {
+function create$61(context) {
 	if (!context.sourceCode.text.includes("try")) return {};
 	const { ctx, visitor } = core.useComponentCollector(context);
 	const reported = /* @__PURE__ */ new Set();
@@ -1319,7 +1317,7 @@ function getUnknownDependenciesMessage(reactiveHookName) {
 
 //#endregion
 //#region src/rules/immutability/immutability.ts
-const RULE_NAME$61 = "immutability";
+const RULE_NAME$60 = "immutability";
 /**
 * Array methods that mutate the array in place.
 * @see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array
@@ -1345,11 +1343,11 @@ var immutability_default = createRule({
 		},
 		schema: []
 	},
-	name: RULE_NAME$61,
-	create: create$61,
+	name: RULE_NAME$60,
+	create: create$60,
 	defaultOptions: []
 });
-function create$61(context) {
+function create$60(context) {
 	const { additionalStateHooks } = getSettingsFromContext(context);
 	const hCollector = core.useHookCollector(context);
 	const cCollector = core.useComponentCollector(context);
@@ -1470,7 +1468,7 @@ function create$61(context) {
 
 //#endregion
 //#region src/rules/jsx-dollar/jsx-dollar.ts
-const RULE_NAME$60 = "jsx-dollar";
+const RULE_NAME$59 = "jsx-dollar";
 var jsx_dollar_default = createRule({
 	meta: {
 		type: "suggestion",
@@ -1483,11 +1481,11 @@ var jsx_dollar_default = createRule({
 		},
 		schema: []
 	},
-	name: RULE_NAME$60,
-	create: create$60,
+	name: RULE_NAME$59,
+	create: create$59,
 	defaultOptions: []
 });
-function create$60(context) {
+function create$59(context) {
 	/**
 	* Visitor function for JSXElement and JSXFragment nodes
 	* @param node The JSXElement or JSXFragment node to be checked
@@ -1528,7 +1526,7 @@ function create$60(context) {
 
 //#endregion
 //#region src/rules/jsx-key-before-spread/jsx-key-before-spread.ts
-const RULE_NAME$59 = "jsx-key-before-spread";
+const RULE_NAME$58 = "jsx-key-before-spread";
 var jsx_key_before_spread_default = createRule({
 	meta: {
 		type: "problem",
@@ -1536,11 +1534,11 @@ var jsx_key_before_spread_default = createRule({
 		messages: { default: "The 'key' prop must be placed before any spread props when using the new JSX transform." },
 		schema: []
 	},
-	name: RULE_NAME$59,
-	create: create$59,
+	name: RULE_NAME$58,
+	create: create$58,
 	defaultOptions: []
 });
-function create$59(context) {
+function create$58(context) {
 	const { jsx } = JsxInspector.from(context).jsxConfig;
 	if (jsx !== JsxEmit.ReactJSX && jsx !== JsxEmit.ReactJSXDev) return {};
 	return defineRuleListener({ JSXOpeningElement(node) {
@@ -1561,7 +1559,7 @@ function create$59(context) {
 
 //#endregion
 //#region src/rules/jsx-no-comment-textnodes/jsx-no-comment-textnodes.ts
-const RULE_NAME$58 = "jsx-no-comment-textnodes";
+const RULE_NAME$57 = "jsx-no-comment-textnodes";
 var jsx_no_comment_textnodes_default = createRule({
 	meta: {
 		type: "problem",
@@ -1569,11 +1567,11 @@ var jsx_no_comment_textnodes_default = createRule({
 		messages: { default: "Possible misused comment in text node. Comments inside children section of tag should be placed inside braces." },
 		schema: []
 	},
-	name: RULE_NAME$58,
-	create: create$58,
+	name: RULE_NAME$57,
+	create: create$57,
 	defaultOptions: []
 });
-function create$58(context) {
+function create$57(context) {
 	function hasCommentLike(node) {
 		if (ast.isOneOf([AST_NODE_TYPES.JSXAttribute, AST_NODE_TYPES.JSXExpressionContainer])(node.parent)) return false;
 		return /^\s*\/(?:\/|\*)/mu.test(context.sourceCode.getText(node));
@@ -1592,39 +1590,6 @@ function create$58(context) {
 	});
 }
 
-//#endregion
-//#region src/rules/jsx-no-duplicate-props/jsx-no-duplicate-props.ts
-const RULE_NAME$57 = "jsx-no-duplicate-props";
-var jsx_no_duplicate_props_default = createRule({
-	meta: {
-		type: "problem",
-		docs: { description: "Disallows duplicate props in JSX elements." },
-		messages: { default: "This JSX property is assigned multiple times." },
-		schema: []
-	},
-	name: RULE_NAME$57,
-	create: create$57,
-	defaultOptions: []
-});
-function create$57(context) {
-	return defineRuleListener({ JSXOpeningElement(node) {
-		const props = [];
-		for (const attr of node.attributes) {
-			if (attr.type === AST_NODE_TYPES.JSXSpreadAttribute) continue;
-			const name = attr.name.name;
-			if (typeof name !== "string") continue;
-			if (!props.includes(name)) {
-				props.push(name);
-				continue;
-			}
-			context.report({
-				messageId: "default",
-				node: attr
-			});
-		}
-	} });
-}
-
 //#endregion
 //#region src/rules/jsx-shorthand-boolean/jsx-shorthand-boolean.ts
 const RULE_NAME$56 = "jsx-shorthand-boolean";
@@ -1723,78 +1688,193 @@ function create$55(context) {
 }
 
 //#endregion
-//#region src/rules/jsx-uses-react/jsx-uses-react.ts
-const RULE_NAME$54 = "jsx-uses-react";
-var jsx_uses_react_default = createRule({
-	meta: {
-		type: "problem",
-		docs: { description: "Marks React variables as used when JSX is present." },
-		messages: { default: "Marked {{name}} as used." },
-		schema: []
-	},
-	name: RULE_NAME$54,
-	create: create$54,
-	defaultOptions: []
-});
-function create$54(context) {
-	const { jsx, jsxFactory, jsxFragmentFactory } = JsxInspector.from(context).jsxConfig;
-	if (jsx === JsxEmit.ReactJSX || jsx === JsxEmit.ReactJSXDev) return {};
-	function handleJsxElement(node) {
-		context.sourceCode.markVariableAsUsed(jsxFactory, node);
-		debugReport(context, node, jsxFactory);
-	}
-	function handleJsxFragment(node) {
-		context.sourceCode.markVariableAsUsed(jsxFragmentFactory, node);
-		debugReport(context, node, jsxFragmentFactory);
+//#region ../../../.pkgs/eff/dist/index.js
+function not(predicate) {
+	return (data) => !predicate(data);
+}
+/**
+* Returns its argument.
+*
+* @param x - The value to return.
+* @returns The input value unchanged.
+*/
+function identity(x) {
+	return x;
+}
+/**
+* Creates a function that can be used in a data-last (aka `pipe`able) or
+* data-first style.
+*
+* The first parameter to `dual` is either the arity of the uncurried function
+* or a predicate that determines if the function is being used in a data-first
+* or data-last style.
+*
+* Using the arity is the most common use case, but there are some cases where
+* you may want to use a predicate. For example, if you have a function that
+* takes an optional argument, you can use a predicate to determine if the
+* function is being used in a data-first or data-last style.
+*
+* You can pass either the arity of the uncurried function or a predicate
+* which determines if the function is being used in a data-first or
+* data-last style.
+*
+* **Example** (Using arity to determine data-first or data-last style)
+*
+* ```ts
+* import { dual, pipe } from "effect/Function"
+*
+* const sum = dual<
+*   (that: number) => (self: number) => number,
+*   (self: number, that: number) => number
+* >(2, (self, that) => self + that)
+*
+* console.log(sum(2, 3)) // 5
+* console.log(pipe(2, sum(3))) // 5
+* ```
+*
+* **Example** (Using call signatures to define the overloads)
+*
+* ```ts
+* import { dual, pipe } from "effect/Function"
+*
+* const sum: {
+*   (that: number): (self: number) => number
+*   (self: number, that: number): number
+* } = dual(2, (self: number, that: number): number => self + that)
+*
+* console.log(sum(2, 3)) // 5
+* console.log(pipe(2, sum(3))) // 5
+* ```
+*
+* **Example** (Using a predicate to determine data-first or data-last style)
+*
+* ```ts
+* import { dual, pipe } from "effect/Function"
+*
+* const sum = dual<
+*   (that: number) => (self: number) => number,
+*   (self: number, that: number) => number
+* >(
+*   (args) => args.length === 2,
+*   (self, that) => self + that
+* )
+*
+* console.log(sum(2, 3)) // 5
+* console.log(pipe(2, sum(3))) // 5
+* ```
+*
+* @param arity - The arity of the uncurried function or a predicate that determines if the function is being used in a data-first or data-last style.
+* @param body - The function to be curried.
+* @since 1.0.0
+*/
+const dual = function(arity, body) {
+	if (typeof arity === "function") return function() {
+		return arity(arguments) ? body.apply(this, arguments) : ((self) => body(self, ...arguments));
+	};
+	switch (arity) {
+		case 0:
+		case 1: throw new RangeError(`Invalid arity ${arity}`);
+		case 2: return function(a, b) {
+			if (arguments.length >= 2) return body(a, b);
+			return function(self) {
+				return body(self, a);
+			};
+		};
+		case 3: return function(a, b, c) {
+			if (arguments.length >= 3) return body(a, b, c);
+			return function(self) {
+				return body(self, a, b);
+			};
+		};
+		default: return function() {
+			if (arguments.length >= arity) return body.apply(this, arguments);
+			const args = arguments;
+			return function(self) {
+				return body(self, ...args);
+			};
+		};
 	}
-	return defineRuleListener({
-		JSXFragment: handleJsxFragment,
-		JSXOpeningElement: handleJsxElement,
-		JSXOpeningFragment: handleJsxElement
-	});
+};
+/**
+* Do nothing and return `void`.
+*/
+function constVoid() {}
+/**
+* Do nothing and return `true`.
+*
+* @returns true
+*/
+function constTrue() {
+	return true;
 }
-function debugReport(context, node, name) {
-	if (process.env["ESLINT_REACT_DEBUG"] !== "1") return;
-	context.report({
-		data: { name },
-		messageId: "default",
-		node
-	});
+/**
+* Do nothing and return `false`.
+*
+* @returns false
+*/
+function constFalse() {
+	return false;
 }
-
-//#endregion
-//#region src/rules/jsx-uses-vars/jsx-uses-vars.ts
-const RULE_NAME$53 = "jsx-uses-vars";
-var jsx_uses_vars_default = createRule({
-	meta: {
-		type: "problem",
-		docs: { description: "Marks JSX element variables as used." },
-		messages: { default: "An identifier in JSX is marked as used." },
-		schema: []
-	},
-	name: RULE_NAME$53,
-	create: create$53,
-	defaultOptions: []
-});
-function create$53(context) {
-	return defineRuleListener({ JSXOpeningElement(node) {
-		switch (node.name.type) {
-			case AST_NODE_TYPES.JSXIdentifier:
-				if (/^[a-z]/u.test(node.name.name)) return;
-				context.sourceCode.markVariableAsUsed(node.name.name, node);
-				break;
-			case AST_NODE_TYPES.JSXMemberExpression: {
-				const { object } = node.name;
-				if (object.type === AST_NODE_TYPES.JSXIdentifier) context.sourceCode.markVariableAsUsed(object.name, node);
-				break;
-			}
-		}
-	} });
+/**
+* Composes two functions, `ab` and `bc` into a single function that takes in an argument `a` of type `A` and returns a result of type `C`.
+* The result is obtained by first applying the `ab` function to `a` and then applying the `bc` function to the result of `ab`.
+*
+* @param self - The first function to apply (or the composed function in data-last style).
+* @param bc - The second function to apply.
+* @returns A composed function that applies both functions in sequence.
+* @example
+* ```ts
+* import * as assert from "node:assert"
+* import { compose } from "effect/Function"
+*
+* const increment = (n: number) => n + 1;
+* const square = (n: number) => n * n;
+*
+* assert.strictEqual(compose(increment, square)(2), 9);
+* ```
+*
+* @since 1.0.0
+*/
+const compose = dual(2, (ab, bc) => (a) => bc(ab(a)));
+function flow(ab, bc, cd, de, ef, fg, gh, hi, ij) {
+	switch (arguments.length) {
+		case 1: return ab;
+		case 2: return function() {
+			return bc(ab.apply(this, arguments));
+		};
+		case 3: return function() {
+			return cd(bc(ab.apply(this, arguments)));
+		};
+		case 4: return function() {
+			return de(cd(bc(ab.apply(this, arguments))));
+		};
+		case 5: return function() {
+			return ef(de(cd(bc(ab.apply(this, arguments)))));
+		};
+		case 6: return function() {
+			return fg(ef(de(cd(bc(ab.apply(this, arguments))))));
+		};
+		case 7: return function() {
+			return gh(fg(ef(de(cd(bc(ab.apply(this, arguments)))))));
+		};
+		case 8: return function() {
+			return hi(gh(fg(ef(de(cd(bc(ab.apply(this, arguments))))))));
+		};
+		case 9: return function() {
+			return ij(hi(gh(fg(ef(de(cd(bc(ab.apply(this, arguments)))))))));
+		};
+	}
+}
+function getOrElseUpdate(map, key, callback) {
+	if (map.has(key)) return map.get(key);
+	const value = callback();
+	map.set(key, value);
+	return value;
 }
 
 //#endregion
 //#region src/rules/no-access-state-in-setstate/no-access-state-in-setstate.ts
-const RULE_NAME$52 = "no-access-state-in-setstate";
+const RULE_NAME$54 = "no-access-state-in-setstate";
 function isKeyLiteral$2(node, key) {
 	return match(key).with({ type: AST_NODE_TYPES.Literal }, constTrue).with({
 		type: AST_NODE_TYPES.TemplateLiteral,
@@ -1808,11 +1888,11 @@ var no_access_state_in_setstate_default = createRule({
 		messages: { default: "Do not access 'this.state' within 'setState'. Use the update function instead." },
 		schema: []
 	},
-	name: RULE_NAME$52,
-	create: create$52,
+	name: RULE_NAME$54,
+	create: create$54,
 	defaultOptions: []
 });
-function create$52(context) {
+function create$54(context) {
 	if (!context.sourceCode.text.includes("setState")) return {};
 	const classStack = [];
 	const methodStack = [];
@@ -1883,7 +1963,7 @@ function create$52(context) {
 
 //#endregion
 //#region src/rules/no-array-index-key/no-array-index-key.ts
-const RULE_NAME$51 = "no-array-index-key";
+const RULE_NAME$53 = "no-array-index-key";
 function getIndexParamPosition(methodName) {
 	switch (methodName) {
 		case "every":
@@ -1928,11 +2008,11 @@ var no_array_index_key_default = createRule({
 		messages: { default: "Do not use item index in the array as its key." },
 		schema: []
 	},
-	name: RULE_NAME$51,
-	create: create$51,
+	name: RULE_NAME$53,
+	create: create$53,
 	defaultOptions: []
 });
-function create$51(context) {
+function create$53(context) {
 	const indexParamNames = [];
 	function isArrayIndex(node) {
 		return node.type === AST_NODE_TYPES.Identifier && indexParamNames.some((name) => name != null && name === node.name);
@@ -1998,7 +2078,7 @@ function create$51(context) {
 
 //#endregion
 //#region src/rules/no-children-count/no-children-count.ts
-const RULE_NAME$50 = "no-children-count";
+const RULE_NAME$52 = "no-children-count";
 var no_children_count_default = createRule({
 	meta: {
 		type: "suggestion",
@@ -2006,11 +2086,11 @@ var no_children_count_default = createRule({
 		messages: { default: "Using 'Children.count' is uncommon and can lead to fragile code. Use alternatives instead." },
 		schema: []
 	},
-	name: RULE_NAME$50,
-	create: create$50,
+	name: RULE_NAME$52,
+	create: create$52,
 	defaultOptions: []
 });
-function create$50(context) {
+function create$52(context) {
 	return defineRuleListener({ MemberExpression(node) {
 		if (core.isChildrenCount(context, node)) context.report({
 			messageId: "default",
@@ -2021,7 +2101,7 @@ function create$50(context) {
 
 //#endregion
 //#region src/rules/no-children-for-each/no-children-for-each.ts
-const RULE_NAME$49 = "no-children-for-each";
+const RULE_NAME$51 = "no-children-for-each";
 var no_children_for_each_default = createRule({
 	meta: {
 		type: "suggestion",
@@ -2029,11 +2109,11 @@ var no_children_for_each_default = createRule({
 		messages: { default: "Using 'Children.forEach' is uncommon and can lead to fragile code. Use alternatives instead." },
 		schema: []
 	},
-	name: RULE_NAME$49,
-	create: create$49,
+	name: RULE_NAME$51,
+	create: create$51,
 	defaultOptions: []
 });
-function create$49(context) {
+function create$51(context) {
 	return defineRuleListener({ MemberExpression(node) {
 		if (core.isChildrenForEach(context, node)) context.report({
 			messageId: "default",
@@ -2044,7 +2124,7 @@ function create$49(context) {
 
 //#endregion
 //#region src/rules/no-children-map/no-children-map.ts
-const RULE_NAME$48 = "no-children-map";
+const RULE_NAME$50 = "no-children-map";
 var no_children_map_default = createRule({
 	meta: {
 		type: "suggestion",
@@ -2052,11 +2132,11 @@ var no_children_map_default = createRule({
 		messages: { default: "Using 'Children.map' is uncommon and can lead to fragile code. Use alternatives instead." },
 		schema: []
 	},
-	name: RULE_NAME$48,
-	create: create$48,
+	name: RULE_NAME$50,
+	create: create$50,
 	defaultOptions: []
 });
-function create$48(context) {
+function create$50(context) {
 	return defineRuleListener({ MemberExpression(node) {
 		if (core.isChildrenMap(context, node)) context.report({
 			messageId: "default",
@@ -2067,7 +2147,7 @@ function create$48(context) {
 
 //#endregion
 //#region src/rules/no-children-only/no-children-only.ts
-const RULE_NAME$47 = "no-children-only";
+const RULE_NAME$49 = "no-children-only";
 var no_children_only_default = createRule({
 	meta: {
 		type: "suggestion",
@@ -2075,11 +2155,11 @@ var no_children_only_default = createRule({
 		messages: { default: "Using 'Children.only' is uncommon and can lead to fragile code. Use alternatives instead." },
 		schema: []
 	},
-	name: RULE_NAME$47,
-	create: create$47,
+	name: RULE_NAME$49,
+	create: create$49,
 	defaultOptions: []
 });
-function create$47(context) {
+function create$49(context) {
 	return defineRuleListener({ MemberExpression(node) {
 		if (core.isChildrenOnly(context, node)) context.report({
 			messageId: "default",
@@ -2090,7 +2170,7 @@ function create$47(context) {
 
 //#endregion
 //#region src/rules/no-children-prop/no-children-prop.ts
-const RULE_NAME$46 = "no-children-prop";
+const RULE_NAME$48 = "no-children-prop";
 var no_children_prop_default = createRule({
 	meta: {
 		type: "suggestion",
@@ -2098,11 +2178,11 @@ var no_children_prop_default = createRule({
 		messages: { default: "Do not pass 'children' as props." },
 		schema: []
 	},
-	name: RULE_NAME$46,
-	create: create$46,
+	name: RULE_NAME$48,
+	create: create$48,
 	defaultOptions: []
 });
-function create$46(context) {
+function create$48(context) {
 	const jsx = JsxInspector.from(context);
 	return defineRuleListener({ JSXElement(node) {
 		const childrenProp = jsx.findAttribute(node, "children");
@@ -2115,7 +2195,7 @@ function create$46(context) {
 
 //#endregion
 //#region src/rules/no-children-to-array/no-children-to-array.ts
-const RULE_NAME$45 = "no-children-to-array";
+const RULE_NAME$47 = "no-children-to-array";
 var no_children_to_array_default = createRule({
 	meta: {
 		type: "suggestion",
@@ -2123,11 +2203,11 @@ var no_children_to_array_default = createRule({
 		messages: { default: "Using 'Children.toArray' is uncommon and can lead to fragile code. Use alternatives instead." },
 		schema: []
 	},
-	name: RULE_NAME$45,
-	create: create$45,
+	name: RULE_NAME$47,
+	create: create$47,
 	defaultOptions: []
 });
-function create$45(context) {
+function create$47(context) {
 	return defineRuleListener({ MemberExpression(node) {
 		if (core.isChildrenToArray(context, node)) context.report({
 			messageId: "default",
@@ -2138,7 +2218,7 @@ function create$45(context) {
 
 //#endregion
 //#region src/rules/no-class-component/no-class-component.ts
-const RULE_NAME$44 = "no-class-component";
+const RULE_NAME$46 = "no-class-component";
 var no_class_component_default = createRule({
 	meta: {
 		type: "suggestion",
@@ -2146,11 +2226,11 @@ var no_class_component_default = createRule({
 		messages: { default: "Avoid using class components. Use function components instead." },
 		schema: []
 	},
-	name: RULE_NAME$44,
-	create: create$44,
+	name: RULE_NAME$46,
+	create: create$46,
 	defaultOptions: []
 });
-function create$44(context) {
+function create$46(context) {
 	if (!context.sourceCode.text.includes("Component")) return {};
 	const { ctx, visitor } = core.useComponentCollectorLegacy(context);
 	return defineRuleListener(visitor, { "Program:exit"(program) {
@@ -2167,7 +2247,7 @@ function create$44(context) {
 
 //#endregion
 //#region src/rules/no-clone-element/no-clone-element.ts
-const RULE_NAME$43 = "no-clone-element";
+const RULE_NAME$45 = "no-clone-element";
 var no_clone_element_default = createRule({
 	meta: {
 		type: "suggestion",
@@ -2175,11 +2255,11 @@ var no_clone_element_default = createRule({
 		messages: { default: "Using 'cloneElement' is uncommon and can lead to fragile code. Use alternatives instead." },
 		schema: []
 	},
-	name: RULE_NAME$43,
-	create: create$43,
+	name: RULE_NAME$45,
+	create: create$45,
 	defaultOptions: []
 });
-function create$43(context) {
+function create$45(context) {
 	return defineRuleListener({ CallExpression(node) {
 		if (core.isCloneElementCall(context, node)) context.report({
 			messageId: "default",
@@ -2190,7 +2270,7 @@ function create$43(context) {
 
 //#endregion
 //#region src/rules/no-component-will-mount/no-component-will-mount.ts
-const RULE_NAME$42 = "no-component-will-mount";
+const RULE_NAME$44 = "no-component-will-mount";
 var no_component_will_mount_default = createRule({
 	meta: {
 		type: "problem",
@@ -2199,11 +2279,11 @@ var no_component_will_mount_default = createRule({
 		messages: { default: "[Deprecated] Use 'UNSAFE_componentWillMount' instead." },
 		schema: []
 	},
-	name: RULE_NAME$42,
-	create: create$42,
+	name: RULE_NAME$44,
+	create: create$44,
 	defaultOptions: []
 });
-function create$42(context) {
+function create$44(context) {
 	if (!context.sourceCode.text.includes("componentWillMount")) return {};
 	const { ctx, visitor } = core.useComponentCollectorLegacy(context);
 	return defineRuleListener(visitor, { "Program:exit"(program) {
@@ -2223,7 +2303,7 @@ function create$42(context) {
 
 //#endregion
 //#region src/rules/no-component-will-receive-props/no-component-will-receive-props.ts
-const RULE_NAME$41 = "no-component-will-receive-props";
+const RULE_NAME$43 = "no-component-will-receive-props";
 var no_component_will_receive_props_default = createRule({
 	meta: {
 		type: "problem",
@@ -2232,11 +2312,11 @@ var no_component_will_receive_props_default = createRule({
 		messages: { default: "[Deprecated] Use 'UNSAFE_componentWillReceiveProps' instead." },
 		schema: []
 	},
-	name: RULE_NAME$41,
-	create: create$41,
+	name: RULE_NAME$43,
+	create: create$43,
 	defaultOptions: []
 });
-function create$41(context) {
+function create$43(context) {
 	if (!context.sourceCode.text.includes("componentWillReceiveProps")) return {};
 	const { ctx, visitor } = core.useComponentCollectorLegacy(context);
 	return defineRuleListener(visitor, { "Program:exit"(program) {
@@ -2256,7 +2336,7 @@ function create$41(context) {
 
 //#endregion
 //#region src/rules/no-component-will-update/no-component-will-update.ts
-const RULE_NAME$40 = "no-component-will-update";
+const RULE_NAME$42 = "no-component-will-update";
 var no_component_will_update_default = createRule({
 	meta: {
 		type: "problem",
@@ -2265,11 +2345,11 @@ var no_component_will_update_default = createRule({
 		messages: { default: "[Deprecated] Use 'UNSAFE_componentWillUpdate' instead." },
 		schema: []
 	},
-	name: RULE_NAME$40,
-	create: create$40,
+	name: RULE_NAME$42,
+	create: create$42,
 	defaultOptions: []
 });
-function create$40(context) {
+function create$42(context) {
 	if (!context.sourceCode.text.includes("componentWillUpdate")) return {};
 	const { ctx, visitor } = core.useComponentCollectorLegacy(context);
 	return defineRuleListener(visitor, { "Program:exit"(program) {
@@ -2289,7 +2369,7 @@ function create$40(context) {
 
 //#endregion
 //#region src/rules/no-context-provider/no-context-provider.ts
-const RULE_NAME$39 = "no-context-provider";
+const RULE_NAME$41 = "no-context-provider";
 var no_context_provider_default = createRule({
 	meta: {
 		type: "suggestion",
@@ -2302,11 +2382,11 @@ var no_context_provider_default = createRule({
 		},
 		schema: []
 	},
-	name: RULE_NAME$39,
-	create: create$39,
+	name: RULE_NAME$41,
+	create: create$41,
 	defaultOptions: []
 });
-function create$39(context) {
+function create$41(context) {
 	if (!context.sourceCode.text.includes("Provider")) return {};
 	const { version } = getSettingsFromContext(context);
 	if (compare(version, "19.0.0", "<")) return {};
@@ -2337,7 +2417,7 @@ function create$39(context) {
 
 //#endregion
 //#region src/rules/no-create-ref/no-create-ref.ts
-const RULE_NAME$38 = "no-create-ref";
+const RULE_NAME$40 = "no-create-ref";
 var no_create_ref_default = createRule({
 	meta: {
 		type: "suggestion",
@@ -2345,11 +2425,11 @@ var no_create_ref_default = createRule({
 		messages: { default: "[Deprecated] Use 'useRef' instead." },
 		schema: []
 	},
-	name: RULE_NAME$38,
-	create: create$38,
+	name: RULE_NAME$40,
+	create: create$40,
 	defaultOptions: []
 });
-function create$38(context) {
+function create$40(context) {
 	return defineRuleListener({ CallExpression(node) {
 		if (core.isCreateRefCall(context, node) && ast.findParentNode(node, core.isClassComponent) == null) context.report({
 			messageId: "default",
@@ -2360,7 +2440,7 @@ function create$38(context) {
 
 //#endregion
 //#region src/rules/no-direct-mutation-state/no-direct-mutation-state.ts
-const RULE_NAME$37 = "no-direct-mutation-state";
+const RULE_NAME$39 = "no-direct-mutation-state";
 function isConstructorFunction(node) {
 	return ast.isOneOf([AST_NODE_TYPES.FunctionDeclaration, AST_NODE_TYPES.FunctionExpression])(node) && ast.isMethodOrProperty(node.parent) && node.parent.key.type === AST_NODE_TYPES.Identifier && node.parent.key.name === "constructor";
 }
@@ -2371,11 +2451,11 @@ var no_direct_mutation_state_default = createRule({
 		messages: { default: "Do not mutate state directly. Use 'setState()' instead." },
 		schema: []
 	},
-	name: RULE_NAME$37,
-	create: create$37,
+	name: RULE_NAME$39,
+	create: create$39,
 	defaultOptions: []
 });
-function create$37(context) {
+function create$39(context) {
 	return defineRuleListener({ AssignmentExpression(node) {
 		if (!core.isAssignmentToThisState(node)) return;
 		const parentClass = ast.findParentNode(node, ast.isOneOf([AST_NODE_TYPES.ClassDeclaration, AST_NODE_TYPES.ClassExpression]));
@@ -2389,7 +2469,7 @@ function create$37(context) {
 
 //#endregion
 //#region src/rules/no-duplicate-key/no-duplicate-key.ts
-const RULE_NAME$36 = "no-duplicate-key";
+const RULE_NAME$38 = "no-duplicate-key";
 var no_duplicate_key_default = createRule({
 	meta: {
 		type: "problem",
@@ -2397,11 +2477,11 @@ var no_duplicate_key_default = createRule({
 		messages: { default: "The 'key' prop must be unique to its sibling elements." },
 		schema: []
 	},
-	name: RULE_NAME$36,
-	create: create$36,
+	name: RULE_NAME$38,
+	create: create$38,
 	defaultOptions: []
 });
-function create$36(context) {
+function create$38(context) {
 	if (!context.sourceCode.text.includes("key=")) return {};
 	const keyedEntries = /* @__PURE__ */ new Map();
 	function isKeyValueEqual(a, b) {
@@ -2456,7 +2536,7 @@ function create$36(context) {
 
 //#endregion
 //#region src/rules/no-forward-ref/no-forward-ref.ts
-const RULE_NAME$35 = "no-forward-ref";
+const RULE_NAME$37 = "no-forward-ref";
 var no_forward_ref_default = createRule({
 	meta: {
 		type: "suggestion",
@@ -2469,11 +2549,11 @@ var no_forward_ref_default = createRule({
 		},
 		schema: []
 	},
-	name: RULE_NAME$35,
-	create: create$35,
+	name: RULE_NAME$37,
+	create: create$37,
 	defaultOptions: []
 });
-function create$35(context) {
+function create$37(context) {
 	if (!context.sourceCode.text.includes("forwardRef")) return {};
 	const { version } = getSettingsFromContext(context);
 	if (compare(version, "19.0.0", "<")) return {};
@@ -2576,9 +2656,47 @@ function getComponentPropsFixes(context, fixer, node, typeArguments) {
 	].join(" ")), ...arg1 == null ? [] : [fixer.remove(arg1), fixer.removeRange([arg0.range[1], arg1.range[0]])]];
 }
 
+//#endregion
+//#region src/rules/no-implicit-children/no-implicit-children.ts
+const RULE_NAME$36 = "no-implicit-children";
+const RE_REACT_CHILDREN_TYPE = /react\.(reactnode|reactelement|reactportal)$/i;
+var no_implicit_children_default = createRule({
+	meta: {
+		type: "problem",
+		docs: { description: "Prevents implicitly passing the 'children' prop to components." },
+		messages: { default: "This spread attribute implicitly passes the 'children' prop to a component, this could lead to unexpected behavior. If you intend to pass the 'children' prop, use 'children={value}'." },
+		schema: []
+	},
+	name: RULE_NAME$36,
+	create: create$36,
+	defaultOptions: []
+});
+function create$36(context) {
+	const services = ESLintUtils.getParserServices(context, false);
+	const checker = services.program.getTypeChecker();
+	return defineRuleListener({ JSXSpreadAttribute(node) {
+		for (const type of unionConstituents(getConstrainedTypeAtLocation(services, node.argument))) {
+			const children = type.getProperty("children");
+			if (children == null) continue;
+			const fqn = getFullyQualifiedNameEx(checker, children).toLowerCase();
+			if (fqn.endsWith("attributes.children") || fqn.endsWith("propswithchildren.children")) continue;
+			const childrenType = checker.getTypeOfSymbol(children);
+			const typeSymbol = childrenType.aliasSymbol ?? childrenType.symbol;
+			if (typeSymbol != null) {
+				const typeFqn = checker.getFullyQualifiedName(typeSymbol);
+				if (RE_REACT_CHILDREN_TYPE.test(typeFqn) || /^JSX\.Element$/i.test(typeFqn)) continue;
+			}
+			context.report({
+				messageId: "default",
+				node
+			});
+		}
+	} });
+}
+
 //#endregion
 //#region src/rules/no-implicit-key/no-implicit-key.ts
-const RULE_NAME$34 = "no-implicit-key";
+const RULE_NAME$35 = "no-implicit-key";
 var no_implicit_key_default = createRule({
 	meta: {
 		type: "problem",
@@ -2586,11 +2704,11 @@ var no_implicit_key_default = createRule({
 		messages: { default: "This spread attribute implicitly passes the 'key' prop to a component, this could lead to unexpected behavior. If you intend to pass the 'key' prop, use 'key={value}'." },
 		schema: []
 	},
-	name: RULE_NAME$34,
-	create: create$34,
+	name: RULE_NAME$35,
+	create: create$35,
 	defaultOptions: []
 });
-function create$34(context) {
+function create$35(context) {
 	const services = ESLintUtils.getParserServices(context, false);
 	const checker = services.program.getTypeChecker();
 	return defineRuleListener({ JSXSpreadAttribute(node) {
@@ -2598,6 +2716,47 @@ function create$34(context) {
 			const key = type.getProperty("key");
 			if (key == null) continue;
 			if (getFullyQualifiedNameEx(checker, key).toLowerCase().endsWith("react.attributes.key")) continue;
+			const keyType = checker.getTypeOfSymbol(key);
+			if (keyType.aliasSymbol != null) {
+				if (checker.getFullyQualifiedName(keyType.aliasSymbol).toLowerCase().endsWith("react.key")) continue;
+			}
+			context.report({
+				messageId: "default",
+				node
+			});
+		}
+	} });
+}
+
+//#endregion
+//#region src/rules/no-implicit-ref/no-implicit-ref.ts
+const RULE_NAME$34 = "no-implicit-ref";
+const RE_REACT_REF_TYPE = /react\.(ref|legacyref|refcallback|refobject)$/i;
+var no_implicit_ref_default = createRule({
+	meta: {
+		type: "problem",
+		docs: { description: "Prevents implicitly passing the 'ref' prop to components." },
+		messages: { default: "This spread attribute implicitly passes the 'ref' prop to a component, this could lead to unexpected behavior. If you intend to pass the 'ref' prop, use 'ref={value}'." },
+		schema: []
+	},
+	name: RULE_NAME$34,
+	create: create$34,
+	defaultOptions: []
+});
+function create$34(context) {
+	const services = ESLintUtils.getParserServices(context, false);
+	const checker = services.program.getTypeChecker();
+	return defineRuleListener({ JSXSpreadAttribute(node) {
+		for (const type of unionConstituents(getConstrainedTypeAtLocation(services, node.argument))) {
+			const ref = type.getProperty("ref");
+			if (ref == null) continue;
+			if (getFullyQualifiedNameEx(checker, ref).toLowerCase().endsWith("attributes.ref")) continue;
+			const refType = checker.getTypeOfSymbol(ref);
+			const typeSymbol = refType.aliasSymbol ?? refType.symbol;
+			if (typeSymbol != null) {
+				const typeFqn = checker.getFullyQualifiedName(typeSymbol);
+				if (RE_REACT_REF_TYPE.test(typeFqn)) continue;
+			}
 			context.report({
 				messageId: "default",
 				node
@@ -4235,24 +4394,35 @@ var purity_default = createRule({
 function create$7(context) {
 	const hCollector = core.useHookCollector(context);
 	const cCollector = core.useComponentCollector(context);
-	const cExprs = [];
-	const nExprs = [];
+	const cEntries = [];
+	const nEntries = [];
 	return defineRuleListener(hCollector.visitor, cCollector.visitor, {
 		CallExpression(node) {
-			if (node.callee.type !== AST_NODE_TYPES.MemberExpression) return;
 			const expr = ast.getUnderlyingExpression(node.callee);
-			if (expr.type !== AST_NODE_TYPES.MemberExpression) return;
-			if (expr.object.type !== AST_NODE_TYPES.Identifier) return;
-			if (expr.property.type !== AST_NODE_TYPES.Identifier) return;
-			const objectName = expr.object.name;
-			const propertyName = expr.property.name;
-			if (!IMPURE_FUNCS.get(objectName)?.has(propertyName)) return;
-			const func = ast.findParentNode(node, ast.isFunction);
-			if (func == null) return;
-			cExprs.push({
-				func,
-				node
-			});
+			switch (true) {
+				case expr.type === AST_NODE_TYPES.Identifier: {
+					if (!IMPURE_FUNCS.get("globalThis")?.has(expr.name)) return;
+					const func = ast.findParentNode(node, ast.isFunction);
+					if (func == null) return;
+					cEntries.push({
+						func,
+						node
+					});
+					break;
+				}
+				case expr.type === AST_NODE_TYPES.MemberExpression && expr.object.type === AST_NODE_TYPES.Identifier && expr.property.type === AST_NODE_TYPES.Identifier: {
+					const objectName = expr.object.name;
+					const propertyName = expr.property.name;
+					if (!IMPURE_FUNCS.get(objectName)?.has(propertyName)) return;
+					const func = ast.findParentNode(node, ast.isFunction);
+					if (func == null) return;
+					cEntries.push({
+						func,
+						node
+					});
+					break;
+				}
+			}
 		},
 		NewExpression(node) {
 			const expr = ast.getUnderlyingExpression(node.callee);
@@ -4260,7 +4430,7 @@ function create$7(context) {
 			if (!IMPURE_CTORS.has(expr.name)) return;
 			const func = ast.findParentNode(node, ast.isFunction);
 			if (func == null) return;
-			nExprs.push({
+			nEntries.push({
 				func,
 				node
 			});
@@ -4269,7 +4439,7 @@ function create$7(context) {
 			const comps = cCollector.ctx.getAllComponents(node);
 			const hooks = hCollector.ctx.getAllHooks(node);
 			const funcs = [...comps, ...hooks];
-			for (const { func, node } of [...cExprs, ...nExprs]) {
+			for (const { func, node } of [...cEntries, ...nEntries]) {
 				if (!funcs.some((f) => f.node === func)) continue;
 				context.report({
 					data: { name: context.sourceCode.getText(node) },
@@ -7232,11 +7402,8 @@ const plugin = {
 		"jsx-dollar": jsx_dollar_default,
 		"jsx-key-before-spread": jsx_key_before_spread_default,
 		"jsx-no-comment-textnodes": jsx_no_comment_textnodes_default,
-		"jsx-no-duplicate-props": jsx_no_duplicate_props_default,
 		"jsx-shorthand-boolean": jsx_shorthand_boolean_default,
 		"jsx-shorthand-fragment": jsx_shorthand_fragment_default,
-		"jsx-uses-react": jsx_uses_react_default,
-		"jsx-uses-vars": jsx_uses_vars_default,
 		"no-access-state-in-setstate": no_access_state_in_setstate_default,
 		"no-array-index-key": no_array_index_key_default,
 		"no-children-count": no_children_count_default,
@@ -7255,7 +7422,9 @@ const plugin = {
 		"no-direct-mutation-state": no_direct_mutation_state_default,
 		"no-duplicate-key": no_duplicate_key_default,
 		"no-forward-ref": no_forward_ref_default,
+		"no-implicit-children": no_implicit_children_default,
 		"no-implicit-key": no_implicit_key_default,
+		"no-implicit-ref": no_implicit_ref_default,
 		"no-leaked-conditional-rendering": no_leaked_conditional_rendering_default,
 		"no-missing-component-display-name": no_missing_component_display_name_default,
 		"no-missing-context-display-name": no_missing_context_display_name_default,
@@ -7309,9 +7478,6 @@ const rules$6 = {
 	"react-x/exhaustive-deps": "warn",
 	"react-x/jsx-key-before-spread": "warn",
 	"react-x/jsx-no-comment-textnodes": "warn",
-	"react-x/jsx-no-duplicate-props": "warn",
-	"react-x/jsx-uses-react": "warn",
-	"react-x/jsx-uses-vars": "warn",
 	"react-x/no-access-state-in-setstate": "error",
 	"react-x/no-array-index-key": "warn",
 	"react-x/no-children-count": "warn",
@@ -7356,11 +7522,7 @@ const settings$5 = { "react-x": DEFAULT_ESLINT_REACT_SETTINGS };
 /**
 * Disables rules that are already handled by TypeScript
 */
-const rules$5 = {
-	"react-x/jsx-no-duplicate-props": "off",
-	"react-x/jsx-uses-react": "off",
-	"react-x/jsx-uses-vars": "off"
-};
+const rules$5 = {};
 
 //#endregion
 //#region src/configs/recommended-typescript.ts
@@ -7389,7 +7551,6 @@ var recommended_type_checked_exports = /* @__PURE__ */ __exportAll({
 const name$3 = "react-x/recommended-type-checked";
 const rules$3 = {
 	...rules$4,
-	"react-x/no-implicit-key": "error",
 	"react-x/no-leaked-conditional-rendering": "error"
 };
 const plugins$3 = { ...plugins$4 };
@@ -7447,7 +7608,6 @@ var strict_type_checked_exports = /* @__PURE__ */ __exportAll({
 const name = "react-x/strict-type-checked";
 const rules = {
 	...rules$1,
-	"react-x/no-implicit-key": "error",
 	"react-x/no-leaked-conditional-rendering": "error",
 	"react-x/no-unused-props": "warn"
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "3.0.0-next.71",
+  "version": "3.0.0-next.76",
   "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.71",
-    "@eslint-react/eff": "3.0.0-next.71",
-    "@eslint-react/core": "3.0.0-next.71",
-    "@eslint-react/shared": "3.0.0-next.71",
-    "@eslint-react/var": "3.0.0-next.71"
+    "@eslint-react/ast": "3.0.0-next.76",
+    "@eslint-react/core": "3.0.0-next.76",
+    "@eslint-react/shared": "3.0.0-next.76",
+    "@local/eff": "3.0.0-beta.72",
+    "@eslint-react/var": "3.0.0-next.76"
   },
   "devDependencies": {
     "@types/react": "^19.2.14",
@@ -60,8 +60,8 @@
     "@local/configs": "0.0.0"
   },
   "peerDependencies": {
-    "eslint": "^8.57.0 || ^9.0.0 || ^10.0.0",
-    "typescript": ">=4.8.4 <6.0.0"
+    "eslint": "^10.0.0",
+    "typescript": "*"
   },
   "engines": {
     "node": ">=22.0.0"