eslint 9.22.0 → 9.23.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 +47 -45
- package/bin/eslint.js +92 -90
- package/conf/default-cli-options.js +22 -22
- package/conf/ecma-version.js +1 -1
- package/conf/globals.js +97 -98
- package/conf/replacements.json +24 -20
- package/conf/rule-type-list.json +88 -92
- package/lib/api.js +12 -12
- package/lib/cli-engine/cli-engine.js +828 -808
- package/lib/cli-engine/file-enumerator.js +381 -387
- package/lib/cli-engine/formatters/formatters-meta.json +16 -16
- package/lib/cli-engine/formatters/html.js +107 -99
- package/lib/cli-engine/formatters/json-with-metadata.js +5 -5
- package/lib/cli-engine/formatters/json.js +2 -2
- package/lib/cli-engine/formatters/stylish.js +96 -75
- package/lib/cli-engine/hash.js +1 -1
- package/lib/cli-engine/index.js +1 -1
- package/lib/cli-engine/lint-result-cache.js +144 -145
- package/lib/cli-engine/load-rules.js +16 -16
- package/lib/cli.js +541 -457
- package/lib/config/config-loader.js +648 -618
- package/lib/config/config.js +247 -221
- package/lib/config/default-config.js +54 -45
- package/lib/config/flat-config-array.js +167 -172
- package/lib/config/flat-config-helpers.js +65 -68
- package/lib/config/flat-config-schema.js +375 -368
- package/lib/config/rule-validator.js +139 -144
- package/lib/config-api.js +2 -2
- package/lib/eslint/eslint-helpers.js +709 -679
- package/lib/eslint/eslint.js +944 -886
- package/lib/eslint/index.js +2 -2
- package/lib/eslint/legacy-eslint.js +576 -532
- package/lib/languages/js/index.js +263 -264
- package/lib/languages/js/source-code/index.js +1 -1
- package/lib/languages/js/source-code/source-code.js +1128 -1057
- package/lib/languages/js/source-code/token-store/backward-token-comment-cursor.js +39 -35
- package/lib/languages/js/source-code/token-store/backward-token-cursor.js +35 -36
- package/lib/languages/js/source-code/token-store/cursor.js +36 -36
- package/lib/languages/js/source-code/token-store/cursors.js +80 -52
- package/lib/languages/js/source-code/token-store/decorative-cursor.js +17 -18
- package/lib/languages/js/source-code/token-store/filter-cursor.js +19 -20
- package/lib/languages/js/source-code/token-store/forward-token-comment-cursor.js +40 -32
- package/lib/languages/js/source-code/token-store/forward-token-cursor.js +40 -41
- package/lib/languages/js/source-code/token-store/index.js +592 -498
- package/lib/languages/js/source-code/token-store/limit-cursor.js +17 -18
- package/lib/languages/js/source-code/token-store/padded-token-cursor.js +23 -16
- package/lib/languages/js/source-code/token-store/skip-cursor.js +19 -20
- package/lib/languages/js/source-code/token-store/utils.js +63 -60
- package/lib/languages/js/validate-language-options.js +104 -89
- package/lib/linter/apply-disable-directives.js +467 -383
- package/lib/linter/code-path-analysis/code-path-analyzer.js +650 -672
- package/lib/linter/code-path-analysis/code-path-segment.js +215 -216
- package/lib/linter/code-path-analysis/code-path-state.js +2118 -2096
- package/lib/linter/code-path-analysis/code-path.js +307 -319
- package/lib/linter/code-path-analysis/debug-helpers.js +183 -163
- package/lib/linter/code-path-analysis/fork-context.js +296 -271
- package/lib/linter/code-path-analysis/id-generator.js +22 -23
- package/lib/linter/file-context.js +119 -120
- package/lib/linter/index.js +3 -3
- package/lib/linter/interpolate.js +16 -16
- package/lib/linter/linter.js +2402 -2044
- package/lib/linter/node-event-generator.js +284 -225
- package/lib/linter/report-translator.js +256 -219
- package/lib/linter/rule-fixer.js +122 -124
- package/lib/linter/rules.js +35 -35
- package/lib/linter/safe-emitter.js +18 -18
- package/lib/linter/source-code-fixer.js +94 -92
- package/lib/linter/timing.js +104 -101
- package/lib/linter/vfile.js +70 -73
- package/lib/options.js +375 -361
- package/lib/rule-tester/index.js +1 -1
- package/lib/rule-tester/rule-tester.js +1307 -1045
- package/lib/rules/accessor-pairs.js +297 -262
- package/lib/rules/array-bracket-newline.js +249 -237
- package/lib/rules/array-bracket-spacing.js +262 -223
- package/lib/rules/array-callback-return.js +401 -355
- package/lib/rules/array-element-newline.js +357 -312
- package/lib/rules/arrow-body-style.js +399 -280
- package/lib/rules/arrow-parens.js +205 -172
- package/lib/rules/arrow-spacing.js +168 -162
- package/lib/rules/block-scoped-var.js +124 -122
- package/lib/rules/block-spacing.js +185 -175
- package/lib/rules/brace-style.js +261 -198
- package/lib/rules/callback-return.js +202 -189
- package/lib/rules/camelcase.js +402 -391
- package/lib/rules/capitalized-comments.js +252 -231
- package/lib/rules/class-methods-use-this.js +179 -171
- package/lib/rules/comma-dangle.js +378 -345
- package/lib/rules/comma-spacing.js +192 -194
- package/lib/rules/comma-style.js +374 -315
- package/lib/rules/complexity.js +172 -168
- package/lib/rules/computed-property-spacing.js +235 -210
- package/lib/rules/consistent-return.js +180 -169
- package/lib/rules/consistent-this.js +166 -146
- package/lib/rules/constructor-super.js +411 -403
- package/lib/rules/curly.js +406 -331
- package/lib/rules/default-case-last.js +37 -30
- package/lib/rules/default-case.js +88 -84
- package/lib/rules/default-param-last.js +68 -53
- package/lib/rules/dot-location.js +121 -109
- package/lib/rules/dot-notation.js +191 -155
- package/lib/rules/eol-last.js +121 -119
- package/lib/rules/eqeqeq.js +167 -154
- package/lib/rules/for-direction.js +145 -120
- package/lib/rules/func-call-spacing.js +260 -230
- package/lib/rules/func-name-matching.js +292 -208
- package/lib/rules/func-names.js +164 -163
- package/lib/rules/func-style.js +158 -126
- package/lib/rules/function-call-argument-newline.js +151 -128
- package/lib/rules/function-paren-newline.js +348 -290
- package/lib/rules/generator-star-spacing.js +228 -209
- package/lib/rules/getter-return.js +207 -171
- package/lib/rules/global-require.js +84 -73
- package/lib/rules/grouped-accessor-pairs.js +169 -149
- package/lib/rules/guard-for-in.js +71 -62
- package/lib/rules/handle-callback-err.js +107 -102
- package/lib/rules/id-blacklist.js +181 -198
- package/lib/rules/id-denylist.js +167 -186
- package/lib/rules/id-length.js +196 -170
- package/lib/rules/id-match.js +343 -288
- package/lib/rules/implicit-arrow-linebreak.js +101 -78
- package/lib/rules/indent-legacy.js +1343 -1117
- package/lib/rules/indent.js +2271 -1758
- package/lib/rules/index.js +317 -292
- package/lib/rules/init-declarations.js +115 -106
- package/lib/rules/jsx-quotes.js +93 -81
- package/lib/rules/key-spacing.js +749 -632
- package/lib/rules/keyword-spacing.js +647 -604
- package/lib/rules/line-comment-position.js +141 -127
- package/lib/rules/linebreak-style.js +106 -105
- package/lib/rules/lines-around-comment.js +539 -447
- package/lib/rules/lines-around-directive.js +232 -202
- package/lib/rules/lines-between-class-members.js +304 -233
- package/lib/rules/logical-assignment-operators.js +581 -398
- package/lib/rules/max-classes-per-file.js +68 -67
- package/lib/rules/max-depth.js +145 -142
- package/lib/rules/max-len.js +472 -433
- package/lib/rules/max-lines-per-function.js +200 -175
- package/lib/rules/max-lines.js +157 -161
- package/lib/rules/max-nested-callbacks.js +101 -103
- package/lib/rules/max-params.js +77 -75
- package/lib/rules/max-statements-per-line.js +204 -197
- package/lib/rules/max-statements.js +167 -163
- package/lib/rules/multiline-comment-style.js +636 -478
- package/lib/rules/multiline-ternary.js +240 -175
- package/lib/rules/new-cap.js +232 -212
- package/lib/rules/new-parens.js +87 -78
- package/lib/rules/newline-after-var.js +286 -249
- package/lib/rules/newline-before-return.js +228 -221
- package/lib/rules/newline-per-chained-call.js +141 -126
- package/lib/rules/no-alert.js +89 -78
- package/lib/rules/no-array-constructor.js +121 -112
- package/lib/rules/no-async-promise-executor.js +29 -23
- package/lib/rules/no-await-in-loop.js +68 -70
- package/lib/rules/no-bitwise.js +123 -99
- package/lib/rules/no-buffer-constructor.js +54 -46
- package/lib/rules/no-caller.js +38 -32
- package/lib/rules/no-case-declarations.js +60 -56
- package/lib/rules/no-catch-shadow.js +75 -72
- package/lib/rules/no-class-assign.js +50 -47
- package/lib/rules/no-compare-neg-zero.js +61 -47
- package/lib/rules/no-cond-assign.js +147 -131
- package/lib/rules/no-confusing-arrow.js +97 -80
- package/lib/rules/no-console.js +201 -198
- package/lib/rules/no-const-assign.js +46 -40
- package/lib/rules/no-constant-binary-expression.js +499 -404
- package/lib/rules/no-constant-condition.js +157 -142
- package/lib/rules/no-constructor-return.js +48 -48
- package/lib/rules/no-continue.js +24 -26
- package/lib/rules/no-control-regex.js +124 -120
- package/lib/rules/no-debugger.js +27 -29
- package/lib/rules/no-delete-var.js +28 -28
- package/lib/rules/no-div-regex.js +46 -40
- package/lib/rules/no-dupe-args.js +67 -68
- package/lib/rules/no-dupe-class-members.js +92 -88
- package/lib/rules/no-dupe-else-if.js +99 -76
- package/lib/rules/no-dupe-keys.js +132 -109
- package/lib/rules/no-duplicate-case.js +49 -42
- package/lib/rules/no-duplicate-imports.js +178 -175
- package/lib/rules/no-else-return.js +429 -384
- package/lib/rules/no-empty-character-class.js +56 -49
- package/lib/rules/no-empty-function.js +126 -127
- package/lib/rules/no-empty-pattern.js +62 -57
- package/lib/rules/no-empty-static-block.js +36 -34
- package/lib/rules/no-empty.js +97 -85
- package/lib/rules/no-eq-null.js +36 -31
- package/lib/rules/no-eval.js +255 -249
- package/lib/rules/no-ex-assign.js +41 -38
- package/lib/rules/no-extend-native.js +160 -158
- package/lib/rules/no-extra-bind.js +200 -189
- package/lib/rules/no-extra-boolean-cast.js +397 -347
- package/lib/rules/no-extra-label.js +149 -130
- package/lib/rules/no-extra-parens.js +1653 -1324
- package/lib/rules/no-extra-semi.js +145 -143
- package/lib/rules/no-fallthrough.js +198 -156
- package/lib/rules/no-floating-decimal.js +73 -65
- package/lib/rules/no-func-assign.js +53 -54
- package/lib/rules/no-global-assign.js +77 -72
- package/lib/rules/no-implicit-coercion.js +348 -292
- package/lib/rules/no-implicit-globals.js +157 -134
- package/lib/rules/no-implied-eval.js +139 -111
- package/lib/rules/no-import-assign.js +144 -158
- package/lib/rules/no-inline-comments.js +100 -94
- package/lib/rules/no-inner-declarations.js +114 -100
- package/lib/rules/no-invalid-regexp.js +221 -189
- package/lib/rules/no-invalid-this.js +122 -116
- package/lib/rules/no-irregular-whitespace.js +265 -251
- package/lib/rules/no-iterator.js +28 -32
- package/lib/rules/no-label-var.js +58 -61
- package/lib/rules/no-labels.js +137 -132
- package/lib/rules/no-lone-blocks.js +126 -122
- package/lib/rules/no-lonely-if.js +107 -76
- package/lib/rules/no-loop-func.js +233 -212
- package/lib/rules/no-loss-of-precision.js +215 -200
- package/lib/rules/no-magic-numbers.js +245 -217
- package/lib/rules/no-misleading-character-class.js +498 -445
- package/lib/rules/no-mixed-operators.js +187 -181
- package/lib/rules/no-mixed-requires.js +252 -239
- package/lib/rules/no-mixed-spaces-and-tabs.js +133 -120
- package/lib/rules/no-multi-assign.js +45 -43
- package/lib/rules/no-multi-spaces.js +162 -142
- package/lib/rules/no-multi-str.js +41 -40
- package/lib/rules/no-multiple-empty-lines.js +195 -157
- package/lib/rules/no-native-reassign.js +89 -84
- package/lib/rules/no-negated-condition.js +78 -74
- package/lib/rules/no-negated-in-lhs.js +44 -42
- package/lib/rules/no-nested-ternary.js +32 -31
- package/lib/rules/no-new-func.js +70 -61
- package/lib/rules/no-new-native-nonconstructor.js +42 -38
- package/lib/rules/no-new-object.js +47 -47
- package/lib/rules/no-new-require.js +47 -46
- package/lib/rules/no-new-symbol.js +51 -49
- package/lib/rules/no-new-wrappers.js +42 -40
- package/lib/rules/no-new.js +27 -28
- package/lib/rules/no-nonoctal-decimal-escape.js +140 -120
- package/lib/rules/no-obj-calls.js +65 -52
- package/lib/rules/no-object-constructor.js +103 -96
- package/lib/rules/no-octal-escape.js +39 -42
- package/lib/rules/no-octal.js +31 -31
- package/lib/rules/no-param-reassign.js +234 -216
- package/lib/rules/no-path-concat.js +65 -66
- package/lib/rules/no-plusplus.js +59 -60
- package/lib/rules/no-process-env.js +48 -47
- package/lib/rules/no-process-exit.js +53 -49
- package/lib/rules/no-promise-executor-return.js +213 -181
- package/lib/rules/no-proto.js +25 -28
- package/lib/rules/no-prototype-builtins.js +145 -123
- package/lib/rules/no-redeclare.js +153 -151
- package/lib/rules/no-regex-spaces.js +182 -160
- package/lib/rules/no-restricted-exports.js +207 -184
- package/lib/rules/no-restricted-globals.js +110 -111
- package/lib/rules/no-restricted-imports.js +656 -536
- package/lib/rules/no-restricted-modules.js +221 -201
- package/lib/rules/no-restricted-properties.js +180 -152
- package/lib/rules/no-restricted-syntax.js +55 -51
- package/lib/rules/no-return-assign.js +54 -49
- package/lib/rules/no-return-await.js +147 -123
- package/lib/rules/no-script-url.js +51 -44
- package/lib/rules/no-self-assign.js +147 -145
- package/lib/rules/no-self-compare.js +62 -45
- package/lib/rules/no-sequences.js +134 -115
- package/lib/rules/no-setter-return.js +184 -151
- package/lib/rules/no-shadow-restricted-names.js +60 -45
- package/lib/rules/no-shadow.js +341 -315
- package/lib/rules/no-spaced-func.js +81 -76
- package/lib/rules/no-sparse-arrays.js +53 -58
- package/lib/rules/no-sync.js +60 -59
- package/lib/rules/no-tabs.js +82 -71
- package/lib/rules/no-template-curly-in-string.js +32 -31
- package/lib/rules/no-ternary.js +24 -28
- package/lib/rules/no-this-before-super.js +320 -318
- package/lib/rules/no-throw-literal.js +30 -35
- package/lib/rules/no-trailing-spaces.js +198 -190
- package/lib/rules/no-undef-init.js +75 -60
- package/lib/rules/no-undef.js +50 -47
- package/lib/rules/no-undefined.js +72 -74
- package/lib/rules/no-underscore-dangle.js +369 -326
- package/lib/rules/no-unexpected-multiline.js +111 -101
- package/lib/rules/no-unmodified-loop-condition.js +253 -253
- package/lib/rules/no-unneeded-ternary.js +211 -146
- package/lib/rules/no-unreachable-loop.js +144 -141
- package/lib/rules/no-unreachable.js +254 -247
- package/lib/rules/no-unsafe-finally.js +92 -84
- package/lib/rules/no-unsafe-negation.js +104 -82
- package/lib/rules/no-unsafe-optional-chaining.js +191 -177
- package/lib/rules/no-unused-expressions.js +177 -161
- package/lib/rules/no-unused-labels.js +138 -123
- package/lib/rules/no-unused-private-class-members.js +205 -181
- package/lib/rules/no-unused-vars.js +1668 -1448
- package/lib/rules/no-use-before-define.js +228 -230
- package/lib/rules/no-useless-assignment.js +589 -510
- package/lib/rules/no-useless-backreference.js +211 -192
- package/lib/rules/no-useless-call.js +57 -52
- package/lib/rules/no-useless-catch.js +39 -39
- package/lib/rules/no-useless-computed-key.js +143 -114
- package/lib/rules/no-useless-concat.js +64 -59
- package/lib/rules/no-useless-constructor.js +157 -110
- package/lib/rules/no-useless-escape.js +341 -290
- package/lib/rules/no-useless-rename.js +182 -155
- package/lib/rules/no-useless-return.js +343 -311
- package/lib/rules/no-var.js +232 -211
- package/lib/rules/no-void.js +49 -47
- package/lib/rules/no-warning-comments.js +190 -185
- package/lib/rules/no-whitespace-before-property.js +130 -114
- package/lib/rules/no-with.js +23 -25
- package/lib/rules/nonblock-statement-body-position.js +148 -129
- package/lib/rules/object-curly-newline.js +305 -264
- package/lib/rules/object-curly-spacing.js +359 -313
- package/lib/rules/object-property-newline.js +136 -105
- package/lib/rules/object-shorthand.js +606 -501
- package/lib/rules/one-var-declaration-per-line.js +103 -99
- package/lib/rules/one-var.js +652 -536
- package/lib/rules/operator-assignment.js +218 -160
- package/lib/rules/operator-linebreak.js +294 -250
- package/lib/rules/padded-blocks.js +345 -307
- package/lib/rules/padding-line-between-statements.js +442 -438
- package/lib/rules/prefer-arrow-callback.js +361 -312
- package/lib/rules/prefer-const.js +417 -376
- package/lib/rules/prefer-destructuring.js +300 -278
- package/lib/rules/prefer-exponentiation-operator.js +175 -132
- package/lib/rules/prefer-named-capture-group.js +152 -139
- package/lib/rules/prefer-numeric-literals.js +120 -112
- package/lib/rules/prefer-object-has-own.js +115 -81
- package/lib/rules/prefer-object-spread.js +212 -192
- package/lib/rules/prefer-promise-reject-errors.js +139 -121
- package/lib/rules/prefer-reflect.js +126 -106
- package/lib/rules/prefer-regex-literals.js +577 -465
- package/lib/rules/prefer-rest-params.js +78 -79
- package/lib/rules/prefer-spread.js +46 -43
- package/lib/rules/prefer-template.js +265 -194
- package/lib/rules/quote-props.js +372 -306
- package/lib/rules/quotes.js +373 -325
- package/lib/rules/radix.js +151 -135
- package/lib/rules/require-atomic-updates.js +315 -284
- package/lib/rules/require-await.js +143 -115
- package/lib/rules/require-unicode-regexp.js +281 -176
- package/lib/rules/require-yield.js +52 -53
- package/lib/rules/rest-spread-spacing.js +127 -115
- package/lib/rules/semi-spacing.js +280 -249
- package/lib/rules/semi-style.js +175 -133
- package/lib/rules/semi.js +455 -435
- package/lib/rules/sort-imports.js +305 -232
- package/lib/rules/sort-keys.js +218 -187
- package/lib/rules/sort-vars.js +126 -92
- package/lib/rules/space-before-blocks.js +198 -188
- package/lib/rules/space-before-function-paren.js +185 -165
- package/lib/rules/space-in-parens.js +358 -287
- package/lib/rules/space-infix-ops.js +236 -200
- package/lib/rules/space-unary-ops.js +355 -297
- package/lib/rules/spaced-comment.js +362 -318
- package/lib/rules/strict.js +264 -229
- package/lib/rules/switch-colon-spacing.js +129 -121
- package/lib/rules/symbol-description.js +44 -47
- package/lib/rules/template-curly-spacing.js +147 -141
- package/lib/rules/template-tag-spacing.js +97 -87
- package/lib/rules/unicode-bom.js +53 -55
- package/lib/rules/use-isnan.js +236 -205
- package/lib/rules/utils/ast-utils.js +2039 -1860
- package/lib/rules/utils/char-source.js +162 -155
- package/lib/rules/utils/fix-tracker.js +83 -80
- package/lib/rules/utils/keywords.js +59 -59
- package/lib/rules/utils/lazy-loading-rule-map.js +79 -76
- package/lib/rules/utils/regular-expressions.js +32 -24
- package/lib/rules/utils/unicode/index.js +4 -4
- package/lib/rules/utils/unicode/is-combining-character.js +1 -1
- package/lib/rules/utils/unicode/is-emoji-modifier.js +1 -1
- package/lib/rules/utils/unicode/is-regional-indicator-symbol.js +1 -1
- package/lib/rules/utils/unicode/is-surrogate-pair.js +1 -1
- package/lib/rules/valid-typeof.js +152 -110
- package/lib/rules/vars-on-top.js +151 -144
- package/lib/rules/wrap-iife.js +203 -190
- package/lib/rules/wrap-regex.js +69 -57
- package/lib/rules/yield-star-spacing.js +144 -133
- package/lib/rules/yoda.js +282 -271
- package/lib/services/parser-service.js +35 -35
- package/lib/services/processor-service.js +66 -73
- package/lib/shared/ajv.js +14 -14
- package/lib/shared/assert.js +3 -4
- package/lib/shared/ast-utils.js +7 -6
- package/lib/shared/deep-merge-arrays.js +24 -22
- package/lib/shared/directives.js +3 -2
- package/lib/shared/flags.js +46 -17
- package/lib/shared/logging.js +24 -25
- package/lib/shared/option-utils.js +43 -36
- package/lib/shared/runtime-info.js +136 -127
- package/lib/shared/serialization.js +27 -27
- package/lib/shared/severity.js +22 -22
- package/lib/shared/stats.js +5 -5
- package/lib/shared/string-utils.js +16 -16
- package/lib/shared/text-table.js +28 -27
- package/lib/shared/traverser.js +153 -146
- package/lib/types/index.d.ts +2010 -1559
- package/lib/types/rules.d.ts +5253 -5140
- package/lib/types/use-at-your-own-risk.d.ts +32 -30
- package/lib/unsupported-api.js +5 -5
- package/messages/all-files-ignored.js +3 -3
- package/messages/all-matched-files-ignored.js +3 -3
- package/messages/config-file-missing.js +2 -2
- package/messages/config-plugin-missing.js +3 -3
- package/messages/config-serialize-function.js +9 -7
- package/messages/eslintrc-incompat.js +13 -15
- package/messages/eslintrc-plugins.js +3 -4
- package/messages/extend-config-missing.js +3 -3
- package/messages/failed-to-read-json.js +3 -3
- package/messages/file-not-found.js +3 -3
- package/messages/invalid-rule-options.js +2 -2
- package/messages/invalid-rule-severity.js +2 -2
- package/messages/no-config-found.js +3 -3
- package/messages/plugin-conflict.js +8 -8
- package/messages/plugin-invalid.js +3 -3
- package/messages/plugin-missing.js +3 -3
- package/messages/print-config-with-directory-path.js +2 -2
- package/messages/shared.js +6 -1
- package/messages/whitespace-found.js +3 -3
- package/package.json +11 -17
package/lib/rules/semi.js
CHANGED
@@ -18,439 +18,459 @@ const astUtils = require("./utils/ast-utils");
|
|
18
18
|
|
19
19
|
/** @type {import('../shared/types').Rule} */
|
20
20
|
module.exports = {
|
21
|
-
|
22
|
-
|
23
|
-
|
24
|
-
|
25
|
-
|
26
|
-
|
27
|
-
|
28
|
-
|
29
|
-
|
30
|
-
|
31
|
-
|
32
|
-
|
33
|
-
|
34
|
-
|
35
|
-
|
36
|
-
|
37
|
-
|
38
|
-
|
39
|
-
|
40
|
-
|
41
|
-
|
42
|
-
|
43
|
-
|
44
|
-
|
45
|
-
|
46
|
-
|
47
|
-
|
48
|
-
|
49
|
-
|
50
|
-
|
51
|
-
|
52
|
-
|
53
|
-
|
54
|
-
|
55
|
-
|
56
|
-
|
57
|
-
|
58
|
-
|
59
|
-
|
60
|
-
|
61
|
-
|
62
|
-
|
63
|
-
|
64
|
-
|
65
|
-
|
66
|
-
|
67
|
-
|
68
|
-
|
69
|
-
|
70
|
-
|
71
|
-
|
72
|
-
|
73
|
-
|
74
|
-
|
75
|
-
|
76
|
-
|
77
|
-
|
78
|
-
|
79
|
-
|
80
|
-
|
81
|
-
|
82
|
-
|
83
|
-
|
84
|
-
|
85
|
-
|
86
|
-
|
87
|
-
|
88
|
-
|
89
|
-
|
90
|
-
|
91
|
-
|
92
|
-
|
93
|
-
|
94
|
-
|
95
|
-
|
96
|
-
|
97
|
-
|
98
|
-
|
99
|
-
|
100
|
-
|
101
|
-
|
102
|
-
|
103
|
-
|
104
|
-
|
105
|
-
|
106
|
-
|
107
|
-
|
108
|
-
|
109
|
-
|
110
|
-
|
111
|
-
|
112
|
-
|
113
|
-
|
114
|
-
|
115
|
-
|
116
|
-
|
117
|
-
|
118
|
-
|
119
|
-
|
120
|
-
|
121
|
-
|
122
|
-
|
123
|
-
|
124
|
-
|
125
|
-
|
126
|
-
|
127
|
-
|
128
|
-
|
129
|
-
|
130
|
-
|
131
|
-
|
132
|
-
|
133
|
-
|
134
|
-
|
135
|
-
|
136
|
-
|
137
|
-
|
138
|
-
|
139
|
-
|
140
|
-
|
141
|
-
|
142
|
-
|
143
|
-
|
144
|
-
|
145
|
-
|
146
|
-
|
147
|
-
|
148
|
-
|
149
|
-
|
150
|
-
|
151
|
-
|
152
|
-
|
153
|
-
|
154
|
-
|
155
|
-
|
156
|
-
|
157
|
-
|
158
|
-
|
159
|
-
|
160
|
-
|
161
|
-
|
162
|
-
|
163
|
-
|
164
|
-
|
165
|
-
|
166
|
-
|
167
|
-
|
168
|
-
|
169
|
-
|
170
|
-
|
171
|
-
|
172
|
-
|
173
|
-
|
174
|
-
|
175
|
-
|
176
|
-
|
177
|
-
|
178
|
-
|
179
|
-
|
180
|
-
|
181
|
-
|
182
|
-
|
183
|
-
|
184
|
-
|
185
|
-
|
186
|
-
|
187
|
-
|
188
|
-
|
189
|
-
|
190
|
-
|
191
|
-
|
192
|
-
|
193
|
-
|
194
|
-
|
195
|
-
|
196
|
-
|
197
|
-
|
198
|
-
|
199
|
-
|
200
|
-
|
201
|
-
|
202
|
-
|
203
|
-
|
204
|
-
|
205
|
-
|
206
|
-
|
207
|
-
|
208
|
-
|
209
|
-
|
210
|
-
|
211
|
-
|
212
|
-
|
213
|
-
|
214
|
-
|
215
|
-
|
216
|
-
|
217
|
-
|
218
|
-
|
219
|
-
|
220
|
-
|
221
|
-
|
222
|
-
|
223
|
-
|
224
|
-
|
225
|
-
|
226
|
-
|
227
|
-
|
228
|
-
|
229
|
-
|
230
|
-
|
231
|
-
|
232
|
-
|
233
|
-
|
234
|
-
|
235
|
-
|
236
|
-
|
237
|
-
|
238
|
-
|
239
|
-
|
240
|
-
|
241
|
-
|
242
|
-
|
243
|
-
|
244
|
-
|
245
|
-
|
246
|
-
|
247
|
-
|
248
|
-
|
249
|
-
|
250
|
-
|
251
|
-
|
252
|
-
|
253
|
-
|
254
|
-
|
255
|
-
|
256
|
-
|
257
|
-
|
258
|
-
|
259
|
-
|
260
|
-
|
261
|
-
|
262
|
-
|
263
|
-
|
264
|
-
|
265
|
-
|
266
|
-
|
267
|
-
|
268
|
-
|
269
|
-
|
270
|
-
|
271
|
-
|
272
|
-
|
273
|
-
|
274
|
-
|
275
|
-
|
276
|
-
|
277
|
-
|
278
|
-
|
279
|
-
|
280
|
-
|
281
|
-
|
282
|
-
|
283
|
-
|
284
|
-
|
285
|
-
|
286
|
-
|
287
|
-
|
288
|
-
|
289
|
-
|
290
|
-
|
291
|
-
|
292
|
-
|
293
|
-
|
294
|
-
|
295
|
-
|
296
|
-
|
297
|
-
|
298
|
-
|
299
|
-
|
300
|
-
|
301
|
-
|
302
|
-
|
303
|
-
|
304
|
-
|
305
|
-
|
306
|
-
|
307
|
-
|
308
|
-
|
309
|
-
|
310
|
-
|
311
|
-
|
312
|
-
|
313
|
-
|
314
|
-
|
315
|
-
|
316
|
-
|
317
|
-
|
318
|
-
|
319
|
-
|
320
|
-
|
321
|
-
|
322
|
-
|
323
|
-
|
324
|
-
|
325
|
-
|
326
|
-
|
327
|
-
|
328
|
-
|
329
|
-
|
330
|
-
|
331
|
-
|
332
|
-
|
333
|
-
|
334
|
-
|
335
|
-
|
336
|
-
|
337
|
-
|
338
|
-
|
339
|
-
|
340
|
-
|
341
|
-
|
342
|
-
|
343
|
-
|
344
|
-
|
345
|
-
|
346
|
-
|
347
|
-
|
348
|
-
|
349
|
-
|
350
|
-
|
351
|
-
|
352
|
-
|
353
|
-
|
354
|
-
|
355
|
-
|
356
|
-
|
357
|
-
|
358
|
-
|
359
|
-
|
360
|
-
|
361
|
-
|
362
|
-
|
363
|
-
|
364
|
-
|
365
|
-
|
366
|
-
|
367
|
-
|
368
|
-
|
369
|
-
|
370
|
-
|
371
|
-
|
372
|
-
|
373
|
-
|
374
|
-
|
375
|
-
|
376
|
-
|
377
|
-
|
378
|
-
|
379
|
-
|
380
|
-
|
381
|
-
|
382
|
-
|
383
|
-
|
384
|
-
|
385
|
-
|
386
|
-
|
387
|
-
|
388
|
-
|
389
|
-
|
390
|
-
|
391
|
-
|
392
|
-
|
393
|
-
|
394
|
-
|
395
|
-
|
396
|
-
|
397
|
-
|
398
|
-
|
399
|
-
|
400
|
-
|
401
|
-
|
402
|
-
|
403
|
-
|
404
|
-
|
405
|
-
|
406
|
-
|
407
|
-
|
408
|
-
|
409
|
-
|
410
|
-
|
411
|
-
|
412
|
-
|
413
|
-
|
414
|
-
|
415
|
-
|
416
|
-
|
417
|
-
|
418
|
-
|
419
|
-
|
420
|
-
|
421
|
-
|
422
|
-
|
423
|
-
|
424
|
-
|
425
|
-
|
426
|
-
|
427
|
-
|
428
|
-
|
429
|
-
|
430
|
-
|
431
|
-
|
432
|
-
|
433
|
-
|
434
|
-
|
435
|
-
|
436
|
-
|
437
|
-
|
438
|
-
|
439
|
-
|
440
|
-
|
441
|
-
|
442
|
-
|
443
|
-
|
444
|
-
|
445
|
-
|
446
|
-
|
447
|
-
|
448
|
-
|
449
|
-
|
450
|
-
|
451
|
-
|
452
|
-
|
453
|
-
|
454
|
-
|
455
|
-
|
21
|
+
meta: {
|
22
|
+
deprecated: {
|
23
|
+
message: "Formatting rules are being moved out of ESLint core.",
|
24
|
+
url: "https://eslint.org/blog/2023/10/deprecating-formatting-rules/",
|
25
|
+
deprecatedSince: "8.53.0",
|
26
|
+
availableUntil: "10.0.0",
|
27
|
+
replacedBy: [
|
28
|
+
{
|
29
|
+
message:
|
30
|
+
"ESLint Stylistic now maintains deprecated stylistic core rules.",
|
31
|
+
url: "https://eslint.style/guide/migration",
|
32
|
+
plugin: {
|
33
|
+
name: "@stylistic/eslint-plugin-js",
|
34
|
+
url: "https://eslint.style/packages/js",
|
35
|
+
},
|
36
|
+
rule: {
|
37
|
+
name: "semi",
|
38
|
+
url: "https://eslint.style/rules/js/semi",
|
39
|
+
},
|
40
|
+
},
|
41
|
+
],
|
42
|
+
},
|
43
|
+
type: "layout",
|
44
|
+
|
45
|
+
docs: {
|
46
|
+
description: "Require or disallow semicolons instead of ASI",
|
47
|
+
recommended: false,
|
48
|
+
url: "https://eslint.org/docs/latest/rules/semi",
|
49
|
+
},
|
50
|
+
|
51
|
+
fixable: "code",
|
52
|
+
|
53
|
+
schema: {
|
54
|
+
anyOf: [
|
55
|
+
{
|
56
|
+
type: "array",
|
57
|
+
items: [
|
58
|
+
{
|
59
|
+
enum: ["never"],
|
60
|
+
},
|
61
|
+
{
|
62
|
+
type: "object",
|
63
|
+
properties: {
|
64
|
+
beforeStatementContinuationChars: {
|
65
|
+
enum: ["always", "any", "never"],
|
66
|
+
},
|
67
|
+
},
|
68
|
+
additionalProperties: false,
|
69
|
+
},
|
70
|
+
],
|
71
|
+
minItems: 0,
|
72
|
+
maxItems: 2,
|
73
|
+
},
|
74
|
+
{
|
75
|
+
type: "array",
|
76
|
+
items: [
|
77
|
+
{
|
78
|
+
enum: ["always"],
|
79
|
+
},
|
80
|
+
{
|
81
|
+
type: "object",
|
82
|
+
properties: {
|
83
|
+
omitLastInOneLineBlock: { type: "boolean" },
|
84
|
+
omitLastInOneLineClassBody: { type: "boolean" },
|
85
|
+
},
|
86
|
+
additionalProperties: false,
|
87
|
+
},
|
88
|
+
],
|
89
|
+
minItems: 0,
|
90
|
+
maxItems: 2,
|
91
|
+
},
|
92
|
+
],
|
93
|
+
},
|
94
|
+
|
95
|
+
messages: {
|
96
|
+
missingSemi: "Missing semicolon.",
|
97
|
+
extraSemi: "Extra semicolon.",
|
98
|
+
},
|
99
|
+
},
|
100
|
+
|
101
|
+
create(context) {
|
102
|
+
const OPT_OUT_PATTERN = /^[-[(/+`]/u; // One of [(/+-`
|
103
|
+
const unsafeClassFieldNames = new Set(["get", "set", "static"]);
|
104
|
+
const unsafeClassFieldFollowers = new Set(["*", "in", "instanceof"]);
|
105
|
+
const options = context.options[1];
|
106
|
+
const never = context.options[0] === "never";
|
107
|
+
const exceptOneLine = Boolean(
|
108
|
+
options && options.omitLastInOneLineBlock,
|
109
|
+
);
|
110
|
+
const exceptOneLineClassBody = Boolean(
|
111
|
+
options && options.omitLastInOneLineClassBody,
|
112
|
+
);
|
113
|
+
const beforeStatementContinuationChars =
|
114
|
+
(options && options.beforeStatementContinuationChars) || "any";
|
115
|
+
const sourceCode = context.sourceCode;
|
116
|
+
|
117
|
+
//--------------------------------------------------------------------------
|
118
|
+
// Helpers
|
119
|
+
//--------------------------------------------------------------------------
|
120
|
+
|
121
|
+
/**
|
122
|
+
* Reports a semicolon error with appropriate location and message.
|
123
|
+
* @param {ASTNode} node The node with an extra or missing semicolon.
|
124
|
+
* @param {boolean} missing True if the semicolon is missing.
|
125
|
+
* @returns {void}
|
126
|
+
*/
|
127
|
+
function report(node, missing) {
|
128
|
+
const lastToken = sourceCode.getLastToken(node);
|
129
|
+
let messageId, fix, loc;
|
130
|
+
|
131
|
+
if (!missing) {
|
132
|
+
messageId = "missingSemi";
|
133
|
+
loc = {
|
134
|
+
start: lastToken.loc.end,
|
135
|
+
end: astUtils.getNextLocation(
|
136
|
+
sourceCode,
|
137
|
+
lastToken.loc.end,
|
138
|
+
),
|
139
|
+
};
|
140
|
+
fix = function (fixer) {
|
141
|
+
return fixer.insertTextAfter(lastToken, ";");
|
142
|
+
};
|
143
|
+
} else {
|
144
|
+
messageId = "extraSemi";
|
145
|
+
loc = lastToken.loc;
|
146
|
+
fix = function (fixer) {
|
147
|
+
/*
|
148
|
+
* Expand the replacement range to include the surrounding
|
149
|
+
* tokens to avoid conflicting with no-extra-semi.
|
150
|
+
* https://github.com/eslint/eslint/issues/7928
|
151
|
+
*/
|
152
|
+
return new FixTracker(fixer, sourceCode)
|
153
|
+
.retainSurroundingTokens(lastToken)
|
154
|
+
.remove(lastToken);
|
155
|
+
};
|
156
|
+
}
|
157
|
+
|
158
|
+
context.report({
|
159
|
+
node,
|
160
|
+
loc,
|
161
|
+
messageId,
|
162
|
+
fix,
|
163
|
+
});
|
164
|
+
}
|
165
|
+
|
166
|
+
/**
|
167
|
+
* Check whether a given semicolon token is redundant.
|
168
|
+
* @param {Token} semiToken A semicolon token to check.
|
169
|
+
* @returns {boolean} `true` if the next token is `;` or `}`.
|
170
|
+
*/
|
171
|
+
function isRedundantSemi(semiToken) {
|
172
|
+
const nextToken = sourceCode.getTokenAfter(semiToken);
|
173
|
+
|
174
|
+
return (
|
175
|
+
!nextToken ||
|
176
|
+
astUtils.isClosingBraceToken(nextToken) ||
|
177
|
+
astUtils.isSemicolonToken(nextToken)
|
178
|
+
);
|
179
|
+
}
|
180
|
+
|
181
|
+
/**
|
182
|
+
* Check whether a given token is the closing brace of an arrow function.
|
183
|
+
* @param {Token} lastToken A token to check.
|
184
|
+
* @returns {boolean} `true` if the token is the closing brace of an arrow function.
|
185
|
+
*/
|
186
|
+
function isEndOfArrowBlock(lastToken) {
|
187
|
+
if (!astUtils.isClosingBraceToken(lastToken)) {
|
188
|
+
return false;
|
189
|
+
}
|
190
|
+
const node = sourceCode.getNodeByRangeIndex(lastToken.range[0]);
|
191
|
+
|
192
|
+
return (
|
193
|
+
node.type === "BlockStatement" &&
|
194
|
+
node.parent.type === "ArrowFunctionExpression"
|
195
|
+
);
|
196
|
+
}
|
197
|
+
|
198
|
+
/**
|
199
|
+
* Checks if a given PropertyDefinition node followed by a semicolon
|
200
|
+
* can safely remove that semicolon. It is not to safe to remove if
|
201
|
+
* the class field name is "get", "set", or "static", or if
|
202
|
+
* followed by a generator method.
|
203
|
+
* @param {ASTNode} node The node to check.
|
204
|
+
* @returns {boolean} `true` if the node cannot have the semicolon
|
205
|
+
* removed.
|
206
|
+
*/
|
207
|
+
function maybeClassFieldAsiHazard(node) {
|
208
|
+
if (node.type !== "PropertyDefinition") {
|
209
|
+
return false;
|
210
|
+
}
|
211
|
+
|
212
|
+
/*
|
213
|
+
* Computed property names and non-identifiers are always safe
|
214
|
+
* as they can be distinguished from keywords easily.
|
215
|
+
*/
|
216
|
+
const needsNameCheck =
|
217
|
+
!node.computed && node.key.type === "Identifier";
|
218
|
+
|
219
|
+
/*
|
220
|
+
* Certain names are problematic unless they also have a
|
221
|
+
* a way to distinguish between keywords and property
|
222
|
+
* names.
|
223
|
+
*/
|
224
|
+
if (needsNameCheck && unsafeClassFieldNames.has(node.key.name)) {
|
225
|
+
/*
|
226
|
+
* Special case: If the field name is `static`,
|
227
|
+
* it is only valid if the field is marked as static,
|
228
|
+
* so "static static" is okay but "static" is not.
|
229
|
+
*/
|
230
|
+
const isStaticStatic =
|
231
|
+
node.static && node.key.name === "static";
|
232
|
+
|
233
|
+
/*
|
234
|
+
* For other unsafe names, we only care if there is no
|
235
|
+
* initializer. No initializer = hazard.
|
236
|
+
*/
|
237
|
+
if (!isStaticStatic && !node.value) {
|
238
|
+
return true;
|
239
|
+
}
|
240
|
+
}
|
241
|
+
|
242
|
+
const followingToken = sourceCode.getTokenAfter(node);
|
243
|
+
|
244
|
+
return unsafeClassFieldFollowers.has(followingToken.value);
|
245
|
+
}
|
246
|
+
|
247
|
+
/**
|
248
|
+
* Check whether a given node is on the same line with the next token.
|
249
|
+
* @param {Node} node A statement node to check.
|
250
|
+
* @returns {boolean} `true` if the node is on the same line with the next token.
|
251
|
+
*/
|
252
|
+
function isOnSameLineWithNextToken(node) {
|
253
|
+
const prevToken = sourceCode.getLastToken(node, 1);
|
254
|
+
const nextToken = sourceCode.getTokenAfter(node);
|
255
|
+
|
256
|
+
return (
|
257
|
+
!!nextToken && astUtils.isTokenOnSameLine(prevToken, nextToken)
|
258
|
+
);
|
259
|
+
}
|
260
|
+
|
261
|
+
/**
|
262
|
+
* Check whether a given node can connect the next line if the next line is unreliable.
|
263
|
+
* @param {Node} node A statement node to check.
|
264
|
+
* @returns {boolean} `true` if the node can connect the next line.
|
265
|
+
*/
|
266
|
+
function maybeAsiHazardAfter(node) {
|
267
|
+
const t = node.type;
|
268
|
+
|
269
|
+
if (
|
270
|
+
t === "DoWhileStatement" ||
|
271
|
+
t === "BreakStatement" ||
|
272
|
+
t === "ContinueStatement" ||
|
273
|
+
t === "DebuggerStatement" ||
|
274
|
+
t === "ImportDeclaration" ||
|
275
|
+
t === "ExportAllDeclaration"
|
276
|
+
) {
|
277
|
+
return false;
|
278
|
+
}
|
279
|
+
if (t === "ReturnStatement") {
|
280
|
+
return Boolean(node.argument);
|
281
|
+
}
|
282
|
+
if (t === "ExportNamedDeclaration") {
|
283
|
+
return Boolean(node.declaration);
|
284
|
+
}
|
285
|
+
if (isEndOfArrowBlock(sourceCode.getLastToken(node, 1))) {
|
286
|
+
return false;
|
287
|
+
}
|
288
|
+
|
289
|
+
return true;
|
290
|
+
}
|
291
|
+
|
292
|
+
/**
|
293
|
+
* Check whether a given token can connect the previous statement.
|
294
|
+
* @param {Token} token A token to check.
|
295
|
+
* @returns {boolean} `true` if the token is one of `[`, `(`, `/`, `+`, `-`, ```, `++`, and `--`.
|
296
|
+
*/
|
297
|
+
function maybeAsiHazardBefore(token) {
|
298
|
+
return (
|
299
|
+
Boolean(token) &&
|
300
|
+
OPT_OUT_PATTERN.test(token.value) &&
|
301
|
+
token.value !== "++" &&
|
302
|
+
token.value !== "--"
|
303
|
+
);
|
304
|
+
}
|
305
|
+
|
306
|
+
/**
|
307
|
+
* Check if the semicolon of a given node is unnecessary, only true if:
|
308
|
+
* - next token is a valid statement divider (`;` or `}`).
|
309
|
+
* - next token is on a new line and the node is not connectable to the new line.
|
310
|
+
* @param {Node} node A statement node to check.
|
311
|
+
* @returns {boolean} whether the semicolon is unnecessary.
|
312
|
+
*/
|
313
|
+
function canRemoveSemicolon(node) {
|
314
|
+
if (isRedundantSemi(sourceCode.getLastToken(node))) {
|
315
|
+
return true; // `;;` or `;}`
|
316
|
+
}
|
317
|
+
if (maybeClassFieldAsiHazard(node)) {
|
318
|
+
return false;
|
319
|
+
}
|
320
|
+
if (isOnSameLineWithNextToken(node)) {
|
321
|
+
return false; // One liner.
|
322
|
+
}
|
323
|
+
|
324
|
+
// continuation characters should not apply to class fields
|
325
|
+
if (
|
326
|
+
node.type !== "PropertyDefinition" &&
|
327
|
+
beforeStatementContinuationChars === "never" &&
|
328
|
+
!maybeAsiHazardAfter(node)
|
329
|
+
) {
|
330
|
+
return true; // ASI works. This statement doesn't connect to the next.
|
331
|
+
}
|
332
|
+
if (!maybeAsiHazardBefore(sourceCode.getTokenAfter(node))) {
|
333
|
+
return true; // ASI works. The next token doesn't connect to this statement.
|
334
|
+
}
|
335
|
+
|
336
|
+
return false;
|
337
|
+
}
|
338
|
+
|
339
|
+
/**
|
340
|
+
* Checks a node to see if it's the last item in a one-liner block.
|
341
|
+
* Block is any `BlockStatement` or `StaticBlock` node. Block is a one-liner if its
|
342
|
+
* braces (and consequently everything between them) are on the same line.
|
343
|
+
* @param {ASTNode} node The node to check.
|
344
|
+
* @returns {boolean} whether the node is the last item in a one-liner block.
|
345
|
+
*/
|
346
|
+
function isLastInOneLinerBlock(node) {
|
347
|
+
const parent = node.parent;
|
348
|
+
const nextToken = sourceCode.getTokenAfter(node);
|
349
|
+
|
350
|
+
if (!nextToken || nextToken.value !== "}") {
|
351
|
+
return false;
|
352
|
+
}
|
353
|
+
|
354
|
+
if (parent.type === "BlockStatement") {
|
355
|
+
return parent.loc.start.line === parent.loc.end.line;
|
356
|
+
}
|
357
|
+
|
358
|
+
if (parent.type === "StaticBlock") {
|
359
|
+
const openingBrace = sourceCode.getFirstToken(parent, {
|
360
|
+
skip: 1,
|
361
|
+
}); // skip the `static` token
|
362
|
+
|
363
|
+
return openingBrace.loc.start.line === parent.loc.end.line;
|
364
|
+
}
|
365
|
+
|
366
|
+
return false;
|
367
|
+
}
|
368
|
+
|
369
|
+
/**
|
370
|
+
* Checks a node to see if it's the last item in a one-liner `ClassBody` node.
|
371
|
+
* ClassBody is a one-liner if its braces (and consequently everything between them) are on the same line.
|
372
|
+
* @param {ASTNode} node The node to check.
|
373
|
+
* @returns {boolean} whether the node is the last item in a one-liner ClassBody.
|
374
|
+
*/
|
375
|
+
function isLastInOneLinerClassBody(node) {
|
376
|
+
const parent = node.parent;
|
377
|
+
const nextToken = sourceCode.getTokenAfter(node);
|
378
|
+
|
379
|
+
if (!nextToken || nextToken.value !== "}") {
|
380
|
+
return false;
|
381
|
+
}
|
382
|
+
|
383
|
+
if (parent.type === "ClassBody") {
|
384
|
+
return parent.loc.start.line === parent.loc.end.line;
|
385
|
+
}
|
386
|
+
|
387
|
+
return false;
|
388
|
+
}
|
389
|
+
|
390
|
+
/**
|
391
|
+
* Checks a node to see if it's followed by a semicolon.
|
392
|
+
* @param {ASTNode} node The node to check.
|
393
|
+
* @returns {void}
|
394
|
+
*/
|
395
|
+
function checkForSemicolon(node) {
|
396
|
+
const isSemi = astUtils.isSemicolonToken(
|
397
|
+
sourceCode.getLastToken(node),
|
398
|
+
);
|
399
|
+
|
400
|
+
if (never) {
|
401
|
+
if (isSemi && canRemoveSemicolon(node)) {
|
402
|
+
report(node, true);
|
403
|
+
} else if (
|
404
|
+
!isSemi &&
|
405
|
+
beforeStatementContinuationChars === "always" &&
|
406
|
+
node.type !== "PropertyDefinition" &&
|
407
|
+
maybeAsiHazardBefore(sourceCode.getTokenAfter(node))
|
408
|
+
) {
|
409
|
+
report(node);
|
410
|
+
}
|
411
|
+
} else {
|
412
|
+
const oneLinerBlock =
|
413
|
+
exceptOneLine && isLastInOneLinerBlock(node);
|
414
|
+
const oneLinerClassBody =
|
415
|
+
exceptOneLineClassBody && isLastInOneLinerClassBody(node);
|
416
|
+
const oneLinerBlockOrClassBody =
|
417
|
+
oneLinerBlock || oneLinerClassBody;
|
418
|
+
|
419
|
+
if (isSemi && oneLinerBlockOrClassBody) {
|
420
|
+
report(node, true);
|
421
|
+
} else if (!isSemi && !oneLinerBlockOrClassBody) {
|
422
|
+
report(node);
|
423
|
+
}
|
424
|
+
}
|
425
|
+
}
|
426
|
+
|
427
|
+
/**
|
428
|
+
* Checks to see if there's a semicolon after a variable declaration.
|
429
|
+
* @param {ASTNode} node The node to check.
|
430
|
+
* @returns {void}
|
431
|
+
*/
|
432
|
+
function checkForSemicolonForVariableDeclaration(node) {
|
433
|
+
const parent = node.parent;
|
434
|
+
|
435
|
+
if (
|
436
|
+
(parent.type !== "ForStatement" || parent.init !== node) &&
|
437
|
+
(!/^For(?:In|Of)Statement/u.test(parent.type) ||
|
438
|
+
parent.left !== node)
|
439
|
+
) {
|
440
|
+
checkForSemicolon(node);
|
441
|
+
}
|
442
|
+
}
|
443
|
+
|
444
|
+
//--------------------------------------------------------------------------
|
445
|
+
// Public API
|
446
|
+
//--------------------------------------------------------------------------
|
447
|
+
|
448
|
+
return {
|
449
|
+
VariableDeclaration: checkForSemicolonForVariableDeclaration,
|
450
|
+
ExpressionStatement: checkForSemicolon,
|
451
|
+
ReturnStatement: checkForSemicolon,
|
452
|
+
ThrowStatement: checkForSemicolon,
|
453
|
+
DoWhileStatement: checkForSemicolon,
|
454
|
+
DebuggerStatement: checkForSemicolon,
|
455
|
+
BreakStatement: checkForSemicolon,
|
456
|
+
ContinueStatement: checkForSemicolon,
|
457
|
+
ImportDeclaration: checkForSemicolon,
|
458
|
+
ExportAllDeclaration: checkForSemicolon,
|
459
|
+
ExportNamedDeclaration(node) {
|
460
|
+
if (!node.declaration) {
|
461
|
+
checkForSemicolon(node);
|
462
|
+
}
|
463
|
+
},
|
464
|
+
ExportDefaultDeclaration(node) {
|
465
|
+
if (
|
466
|
+
!/(?:Class|Function)Declaration/u.test(
|
467
|
+
node.declaration.type,
|
468
|
+
)
|
469
|
+
) {
|
470
|
+
checkForSemicolon(node);
|
471
|
+
}
|
472
|
+
},
|
473
|
+
PropertyDefinition: checkForSemicolon,
|
474
|
+
};
|
475
|
+
},
|
456
476
|
};
|