eslint 9.23.0 → 9.25.0
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 +3 -2
- package/lib/cli-engine/cli-engine.js +2 -2
- package/lib/cli-engine/formatters/html.js +3 -3
- package/lib/cli-engine/formatters/stylish.js +1 -1
- package/lib/cli.js +98 -1
- package/lib/config/config-loader.js +108 -34
- package/lib/config/flat-config-helpers.js +7 -4
- package/lib/eslint/eslint-helpers.js +50 -5
- package/lib/eslint/eslint.js +17 -53
- package/lib/eslint/legacy-eslint.js +1 -1
- package/lib/languages/js/source-code/source-code.js +3 -3
- package/lib/languages/js/source-code/token-store/index.js +3 -3
- package/lib/linter/code-path-analysis/code-path-state.js +1 -1
- package/lib/linter/linter.js +1 -1
- package/lib/linter/node-event-generator.js +3 -1
- package/lib/linter/report-translator.js +1 -1
- package/lib/linter/rule-fixer.js +13 -5
- package/lib/linter/rules.js +1 -1
- package/lib/linter/source-code-fixer.js +2 -2
- package/lib/linter/timing.js +2 -2
- package/lib/options.js +29 -0
- package/lib/rule-tester/rule-tester.js +1 -1
- package/lib/rules/accessor-pairs.js +1 -1
- package/lib/rules/array-bracket-newline.js +1 -1
- package/lib/rules/array-bracket-spacing.js +1 -1
- package/lib/rules/array-callback-return.js +1 -1
- package/lib/rules/array-element-newline.js +1 -1
- package/lib/rules/arrow-body-style.js +1 -1
- package/lib/rules/arrow-parens.js +1 -1
- package/lib/rules/arrow-spacing.js +1 -1
- package/lib/rules/block-scoped-var.js +1 -1
- package/lib/rules/block-spacing.js +1 -1
- package/lib/rules/brace-style.js +1 -1
- package/lib/rules/callback-return.js +1 -1
- package/lib/rules/camelcase.js +1 -1
- package/lib/rules/capitalized-comments.js +1 -1
- package/lib/rules/class-methods-use-this.js +46 -2
- package/lib/rules/comma-dangle.js +1 -1
- package/lib/rules/comma-spacing.js +1 -1
- package/lib/rules/comma-style.js +1 -1
- package/lib/rules/complexity.js +1 -1
- package/lib/rules/computed-property-spacing.js +1 -1
- package/lib/rules/consistent-return.js +1 -1
- package/lib/rules/consistent-this.js +1 -1
- package/lib/rules/constructor-super.js +1 -1
- package/lib/rules/curly.js +1 -1
- package/lib/rules/default-case-last.js +1 -1
- package/lib/rules/default-case.js +1 -1
- package/lib/rules/default-param-last.js +1 -1
- package/lib/rules/dot-location.js +1 -1
- package/lib/rules/dot-notation.js +1 -1
- package/lib/rules/eol-last.js +1 -1
- package/lib/rules/eqeqeq.js +1 -1
- package/lib/rules/for-direction.js +5 -5
- package/lib/rules/func-call-spacing.js +1 -1
- package/lib/rules/func-name-matching.js +2 -2
- package/lib/rules/func-names.js +1 -1
- package/lib/rules/func-style.js +1 -1
- package/lib/rules/function-call-argument-newline.js +1 -1
- package/lib/rules/function-paren-newline.js +1 -1
- package/lib/rules/generator-star-spacing.js +1 -1
- package/lib/rules/getter-return.js +1 -1
- package/lib/rules/global-require.js +1 -1
- package/lib/rules/grouped-accessor-pairs.js +1 -1
- package/lib/rules/guard-for-in.js +1 -1
- package/lib/rules/handle-callback-err.js +1 -1
- package/lib/rules/id-blacklist.js +1 -1
- package/lib/rules/id-denylist.js +1 -1
- package/lib/rules/id-length.js +1 -1
- package/lib/rules/id-match.js +1 -1
- package/lib/rules/implicit-arrow-linebreak.js +1 -1
- package/lib/rules/indent-legacy.js +14 -14
- package/lib/rules/indent.js +4 -4
- package/lib/rules/init-declarations.js +22 -1
- package/lib/rules/jsx-quotes.js +1 -1
- package/lib/rules/key-spacing.js +3 -3
- package/lib/rules/keyword-spacing.js +1 -1
- package/lib/rules/line-comment-position.js +1 -1
- package/lib/rules/linebreak-style.js +10 -2
- package/lib/rules/lines-around-comment.js +1 -1
- package/lib/rules/lines-around-directive.js +1 -1
- package/lib/rules/lines-between-class-members.js +1 -1
- package/lib/rules/logical-assignment-operators.js +1 -1
- package/lib/rules/max-classes-per-file.js +1 -1
- package/lib/rules/max-depth.js +1 -1
- package/lib/rules/max-len.js +3 -3
- package/lib/rules/max-lines-per-function.js +1 -1
- package/lib/rules/max-lines.js +1 -1
- package/lib/rules/max-nested-callbacks.js +1 -1
- package/lib/rules/max-params.js +1 -1
- package/lib/rules/max-statements-per-line.js +1 -1
- package/lib/rules/max-statements.js +3 -3
- package/lib/rules/multiline-comment-style.js +1 -1
- package/lib/rules/multiline-ternary.js +1 -1
- package/lib/rules/new-cap.js +1 -1
- package/lib/rules/new-parens.js +1 -1
- package/lib/rules/newline-after-var.js +1 -1
- package/lib/rules/newline-before-return.js +1 -1
- package/lib/rules/newline-per-chained-call.js +1 -1
- package/lib/rules/no-alert.js +1 -1
- package/lib/rules/no-array-constructor.js +4 -1
- package/lib/rules/no-async-promise-executor.js +1 -1
- package/lib/rules/no-await-in-loop.js +1 -1
- package/lib/rules/no-bitwise.js +1 -1
- package/lib/rules/no-buffer-constructor.js +1 -1
- package/lib/rules/no-caller.js +1 -1
- package/lib/rules/no-case-declarations.js +1 -1
- package/lib/rules/no-catch-shadow.js +1 -1
- package/lib/rules/no-class-assign.js +1 -1
- package/lib/rules/no-compare-neg-zero.js +1 -1
- package/lib/rules/no-cond-assign.js +1 -1
- package/lib/rules/no-confusing-arrow.js +1 -1
- package/lib/rules/no-console.js +1 -1
- package/lib/rules/no-const-assign.js +1 -1
- package/lib/rules/no-constant-binary-expression.js +2 -2
- package/lib/rules/no-constant-condition.js +1 -1
- package/lib/rules/no-constructor-return.js +1 -1
- package/lib/rules/no-continue.js +1 -1
- package/lib/rules/no-control-regex.js +1 -1
- package/lib/rules/no-debugger.js +1 -1
- package/lib/rules/no-delete-var.js +1 -1
- package/lib/rules/no-div-regex.js +1 -1
- package/lib/rules/no-dupe-args.js +1 -1
- package/lib/rules/no-dupe-class-members.js +10 -1
- package/lib/rules/no-dupe-else-if.js +1 -1
- package/lib/rules/no-dupe-keys.js +1 -1
- package/lib/rules/no-duplicate-case.js +1 -1
- package/lib/rules/no-duplicate-imports.js +1 -1
- package/lib/rules/no-else-return.js +1 -1
- package/lib/rules/no-empty-character-class.js +1 -1
- package/lib/rules/no-empty-function.js +54 -3
- package/lib/rules/no-empty-pattern.js +1 -1
- package/lib/rules/no-empty-static-block.js +1 -1
- package/lib/rules/no-empty.js +1 -1
- package/lib/rules/no-eq-null.js +1 -1
- package/lib/rules/no-eval.js +1 -1
- package/lib/rules/no-ex-assign.js +1 -1
- package/lib/rules/no-extend-native.js +1 -1
- package/lib/rules/no-extra-bind.js +1 -1
- package/lib/rules/no-extra-boolean-cast.js +1 -1
- package/lib/rules/no-extra-label.js +1 -1
- package/lib/rules/no-extra-parens.js +1 -1
- package/lib/rules/no-extra-semi.js +1 -1
- package/lib/rules/no-fallthrough.js +1 -1
- package/lib/rules/no-floating-decimal.js +1 -1
- package/lib/rules/no-func-assign.js +1 -1
- package/lib/rules/no-global-assign.js +2 -2
- package/lib/rules/no-implicit-coercion.js +1 -1
- package/lib/rules/no-implicit-globals.js +1 -1
- package/lib/rules/no-implied-eval.js +1 -1
- package/lib/rules/no-import-assign.js +1 -1
- package/lib/rules/no-inline-comments.js +1 -1
- package/lib/rules/no-inner-declarations.js +1 -1
- package/lib/rules/no-invalid-regexp.js +1 -1
- package/lib/rules/no-invalid-this.js +23 -1
- package/lib/rules/no-irregular-whitespace.js +1 -1
- package/lib/rules/no-iterator.js +1 -1
- package/lib/rules/no-label-var.js +1 -1
- package/lib/rules/no-labels.js +1 -1
- package/lib/rules/no-lone-blocks.js +1 -1
- package/lib/rules/no-lonely-if.js +1 -1
- package/lib/rules/no-loop-func.js +10 -4
- package/lib/rules/no-loss-of-precision.js +3 -1
- package/lib/rules/no-magic-numbers.js +1 -1
- package/lib/rules/no-misleading-character-class.js +1 -1
- package/lib/rules/no-mixed-operators.js +1 -1
- package/lib/rules/no-mixed-requires.js +1 -1
- package/lib/rules/no-mixed-spaces-and-tabs.js +1 -1
- package/lib/rules/no-multi-assign.js +1 -1
- package/lib/rules/no-multi-spaces.js +1 -1
- package/lib/rules/no-multi-str.js +1 -1
- package/lib/rules/no-multiple-empty-lines.js +1 -1
- package/lib/rules/no-native-reassign.js +2 -2
- package/lib/rules/no-negated-condition.js +1 -1
- package/lib/rules/no-negated-in-lhs.js +1 -1
- package/lib/rules/no-nested-ternary.js +1 -1
- package/lib/rules/no-new-func.js +1 -1
- package/lib/rules/no-new-native-nonconstructor.js +1 -1
- package/lib/rules/no-new-object.js +1 -1
- package/lib/rules/no-new-require.js +1 -1
- package/lib/rules/no-new-symbol.js +1 -1
- package/lib/rules/no-new-wrappers.js +1 -1
- package/lib/rules/no-new.js +1 -1
- package/lib/rules/no-nonoctal-decimal-escape.js +10 -2
- package/lib/rules/no-obj-calls.js +1 -1
- package/lib/rules/no-object-constructor.js +1 -1
- package/lib/rules/no-octal-escape.js +1 -1
- package/lib/rules/no-octal.js +1 -1
- package/lib/rules/no-param-reassign.js +2 -2
- package/lib/rules/no-path-concat.js +1 -1
- package/lib/rules/no-plusplus.js +1 -1
- package/lib/rules/no-process-env.js +1 -1
- package/lib/rules/no-process-exit.js +1 -1
- package/lib/rules/no-promise-executor-return.js +1 -1
- package/lib/rules/no-proto.js +1 -1
- package/lib/rules/no-prototype-builtins.js +1 -1
- package/lib/rules/no-redeclare.js +1 -1
- package/lib/rules/no-regex-spaces.js +1 -1
- package/lib/rules/no-restricted-exports.js +1 -1
- package/lib/rules/no-restricted-globals.js +1 -1
- package/lib/rules/no-restricted-imports.js +1 -1
- package/lib/rules/no-restricted-modules.js +1 -1
- package/lib/rules/no-restricted-properties.js +43 -29
- package/lib/rules/no-restricted-syntax.js +1 -1
- package/lib/rules/no-return-assign.js +1 -1
- package/lib/rules/no-return-await.js +1 -2
- package/lib/rules/no-script-url.js +1 -1
- package/lib/rules/no-self-assign.js +1 -1
- package/lib/rules/no-self-compare.js +1 -1
- package/lib/rules/no-sequences.js +1 -1
- package/lib/rules/no-setter-return.js +1 -1
- package/lib/rules/no-shadow-restricted-names.js +1 -1
- package/lib/rules/no-shadow.js +1 -1
- package/lib/rules/no-spaced-func.js +1 -1
- package/lib/rules/no-sparse-arrays.js +1 -1
- package/lib/rules/no-sync.js +1 -1
- package/lib/rules/no-tabs.js +1 -1
- package/lib/rules/no-template-curly-in-string.js +1 -1
- package/lib/rules/no-ternary.js +1 -1
- package/lib/rules/no-this-before-super.js +1 -1
- package/lib/rules/no-throw-literal.js +1 -1
- package/lib/rules/no-trailing-spaces.js +11 -3
- package/lib/rules/no-undef-init.js +1 -1
- package/lib/rules/no-undef.js +1 -1
- package/lib/rules/no-undefined.js +1 -1
- package/lib/rules/no-underscore-dangle.js +1 -1
- package/lib/rules/no-unexpected-multiline.js +1 -1
- package/lib/rules/no-unmodified-loop-condition.js +1 -1
- package/lib/rules/no-unneeded-ternary.js +1 -1
- package/lib/rules/no-unreachable-loop.js +1 -1
- package/lib/rules/no-unreachable.js +1 -1
- package/lib/rules/no-unsafe-finally.js +1 -1
- package/lib/rules/no-unsafe-negation.js +1 -1
- package/lib/rules/no-unsafe-optional-chaining.js +1 -1
- package/lib/rules/no-unused-expressions.js +15 -57
- package/lib/rules/no-unused-labels.js +1 -1
- package/lib/rules/no-unused-private-class-members.js +1 -1
- package/lib/rules/no-unused-vars.js +3 -3
- package/lib/rules/no-use-before-define.js +1 -1
- package/lib/rules/no-useless-assignment.js +1 -1
- package/lib/rules/no-useless-backreference.js +1 -1
- package/lib/rules/no-useless-call.js +1 -1
- package/lib/rules/no-useless-catch.js +1 -1
- package/lib/rules/no-useless-computed-key.js +1 -1
- package/lib/rules/no-useless-concat.js +1 -1
- package/lib/rules/no-useless-constructor.js +1 -1
- package/lib/rules/no-useless-escape.js +1 -1
- package/lib/rules/no-useless-rename.js +1 -1
- package/lib/rules/no-useless-return.js +1 -1
- package/lib/rules/no-var.js +1 -1
- package/lib/rules/no-void.js +1 -1
- package/lib/rules/no-warning-comments.js +1 -1
- package/lib/rules/no-whitespace-before-property.js +1 -1
- package/lib/rules/no-with.js +1 -1
- package/lib/rules/nonblock-statement-body-position.js +1 -1
- package/lib/rules/object-curly-newline.js +1 -1
- package/lib/rules/object-curly-spacing.js +1 -1
- package/lib/rules/object-property-newline.js +1 -1
- package/lib/rules/object-shorthand.js +1 -1
- package/lib/rules/one-var-declaration-per-line.js +1 -1
- package/lib/rules/one-var.js +1 -1
- package/lib/rules/operator-assignment.js +1 -1
- package/lib/rules/operator-linebreak.js +1 -1
- package/lib/rules/padded-blocks.js +1 -1
- package/lib/rules/padding-line-between-statements.js +1 -1
- package/lib/rules/prefer-arrow-callback.js +1 -1
- package/lib/rules/prefer-const.js +1 -1
- package/lib/rules/prefer-destructuring.js +1 -1
- package/lib/rules/prefer-exponentiation-operator.js +1 -1
- package/lib/rules/prefer-named-capture-group.js +1 -1
- package/lib/rules/prefer-numeric-literals.js +1 -1
- package/lib/rules/prefer-object-has-own.js +1 -1
- package/lib/rules/prefer-object-spread.js +1 -1
- package/lib/rules/prefer-promise-reject-errors.js +1 -1
- package/lib/rules/prefer-reflect.js +1 -2
- package/lib/rules/prefer-regex-literals.js +1 -1
- package/lib/rules/prefer-rest-params.js +1 -1
- package/lib/rules/prefer-spread.js +1 -1
- package/lib/rules/prefer-template.js +1 -1
- package/lib/rules/quote-props.js +1 -1
- package/lib/rules/quotes.js +1 -1
- package/lib/rules/radix.js +1 -1
- package/lib/rules/require-atomic-updates.js +1 -1
- package/lib/rules/require-await.js +1 -1
- package/lib/rules/require-unicode-regexp.js +1 -1
- package/lib/rules/require-yield.js +1 -1
- package/lib/rules/rest-spread-spacing.js +1 -1
- package/lib/rules/semi-spacing.js +1 -1
- package/lib/rules/semi-style.js +1 -1
- package/lib/rules/semi.js +1 -1
- package/lib/rules/sort-imports.js +1 -1
- package/lib/rules/sort-keys.js +1 -1
- package/lib/rules/sort-vars.js +1 -1
- package/lib/rules/space-before-blocks.js +1 -1
- package/lib/rules/space-before-function-paren.js +1 -1
- package/lib/rules/space-in-parens.js +1 -1
- package/lib/rules/space-infix-ops.js +1 -1
- package/lib/rules/space-unary-ops.js +1 -1
- package/lib/rules/spaced-comment.js +1 -1
- package/lib/rules/strict.js +1 -1
- package/lib/rules/switch-colon-spacing.js +1 -1
- package/lib/rules/symbol-description.js +1 -1
- package/lib/rules/template-curly-spacing.js +1 -1
- package/lib/rules/template-tag-spacing.js +1 -1
- package/lib/rules/unicode-bom.js +1 -1
- package/lib/rules/use-isnan.js +1 -1
- package/lib/rules/utils/ast-utils.js +13 -2
- package/lib/rules/utils/fix-tracker.js +10 -2
- package/lib/rules/valid-typeof.js +1 -1
- package/lib/rules/vars-on-top.js +1 -1
- package/lib/rules/wrap-iife.js +1 -1
- package/lib/rules/wrap-regex.js +1 -1
- package/lib/rules/yield-star-spacing.js +1 -1
- package/lib/rules/yoda.js +1 -1
- package/lib/services/suppressions-service.js +289 -0
- package/lib/shared/flags.js +4 -0
- package/lib/shared/types.js +4 -27
- package/lib/types/index.d.ts +30 -7
- package/lib/types/rules.d.ts +14 -0
- package/package.json +9 -9
package/lib/rules/new-cap.js
CHANGED
@@ -57,7 +57,7 @@ function calculateCapIsNewExceptions(config) {
|
|
57
57
|
// Rule Definition
|
58
58
|
//------------------------------------------------------------------------------
|
59
59
|
|
60
|
-
/** @type {import('../
|
60
|
+
/** @type {import('../types').Rule.RuleModule} */
|
61
61
|
module.exports = {
|
62
62
|
meta: {
|
63
63
|
type: "suggestion",
|
package/lib/rules/new-parens.js
CHANGED
@@ -20,7 +20,7 @@ const astUtils = require("./utils/ast-utils");
|
|
20
20
|
// Rule Definition
|
21
21
|
//------------------------------------------------------------------------------
|
22
22
|
|
23
|
-
/** @type {import('../
|
23
|
+
/** @type {import('../types').Rule.RuleModule} */
|
24
24
|
module.exports = {
|
25
25
|
meta: {
|
26
26
|
deprecated: {
|
@@ -16,7 +16,7 @@ const astUtils = require("./utils/ast-utils");
|
|
16
16
|
// Rule Definition
|
17
17
|
//------------------------------------------------------------------------------
|
18
18
|
|
19
|
-
/** @type {import('../
|
19
|
+
/** @type {import('../types').Rule.RuleModule} */
|
20
20
|
module.exports = {
|
21
21
|
meta: {
|
22
22
|
type: "layout",
|
@@ -13,7 +13,7 @@ const astUtils = require("./utils/ast-utils");
|
|
13
13
|
// Rule Definition
|
14
14
|
//------------------------------------------------------------------------------
|
15
15
|
|
16
|
-
/** @type {import('../
|
16
|
+
/** @type {import('../types').Rule.RuleModule} */
|
17
17
|
module.exports = {
|
18
18
|
meta: {
|
19
19
|
deprecated: {
|
package/lib/rules/no-alert.js
CHANGED
@@ -86,7 +86,7 @@ function isGlobalThisReferenceOrGlobalWindow(scope, node) {
|
|
86
86
|
// Rule Definition
|
87
87
|
//------------------------------------------------------------------------------
|
88
88
|
|
89
|
-
/** @type {import('../
|
89
|
+
/** @type {import('../types').Rule.RuleModule} */
|
90
90
|
module.exports = {
|
91
91
|
meta: {
|
92
92
|
type: "suggestion",
|
@@ -21,9 +21,11 @@ const {
|
|
21
21
|
// Rule Definition
|
22
22
|
//------------------------------------------------------------------------------
|
23
23
|
|
24
|
-
/** @type {import('../
|
24
|
+
/** @type {import('../types').Rule.RuleModule} */
|
25
25
|
module.exports = {
|
26
26
|
meta: {
|
27
|
+
dialects: ["javascript", "typescript"],
|
28
|
+
language: "javascript",
|
27
29
|
type: "suggestion",
|
28
30
|
|
29
31
|
docs: {
|
@@ -84,6 +86,7 @@ module.exports = {
|
|
84
86
|
if (
|
85
87
|
node.callee.type !== "Identifier" ||
|
86
88
|
node.callee.name !== "Array" ||
|
89
|
+
node.typeArguments ||
|
87
90
|
(node.arguments.length === 1 &&
|
88
91
|
node.arguments[0].type !== "SpreadElement")
|
89
92
|
) {
|
package/lib/rules/no-bitwise.js
CHANGED
@@ -30,7 +30,7 @@ const BITWISE_OPERATORS = [
|
|
30
30
|
// Rule Definition
|
31
31
|
//------------------------------------------------------------------------------
|
32
32
|
|
33
|
-
/** @type {import('../
|
33
|
+
/** @type {import('../types').Rule.RuleModule} */
|
34
34
|
module.exports = {
|
35
35
|
meta: {
|
36
36
|
type: "suggestion",
|
package/lib/rules/no-caller.js
CHANGED
@@ -9,7 +9,7 @@
|
|
9
9
|
// Rule Definition
|
10
10
|
//------------------------------------------------------------------------------
|
11
11
|
|
12
|
-
/** @type {import('../
|
12
|
+
/** @type {import('../types').Rule.RuleModule} */
|
13
13
|
module.exports = {
|
14
14
|
meta: {
|
15
15
|
type: "suggestion",
|
@@ -8,7 +8,7 @@
|
|
8
8
|
// Rule Definition
|
9
9
|
//------------------------------------------------------------------------------
|
10
10
|
|
11
|
-
/** @type {import('../
|
11
|
+
/** @type {import('../types').Rule.RuleModule} */
|
12
12
|
module.exports = {
|
13
13
|
meta: {
|
14
14
|
type: "suggestion",
|
@@ -16,7 +16,7 @@ const astUtils = require("./utils/ast-utils");
|
|
16
16
|
// Rule Definition
|
17
17
|
//------------------------------------------------------------------------------
|
18
18
|
|
19
|
-
/** @type {import('../
|
19
|
+
/** @type {import('../types').Rule.RuleModule} */
|
20
20
|
module.exports = {
|
21
21
|
meta: {
|
22
22
|
type: "suggestion",
|
@@ -11,7 +11,7 @@ const astUtils = require("./utils/ast-utils");
|
|
11
11
|
// Rule Definition
|
12
12
|
//------------------------------------------------------------------------------
|
13
13
|
|
14
|
-
/** @type {import('../
|
14
|
+
/** @type {import('../types').Rule.RuleModule} */
|
15
15
|
module.exports = {
|
16
16
|
meta: {
|
17
17
|
type: "problem",
|
@@ -34,7 +34,7 @@ const NODE_DESCRIPTIONS = {
|
|
34
34
|
// Rule Definition
|
35
35
|
//------------------------------------------------------------------------------
|
36
36
|
|
37
|
-
/** @type {import('../
|
37
|
+
/** @type {import('../types').Rule.RuleModule} */
|
38
38
|
module.exports = {
|
39
39
|
meta: {
|
40
40
|
type: "problem",
|
@@ -26,7 +26,7 @@ function isConditional(node) {
|
|
26
26
|
// Rule Definition
|
27
27
|
//------------------------------------------------------------------------------
|
28
28
|
|
29
|
-
/** @type {import('../
|
29
|
+
/** @type {import('../types').Rule.RuleModule} */
|
30
30
|
module.exports = {
|
31
31
|
meta: {
|
32
32
|
deprecated: {
|
package/lib/rules/no-console.js
CHANGED
@@ -15,7 +15,7 @@ const astUtils = require("./utils/ast-utils");
|
|
15
15
|
// Rule Definition
|
16
16
|
//------------------------------------------------------------------------------
|
17
17
|
|
18
|
-
/** @type {import('../
|
18
|
+
/** @type {import('../types').Rule.RuleModule} */
|
19
19
|
module.exports = {
|
20
20
|
meta: {
|
21
21
|
type: "suggestion",
|
@@ -11,7 +11,7 @@ const astUtils = require("./utils/ast-utils");
|
|
11
11
|
// Rule Definition
|
12
12
|
//------------------------------------------------------------------------------
|
13
13
|
|
14
|
-
/** @type {import('../
|
14
|
+
/** @type {import('../types').Rule.RuleModule} */
|
15
15
|
module.exports = {
|
16
16
|
meta: {
|
17
17
|
type: "problem",
|
@@ -489,7 +489,7 @@ function findBinaryExpressionConstantOperand(scope, a, b, operator) {
|
|
489
489
|
// Rule Definition
|
490
490
|
//------------------------------------------------------------------------------
|
491
491
|
|
492
|
-
/** @type {import('../
|
492
|
+
/** @type {import('../types').Rule.RuleModule} */
|
493
493
|
module.exports = {
|
494
494
|
meta: {
|
495
495
|
type: "problem",
|
@@ -594,7 +594,7 @@ module.exports = {
|
|
594
594
|
* In theory we could handle short-circuiting assignment operators,
|
595
595
|
* for some constant values, but that would require walking the
|
596
596
|
* scope to find the value of the variable being assigned. This is
|
597
|
-
*
|
597
|
+
* dependent on https://github.com/eslint/eslint/issues/13776
|
598
598
|
*
|
599
599
|
* AssignmentExpression() {},
|
600
600
|
*/
|
@@ -15,7 +15,7 @@ const { isConstant } = require("./utils/ast-utils");
|
|
15
15
|
// Rule Definition
|
16
16
|
//------------------------------------------------------------------------------
|
17
17
|
|
18
|
-
/** @type {import('../
|
18
|
+
/** @type {import('../types').Rule.RuleModule} */
|
19
19
|
module.exports = {
|
20
20
|
meta: {
|
21
21
|
type: "problem",
|
package/lib/rules/no-continue.js
CHANGED
@@ -9,7 +9,7 @@
|
|
9
9
|
// Rule Definition
|
10
10
|
//------------------------------------------------------------------------------
|
11
11
|
|
12
|
-
/** @type {import('../
|
12
|
+
/** @type {import('../types').Rule.RuleModule} */
|
13
13
|
module.exports = {
|
14
14
|
meta: {
|
15
15
|
type: "suggestion",
|
@@ -61,7 +61,7 @@ const collector = new (class {
|
|
61
61
|
// Rule Definition
|
62
62
|
//------------------------------------------------------------------------------
|
63
63
|
|
64
|
-
/** @type {import('../
|
64
|
+
/** @type {import('../types').Rule.RuleModule} */
|
65
65
|
module.exports = {
|
66
66
|
meta: {
|
67
67
|
type: "problem",
|
package/lib/rules/no-debugger.js
CHANGED
@@ -9,7 +9,7 @@
|
|
9
9
|
// Rule Definition
|
10
10
|
//------------------------------------------------------------------------------
|
11
11
|
|
12
|
-
/** @type {import('../
|
12
|
+
/** @type {import('../types').Rule.RuleModule} */
|
13
13
|
module.exports = {
|
14
14
|
meta: {
|
15
15
|
type: "suggestion",
|
@@ -9,7 +9,7 @@
|
|
9
9
|
// Rule Definition
|
10
10
|
//------------------------------------------------------------------------------
|
11
11
|
|
12
|
-
/** @type {import('../
|
12
|
+
/** @type {import('../types').Rule.RuleModule} */
|
13
13
|
module.exports = {
|
14
14
|
meta: {
|
15
15
|
type: "suggestion",
|
@@ -11,10 +11,12 @@ const astUtils = require("./utils/ast-utils");
|
|
11
11
|
// Rule Definition
|
12
12
|
//------------------------------------------------------------------------------
|
13
13
|
|
14
|
-
/** @type {import('../
|
14
|
+
/** @type {import('../types').Rule.RuleModule} */
|
15
15
|
module.exports = {
|
16
16
|
meta: {
|
17
17
|
type: "problem",
|
18
|
+
dialects: ["javascript", "typescript"],
|
19
|
+
language: "javascript",
|
18
20
|
|
19
21
|
docs: {
|
20
22
|
description: "Disallow duplicate class members",
|
@@ -73,6 +75,13 @@ module.exports = {
|
|
73
75
|
|
74
76
|
// Reports the node if its name has been declared already.
|
75
77
|
"MethodDefinition, PropertyDefinition"(node) {
|
78
|
+
if (
|
79
|
+
node.value &&
|
80
|
+
node.value.type === "TSEmptyBodyFunctionExpression"
|
81
|
+
) {
|
82
|
+
return;
|
83
|
+
}
|
84
|
+
|
76
85
|
const name = astUtils.getStaticPropertyName(node);
|
77
86
|
const kind =
|
78
87
|
node.type === "MethodDefinition" ? node.kind : "field";
|
@@ -49,7 +49,7 @@ const splitByAnd = splitByLogicalOperator.bind(null, "&&");
|
|
49
49
|
// Rule Definition
|
50
50
|
//------------------------------------------------------------------------------
|
51
51
|
|
52
|
-
/** @type {import('../
|
52
|
+
/** @type {import('../types').Rule.RuleModule} */
|
53
53
|
module.exports = {
|
54
54
|
meta: {
|
55
55
|
type: "problem",
|
@@ -81,7 +81,7 @@ class ObjectInfo {
|
|
81
81
|
// Rule Definition
|
82
82
|
//------------------------------------------------------------------------------
|
83
83
|
|
84
|
-
/** @type {import('../
|
84
|
+
/** @type {import('../types').Rule.RuleModule} */
|
85
85
|
module.exports = {
|
86
86
|
meta: {
|
87
87
|
type: "problem",
|
@@ -16,7 +16,7 @@ const astUtils = require("./utils/ast-utils");
|
|
16
16
|
// Rule Definition
|
17
17
|
//------------------------------------------------------------------------------
|
18
18
|
|
19
|
-
/** @type {import('../
|
19
|
+
/** @type {import('../types').Rule.RuleModule} */
|
20
20
|
module.exports = {
|
21
21
|
meta: {
|
22
22
|
type: "problem",
|
@@ -16,7 +16,7 @@ const FixTracker = require("./utils/fix-tracker");
|
|
16
16
|
// Rule Definition
|
17
17
|
//------------------------------------------------------------------------------
|
18
18
|
|
19
|
-
/** @type {import('../
|
19
|
+
/** @type {import('../types').Rule.RuleModule} */
|
20
20
|
module.exports = {
|
21
21
|
meta: {
|
22
22
|
type: "suggestion",
|
@@ -22,7 +22,7 @@ const QUICK_TEST_REGEX = /\[\]/u;
|
|
22
22
|
// Rule Definition
|
23
23
|
//------------------------------------------------------------------------------
|
24
24
|
|
25
|
-
/** @type {import('../
|
25
|
+
/** @type {import('../types').Rule.RuleModule} */
|
26
26
|
module.exports = {
|
27
27
|
meta: {
|
28
28
|
type: "problem",
|
@@ -26,6 +26,10 @@ const ALLOW_OPTIONS = Object.freeze([
|
|
26
26
|
"constructors",
|
27
27
|
"asyncFunctions",
|
28
28
|
"asyncMethods",
|
29
|
+
"privateConstructors",
|
30
|
+
"protectedConstructors",
|
31
|
+
"decoratedFunctions",
|
32
|
+
"overrideMethods",
|
29
33
|
]);
|
30
34
|
|
31
35
|
/**
|
@@ -83,13 +87,24 @@ function getKind(node) {
|
|
83
87
|
return prefix + kind[0].toUpperCase() + kind.slice(1);
|
84
88
|
}
|
85
89
|
|
90
|
+
/**
|
91
|
+
* Checks if a constructor function has parameter properties.
|
92
|
+
* @param {ASTNode} node The function node to examine.
|
93
|
+
* @returns {boolean} True if the constructor has parameter properties, false otherwise.
|
94
|
+
*/
|
95
|
+
function isParameterPropertiesConstructor(node) {
|
96
|
+
return node.params.some(param => param.type === "TSParameterProperty");
|
97
|
+
}
|
98
|
+
|
86
99
|
//------------------------------------------------------------------------------
|
87
100
|
// Rule Definition
|
88
101
|
//------------------------------------------------------------------------------
|
89
102
|
|
90
|
-
/** @type {import('../
|
103
|
+
/** @type {import('../types').Rule.RuleModule} */
|
91
104
|
module.exports = {
|
92
105
|
meta: {
|
106
|
+
dialects: ["javascript", "typescript"],
|
107
|
+
language: "javascript",
|
93
108
|
type: "suggestion",
|
94
109
|
|
95
110
|
defaultOptions: [{ allow: [] }],
|
@@ -123,6 +138,43 @@ module.exports = {
|
|
123
138
|
const [{ allow }] = context.options;
|
124
139
|
const sourceCode = context.sourceCode;
|
125
140
|
|
141
|
+
/**
|
142
|
+
* Checks if the given function node is allowed to be empty.
|
143
|
+
* @param {ASTNode} node The function node to check.
|
144
|
+
* @returns {boolean} True if the function is allowed to be empty, false otherwise.
|
145
|
+
*/
|
146
|
+
function isAllowedEmptyFunction(node) {
|
147
|
+
const kind = getKind(node);
|
148
|
+
|
149
|
+
if (allow.includes(kind)) {
|
150
|
+
return true;
|
151
|
+
}
|
152
|
+
|
153
|
+
if (kind === "constructors") {
|
154
|
+
if (
|
155
|
+
(node.parent.accessibility === "private" &&
|
156
|
+
allow.includes("privateConstructors")) ||
|
157
|
+
(node.parent.accessibility === "protected" &&
|
158
|
+
allow.includes("protectedConstructors")) ||
|
159
|
+
isParameterPropertiesConstructor(node)
|
160
|
+
) {
|
161
|
+
return true;
|
162
|
+
}
|
163
|
+
}
|
164
|
+
|
165
|
+
if (/(g|s)etters|methods$/iu.test(kind)) {
|
166
|
+
if (
|
167
|
+
(node.parent.decorators?.length &&
|
168
|
+
allow.includes("decoratedFunctions")) ||
|
169
|
+
(node.parent.override && allow.includes("overrideMethods"))
|
170
|
+
) {
|
171
|
+
return true;
|
172
|
+
}
|
173
|
+
}
|
174
|
+
|
175
|
+
return false;
|
176
|
+
}
|
177
|
+
|
126
178
|
/**
|
127
179
|
* Reports a given function node if the node matches the following patterns.
|
128
180
|
*
|
@@ -135,7 +187,6 @@ module.exports = {
|
|
135
187
|
* @returns {void}
|
136
188
|
*/
|
137
189
|
function reportIfEmpty(node) {
|
138
|
-
const kind = getKind(node);
|
139
190
|
const name = astUtils.getFunctionNameWithKind(node);
|
140
191
|
const innerComments = sourceCode.getTokens(node.body, {
|
141
192
|
includeComments: true,
|
@@ -143,7 +194,7 @@ module.exports = {
|
|
143
194
|
});
|
144
195
|
|
145
196
|
if (
|
146
|
-
!
|
197
|
+
!isAllowedEmptyFunction(node) &&
|
147
198
|
node.body.type === "BlockStatement" &&
|
148
199
|
node.body.body.length === 0 &&
|
149
200
|
innerComments.length === 0
|
@@ -10,7 +10,7 @@ const astUtils = require("./utils/ast-utils");
|
|
10
10
|
// Rule Definition
|
11
11
|
//------------------------------------------------------------------------------
|
12
12
|
|
13
|
-
/** @type {import('../
|
13
|
+
/** @type {import('../types').Rule.RuleModule} */
|
14
14
|
module.exports = {
|
15
15
|
meta: {
|
16
16
|
type: "problem",
|
@@ -8,7 +8,7 @@
|
|
8
8
|
// Rule Definition
|
9
9
|
//------------------------------------------------------------------------------
|
10
10
|
|
11
|
-
/** @type {import('../
|
11
|
+
/** @type {import('../types').Rule.RuleModule} */
|
12
12
|
module.exports = {
|
13
13
|
meta: {
|
14
14
|
type: "suggestion",
|
package/lib/rules/no-empty.js
CHANGED
@@ -14,7 +14,7 @@ const astUtils = require("./utils/ast-utils");
|
|
14
14
|
// Rule Definition
|
15
15
|
//------------------------------------------------------------------------------
|
16
16
|
|
17
|
-
/** @type {import('../
|
17
|
+
/** @type {import('../types').Rule.RuleModule} */
|
18
18
|
module.exports = {
|
19
19
|
meta: {
|
20
20
|
hasSuggestions: true,
|
package/lib/rules/no-eq-null.js
CHANGED
@@ -10,7 +10,7 @@
|
|
10
10
|
// Rule Definition
|
11
11
|
//------------------------------------------------------------------------------
|
12
12
|
|
13
|
-
/** @type {import('../
|
13
|
+
/** @type {import('../types').Rule.RuleModule} */
|
14
14
|
module.exports = {
|
15
15
|
meta: {
|
16
16
|
type: "suggestion",
|
package/lib/rules/no-eval.js
CHANGED
@@ -37,7 +37,7 @@ function isMember(node, name) {
|
|
37
37
|
// Rule Definition
|
38
38
|
//------------------------------------------------------------------------------
|
39
39
|
|
40
|
-
/** @type {import('../
|
40
|
+
/** @type {import('../types').Rule.RuleModule} */
|
41
41
|
module.exports = {
|
42
42
|
meta: {
|
43
43
|
type: "suggestion",
|
@@ -11,7 +11,7 @@ const astUtils = require("./utils/ast-utils");
|
|
11
11
|
// Rule Definition
|
12
12
|
//------------------------------------------------------------------------------
|
13
13
|
|
14
|
-
/** @type {import('../
|
14
|
+
/** @type {import('../types').Rule.RuleModule} */
|
15
15
|
module.exports = {
|
16
16
|
meta: {
|
17
17
|
type: "problem",
|
@@ -15,7 +15,7 @@ const astUtils = require("./utils/ast-utils");
|
|
15
15
|
// Rule Definition
|
16
16
|
//------------------------------------------------------------------------------
|
17
17
|
|
18
|
-
/** @type {import('../
|
18
|
+
/** @type {import('../types').Rule.RuleModule} */
|
19
19
|
module.exports = {
|
20
20
|
meta: {
|
21
21
|
type: "suggestion",
|
@@ -25,7 +25,7 @@ const SIDE_EFFECT_FREE_NODE_TYPES = new Set([
|
|
25
25
|
// Rule Definition
|
26
26
|
//------------------------------------------------------------------------------
|
27
27
|
|
28
|
-
/** @type {import('../
|
28
|
+
/** @type {import('../types').Rule.RuleModule} */
|
29
29
|
module.exports = {
|
30
30
|
meta: {
|
31
31
|
type: "suggestion",
|
@@ -18,7 +18,7 @@ const precedence = astUtils.getPrecedence;
|
|
18
18
|
// Rule Definition
|
19
19
|
//------------------------------------------------------------------------------
|
20
20
|
|
21
|
-
/** @type {import('../
|
21
|
+
/** @type {import('../types').Rule.RuleModule} */
|
22
22
|
module.exports = {
|
23
23
|
meta: {
|
24
24
|
type: "suggestion",
|
@@ -15,7 +15,7 @@ const astUtils = require("./utils/ast-utils");
|
|
15
15
|
// Rule Definition
|
16
16
|
//------------------------------------------------------------------------------
|
17
17
|
|
18
|
-
/** @type {import('../
|
18
|
+
/** @type {import('../types').Rule.RuleModule} */
|
19
19
|
module.exports = {
|
20
20
|
meta: {
|
21
21
|
type: "suggestion",
|
@@ -14,7 +14,7 @@ const {
|
|
14
14
|
} = require("@eslint-community/eslint-utils");
|
15
15
|
const astUtils = require("./utils/ast-utils.js");
|
16
16
|
|
17
|
-
/** @type {import('../
|
17
|
+
/** @type {import('../types').Rule.RuleModule} */
|
18
18
|
module.exports = {
|
19
19
|
meta: {
|
20
20
|
deprecated: {
|
@@ -17,7 +17,7 @@ const astUtils = require("./utils/ast-utils");
|
|
17
17
|
// Rule Definition
|
18
18
|
//------------------------------------------------------------------------------
|
19
19
|
|
20
|
-
/** @type {import('../
|
20
|
+
/** @type {import('../types').Rule.RuleModule} */
|
21
21
|
module.exports = {
|
22
22
|
meta: {
|
23
23
|
deprecated: {
|
@@ -108,7 +108,7 @@ function hasBlankLinesBetween(node, token) {
|
|
108
108
|
// Rule Definition
|
109
109
|
//------------------------------------------------------------------------------
|
110
110
|
|
111
|
-
/** @type {import('../
|
111
|
+
/** @type {import('../types').Rule.RuleModule} */
|
112
112
|
module.exports = {
|
113
113
|
meta: {
|
114
114
|
type: "problem",
|