@zayne-labs/eslint-config 0.10.3 → 0.10.5

package/dist/{src-ZlDTU7nv.js → src-CqhpNkRz.js}

@@ -207,7 +207,7 @@ var require_lib = /* @__PURE__ */ __commonJS({ "../../node_modules/.pnpm/eslint-
 }) });
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-utils@3.0.0_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-utils/index.mjs
+//#region ../../node_modules/.pnpm/eslint-utils@3.0.0_eslint@9.37.0_jiti@2.6.1_/node_modules/eslint-utils/index.mjs
 var import_lib = /* @__PURE__ */ __toESM(require_lib(), 1);
 /**
 * Get the innermost scope which contains a given location.
@@ -1318,7 +1318,7 @@ function exceptDefault(name, index) {
 }
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.4_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/util/ast.js
+//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.6_eslint@9.37.0_jiti@2.6.1_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/util/ast.js
 const traverse = (context, node, visit, visited = /* @__PURE__ */ new Set()) => {
 	if (visited.has(node)) return;
 	visited.add(node);
@@ -1350,7 +1350,7 @@ const getCallExpr = (ref, current = ref.identifier.parent) => {
 };
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.4_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/util/react.js
+//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.6_eslint@9.37.0_jiti@2.6.1_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/util/react.js
 const isReactFunctionalComponent = (node) => (node.type === "FunctionDeclaration" || node.type === "VariableDeclarator" && (node.init.type === "ArrowFunctionExpression" || node.init.type === "CallExpression")) && node.id.type === "Identifier" && node.id.name[0].toUpperCase() === node.id.name[0];
 const isReactFunctionalHOC = (node) => node.type === "VariableDeclarator" && node.init && node.init.type === "CallExpression" && node.init.callee.type === "Identifier" && !["memo", "forwardRef"].includes(node.init.callee.name) && node.init.arguments.length > 0 && (node.init.arguments[0].type === "ArrowFunctionExpression" || node.init.arguments[0].type === "FunctionExpression") && node.id.type === "Identifier" && node.id.name[0].toUpperCase() === node.id.name[0];
 const isCustomHook = (node) => (node.type === "FunctionDeclaration" || node.type === "VariableDeclarator" && node.init && (node.init.type === "ArrowFunctionExpression" || node.init.type === "FunctionExpression")) && node.id.type === "Identifier" && node.id.name.startsWith("use") && node.id.name[3] === node.id.name[3].toUpperCase();
@@ -1368,15 +1368,13 @@ const isPropDef = (def) => {
 const isUseRef = (node) => node.type === "VariableDeclarator" && node.init && node.init.type === "CallExpression" && node.init.callee.name === "useRef" && node.id.type === "Identifier";
 const isUseEffect = (node) => node.type === "CallExpression" && (node.callee.type === "Identifier" && node.callee.name === "useEffect" || node.callee.type === "MemberExpression" && node.callee.object.name === "React" && node.callee.property.name === "useEffect");
 const getEffectFnRefs = (context, node) => {
-	if (!isUseEffect(node) || node.arguments.length < 1) return;
 	const effectFn = node.arguments[0];
-	if (effectFn.type !== "ArrowFunctionExpression" && effectFn.type !== "FunctionExpression") return;
+	if (effectFn?.type !== "ArrowFunctionExpression" && effectFn?.type !== "FunctionExpression") return;
 	return getDownstreamRefs(context, effectFn);
 };
 function getEffectDepsRefs(context, node) {
-	if (!isUseEffect(node) || node.arguments.length < 2) return;
 	const depsArr = node.arguments[1];
-	if (depsArr.type !== "ArrayExpression") return;
+	if (depsArr?.type !== "ArrayExpression") return;
 	return getDownstreamRefs(context, depsArr);
 }
 const isStateSetter = (context, ref) => getCallExpr(ref) !== void 0 && isState(context, ref);
@@ -1407,7 +1405,7 @@ const isArgsAllLiterals = (context, callExpr) => callExpr.arguments.flatMap((arg
 const getUpstreamReactVariables = (context, variable) => getUpstreamVariables(context, variable, (node) => !isUseState(node)).filter((variable$1) => variable$1.defs.every((def) => isPropDef(def) || def.type !== "Parameter"));
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.4_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-empty-effect.js
+//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.6_eslint@9.37.0_jiti@2.6.1_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-empty-effect.js
 /**
 * @type {import("eslint").Rule.RuleModule}
 */
@@ -1428,7 +1426,7 @@ var no_empty_effect_default = {
 };
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.4_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-adjust-state-on-prop-change.js
+//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.6_eslint@9.37.0_jiti@2.6.1_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-adjust-state-on-prop-change.js
 /**
 * @type {import("eslint").Rule.RuleModule}
 */
@@ -1443,6 +1441,7 @@ var no_adjust_state_on_prop_change_default = {
 		messages: { avoidAdjustingStateWhenAPropChanges: "Avoid adjusting state when a prop changes. Instead, adjust the state directly during render, or refactor your state to avoid this need entirely." }
 	},
 	create: (context) => ({ CallExpression: (node) => {
+		if (!isUseEffect(node)) return;
 		const effectFnRefs = getEffectFnRefs(context, node);
 		const depsRefs = getEffectDepsRefs(context, node);
 		if (!effectFnRefs || !depsRefs) return;
@@ -1458,7 +1457,7 @@ var no_adjust_state_on_prop_change_default = {
 };
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.4_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-reset-all-state-on-prop-change.js
+//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.6_eslint@9.37.0_jiti@2.6.1_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-reset-all-state-on-prop-change.js
 /**
 * @type {import("eslint").Rule.RuleModule}
 */
@@ -1473,9 +1472,12 @@ var no_reset_all_state_on_prop_change_default = {
 		messages: { avoidResettingAllStateWhenAPropChanges: "Avoid resetting all state when a prop changes. If \"{{prop}}\" is a key, pass it as `key` instead so React will reset the component." }
 	},
 	create: (context) => ({ CallExpression: (node) => {
+		if (!isUseEffect(node)) return;
 		const effectFnRefs = getEffectFnRefs(context, node);
 		const depsRefs = getEffectDepsRefs(context, node);
 		if (!effectFnRefs || !depsRefs) return;
+		const containingNode = findContainingNode(node);
+		if (containingNode && isCustomHook(containingNode)) return;
 		const propUsedToResetAllState = findPropUsedToResetAllState(context, effectFnRefs, depsRefs, node);
 		if (propUsedToResetAllState) context.report({
 			node,
@@ -1512,7 +1514,7 @@ const findContainingNode = (node) => {
 };
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.4_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-event-handler.js
+//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.6_eslint@9.37.0_jiti@2.6.1_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-event-handler.js
 /**
 * @type {import("eslint").Rule.RuleModule}
 */
@@ -1527,10 +1529,10 @@ var no_event_handler_default = {
 		messages: { avoidEventHandler: "Avoid using state and effects as an event handler. Instead, call the event handling code directly when the event occurs." }
 	},
 	create: (context) => ({ CallExpression: (node) => {
+		if (!isUseEffect(node) || hasCleanup(node)) return;
 		const effectFnRefs = getEffectFnRefs(context, node);
 		const depsRefs = getEffectDepsRefs(context, node);
 		if (!effectFnRefs || !depsRefs) return;
-		if (hasCleanup(node)) return;
 		findDownstreamNodes(context, node, "IfStatement").filter((ifNode) => !ifNode.alternate).filter((ifNode) => getDownstreamRefs(context, ifNode.test).notEmptyEvery((ref) => isState(context, ref))).forEach((ifNode) => {
 			context.report({
 				node: ifNode.test,
@@ -1541,7 +1543,7 @@ var no_event_handler_default = {
 };
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.4_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-pass-live-state-to-parent.js
+//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.6_eslint@9.37.0_jiti@2.6.1_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-pass-live-state-to-parent.js
 /**
 * @type {import("eslint").Rule.RuleModule}
 */
@@ -1556,6 +1558,7 @@ var no_pass_live_state_to_parent_default = {
 		messages: { avoidPassingLiveStateToParent: "Avoid passing live state to parents in an effect. Instead, lift the state to the parent and pass it down to the child as a prop." }
 	},
 	create: (context) => ({ CallExpression: (node) => {
+		if (!isUseEffect(node)) return;
 		const effectFnRefs = getEffectFnRefs(context, node);
 		const depsRefs = getEffectDepsRefs(context, node);
 		if (!effectFnRefs || !depsRefs) return;
@@ -1570,7 +1573,7 @@ var no_pass_live_state_to_parent_default = {
 };
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.4_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-initialize-state.js
+//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.6_eslint@9.37.0_jiti@2.6.1_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-initialize-state.js
 /**
 * @type {import("eslint").Rule.RuleModule}
 */
@@ -1582,6 +1585,7 @@ var no_initialize_state_default = {
 		messages: { avoidInitializingState: "Avoid initializing state in an effect. Instead, pass \"{{state}}\"'s initial value to its `useState`." }
 	},
 	create: (context) => ({ CallExpression: (node) => {
+		if (!isUseEffect(node)) return;
 		const effectFnRefs = getEffectFnRefs(context, node);
 		const depsRefs = getEffectDepsRefs(context, node);
 		if (!effectFnRefs || !depsRefs) return;
@@ -1599,7 +1603,7 @@ var no_initialize_state_default = {
 };
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.4_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-chain-state-updates.js
+//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.6_eslint@9.37.0_jiti@2.6.1_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-chain-state-updates.js
 /**
 * @type {import("eslint").Rule.RuleModule}
 */
@@ -1614,10 +1618,10 @@ var no_chain_state_updates_default = {
 		messages: { avoidChainingStateUpdates: "Avoid chaining state changes. When possible, update all relevant state simultaneously." }
 	},
 	create: (context) => ({ CallExpression: (node) => {
+		if (!isUseEffect(node) || hasCleanup(node)) return;
 		const effectFnRefs = getEffectFnRefs(context, node);
 		const depsRefs = getEffectDepsRefs(context, node);
 		if (!effectFnRefs || !depsRefs) return;
-		if (hasCleanup(node)) return;
 		const isAllDepsState = depsRefs.notEmptyEvery((ref) => isState(context, ref));
 		effectFnRefs.filter((ref) => isStateSetter(context, ref)).filter((ref) => isImmediateCall(ref.identifier)).forEach((ref) => {
 			const callExpr = getCallExpr(ref);
@@ -1630,7 +1634,7 @@ var no_chain_state_updates_default = {
 };
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.4_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-derived-state.js
+//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.6_eslint@9.37.0_jiti@2.6.1_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-derived-state.js
 /**
 * @type {import('eslint').Rule.RuleModule}
 */
@@ -1645,10 +1649,10 @@ var no_derived_state_default = {
 		messages: { avoidDerivedState: "Avoid storing derived state. Compute \"{{state}}\" directly during render, optionally with `useMemo` if it's expensive." }
 	},
 	create: (context) => ({ CallExpression: (node) => {
+		if (!isUseEffect(node) || hasCleanup(node)) return;
 		const effectFnRefs = getEffectFnRefs(context, node);
 		const depsRefs = getEffectDepsRefs(context, node);
 		if (!effectFnRefs || !depsRefs) return;
-		if (hasCleanup(node)) return;
 		effectFnRefs.filter((ref) => isStateSetter(context, ref)).filter((ref) => isImmediateCall(ref.identifier)).forEach((ref) => {
 			const callExpr = getCallExpr(ref);
 			const useStateNode = getUseStateNode(context, ref);
@@ -1669,7 +1673,7 @@ var no_derived_state_default = {
 const countCalls = (ref) => ref.resolved.references.filter((ref$1) => ref$1.identifier.parent.type === "CallExpression").length;
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.4_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-pass-data-to-parent.js
+//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.6_eslint@9.37.0_jiti@2.6.1_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-pass-data-to-parent.js
 /**
 * @type {import("eslint").Rule.RuleModule}
 */
@@ -1684,10 +1688,10 @@ var no_pass_data_to_parent_default = {
 		messages: { avoidPassingDataToParent: "Avoid passing data to parents in an effect. Instead, let the parent fetch the data itself and pass it down to the child as a prop." }
 	},
 	create: (context) => ({ CallExpression: (node) => {
+		if (!isUseEffect(node) || hasCleanup(node)) return;
 		const effectFnRefs = getEffectFnRefs(context, node);
 		const depsRefs = getEffectDepsRefs(context, node);
 		if (!effectFnRefs || !depsRefs) return;
-		if (hasCleanup(node)) return;
 		effectFnRefs.filter((ref) => isPropCallback(context, ref)).forEach((ref) => {
 			const callExpr = getCallExpr(ref);
 			if (callExpr.arguments.flatMap((arg) => getDownstreamRefs(context, arg)).notEmptyEvery((ref$1) => !isState(context, ref$1) && !isProp(context, ref$1) && !isRef(context, ref$1))) context.report({
@@ -1699,7 +1703,7 @@ var no_pass_data_to_parent_default = {
 };
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.4_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-manage-parent.js
+//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.6_eslint@9.37.0_jiti@2.6.1_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-manage-parent.js
 /**
 * @type {import("eslint").Rule.RuleModule}
 */
@@ -1711,6 +1715,7 @@ var no_manage_parent_default = {
 		messages: { avoidManagingParent: "This effect only uses props. Consider lifting the logic up to the parent." }
 	},
 	create: (context) => ({ CallExpression: (node) => {
+		if (!isUseEffect(node)) return;
 		const effectFnRefs = getEffectFnRefs(context, node);
 		const depsRefs = getEffectDepsRefs(context, node);
 		if (!effectFnRefs || !depsRefs) return;
@@ -1723,13 +1728,13 @@ var no_manage_parent_default = {
 };
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.4_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/util/javascript.js
+//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.6_eslint@9.37.0_jiti@2.6.1_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/util/javascript.js
 Array.prototype.notEmptyEvery = function(predicate) {
 	return this.length > 0 && this.every(predicate);
 };
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.4_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/index.js
+//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.6_eslint@9.37.0_jiti@2.6.1_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/index.js
 /**
 * @type {import("eslint").ESLint.Plugin}
 */
@@ -1774,4 +1779,4 @@ var src_default = plugin;
 
 //#endregion
 export { src_default as default };
-//# sourceMappingURL=src-ZlDTU7nv.js.map
+//# sourceMappingURL=src-CqhpNkRz.js.map