eslint-plugin-react-web-api 2.7.4-beta.0 → 2.7.4-beta.10

package/README.md

@@ -36,6 +36,18 @@ export default defineConfig(
 );
 ```
 
+## Web API Rules
+
+> [!NOTE]
+> Web API rules prevent resource leaks by detecting uncleaned Web API subscriptions in components and hooks.
+
+**Implemented Rules:**
+
+- [`no-leaked-event-listener`](./web-api-no-leaked-event-listener) - Prevents leaked `addEventListener` calls
+- [`no-leaked-interval`](./web-api-no-leaked-interval) - Prevents leaked `setInterval` calls
+- [`no-leaked-resize-observer`](./web-api-no-leaked-resize-observer) - Prevents leaked `ResizeObserver` instances
+- [`no-leaked-timeout`](./web-api-no-leaked-timeout) - Prevents leaked `setTimeout` calls
+
 ## Rules
 
 <https://eslint-react.xyz/docs/rules/overview#web-api-rules>

package/dist/index.js

@@ -1,8 +1,8 @@
 import { DEFAULT_ESLINT_REACT_SETTINGS, WEBSITE_URL, getConfigAdapters } from "@eslint-react/shared";
 import * as AST from "@eslint-react/ast";
-import { ComponentPhaseRelevance, findEnclosingAssignmentTarget, getPhaseKindOfFunction, isInitializedFromReactNative, isInstanceIdEqual, isInversePhase } from "@eslint-react/core";
+import { ComponentPhaseRelevance, getPhaseKindOfFunction, isInitializedFromReactNative, isInversePhase } from "@eslint-react/core";
 import { or, unit } from "@eslint-react/eff";
-import { findAssignmentTarget, findProperty, findVariable, getVariableDefinitionNode, isNodeValueEqual } from "@eslint-react/var";
+import { findEnclosingAssignmentTarget, findProperty, findVariable, getVariableDefinitionNode, isAssignmentTargetEqual, isNodeValueEqual } from "@eslint-react/var";
 import { AST_NODE_TYPES, ESLintUtils } from "@typescript-eslint/utils";
 import { getStaticValue } from "@typescript-eslint/utils/ast-utils";
 import { P, isMatching, match } from "ts-pattern";
@@ -11,9 +11,9 @@ import { P, isMatching, match } from "ts-pattern";
 var __defProp = Object.defineProperty;
 var __exportAll = (all, symbols) => {
 	let target = {};
-	for (var name$2 in all) {
-		__defProp(target, name$2, {
-			get: all[name$2],
+	for (var name in all) {
+		__defProp(target, name, {
+			get: all[name],
 			enumerable: true
 		});
 	}
@@ -42,7 +42,7 @@ const settings = { "react-x": DEFAULT_ESLINT_REACT_SETTINGS };
 //#endregion
 //#region package.json
 var name = "eslint-plugin-react-web-api";
-var version = "2.7.4-beta.0";
+var version = "2.7.4-beta.10";
 
 //#endregion
 //#region src/utils/create-rule.ts
@@ -94,20 +94,20 @@ function getOptions(node, initialScope) {
 		}
 		return filter(v) ? v : unit;
 	}
-	function getOpts(node$1) {
-		switch (node$1.type) {
+	function getOpts(node) {
+		switch (node.type) {
 			case AST_NODE_TYPES.Identifier: {
-				const variableNode = getVariableDefinitionNode(findVariable(node$1, initialScope), 0);
+				const variableNode = getVariableDefinitionNode(findVariable(node, initialScope), 0);
 				if (variableNode?.type === AST_NODE_TYPES.ObjectExpression) return getOpts(variableNode);
 				return defaultOptions;
 			}
 			case AST_NODE_TYPES.Literal: return {
 				...defaultOptions,
-				capture: Boolean(node$1.value)
+				capture: Boolean(node.value)
 			};
 			case AST_NODE_TYPES.ObjectExpression: {
-				const vCapture = !!getPropValue(findProp(node$1.properties, "capture"));
-				const pSignal = findProp(node$1.properties, "signal");
+				const vCapture = !!getPropValue(findProp(node.properties, "capture"));
+				const pSignal = findProp(node.properties, "signal");
 				return {
 					capture: vCapture,
 					signal: pSignal?.type === AST_NODE_TYPES.Property ? getSignalValueExpression(pSignal.value, initialScope) : unit
@@ -269,7 +269,7 @@ function create$2(context) {
 	const sEntries = [];
 	const cEntries = [];
 	function isInverseEntry(a, b) {
-		return isInstanceIdEqual(context, a.timerId, b.timerId);
+		return isAssignmentTargetEqual(context, a.timerId, b.timerId);
 	}
 	return {
 		[":function"](node) {
@@ -288,7 +288,7 @@ function create$2(context) {
 					const fEntry = fEntries.findLast((x) => x.kind !== "other");
 					if (fEntry == null) break;
 					if (!ComponentPhaseRelevance.has(fEntry.kind)) break;
-					const intervalIdNode = findAssignmentTarget(node);
+					const intervalIdNode = findEnclosingAssignmentTarget(node);
 					if (intervalIdNode == null) {
 						context.report({
 							messageId: "expectedIntervalId",
@@ -466,11 +466,11 @@ function create$1(context) {
 		},
 		["Program:exit"]() {
 			for (const { id, node, phaseNode } of observers) {
-				if (dEntries.some((e) => isInstanceIdEqual(context, e.observer, id))) continue;
-				const oentries = oEntries.filter((e) => isInstanceIdEqual(context, e.observer, id));
-				const uentries = uEntries.filter((e) => isInstanceIdEqual(context, e.observer, id));
-				const isDynamic = (node$1) => node$1?.type === AST_NODE_TYPES.CallExpression || AST.isConditional(node$1);
-				const isPhaseNode = (node$1) => node$1 === phaseNode;
+				if (dEntries.some((e) => isAssignmentTargetEqual(context, e.observer, id))) continue;
+				const oentries = oEntries.filter((e) => isAssignmentTargetEqual(context, e.observer, id));
+				const uentries = uEntries.filter((e) => isAssignmentTargetEqual(context, e.observer, id));
+				const isDynamic = (node) => node?.type === AST_NODE_TYPES.CallExpression || AST.isConditional(node);
+				const isPhaseNode = (node) => node === phaseNode;
 				if (oentries.some((e) => !isPhaseNode(AST.findParentNode(e.node, or(isDynamic, isPhaseNode))))) {
 					context.report({
 						messageId: "expectedDisconnectInControlFlow",
@@ -479,7 +479,7 @@ function create$1(context) {
 					continue;
 				}
 				for (const oEntry of oentries) {
-					if (uentries.some((uEntry) => isInstanceIdEqual(context, uEntry.element, oEntry.element))) continue;
+					if (uentries.some((uEntry) => isAssignmentTargetEqual(context, uEntry.element, oEntry.element))) continue;
 					context.report({
 						messageId: "expectedDisconnectOrUnobserveInCleanup",
 						node: oEntry.node
@@ -521,7 +521,7 @@ function create(context) {
 	const sEntries = [];
 	const rEntries = [];
 	function isInverseEntry(a, b) {
-		return isInstanceIdEqual(context, a.timerId, b.timerId);
+		return isAssignmentTargetEqual(context, a.timerId, b.timerId);
 	}
 	return {
 		[":function"](node) {
@@ -539,7 +539,7 @@ function create(context) {
 			if (!ComponentPhaseRelevance.has(fEntry?.kind)) return;
 			switch (getCallKind(node)) {
 				case "setTimeout": {
-					const timeoutIdNode = findAssignmentTarget(node);
+					const timeoutIdNode = findEnclosingAssignmentTarget(node);
 					if (timeoutIdNode == null) {
 						context.report({
 							messageId: "expectedTimeoutId",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-web-api",
-  "version": "2.7.4-beta.0",
+  "version": "2.7.4-beta.10",
   "description": "ESLint React's ESLint plugin for interacting with Web APIs",
   "keywords": [
     "react",
@@ -20,7 +20,7 @@
     "directory": "packages/plugins/eslint-plugin-react-web-api"
   },
   "license": "MIT",
-  "author": "Rel1cx<rel1cx@proton.me>",
+  "author": "Rel1cx",
   "sideEffects": false,
   "type": "module",
   "exports": {
@@ -43,16 +43,16 @@
     "@typescript-eslint/utils": "^8.53.1",
     "string-ts": "^2.3.1",
     "ts-pattern": "^5.9.0",
-    "@eslint-react/ast": "2.7.4-beta.0",
-    "@eslint-react/core": "2.7.4-beta.0",
-    "@eslint-react/eff": "2.7.4-beta.0",
-    "@eslint-react/var": "2.7.4-beta.0",
-    "@eslint-react/shared": "2.7.4-beta.0"
+    "@eslint-react/ast": "2.7.4-beta.10",
+    "@eslint-react/core": "2.7.4-beta.10",
+    "@eslint-react/eff": "2.7.4-beta.10",
+    "@eslint-react/shared": "2.7.4-beta.10",
+    "@eslint-react/var": "2.7.4-beta.10"
   },
   "devDependencies": {
     "@types/react": "^19.2.9",
     "@types/react-dom": "^19.2.3",
-    "tsdown": "^0.20.0-beta.4",
+    "tsdown": "^0.20.1",
     "@local/configs": "0.0.0"
   },
   "peerDependencies": {