eslint-plugin-react-web-api 2.7.4-beta.5 → 2.7.4-beta.7
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +12 -0
- package/dist/index.js +12 -12
- package/package.json +7 -7
package/README.md
CHANGED
|
@@ -36,6 +36,18 @@ export default defineConfig(
|
|
|
36
36
|
);
|
|
37
37
|
```
|
|
38
38
|
|
|
39
|
+
## Web API Rules
|
|
40
|
+
|
|
41
|
+
> [!NOTE]
|
|
42
|
+
> Web API rules prevent resource leaks by detecting uncleaned Web API subscriptions in components and hooks.
|
|
43
|
+
|
|
44
|
+
**Implemented Rules:**
|
|
45
|
+
|
|
46
|
+
- [`no-leaked-event-listener`](./web-api-no-leaked-event-listener) - Prevents leaked `addEventListener` calls
|
|
47
|
+
- [`no-leaked-interval`](./web-api-no-leaked-interval) - Prevents leaked `setInterval` calls
|
|
48
|
+
- [`no-leaked-resize-observer`](./web-api-no-leaked-resize-observer) - Prevents leaked `ResizeObserver` instances
|
|
49
|
+
- [`no-leaked-timeout`](./web-api-no-leaked-timeout) - Prevents leaked `setTimeout` calls
|
|
50
|
+
|
|
39
51
|
## Rules
|
|
40
52
|
|
|
41
53
|
<https://eslint-react.xyz/docs/rules/overview#web-api-rules>
|
package/dist/index.js
CHANGED
|
@@ -11,9 +11,9 @@ import { P, isMatching, match } from "ts-pattern";
|
|
|
11
11
|
var __defProp = Object.defineProperty;
|
|
12
12
|
var __exportAll = (all, symbols) => {
|
|
13
13
|
let target = {};
|
|
14
|
-
for (var name
|
|
15
|
-
__defProp(target, name
|
|
16
|
-
get: all[name
|
|
14
|
+
for (var name in all) {
|
|
15
|
+
__defProp(target, name, {
|
|
16
|
+
get: all[name],
|
|
17
17
|
enumerable: true
|
|
18
18
|
});
|
|
19
19
|
}
|
|
@@ -42,7 +42,7 @@ const settings = { "react-x": DEFAULT_ESLINT_REACT_SETTINGS };
|
|
|
42
42
|
//#endregion
|
|
43
43
|
//#region package.json
|
|
44
44
|
var name = "eslint-plugin-react-web-api";
|
|
45
|
-
var version = "2.7.4-beta.
|
|
45
|
+
var version = "2.7.4-beta.7";
|
|
46
46
|
|
|
47
47
|
//#endregion
|
|
48
48
|
//#region src/utils/create-rule.ts
|
|
@@ -94,20 +94,20 @@ function getOptions(node, initialScope) {
|
|
|
94
94
|
}
|
|
95
95
|
return filter(v) ? v : unit;
|
|
96
96
|
}
|
|
97
|
-
function getOpts(node
|
|
98
|
-
switch (node
|
|
97
|
+
function getOpts(node) {
|
|
98
|
+
switch (node.type) {
|
|
99
99
|
case AST_NODE_TYPES.Identifier: {
|
|
100
|
-
const variableNode = getVariableDefinitionNode(findVariable(node
|
|
100
|
+
const variableNode = getVariableDefinitionNode(findVariable(node, initialScope), 0);
|
|
101
101
|
if (variableNode?.type === AST_NODE_TYPES.ObjectExpression) return getOpts(variableNode);
|
|
102
102
|
return defaultOptions;
|
|
103
103
|
}
|
|
104
104
|
case AST_NODE_TYPES.Literal: return {
|
|
105
105
|
...defaultOptions,
|
|
106
|
-
capture: Boolean(node
|
|
106
|
+
capture: Boolean(node.value)
|
|
107
107
|
};
|
|
108
108
|
case AST_NODE_TYPES.ObjectExpression: {
|
|
109
|
-
const vCapture = !!getPropValue(findProp(node
|
|
110
|
-
const pSignal = findProp(node
|
|
109
|
+
const vCapture = !!getPropValue(findProp(node.properties, "capture"));
|
|
110
|
+
const pSignal = findProp(node.properties, "signal");
|
|
111
111
|
return {
|
|
112
112
|
capture: vCapture,
|
|
113
113
|
signal: pSignal?.type === AST_NODE_TYPES.Property ? getSignalValueExpression(pSignal.value, initialScope) : unit
|
|
@@ -469,8 +469,8 @@ function create$1(context) {
|
|
|
469
469
|
if (dEntries.some((e) => isAssignmentTargetEqual(context, e.observer, id))) continue;
|
|
470
470
|
const oentries = oEntries.filter((e) => isAssignmentTargetEqual(context, e.observer, id));
|
|
471
471
|
const uentries = uEntries.filter((e) => isAssignmentTargetEqual(context, e.observer, id));
|
|
472
|
-
const isDynamic = (node
|
|
473
|
-
const isPhaseNode = (node
|
|
472
|
+
const isDynamic = (node) => node?.type === AST_NODE_TYPES.CallExpression || AST.isConditional(node);
|
|
473
|
+
const isPhaseNode = (node) => node === phaseNode;
|
|
474
474
|
if (oentries.some((e) => !isPhaseNode(AST.findParentNode(e.node, or(isDynamic, isPhaseNode))))) {
|
|
475
475
|
context.report({
|
|
476
476
|
messageId: "expectedDisconnectInControlFlow",
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "eslint-plugin-react-web-api",
|
|
3
|
-
"version": "2.7.4-beta.
|
|
3
|
+
"version": "2.7.4-beta.7",
|
|
4
4
|
"description": "ESLint React's ESLint plugin for interacting with Web APIs",
|
|
5
5
|
"keywords": [
|
|
6
6
|
"react",
|
|
@@ -43,16 +43,16 @@
|
|
|
43
43
|
"@typescript-eslint/utils": "^8.53.1",
|
|
44
44
|
"string-ts": "^2.3.1",
|
|
45
45
|
"ts-pattern": "^5.9.0",
|
|
46
|
-
"@eslint-react/ast": "2.7.4-beta.
|
|
47
|
-
"@eslint-react/
|
|
48
|
-
"@eslint-react/shared": "2.7.4-beta.
|
|
49
|
-
"@eslint-react/
|
|
50
|
-
"@eslint-react/
|
|
46
|
+
"@eslint-react/ast": "2.7.4-beta.7",
|
|
47
|
+
"@eslint-react/core": "2.7.4-beta.7",
|
|
48
|
+
"@eslint-react/shared": "2.7.4-beta.7",
|
|
49
|
+
"@eslint-react/var": "2.7.4-beta.7",
|
|
50
|
+
"@eslint-react/eff": "2.7.4-beta.7"
|
|
51
51
|
},
|
|
52
52
|
"devDependencies": {
|
|
53
53
|
"@types/react": "^19.2.9",
|
|
54
54
|
"@types/react-dom": "^19.2.3",
|
|
55
|
-
"tsdown": "^0.20.
|
|
55
|
+
"tsdown": "^0.20.1",
|
|
56
56
|
"@local/configs": "0.0.0"
|
|
57
57
|
},
|
|
58
58
|
"peerDependencies": {
|