eslint-plugin-react-web-api 5.24.0-beta.1 → 5.24.0

package/dist/index.js

@@ -27,7 +27,7 @@ var __exportAll = (all, no_symbols) => {
 //#endregion
 //#region package.json
 var name$1 = "eslint-plugin-react-web-api";
-var version = "5.24.0-beta.1";
+var version = "5.24.0";
 
 //#endregion
 //#region src/utils/create-rule.ts
@@ -38,10 +38,23 @@ const createRule = ESLintUtils.RuleCreator(getDocsUrl);
 
 //#endregion
 //#region ../../.pkgs/eff/dist/index.js
+function and(a, b) {
+	return (data) => a(data) && b(data);
+}
 function or(a, b) {
 	return (data) => a(data) || b(data);
 }
 /**
+* A function that checks if the passed parameter is a string and narrows its type accordingly.
+*
+* @param data - The variable to check.
+* @returns True if the passed input is a string, false otherwise.
+* @category guards
+*/
+function isString(data) {
+	return typeof data === "string";
+}
+/**
 * Applies a `pipe` method's variadic arguments to an initial value from left
 * to right.
 *
@@ -574,6 +587,7 @@ function getOptions(context, node) {
 //#endregion
 //#region src/rules/no-leaked-event-listener/no-leaked-event-listener.ts
 const RULE_NAME$5 = "no-leaked-event-listener";
+const isUseEffectCallback$5 = or(isUseEffectSetupCallback, isUseEffectCleanupCallback);
 function getCallKind$5(node) {
 	const name = Extract.getCalleeName(node);
 	if (name != null && isMatching(P.union("addEventListener", "removeEventListener"))(name)) return name;
@@ -616,7 +630,7 @@ function resolveCleanupCallee(context, node) {
 			for (const prop of init.properties) {
 				if (prop.type !== AST_NODE_TYPES.Property || prop.computed) continue;
 				const key = Extract.unwrap(prop.key);
-				if ((key.type === AST_NODE_TYPES.Identifier ? key.name : key.type === AST_NODE_TYPES.Literal && typeof key.value === "string" ? key.value : null) !== property.name) continue;
+				if ((key.type === AST_NODE_TYPES.Identifier ? key.name : key.type === AST_NODE_TYPES.Literal && isString(key.value) ? key.value : null) !== property.name) continue;
 				const value = Extract.unwrap(prop.value);
 				if (Check.isFunction(value)) return {
 					kind: "function",
@@ -681,7 +695,7 @@ function create$5(context) {
 	}
 	return {
 		["CallExpression"](node) {
-			const fn = Traverse.findParent(node, (n) => isUseEffectSetupCallback(n) || isUseEffectCleanupCallback(n));
+			const fn = Traverse.findParent(node, isUseEffectCallback$5);
 			if (fn == null) return;
 			const fKind = isUseEffectSetupCallback(fn) ? "setup" : "cleanup";
 			const setupFn = fKind === "setup" ? fn : Traverse.findParent(fn, Check.isFunction);
@@ -763,6 +777,7 @@ function resolveToObjectExpression(context, node) {
 //#endregion
 //#region src/rules/no-leaked-fetch/no-leaked-fetch.ts
 const RULE_NAME$4 = "no-leaked-fetch";
+const isUseEffectCallback$4 = or(isUseEffectSetupCallback, isUseEffectCleanupCallback);
 function getCallKind$4(node) {
 	const name = Extract.getCalleeName(node);
 	if (name != null && isMatching(P.union("fetch", "abort"))(name)) return name;
@@ -851,7 +866,7 @@ function create$4(context) {
 					node
 				});
 			}).with("abort", () => {
-				const fn = Traverse.findParent(node, (n) => isUseEffectSetupCallback(n) || isUseEffectCleanupCallback(n));
+				const fn = Traverse.findParent(node, isUseEffectCallback$4);
 				if (fn == null || !isUseEffectCleanupCallback(fn)) return;
 				const controller = getAbortController(node);
 				if (controller == null) return;
@@ -902,6 +917,7 @@ function isFromObserver$1(context, node, name) {
 //#endregion
 //#region src/rules/no-leaked-intersection-observer/no-leaked-intersection-observer.ts
 const RULE_NAME$3 = "no-leaked-intersection-observer";
+const isUseEffectCallback$3 = or(isUseEffectSetupCallback, isUseEffectCleanupCallback);
 function getCallKind$3(context, node) {
 	const callee = Extract.unwrap(node.callee);
 	if (callee.type !== AST_NODE_TYPES.MemberExpression) return "other";
@@ -932,7 +948,7 @@ function create$3(context) {
 	const dEntries = [];
 	return {
 		["CallExpression"](node) {
-			if (Traverse.findParent(node, (n) => isUseEffectSetupCallback(n) || isUseEffectCleanupCallback(n)) == null) return;
+			if (Traverse.findParent(node, isUseEffectCallback$3) == null) return;
 			const callee = Extract.unwrap(node.callee);
 			if (callee.type !== AST_NODE_TYPES.MemberExpression) return;
 			const { object } = callee;
@@ -963,7 +979,7 @@ function create$3(context) {
 			}).otherwise(() => null);
 		},
 		["NewExpression"](node) {
-			const fn = Traverse.findParent(node, (n) => Check.isFunction(n) && (isUseEffectSetupCallback(n) || isUseEffectCleanupCallback(n)));
+			const fn = Traverse.findParent(node, and(Check.isFunction, isUseEffectCallback$3));
 			if (fn == null) return;
 			if (!isNewObserver$1(node, "IntersectionObserver")) return;
 			const id = resolveEnclosingAssignmentTarget(node);
@@ -1010,6 +1026,7 @@ function create$3(context) {
 //#endregion
 //#region src/rules/no-leaked-interval/no-leaked-interval.ts
 const RULE_NAME$2 = "no-leaked-interval";
+const isUseEffectCallback$2 = or(isUseEffectSetupCallback, isUseEffectCleanupCallback);
 function getCallKind$2(node) {
 	const name = Extract.getCalleeName(node);
 	if (name != null && isMatching(P.union("setInterval", "clearInterval"))(name)) return name;
@@ -1038,7 +1055,7 @@ function create$2(context) {
 	}
 	return {
 		["CallExpression"](node) {
-			if (Traverse.findParent(node, (n) => isUseEffectSetupCallback(n) || isUseEffectCleanupCallback(n)) == null) return;
+			if (Traverse.findParent(node, isUseEffectCallback$2) == null) return;
 			match(getCallKind$2(node)).with("setInterval", () => {
 				const intervalIdNode = resolveEnclosingAssignmentTarget(node);
 				if (intervalIdNode == null) {
@@ -1095,6 +1112,7 @@ function isFromObserver(context, node, name) {
 //#endregion
 //#region src/rules/no-leaked-resize-observer/no-leaked-resize-observer.ts
 const RULE_NAME$1 = "no-leaked-resize-observer";
+const isUseEffectCallback$1 = or(isUseEffectSetupCallback, isUseEffectCleanupCallback);
 function getCallKind$1(context, node) {
 	const callee = Extract.unwrap(node.callee);
 	if (callee.type !== AST_NODE_TYPES.MemberExpression) return "other";
@@ -1125,7 +1143,7 @@ function create$1(context) {
 	const dEntries = [];
 	return {
 		["CallExpression"](node) {
-			if (Traverse.findParent(node, (n) => isUseEffectSetupCallback(n) || isUseEffectCleanupCallback(n)) == null) return;
+			if (Traverse.findParent(node, isUseEffectCallback$1) == null) return;
 			const callee = Extract.unwrap(node.callee);
 			if (callee.type !== AST_NODE_TYPES.MemberExpression) return;
 			const { object } = callee;
@@ -1156,7 +1174,7 @@ function create$1(context) {
 			}).otherwise(() => null);
 		},
 		["NewExpression"](node) {
-			const fn = Traverse.findParent(node, (n) => Check.isFunction(n) && (isUseEffectSetupCallback(n) || isUseEffectCleanupCallback(n)));
+			const fn = Traverse.findParent(node, and(Check.isFunction, isUseEffectCallback$1));
 			if (fn == null) return;
 			if (!isNewObserver(node, "ResizeObserver")) return;
 			const id = resolveEnclosingAssignmentTarget(node);
@@ -1203,6 +1221,7 @@ function create$1(context) {
 //#endregion
 //#region src/rules/no-leaked-timeout/no-leaked-timeout.ts
 const RULE_NAME = "no-leaked-timeout";
+const isUseEffectCallback = or(isUseEffectSetupCallback, isUseEffectCleanupCallback);
 function getCallKind(node) {
 	const name = Extract.getCalleeName(node);
 	if (name != null && isMatching(P.union("setTimeout", "clearTimeout"))(name)) return name;
@@ -1231,7 +1250,7 @@ function create(context) {
 	}
 	return {
 		["CallExpression"](node) {
-			if (Traverse.findParent(node, (n) => isUseEffectSetupCallback(n) || isUseEffectCleanupCallback(n)) == null) return;
+			if (Traverse.findParent(node, isUseEffectCallback) == null) return;
 			match(getCallKind(node)).with("setTimeout", () => {
 				const timeoutIdNode = resolveEnclosingAssignmentTarget(node);
 				if (timeoutIdNode == null) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-web-api",
-  "version": "5.24.0-beta.1",
+  "version": "5.24.0",
   "description": "ESLint React's ESLint plugin for interacting with Web APIs",
   "keywords": [
     "react",
@@ -37,11 +37,11 @@
     "dist"
   ],
   "dependencies": {
-    "@eslint-react/ast": "5.24.0-beta.1",
-    "@eslint-react/core": "5.24.0-beta.1",
-    "@eslint-react/eslint": "5.24.0-beta.1",
-    "@eslint-react/shared": "5.24.0-beta.1",
-    "@eslint-react/var": "5.24.0-beta.1",
+    "@eslint-react/ast": "5.24.0",
+    "@eslint-react/core": "5.24.0",
+    "@eslint-react/eslint": "5.24.0",
+    "@eslint-react/shared": "5.24.0",
+    "@eslint-react/var": "5.24.0",
     "@typescript-eslint/types": "^8.71.0",
     "@typescript-eslint/utils": "^8.71.0",
     "ts-pattern": "^5.9.0"