eslint-plugin-unicorn 44.0.2 → 45.0.1
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/configs/recommended.js +4 -0
- package/package.json +19 -16
- package/readme.md +122 -117
- package/rules/ast/index.js +1 -0
- package/rules/ast/is-new-expression.js +22 -0
- package/rules/better-regex.js +2 -2
- package/rules/catch-error-name.js +1 -1
- package/rules/consistent-function-scoping.js +1 -1
- package/rules/empty-brace-spaces.js +1 -1
- package/rules/error-message.js +1 -1
- package/rules/explicit-length-check.js +1 -1
- package/rules/fix/add-parenthesizes-to-return-or-throw-expression.js +1 -1
- package/rules/fix/append-argument.js +1 -1
- package/rules/fix/remove-argument.js +1 -1
- package/rules/import-style.js +1 -1
- package/rules/no-array-callback-reference.js +1 -1
- package/rules/no-array-for-each.js +4 -39
- package/rules/no-array-method-this-argument.js +1 -1
- package/rules/no-array-push-push.js +1 -1
- package/rules/no-for-loop.js +1 -1
- package/rules/no-invalid-remove-event-listener.js +1 -1
- package/rules/no-lonely-if.js +1 -1
- package/rules/no-negated-condition.js +138 -0
- package/rules/no-nested-ternary.js +1 -1
- package/rules/no-new-array.js +1 -1
- package/rules/no-new-buffer.js +1 -1
- package/rules/no-static-only-class.js +2 -2
- package/rules/no-thenable.js +1 -1
- package/rules/no-typeof-undefined.js +140 -0
- package/rules/no-unreadable-array-destructuring.js +1 -1
- package/rules/no-unsafe-regex.js +2 -4
- package/rules/no-useless-spread.js +135 -43
- package/rules/no-useless-undefined.js +16 -8
- package/rules/no-zero-fractions.js +1 -1
- package/rules/prefer-add-event-listener.js +7 -3
- package/rules/prefer-array-find.js +1 -1
- package/rules/prefer-at.js +1 -1
- package/rules/prefer-default-parameters.js +1 -1
- package/rules/prefer-dom-node-dataset.js +4 -4
- package/rules/prefer-dom-node-remove.js +1 -1
- package/rules/prefer-export-from.js +1 -1
- package/rules/prefer-json-parse-buffer.js +1 -1
- package/rules/prefer-keyboard-event-key.js +2 -2
- package/rules/prefer-math-trunc.js +1 -1
- package/rules/prefer-module.js +1 -1
- package/rules/prefer-native-coercion-functions.js +1 -1
- package/rules/prefer-object-from-entries.js +1 -1
- package/rules/prefer-optional-catch-binding.js +1 -1
- package/rules/prefer-prototype-methods.js +1 -1
- package/rules/prefer-reflect-apply.js +1 -1
- package/rules/prefer-regexp-test.js +34 -20
- package/rules/prefer-set-has.js +1 -1
- package/rules/prefer-set-size.js +99 -0
- package/rules/prefer-spread.js +1 -1
- package/rules/prefer-string-replace-all.js +96 -33
- package/rules/prefer-string-slice.js +1 -1
- package/rules/prefer-string-starts-ends-with.js +3 -3
- package/rules/prefer-switch.js +1 -1
- package/rules/prefer-ternary.js +1 -1
- package/rules/prefer-top-level-await.js +8 -4
- package/rules/prevent-abbreviations.js +10 -0
- package/rules/relative-url-style.js +1 -1
- package/rules/shared/simple-array-search-rule.js +1 -1
- package/rules/string-content.js +2 -2
- package/rules/switch-case-braces.js +1 -1
- package/rules/utils/escape-string.js +26 -0
- package/rules/utils/get-call-expression-arguments-text.js +1 -1
- package/rules/utils/get-switch-case-head-location.js +1 -1
- package/rules/utils/global-reference-tracker.js +1 -1
- package/rules/utils/is-function-self-used-inside.js +1 -1
- package/rules/utils/is-new-expression-with-parentheses.js +1 -1
- package/rules/utils/is-number.js +1 -1
- package/rules/utils/is-same-reference.js +19 -6
- package/rules/utils/parentheses.js +1 -1
- package/rules/utils/should-add-parentheses-to-logical-expression-child.js +1 -1
- package/rules/utils/is-valid-variable-name.js +0 -3
- package/rules/utils/quote-string.js +0 -17
package/configs/recommended.js
CHANGED
|
@@ -40,6 +40,8 @@ module.exports = {
|
|
|
40
40
|
'unicorn/no-invalid-remove-event-listener': 'error',
|
|
41
41
|
'unicorn/no-keyword-prefix': 'off',
|
|
42
42
|
'unicorn/no-lonely-if': 'error',
|
|
43
|
+
'no-negated-condition': 'off',
|
|
44
|
+
'unicorn/no-negated-condition': 'error',
|
|
43
45
|
'no-nested-ternary': 'off',
|
|
44
46
|
'unicorn/no-nested-ternary': 'error',
|
|
45
47
|
'unicorn/no-new-array': 'error',
|
|
@@ -50,6 +52,7 @@ module.exports = {
|
|
|
50
52
|
'unicorn/no-static-only-class': 'error',
|
|
51
53
|
'unicorn/no-thenable': 'error',
|
|
52
54
|
'unicorn/no-this-assignment': 'error',
|
|
55
|
+
'unicorn/no-typeof-undefined': 'error',
|
|
53
56
|
'unicorn/no-unnecessary-await': 'error',
|
|
54
57
|
'unicorn/no-unreadable-array-destructuring': 'error',
|
|
55
58
|
'unicorn/no-unreadable-iife': 'error',
|
|
@@ -101,6 +104,7 @@ module.exports = {
|
|
|
101
104
|
'unicorn/prefer-reflect-apply': 'error',
|
|
102
105
|
'unicorn/prefer-regexp-test': 'error',
|
|
103
106
|
'unicorn/prefer-set-has': 'error',
|
|
107
|
+
'unicorn/prefer-set-size': 'error',
|
|
104
108
|
'unicorn/prefer-spread': 'error',
|
|
105
109
|
// TODO: Enable this by default when targeting Node.js 16.
|
|
106
110
|
'unicorn/prefer-string-replace-all': 'off',
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "eslint-plugin-unicorn",
|
|
3
|
-
"version": "
|
|
3
|
+
"version": "45.0.1",
|
|
4
4
|
"description": "More than 100 powerful ESLint rules",
|
|
5
5
|
"license": "MIT",
|
|
6
6
|
"repository": "sindresorhus/eslint-plugin-unicorn",
|
|
@@ -14,14 +14,14 @@
|
|
|
14
14
|
"node": ">=14.18"
|
|
15
15
|
},
|
|
16
16
|
"scripts": {
|
|
17
|
-
"create-rule": "node ./scripts/create-rule.mjs && npm run
|
|
17
|
+
"create-rule": "node ./scripts/create-rule.mjs && npm run fix:eslint-docs",
|
|
18
18
|
"fix": "run-p --continue-on-error fix:*",
|
|
19
|
+
"fix:eslint-docs": "eslint-doc-generator --ignore-deprecated-rules --ignore-config all --rule-doc-title-format desc --url-configs \"https://github.com/sindresorhus/eslint-plugin-unicorn#preset-configs\"",
|
|
19
20
|
"fix:js": "npm run lint:js -- --fix",
|
|
20
21
|
"fix:md": "npm run lint:md -- --fix",
|
|
21
|
-
"generate-rule-notices": "node ./scripts/generate-rule-notices.mjs",
|
|
22
|
-
"generate-rules-table": "node ./scripts/generate-rules-table.mjs",
|
|
23
22
|
"integration": "node ./test/integration/test.mjs",
|
|
24
23
|
"lint": "run-p --continue-on-error lint:*",
|
|
24
|
+
"lint:eslint-docs": "npm run fix:eslint-docs -- --check",
|
|
25
25
|
"lint:js": "xo",
|
|
26
26
|
"lint:md": "markdownlint \"**/*.md\"",
|
|
27
27
|
"lint:package-json": "npmPkgJsonLint .",
|
|
@@ -47,36 +47,39 @@
|
|
|
47
47
|
],
|
|
48
48
|
"dependencies": {
|
|
49
49
|
"@babel/helper-validator-identifier": "^7.19.1",
|
|
50
|
-
"
|
|
50
|
+
"@eslint-community/eslint-utils": "^4.1.0",
|
|
51
|
+
"ci-info": "^3.6.1",
|
|
51
52
|
"clean-regexp": "^1.0.0",
|
|
52
|
-
"eslint-utils": "^3.0.0",
|
|
53
53
|
"esquery": "^1.4.0",
|
|
54
54
|
"indent-string": "^4.0.0",
|
|
55
55
|
"is-builtin-module": "^3.2.0",
|
|
56
|
+
"jsesc": "^3.0.2",
|
|
56
57
|
"lodash": "^4.17.21",
|
|
57
58
|
"pluralize": "^8.0.0",
|
|
58
59
|
"read-pkg-up": "^7.0.1",
|
|
59
60
|
"regexp-tree": "^0.1.24",
|
|
61
|
+
"regjsparser": "^0.9.1",
|
|
60
62
|
"safe-regex": "^2.1.1",
|
|
61
|
-
"semver": "^7.3.
|
|
63
|
+
"semver": "^7.3.8",
|
|
62
64
|
"strip-indent": "^3.0.0"
|
|
63
65
|
},
|
|
64
66
|
"devDependencies": {
|
|
65
67
|
"@babel/code-frame": "^7.18.6",
|
|
66
|
-
"@babel/core": "^7.
|
|
68
|
+
"@babel/core": "^7.20.2",
|
|
67
69
|
"@babel/eslint-parser": "^7.19.1",
|
|
68
70
|
"@lubien/fixture-beta-package": "^1.0.0-beta.1",
|
|
69
|
-
"@typescript-eslint/parser": "^5.
|
|
71
|
+
"@typescript-eslint/parser": "^5.43.0",
|
|
70
72
|
"ava": "^3.15.0",
|
|
71
73
|
"c8": "^7.12.0",
|
|
72
|
-
"chalk": "^5.
|
|
74
|
+
"chalk": "^5.1.2",
|
|
73
75
|
"enquirer": "^2.3.6",
|
|
74
|
-
"eslint": "^8.
|
|
76
|
+
"eslint": "^8.28.0",
|
|
75
77
|
"eslint-ava-rule-tester": "^4.0.0",
|
|
78
|
+
"eslint-doc-generator": "^0.24.0",
|
|
76
79
|
"eslint-plugin-eslint-plugin": "^5.0.6",
|
|
77
80
|
"eslint-plugin-internal-rules": "file:./scripts/internal-rules/",
|
|
78
81
|
"eslint-remote-tester": "^3.0.0",
|
|
79
|
-
"eslint-remote-tester-repositories": "^0.0.
|
|
82
|
+
"eslint-remote-tester-repositories": "^0.0.7",
|
|
80
83
|
"execa": "^6.1.0",
|
|
81
84
|
"listr": "^0.14.3",
|
|
82
85
|
"lodash-es": "^4.17.21",
|
|
@@ -85,13 +88,13 @@
|
|
|
85
88
|
"npm-package-json-lint": "^6.3.0",
|
|
86
89
|
"npm-run-all": "^4.1.5",
|
|
87
90
|
"outdent": "^0.8.0",
|
|
88
|
-
"typescript": "^4.
|
|
91
|
+
"typescript": "^4.9.3",
|
|
89
92
|
"vue-eslint-parser": "^9.1.0",
|
|
90
|
-
"xo": "^0.
|
|
91
|
-
"yaml": "^1.
|
|
93
|
+
"xo": "^0.53.1",
|
|
94
|
+
"yaml": "^2.1.3"
|
|
92
95
|
},
|
|
93
96
|
"peerDependencies": {
|
|
94
|
-
"eslint": ">=8.
|
|
97
|
+
"eslint": ">=8.28.0"
|
|
95
98
|
},
|
|
96
99
|
"ava": {
|
|
97
100
|
"files": [
|
package/readme.md
CHANGED
|
@@ -43,123 +43,128 @@ Use a [preset config](#preset-configs) or configure each rules in `package.json`
|
|
|
43
43
|
|
|
44
44
|
## Rules
|
|
45
45
|
|
|
46
|
-
|
|
47
|
-
|
|
48
|
-
|
|
49
|
-
|
|
50
|
-
|
|
51
|
-
|
|
52
|
-
|
|
53
|
-
|
|
54
|
-
|
|
55
|
-
|
|
|
56
|
-
|
|
|
57
|
-
| [
|
|
58
|
-
| [
|
|
59
|
-
| [consistent-
|
|
60
|
-
| [
|
|
61
|
-
| [
|
|
62
|
-
| [
|
|
63
|
-
| [
|
|
64
|
-
| [
|
|
65
|
-
| [
|
|
66
|
-
| [
|
|
67
|
-
| [
|
|
68
|
-
| [
|
|
69
|
-
| [
|
|
70
|
-
| [no-
|
|
71
|
-
| [no-array-
|
|
72
|
-
| [no-array-
|
|
73
|
-
| [no-array-
|
|
74
|
-
| [no-array-
|
|
75
|
-
| [no-
|
|
76
|
-
| [no-
|
|
77
|
-
| [no-
|
|
78
|
-
| [no-
|
|
79
|
-
| [no-
|
|
80
|
-
| [no-
|
|
81
|
-
| [no-
|
|
82
|
-
| [no-
|
|
83
|
-
| [no-
|
|
84
|
-
| [no-
|
|
85
|
-
| [no-
|
|
86
|
-
| [no-
|
|
87
|
-
| [no-
|
|
88
|
-
| [no-
|
|
89
|
-
| [no-
|
|
90
|
-
| [no-
|
|
91
|
-
| [no-
|
|
92
|
-
| [no-
|
|
93
|
-
| [no-
|
|
94
|
-
| [no-
|
|
95
|
-
| [no-
|
|
96
|
-
| [no-
|
|
97
|
-
| [no-
|
|
98
|
-
| [no-
|
|
99
|
-
| [no-
|
|
100
|
-
| [no-
|
|
101
|
-
| [no-
|
|
102
|
-
| [no-useless-spread](docs/rules/no-useless-spread.md)
|
|
103
|
-
| [no-useless-
|
|
104
|
-
| [no-useless-
|
|
105
|
-
| [no-
|
|
106
|
-
| [
|
|
107
|
-
| [
|
|
108
|
-
| [
|
|
109
|
-
| [
|
|
110
|
-
| [
|
|
111
|
-
| [prefer-
|
|
112
|
-
| [prefer-array-
|
|
113
|
-
| [prefer-array-
|
|
114
|
-
| [prefer-
|
|
115
|
-
| [prefer-
|
|
116
|
-
| [prefer-
|
|
117
|
-
| [prefer-
|
|
118
|
-
| [prefer-
|
|
119
|
-
| [prefer-
|
|
120
|
-
| [prefer-
|
|
121
|
-
| [prefer-dom-node-
|
|
122
|
-
| [prefer-
|
|
123
|
-
| [prefer-
|
|
124
|
-
| [prefer-
|
|
125
|
-
| [prefer-
|
|
126
|
-
| [prefer-
|
|
127
|
-
| [prefer-
|
|
128
|
-
| [prefer-
|
|
129
|
-
| [prefer-
|
|
130
|
-
| [prefer-
|
|
131
|
-
| [prefer-
|
|
132
|
-
| [prefer-
|
|
133
|
-
| [prefer-
|
|
134
|
-
| [prefer-
|
|
135
|
-
| [prefer-
|
|
136
|
-
| [prefer-
|
|
137
|
-
| [prefer-
|
|
138
|
-
| [prefer-
|
|
139
|
-
| [prefer-
|
|
140
|
-
| [prefer-
|
|
141
|
-
| [prefer-
|
|
142
|
-
| [prefer-
|
|
143
|
-
| [prefer-
|
|
144
|
-
| [prefer-
|
|
145
|
-
| [prefer-
|
|
146
|
-
| [prefer-
|
|
147
|
-
| [prefer-
|
|
148
|
-
| [prefer-
|
|
149
|
-
| [prefer-
|
|
150
|
-
| [prefer-
|
|
151
|
-
| [prefer-
|
|
152
|
-
| [
|
|
153
|
-
| [
|
|
154
|
-
| [
|
|
155
|
-
| [
|
|
156
|
-
| [
|
|
157
|
-
| [
|
|
158
|
-
| [
|
|
159
|
-
| [
|
|
160
|
-
| [
|
|
161
|
-
| [
|
|
162
|
-
|
|
46
|
+
<!-- Do not manually modify this list. Run: `npm run fix:eslint-docs` -->
|
|
47
|
+
<!-- begin auto-generated rules list -->
|
|
48
|
+
|
|
49
|
+
💼 [Configurations](https://github.com/sindresorhus/eslint-plugin-unicorn#preset-configs) enabled in.\
|
|
50
|
+
🚫 [Configurations](https://github.com/sindresorhus/eslint-plugin-unicorn#preset-configs) disabled in.\
|
|
51
|
+
✅ Set in the `recommended` [configuration](https://github.com/sindresorhus/eslint-plugin-unicorn#preset-configs).\
|
|
52
|
+
🔧 Automatically fixable by the [`--fix` CLI option](https://eslint.org/docs/user-guide/command-line-interface#--fix).\
|
|
53
|
+
💡 Manually fixable by [editor suggestions](https://eslint.org/docs/developer-guide/working-with-rules#providing-suggestions).
|
|
54
|
+
|
|
55
|
+
| Name | Description | 💼 | 🚫 | 🔧 | 💡 |
|
|
56
|
+
| :----------------------------------------------------------------------------------------------- | :---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | :- | :- | :- | :- |
|
|
57
|
+
| [better-regex](docs/rules/better-regex.md) | Improve regexes by making them shorter, consistent, and safer. | ✅ | | 🔧 | |
|
|
58
|
+
| [catch-error-name](docs/rules/catch-error-name.md) | Enforce a specific parameter name in catch clauses. | ✅ | | 🔧 | |
|
|
59
|
+
| [consistent-destructuring](docs/rules/consistent-destructuring.md) | Use destructured variables over properties. | ✅ | | 🔧 | 💡 |
|
|
60
|
+
| [consistent-function-scoping](docs/rules/consistent-function-scoping.md) | Move function definitions to the highest possible scope. | ✅ | | | |
|
|
61
|
+
| [custom-error-definition](docs/rules/custom-error-definition.md) | Enforce correct `Error` subclassing. | | ✅ | 🔧 | |
|
|
62
|
+
| [empty-brace-spaces](docs/rules/empty-brace-spaces.md) | Enforce no spaces between braces. | ✅ | | 🔧 | |
|
|
63
|
+
| [error-message](docs/rules/error-message.md) | Enforce passing a `message` value when creating a built-in error. | ✅ | | | |
|
|
64
|
+
| [escape-case](docs/rules/escape-case.md) | Require escape sequences to use uppercase values. | ✅ | | 🔧 | |
|
|
65
|
+
| [expiring-todo-comments](docs/rules/expiring-todo-comments.md) | Add expiration conditions to TODO comments. | ✅ | | | |
|
|
66
|
+
| [explicit-length-check](docs/rules/explicit-length-check.md) | Enforce explicitly comparing the `length` or `size` property of a value. | ✅ | | 🔧 | 💡 |
|
|
67
|
+
| [filename-case](docs/rules/filename-case.md) | Enforce a case style for filenames. | ✅ | | | |
|
|
68
|
+
| [import-style](docs/rules/import-style.md) | Enforce specific import styles per module. | ✅ | | | |
|
|
69
|
+
| [new-for-builtins](docs/rules/new-for-builtins.md) | Enforce the use of `new` for all builtins, except `String`, `Number`, `Boolean`, `Symbol` and `BigInt`. | ✅ | | 🔧 | |
|
|
70
|
+
| [no-abusive-eslint-disable](docs/rules/no-abusive-eslint-disable.md) | Enforce specifying rules to disable in `eslint-disable` comments. | ✅ | | | |
|
|
71
|
+
| [no-array-callback-reference](docs/rules/no-array-callback-reference.md) | Prevent passing a function reference directly to iterator methods. | ✅ | | | 💡 |
|
|
72
|
+
| [no-array-for-each](docs/rules/no-array-for-each.md) | Prefer `for…of` over the `forEach` method. | ✅ | | 🔧 | 💡 |
|
|
73
|
+
| [no-array-method-this-argument](docs/rules/no-array-method-this-argument.md) | Disallow using the `this` argument in array methods. | ✅ | | 🔧 | 💡 |
|
|
74
|
+
| [no-array-push-push](docs/rules/no-array-push-push.md) | Enforce combining multiple `Array#push()` into one call. | ✅ | | 🔧 | 💡 |
|
|
75
|
+
| [no-array-reduce](docs/rules/no-array-reduce.md) | Disallow `Array#reduce()` and `Array#reduceRight()`. | ✅ | | | |
|
|
76
|
+
| [no-await-expression-member](docs/rules/no-await-expression-member.md) | Disallow member access from await expression. | ✅ | | 🔧 | |
|
|
77
|
+
| [no-console-spaces](docs/rules/no-console-spaces.md) | Do not use leading/trailing space between `console.log` parameters. | ✅ | | 🔧 | |
|
|
78
|
+
| [no-document-cookie](docs/rules/no-document-cookie.md) | Do not use `document.cookie` directly. | ✅ | | | |
|
|
79
|
+
| [no-empty-file](docs/rules/no-empty-file.md) | Disallow empty files. | ✅ | | | |
|
|
80
|
+
| [no-for-loop](docs/rules/no-for-loop.md) | Do not use a `for` loop that can be replaced with a `for-of` loop. | ✅ | | 🔧 | |
|
|
81
|
+
| [no-hex-escape](docs/rules/no-hex-escape.md) | Enforce the use of Unicode escapes instead of hexadecimal escapes. | ✅ | | 🔧 | |
|
|
82
|
+
| [no-instanceof-array](docs/rules/no-instanceof-array.md) | Require `Array.isArray()` instead of `instanceof Array`. | ✅ | | 🔧 | |
|
|
83
|
+
| [no-invalid-remove-event-listener](docs/rules/no-invalid-remove-event-listener.md) | Prevent calling `EventTarget#removeEventListener()` with the result of an expression. | ✅ | | | |
|
|
84
|
+
| [no-keyword-prefix](docs/rules/no-keyword-prefix.md) | Disallow identifiers starting with `new` or `class`. | | ✅ | | |
|
|
85
|
+
| [no-lonely-if](docs/rules/no-lonely-if.md) | Disallow `if` statements as the only statement in `if` blocks without `else`. | ✅ | | 🔧 | |
|
|
86
|
+
| [no-negated-condition](docs/rules/no-negated-condition.md) | Disallow negated conditions. | ✅ | | 🔧 | |
|
|
87
|
+
| [no-nested-ternary](docs/rules/no-nested-ternary.md) | Disallow nested ternary expressions. | ✅ | | 🔧 | |
|
|
88
|
+
| [no-new-array](docs/rules/no-new-array.md) | Disallow `new Array()`. | ✅ | | 🔧 | 💡 |
|
|
89
|
+
| [no-new-buffer](docs/rules/no-new-buffer.md) | Enforce the use of `Buffer.from()` and `Buffer.alloc()` instead of the deprecated `new Buffer()`. | ✅ | | 🔧 | 💡 |
|
|
90
|
+
| [no-null](docs/rules/no-null.md) | Disallow the use of the `null` literal. | ✅ | | 🔧 | 💡 |
|
|
91
|
+
| [no-object-as-default-parameter](docs/rules/no-object-as-default-parameter.md) | Disallow the use of objects as default parameters. | ✅ | | | |
|
|
92
|
+
| [no-process-exit](docs/rules/no-process-exit.md) | Disallow `process.exit()`. | ✅ | | | |
|
|
93
|
+
| [no-static-only-class](docs/rules/no-static-only-class.md) | Disallow classes that only have static members. | ✅ | | 🔧 | |
|
|
94
|
+
| [no-thenable](docs/rules/no-thenable.md) | Disallow `then` property. | ✅ | | | |
|
|
95
|
+
| [no-this-assignment](docs/rules/no-this-assignment.md) | Disallow assigning `this` to a variable. | ✅ | | | |
|
|
96
|
+
| [no-typeof-undefined](docs/rules/no-typeof-undefined.md) | Disallow comparing `undefined` using `typeof`. | ✅ | | 🔧 | 💡 |
|
|
97
|
+
| [no-unnecessary-await](docs/rules/no-unnecessary-await.md) | Disallow awaiting non-promise values. | ✅ | | 🔧 | |
|
|
98
|
+
| [no-unreadable-array-destructuring](docs/rules/no-unreadable-array-destructuring.md) | Disallow unreadable array destructuring. | ✅ | | 🔧 | |
|
|
99
|
+
| [no-unreadable-iife](docs/rules/no-unreadable-iife.md) | Disallow unreadable IIFEs. | ✅ | | | |
|
|
100
|
+
| [no-unsafe-regex](docs/rules/no-unsafe-regex.md) | Disallow unsafe regular expressions. | | ✅ | | |
|
|
101
|
+
| [no-unused-properties](docs/rules/no-unused-properties.md) | Disallow unused object properties. | | ✅ | | |
|
|
102
|
+
| [no-useless-fallback-in-spread](docs/rules/no-useless-fallback-in-spread.md) | Disallow useless fallback when spreading in object literals. | ✅ | | 🔧 | |
|
|
103
|
+
| [no-useless-length-check](docs/rules/no-useless-length-check.md) | Disallow useless array length check. | ✅ | | 🔧 | |
|
|
104
|
+
| [no-useless-promise-resolve-reject](docs/rules/no-useless-promise-resolve-reject.md) | Disallow returning/yielding `Promise.resolve/reject()` in async functions or promise callbacks | ✅ | | 🔧 | |
|
|
105
|
+
| [no-useless-spread](docs/rules/no-useless-spread.md) | Disallow unnecessary spread. | ✅ | | 🔧 | |
|
|
106
|
+
| [no-useless-switch-case](docs/rules/no-useless-switch-case.md) | Disallow useless case in switch statements. | ✅ | | | 💡 |
|
|
107
|
+
| [no-useless-undefined](docs/rules/no-useless-undefined.md) | Disallow useless `undefined`. | ✅ | | 🔧 | |
|
|
108
|
+
| [no-zero-fractions](docs/rules/no-zero-fractions.md) | Disallow number literals with zero fractions or dangling dots. | ✅ | | 🔧 | |
|
|
109
|
+
| [number-literal-case](docs/rules/number-literal-case.md) | Enforce proper case for numeric literals. | ✅ | | 🔧 | |
|
|
110
|
+
| [numeric-separators-style](docs/rules/numeric-separators-style.md) | Enforce the style of numeric separators by correctly grouping digits. | ✅ | | 🔧 | |
|
|
111
|
+
| [prefer-add-event-listener](docs/rules/prefer-add-event-listener.md) | Prefer `.addEventListener()` and `.removeEventListener()` over `on`-functions. | ✅ | | 🔧 | |
|
|
112
|
+
| [prefer-array-find](docs/rules/prefer-array-find.md) | Prefer `.find(…)` and `.findLast(…)` over the first or last element from `.filter(…)`. | ✅ | | 🔧 | 💡 |
|
|
113
|
+
| [prefer-array-flat](docs/rules/prefer-array-flat.md) | Prefer `Array#flat()` over legacy techniques to flatten arrays. | ✅ | | 🔧 | |
|
|
114
|
+
| [prefer-array-flat-map](docs/rules/prefer-array-flat-map.md) | Prefer `.flatMap(…)` over `.map(…).flat()`. | ✅ | | 🔧 | |
|
|
115
|
+
| [prefer-array-index-of](docs/rules/prefer-array-index-of.md) | Prefer `Array#{indexOf,lastIndexOf}()` over `Array#{findIndex,findLastIndex}()` when looking for the index of an item. | ✅ | | 🔧 | 💡 |
|
|
116
|
+
| [prefer-array-some](docs/rules/prefer-array-some.md) | Prefer `.some(…)` over `.filter(…).length` check and `.{find,findLast}(…)`. | ✅ | | 🔧 | 💡 |
|
|
117
|
+
| [prefer-at](docs/rules/prefer-at.md) | Prefer `.at()` method for index access and `String#charAt()`. | | ✅ | 🔧 | 💡 |
|
|
118
|
+
| [prefer-code-point](docs/rules/prefer-code-point.md) | Prefer `String#codePointAt(…)` over `String#charCodeAt(…)` and `String.fromCodePoint(…)` over `String.fromCharCode(…)`. | ✅ | | | 💡 |
|
|
119
|
+
| [prefer-date-now](docs/rules/prefer-date-now.md) | Prefer `Date.now()` to get the number of milliseconds since the Unix Epoch. | ✅ | | 🔧 | |
|
|
120
|
+
| [prefer-default-parameters](docs/rules/prefer-default-parameters.md) | Prefer default parameters over reassignment. | ✅ | | 🔧 | 💡 |
|
|
121
|
+
| [prefer-dom-node-append](docs/rules/prefer-dom-node-append.md) | Prefer `Node#append()` over `Node#appendChild()`. | ✅ | | 🔧 | |
|
|
122
|
+
| [prefer-dom-node-dataset](docs/rules/prefer-dom-node-dataset.md) | Prefer using `.dataset` on DOM elements over calling attribute methods. | ✅ | | 🔧 | |
|
|
123
|
+
| [prefer-dom-node-remove](docs/rules/prefer-dom-node-remove.md) | Prefer `childNode.remove()` over `parentNode.removeChild(childNode)`. | ✅ | | 🔧 | 💡 |
|
|
124
|
+
| [prefer-dom-node-text-content](docs/rules/prefer-dom-node-text-content.md) | Prefer `.textContent` over `.innerText`. | ✅ | | | 💡 |
|
|
125
|
+
| [prefer-event-target](docs/rules/prefer-event-target.md) | Prefer `EventTarget` over `EventEmitter`. | | ✅ | | |
|
|
126
|
+
| [prefer-export-from](docs/rules/prefer-export-from.md) | Prefer `export…from` when re-exporting. | ✅ | | 🔧 | 💡 |
|
|
127
|
+
| [prefer-includes](docs/rules/prefer-includes.md) | Prefer `.includes()` over `.indexOf()` and `Array#some()` when checking for existence or non-existence. | ✅ | | 🔧 | 💡 |
|
|
128
|
+
| [prefer-json-parse-buffer](docs/rules/prefer-json-parse-buffer.md) | Prefer reading a JSON file as a buffer. | | ✅ | 🔧 | |
|
|
129
|
+
| [prefer-keyboard-event-key](docs/rules/prefer-keyboard-event-key.md) | Prefer `KeyboardEvent#key` over `KeyboardEvent#keyCode`. | ✅ | | 🔧 | |
|
|
130
|
+
| [prefer-logical-operator-over-ternary](docs/rules/prefer-logical-operator-over-ternary.md) | Prefer using a logical operator over a ternary. | ✅ | | | 💡 |
|
|
131
|
+
| [prefer-math-trunc](docs/rules/prefer-math-trunc.md) | Enforce the use of `Math.trunc` instead of bitwise operators. | ✅ | | 🔧 | 💡 |
|
|
132
|
+
| [prefer-modern-dom-apis](docs/rules/prefer-modern-dom-apis.md) | Prefer `.before()` over `.insertBefore()`, `.replaceWith()` over `.replaceChild()`, prefer one of `.before()`, `.after()`, `.append()` or `.prepend()` over `insertAdjacentText()` and `insertAdjacentElement()`. | ✅ | | 🔧 | |
|
|
133
|
+
| [prefer-modern-math-apis](docs/rules/prefer-modern-math-apis.md) | Prefer modern `Math` APIs over legacy patterns. | ✅ | | 🔧 | |
|
|
134
|
+
| [prefer-module](docs/rules/prefer-module.md) | Prefer JavaScript modules (ESM) over CommonJS. | ✅ | | 🔧 | 💡 |
|
|
135
|
+
| [prefer-native-coercion-functions](docs/rules/prefer-native-coercion-functions.md) | Prefer using `String`, `Number`, `BigInt`, `Boolean`, and `Symbol` directly. | ✅ | | 🔧 | |
|
|
136
|
+
| [prefer-negative-index](docs/rules/prefer-negative-index.md) | Prefer negative index over `.length - index` for `{String,Array,TypedArray}#{slice,at}()` and `Array#splice()`. | ✅ | | 🔧 | |
|
|
137
|
+
| [prefer-node-protocol](docs/rules/prefer-node-protocol.md) | Prefer using the `node:` protocol when importing Node.js builtin modules. | ✅ | | 🔧 | |
|
|
138
|
+
| [prefer-number-properties](docs/rules/prefer-number-properties.md) | Prefer `Number` static properties over global ones. | ✅ | | 🔧 | 💡 |
|
|
139
|
+
| [prefer-object-from-entries](docs/rules/prefer-object-from-entries.md) | Prefer using `Object.fromEntries(…)` to transform a list of key-value pairs into an object. | ✅ | | 🔧 | |
|
|
140
|
+
| [prefer-optional-catch-binding](docs/rules/prefer-optional-catch-binding.md) | Prefer omitting the `catch` binding parameter. | ✅ | | 🔧 | |
|
|
141
|
+
| [prefer-prototype-methods](docs/rules/prefer-prototype-methods.md) | Prefer borrowing methods from the prototype instead of the instance. | ✅ | | 🔧 | |
|
|
142
|
+
| [prefer-query-selector](docs/rules/prefer-query-selector.md) | Prefer `.querySelector()` over `.getElementById()`, `.querySelectorAll()` over `.getElementsByClassName()` and `.getElementsByTagName()`. | ✅ | | 🔧 | |
|
|
143
|
+
| [prefer-reflect-apply](docs/rules/prefer-reflect-apply.md) | Prefer `Reflect.apply()` over `Function#apply()`. | ✅ | | 🔧 | |
|
|
144
|
+
| [prefer-regexp-test](docs/rules/prefer-regexp-test.md) | Prefer `RegExp#test()` over `String#match()` and `RegExp#exec()`. | ✅ | | 🔧 | 💡 |
|
|
145
|
+
| [prefer-set-has](docs/rules/prefer-set-has.md) | Prefer `Set#has()` over `Array#includes()` when checking for existence or non-existence. | ✅ | | 🔧 | 💡 |
|
|
146
|
+
| [prefer-set-size](docs/rules/prefer-set-size.md) | Prefer using `Set#size` instead of `Array#length`. | ✅ | | 🔧 | |
|
|
147
|
+
| [prefer-spread](docs/rules/prefer-spread.md) | Prefer the spread operator over `Array.from(…)`, `Array#concat(…)`, `Array#slice()` and `String#split('')`. | ✅ | | 🔧 | 💡 |
|
|
148
|
+
| [prefer-string-replace-all](docs/rules/prefer-string-replace-all.md) | Prefer `String#replaceAll()` over regex searches with the global flag. | | ✅ | 🔧 | |
|
|
149
|
+
| [prefer-string-slice](docs/rules/prefer-string-slice.md) | Prefer `String#slice()` over `String#substr()` and `String#substring()`. | ✅ | | 🔧 | |
|
|
150
|
+
| [prefer-string-starts-ends-with](docs/rules/prefer-string-starts-ends-with.md) | Prefer `String#startsWith()` & `String#endsWith()` over `RegExp#test()`. | ✅ | | 🔧 | 💡 |
|
|
151
|
+
| [prefer-string-trim-start-end](docs/rules/prefer-string-trim-start-end.md) | Prefer `String#trimStart()` / `String#trimEnd()` over `String#trimLeft()` / `String#trimRight()`. | ✅ | | 🔧 | |
|
|
152
|
+
| [prefer-switch](docs/rules/prefer-switch.md) | Prefer `switch` over multiple `else-if`. | ✅ | | 🔧 | |
|
|
153
|
+
| [prefer-ternary](docs/rules/prefer-ternary.md) | Prefer ternary expressions over simple `if-else` statements. | ✅ | | 🔧 | |
|
|
154
|
+
| [prefer-top-level-await](docs/rules/prefer-top-level-await.md) | Prefer top-level await over top-level promises and async function calls. | ✅ | | | 💡 |
|
|
155
|
+
| [prefer-type-error](docs/rules/prefer-type-error.md) | Enforce throwing `TypeError` in type checking conditions. | ✅ | | 🔧 | |
|
|
156
|
+
| [prevent-abbreviations](docs/rules/prevent-abbreviations.md) | Prevent abbreviations. | ✅ | | 🔧 | |
|
|
157
|
+
| [relative-url-style](docs/rules/relative-url-style.md) | Enforce consistent relative URL style. | ✅ | | 🔧 | 💡 |
|
|
158
|
+
| [require-array-join-separator](docs/rules/require-array-join-separator.md) | Enforce using the separator argument with `Array#join()`. | ✅ | | 🔧 | |
|
|
159
|
+
| [require-number-to-fixed-digits-argument](docs/rules/require-number-to-fixed-digits-argument.md) | Enforce using the digits argument with `Number#toFixed()`. | ✅ | | 🔧 | |
|
|
160
|
+
| [require-post-message-target-origin](docs/rules/require-post-message-target-origin.md) | Enforce using the `targetOrigin` argument with `window.postMessage()`. | | ✅ | | 💡 |
|
|
161
|
+
| [string-content](docs/rules/string-content.md) | Enforce better string content. | | ✅ | 🔧 | 💡 |
|
|
162
|
+
| [switch-case-braces](docs/rules/switch-case-braces.md) | Enforce consistent brace style for `case` clauses. | ✅ | | 🔧 | |
|
|
163
|
+
| [template-indent](docs/rules/template-indent.md) | Fix whitespace-insensitive template indentation. | ✅ | | 🔧 | |
|
|
164
|
+
| [text-encoding-identifier-case](docs/rules/text-encoding-identifier-case.md) | Enforce consistent case for text encoding identifiers. | ✅ | | 🔧 | 💡 |
|
|
165
|
+
| [throw-new-error](docs/rules/throw-new-error.md) | Require `new` when throwing an error. | ✅ | | 🔧 | |
|
|
166
|
+
|
|
167
|
+
<!-- end auto-generated rules list -->
|
|
163
168
|
|
|
164
169
|
### Deprecated Rules
|
|
165
170
|
|
package/rules/ast/index.js
CHANGED
|
@@ -0,0 +1,22 @@
|
|
|
1
|
+
'use strict';
|
|
2
|
+
|
|
3
|
+
function isNewExpression(node, options) {
|
|
4
|
+
if (node?.type !== 'NewExpression') {
|
|
5
|
+
return false;
|
|
6
|
+
}
|
|
7
|
+
|
|
8
|
+
const {
|
|
9
|
+
name,
|
|
10
|
+
} = {
|
|
11
|
+
...options,
|
|
12
|
+
};
|
|
13
|
+
|
|
14
|
+
if (name) {
|
|
15
|
+
return node.callee.type === 'Identifier' && node.callee.name === name;
|
|
16
|
+
}
|
|
17
|
+
|
|
18
|
+
/* c8 ignore next */
|
|
19
|
+
return true;
|
|
20
|
+
}
|
|
21
|
+
|
|
22
|
+
module.exports = isNewExpression;
|
package/rules/better-regex.js
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
'use strict';
|
|
2
2
|
const cleanRegexp = require('clean-regexp');
|
|
3
3
|
const {optimize} = require('regexp-tree');
|
|
4
|
-
const
|
|
4
|
+
const escapeString = require('./utils/escape-string.js');
|
|
5
5
|
const {newExpressionSelector} = require('./selectors/index.js');
|
|
6
6
|
const {isStringLiteral} = require('./ast/index.js');
|
|
7
7
|
|
|
@@ -102,7 +102,7 @@ const create = context => {
|
|
|
102
102
|
},
|
|
103
103
|
fix: fixer => fixer.replaceText(
|
|
104
104
|
patternNode,
|
|
105
|
-
|
|
105
|
+
escapeString(newPattern, patternNode.raw.charAt(0)),
|
|
106
106
|
),
|
|
107
107
|
};
|
|
108
108
|
}
|
|
@@ -1,5 +1,5 @@
|
|
|
1
1
|
'use strict';
|
|
2
|
-
const {findVariable} = require('eslint-utils');
|
|
2
|
+
const {findVariable} = require('@eslint-community/eslint-utils');
|
|
3
3
|
const avoidCapture = require('./utils/avoid-capture.js');
|
|
4
4
|
const {renameVariable} = require('./fix/index.js');
|
|
5
5
|
const {matches, methodCallSelector} = require('./selectors/index.js');
|
|
@@ -1,5 +1,5 @@
|
|
|
1
1
|
'use strict';
|
|
2
|
-
const {getFunctionHeadLocation, getFunctionNameWithKind} = require('eslint-utils');
|
|
2
|
+
const {getFunctionHeadLocation, getFunctionNameWithKind} = require('@eslint-community/eslint-utils');
|
|
3
3
|
const getReferences = require('./utils/get-references.js');
|
|
4
4
|
const {isNodeMatches} = require('./utils/is-node-matches.js');
|
|
5
5
|
|
package/rules/error-message.js
CHANGED
|
@@ -1,5 +1,5 @@
|
|
|
1
1
|
'use strict';
|
|
2
|
-
const {isParenthesized, getStaticValue} = require('eslint-utils');
|
|
2
|
+
const {isParenthesized, getStaticValue} = require('@eslint-community/eslint-utils');
|
|
3
3
|
const {checkVueTemplate} = require('./utils/rule.js');
|
|
4
4
|
const isLogicalExpression = require('./utils/is-logical-expression.js');
|
|
5
5
|
const {isBooleanNode, getBooleanAncestor} = require('./utils/boolean.js');
|