@zayne-labs/eslint-config 0.10.2 → 0.10.4

package/dist/{src-DwSufEpw.js → src-tkLps0qA.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.5.1_/node_modules/eslint-utils/index.mjs
+//#region ../../node_modules/.pnpm/eslint-utils@3.0.0_eslint@9.36.0_jiti@2.6.0_/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.3_eslint@9.36.0_jiti@2.5.1_/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.5_eslint@9.36.0_jiti@2.6.0_/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,13 +1350,17 @@ const getCallExpr = (ref, current = ref.identifier.parent) => {
 };
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.3_eslint@9.36.0_jiti@2.5.1_/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.5_eslint@9.36.0_jiti@2.6.0_/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();
 const isUseState = (node) => node.type === "VariableDeclarator" && node.init && node.init.type === "CallExpression" && node.init.callee.name === "useState" && node.id.type === "ArrayPattern" && (node.id.elements.length === 1 || node.id.elements.length === 2) && node.id.elements.every((el) => {
 	return !el || el.type === "Identifier";
 });
+const hasCleanup = (node) => {
+	const effectFn = node.arguments[0];
+	return (effectFn.type === "ArrowFunctionExpression" || effectFn.type === "FunctionExpression") && effectFn.body.type === "BlockStatement" && effectFn.body.body.some((stmt) => stmt.type === "ReturnStatement" && stmt.argument);
+};
 const isPropDef = (def) => {
 	const declaringNode = def.node.type === "ArrowFunctionExpression" ? def.node.parent.type === "CallExpression" ? def.node.parent.parent : def.node.parent : def.node;
 	return def.type === "Parameter" && (isReactFunctionalComponent(declaringNode) && !isReactFunctionalHOC(declaringNode) || isCustomHook(declaringNode));
@@ -1364,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);
@@ -1403,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.3_eslint@9.36.0_jiti@2.5.1_/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.5_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-empty-effect.js
 /**
 * @type {import("eslint").Rule.RuleModule}
 */
@@ -1424,7 +1426,7 @@ var no_empty_effect_default = {
 };
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.3_eslint@9.36.0_jiti@2.5.1_/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.5_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
 /**
 * @type {import("eslint").Rule.RuleModule}
 */
@@ -1439,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;
@@ -1454,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.3_eslint@9.36.0_jiti@2.5.1_/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.5_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
 /**
 * @type {import("eslint").Rule.RuleModule}
 */
@@ -1469,6 +1472,7 @@ 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;
@@ -1508,7 +1512,7 @@ const findContainingNode = (node) => {
 };
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.3_eslint@9.36.0_jiti@2.5.1_/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.5_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-event-handler.js
 /**
 * @type {import("eslint").Rule.RuleModule}
 */
@@ -1523,6 +1527,7 @@ 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;
@@ -1536,7 +1541,7 @@ var no_event_handler_default = {
 };
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.3_eslint@9.36.0_jiti@2.5.1_/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.5_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
 /**
 * @type {import("eslint").Rule.RuleModule}
 */
@@ -1551,6 +1556,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;
@@ -1565,7 +1571,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.3_eslint@9.36.0_jiti@2.5.1_/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.5_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-initialize-state.js
 /**
 * @type {import("eslint").Rule.RuleModule}
 */
@@ -1577,6 +1583,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;
@@ -1594,7 +1601,7 @@ var no_initialize_state_default = {
 };
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.3_eslint@9.36.0_jiti@2.5.1_/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.5_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
 /**
 * @type {import("eslint").Rule.RuleModule}
 */
@@ -1609,6 +1616,7 @@ 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;
@@ -1624,7 +1632,7 @@ var no_chain_state_updates_default = {
 };
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.3_eslint@9.36.0_jiti@2.5.1_/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.5_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-derived-state.js
 /**
 * @type {import('eslint').Rule.RuleModule}
 */
@@ -1639,6 +1647,7 @@ 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;
@@ -1662,7 +1671,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.3_eslint@9.36.0_jiti@2.5.1_/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.5_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
 /**
 * @type {import("eslint").Rule.RuleModule}
 */
@@ -1677,6 +1686,7 @@ 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;
@@ -1691,7 +1701,7 @@ var no_pass_data_to_parent_default = {
 };
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.3_eslint@9.36.0_jiti@2.5.1_/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.5_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/no-manage-parent.js
 /**
 * @type {import("eslint").Rule.RuleModule}
 */
@@ -1703,6 +1713,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;
@@ -1715,13 +1726,13 @@ var no_manage_parent_default = {
 };
 
 //#endregion
-//#region ../../node_modules/.pnpm/eslint-plugin-react-you-might-not-need-an-effect@0.5.3_eslint@9.36.0_jiti@2.5.1_/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.5_eslint@9.36.0_jiti@2.6.0_/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.3_eslint@9.36.0_jiti@2.5.1_/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.5_eslint@9.36.0_jiti@2.6.0_/node_modules/eslint-plugin-react-you-might-not-need-an-effect/src/index.js
 /**
 * @type {import("eslint").ESLint.Plugin}
 */
@@ -1766,4 +1777,4 @@ var src_default = plugin;
 
 //#endregion
 export { src_default as default };
-//# sourceMappingURL=src-DwSufEpw.js.map
+//# sourceMappingURL=src-tkLps0qA.js.map