Jack Franklin | aba8800 | 2020-11-26 16:23:05 | [diff] [blame] | 1 | // Copyright 2020 The Chromium Authors. All rights reserved. |
| 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
Jack Franklin | df0e931 | 2022-12-05 10:29:52 | [diff] [blame] | 5 | // clang-format off |
| 6 | |
Tim van der Lippe | d6c33c4 | 2020-03-10 16:57:22 | [diff] [blame] | 7 | const path = require('path'); |
Jack Franklin | aba8800 | 2020-11-26 16:23:05 | [diff] [blame] | 8 | const rulesDirPlugin = require('eslint-plugin-rulesdir'); |
Tim van der Lippe | d6c33c4 | 2020-03-10 16:57:22 | [diff] [blame] | 9 | rulesDirPlugin.RULES_DIR = path.join(__dirname, 'scripts', 'eslint_rules', 'lib'); |
| 10 | |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 | [diff] [blame] | 11 | module.exports = { |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 12 | 'root': true, |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 | [diff] [blame] | 13 | |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 14 | 'env': {'browser': true, 'es6': true}, |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 | [diff] [blame] | 15 | |
Tim van der Lippe | 399a956 | 2020-01-16 10:53:28 | [diff] [blame] | 16 | 'parser': '@typescript-eslint/parser', |
| 17 | |
| 18 | 'plugins': [ |
| 19 | '@typescript-eslint', |
Jack Franklin | 8b9aa2f | 2020-02-12 16:35:15 | [diff] [blame] | 20 | 'mocha', |
Tim van der Lippe | d6c33c4 | 2020-03-10 16:57:22 | [diff] [blame] | 21 | 'rulesdir', |
Tim van der Lippe | 298b8cc | 2020-04-03 11:55:10 | [diff] [blame] | 22 | 'import', |
Jack Franklin | fd72c07 | 2022-12-21 11:45:01 | [diff] [blame] | 23 | 'jsdoc', |
Tim van der Lippe | 399a956 | 2020-01-16 10:53:28 | [diff] [blame] | 24 | ], |
| 25 | |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 26 | 'parserOptions': {'ecmaVersion': 9, 'sourceType': 'module'}, |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 | [diff] [blame] | 27 | |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 28 | /** |
Tim van der Lippe | 406249f | 2020-12-14 14:59:10 | [diff] [blame] | 29 | * ESLint rules |
| 30 | * |
| 31 | * All available rules: http://eslint.org/docs/rules/ |
| 32 | * |
| 33 | * Rules take the following form: |
| 34 | * 'rule-name', [severity, { opts }] |
| 35 | * Severity: 2 == error, 1 == warning, 0 == off. |
| 36 | */ |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 37 | 'rules': { |
| 38 | /** |
Tim van der Lippe | 406249f | 2020-12-14 14:59:10 | [diff] [blame] | 39 | * Enforced rules |
| 40 | */ |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 | [diff] [blame] | 41 | |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 42 | // syntax preferences |
Mathias Bynens | 23ee1aa | 2020-03-02 12:06:38 | [diff] [blame] | 43 | 'quotes': [2, 'single', {'avoidEscape': true, 'allowTemplateLiterals': false}], |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 44 | 'semi': 2, |
| 45 | 'no-extra-semi': 2, |
| 46 | 'comma-style': [2, 'last'], |
| 47 | 'wrap-iife': [2, 'inside'], |
| 48 | 'spaced-comment': [2, 'always', {'markers': ['*']}], |
| 49 | 'eqeqeq': [2], |
| 50 | 'accessor-pairs': [2, {'getWithoutSet': false, 'setWithoutGet': false}], |
| 51 | 'curly': 2, |
| 52 | 'new-parens': 2, |
| 53 | 'func-call-spacing': 2, |
| 54 | 'arrow-parens': [2, 'as-needed'], |
Tim van der Lippe | 20b29c2 | 2019-11-04 14:36:15 | [diff] [blame] | 55 | 'eol-last': 2, |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 | [diff] [blame] | 56 | |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 57 | // anti-patterns |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 58 | 'no-caller': 2, |
Mathias Bynens | 88e8f15 | 2020-03-25 14:33:12 | [diff] [blame] | 59 | 'no-case-declarations': 2, |
Mathias Bynens | f06e8c0 | 2020-02-28 13:58:28 | [diff] [blame] | 60 | 'no-cond-assign': 2, |
| 61 | 'no-console': [2, {'allow': ['assert', 'context', 'error', 'timeStamp', 'time', 'timeEnd', 'warn']}], |
| 62 | 'no-debugger': 2, |
| 63 | 'no-dupe-keys': 2, |
| 64 | 'no-duplicate-case': 2, |
| 65 | 'no-else-return': [2, {'allowElseIf': false}], |
| 66 | 'no-empty-character-class': 2, |
Mathias Bynens | 15213d0 | 2020-05-14 11:55:11 | [diff] [blame] | 67 | 'no-global-assign': 2, |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 68 | 'no-implied-eval': 2, |
| 69 | 'no-labels': 2, |
Mathias Bynens | f06e8c0 | 2020-02-28 13:58:28 | [diff] [blame] | 70 | 'no-multi-str': 2, |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 71 | 'no-new-object': 2, |
| 72 | 'no-octal-escape': 2, |
| 73 | 'no-self-compare': 2, |
| 74 | 'no-shadow-restricted-names': 2, |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 75 | 'no-unreachable': 2, |
| 76 | 'no-unsafe-negation': 2, |
Mathias Bynens | f06e8c0 | 2020-02-28 13:58:28 | [diff] [blame] | 77 | 'no-unused-vars': [2, {'args': 'none', 'vars': 'local'}], |
| 78 | 'no-var': 2, |
| 79 | 'no-with': 2, |
| 80 | 'prefer-const': 2, |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 81 | 'radix': 2, |
| 82 | 'valid-typeof': 2, |
Patrick Brosset | e65aaac | 2020-06-22 08:04:40 | [diff] [blame] | 83 | 'no-return-assign': [2, 'always'], |
Tim van der Lippe | 4075559 | 2021-01-07 13:55:08 | [diff] [blame] | 84 | 'no-implicit-coercion': 2, |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 | [diff] [blame] | 85 | |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 86 | // es2015 features |
| 87 | 'require-yield': 2, |
| 88 | 'template-curly-spacing': [2, 'never'], |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 | [diff] [blame] | 89 | |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 90 | // file whitespace |
Kriti Sapra | 56daf78 | 2021-07-28 12:05:19 | [diff] [blame] | 91 | 'no-multiple-empty-lines': [2, {'max': 1}], |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 92 | 'no-mixed-spaces-and-tabs': 2, |
| 93 | 'no-trailing-spaces': 2, |
| 94 | 'linebreak-style': [2, 'unix'], |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 | [diff] [blame] | 95 | |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 96 | /** |
Tim van der Lippe | 406249f | 2020-12-14 14:59:10 | [diff] [blame] | 97 | * Disabled, aspirational rules |
| 98 | */ |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 | [diff] [blame] | 99 | |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 100 | 'indent': [0, 2, {'SwitchCase': 1, 'CallExpression': {'arguments': 2}, 'MemberExpression': 2}], |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 | [diff] [blame] | 101 | |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 102 | // brace-style is disabled, as eslint cannot enforce 1tbs as default, but allman for functions |
| 103 | 'brace-style': [0, 'allman', {'allowSingleLine': true}], |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 | [diff] [blame] | 104 | |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 105 | // key-spacing is disabled, as some objects use value-aligned spacing, some not. |
| 106 | 'key-spacing': [0, {'beforeColon': false, 'afterColon': true, 'align': 'value'}], |
| 107 | // quote-props is diabled, as property quoting styles are too varied to enforce. |
| 108 | 'quote-props': [0, 'as-needed'], |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 | [diff] [blame] | 109 | |
Tim van der Lippe | 1d6e57a | 2019-09-30 11:55:34 | [diff] [blame] | 110 | // no-implicit-globals will prevent accidental globals |
Tim van der Lippe | 399a956 | 2020-01-16 10:53:28 | [diff] [blame] | 111 | 'no-implicit-globals': [0], |
Tim van der Lippe | 4f11084 | 2021-12-01 14:55:10 | [diff] [blame] | 112 | 'no-unused-private-class-members': 2, |
Tim van der Lippe | 399a956 | 2020-01-16 10:53:28 | [diff] [blame] | 113 | |
Jack Franklin | 264237a | 2020-07-15 09:02:26 | [diff] [blame] | 114 | // forbids interfaces starting with an I prefix. |
Paul Lewis | 839037f | 2020-07-21 12:25:19 | [diff] [blame] | 115 | '@typescript-eslint/naming-convention': |
| 116 | [2, {'selector': 'interface', 'format': ['PascalCase'], 'custom': {'regex': '^I[A-Z]', 'match': false}}], |
Tim van der Lippe | 399a956 | 2020-01-16 10:53:28 | [diff] [blame] | 117 | '@typescript-eslint/explicit-member-accessibility': [0], |
Benedikt Meurer | cfc8cd6 | 2024-01-17 10:08:28 | [diff] [blame^] | 118 | '@typescript-eslint/no-explicit-any': [ |
| 119 | "error", |
| 120 | { |
| 121 | "ignoreRestArgs": true |
| 122 | } |
| 123 | ], |
Jack Franklin | 8b9aa2f | 2020-02-12 16:35:15 | [diff] [blame] | 124 | |
Tim van der Lippe | 298b8cc | 2020-04-03 11:55:10 | [diff] [blame] | 125 | // Closure does not properly typecheck default exports |
| 126 | 'import/no-default-export': 2, |
| 127 | |
Jack Franklin | 6d9f877 | 2023-07-11 10:55:01 | [diff] [blame] | 128 | /** |
| 129 | * Catch duplicate import paths. For example this would catch the following example: |
| 130 | * import {Foo} from './foo.js' |
| 131 | * import * as FooModule from './foo.js' |
| 132 | **/ |
| 133 | 'import/no-duplicates': 2, |
| 134 | |
Jack Franklin | 02f0dbb | 2021-03-02 14:10:49 | [diff] [blame] | 135 | // Try to spot '// console.log()' left over from debugging |
| 136 | 'rulesdir/commented_out_console': 2, |
| 137 | |
Jack Franklin | df0e931 | 2022-12-05 10:29:52 | [diff] [blame] | 138 | // Prevent imports being commented out rather than deleted. |
| 139 | 'rulesdir/commented_out_import': 2, |
| 140 | |
Tim van der Lippe | d6c33c4 | 2020-03-10 16:57:22 | [diff] [blame] | 141 | // DevTools specific rules |
| 142 | 'rulesdir/es_modules_import': 2, |
Tim van der Lippe | 4a8441c | 2020-03-12 15:45:18 | [diff] [blame] | 143 | 'rulesdir/check_license_header': 2, |
Jack Franklin | c8d5dd2 | 2022-12-21 14:58:15 | [diff] [blame] | 144 | /** |
| 145 | * Ensures that JS Doc comments are properly aligned - all the starting |
| 146 | * `*` are in the right place. |
| 147 | */ |
| 148 | 'jsdoc/check-alignment': 2, |
Tim van der Lippe | 399a956 | 2020-01-16 10:53:28 | [diff] [blame] | 149 | }, |
| 150 | 'overrides': [{ |
| 151 | 'files': ['*.ts'], |
Simon Zünd | 607774a | 2021-06-16 10:55:53 | [diff] [blame] | 152 | 'parserOptions': { |
| 153 | 'allowAutomaticSingleRunInference': true, |
Jack Franklin | 468d759 | 2023-06-02 13:17:33 | [diff] [blame] | 154 | 'project': path.join(__dirname, 'config', 'typescript', 'tsconfig.eslint.json'), |
Simon Zünd | 607774a | 2021-06-16 10:55:53 | [diff] [blame] | 155 | }, |
Tim van der Lippe | 399a956 | 2020-01-16 10:53:28 | [diff] [blame] | 156 | 'rules': { |
Jack Franklin | 173a1ae | 2020-06-12 13:45:40 | [diff] [blame] | 157 | '@typescript-eslint/explicit-member-accessibility': [2, {'accessibility': 'no-public'}], |
Sigurd Schneider | 9c07567 | 2021-02-03 13:16:39 | [diff] [blame] | 158 | 'comma-dangle': 'off', |
| 159 | '@typescript-eslint/comma-dangle': [2, 'always-multiline'], |
Tim van der Lippe | 0ebbf49 | 2020-12-03 12:13:21 | [diff] [blame] | 160 | |
Jack Franklin | 9531032 | 2020-03-20 14:34:04 | [diff] [blame] | 161 | // run just the TypeScript unused-vars rule, else we get duplicate errors |
| 162 | 'no-unused-vars': 0, |
Tim van der Lippe | 159b890 | 2020-09-21 10:55:07 | [diff] [blame] | 163 | '@typescript-eslint/no-unused-vars': [2, {'argsIgnorePattern': '^_'}], |
Tim van der Lippe | 0ebbf49 | 2020-12-03 12:13:21 | [diff] [blame] | 164 | // run just the TypeScript semi rule, else we get duplicate errors |
| 165 | 'semi': 0, |
Sigurd Schneider | 1576186 | 2021-02-04 08:05:36 | [diff] [blame] | 166 | '@typescript-eslint/semi': ['error'], |
| 167 | '@typescript-eslint/member-delimiter-style': [ |
| 168 | 'error', { |
| 169 | 'multiline': {'delimiter': 'semi', 'requireLast': true}, |
| 170 | 'singleline': {'delimiter': 'comma', 'requireLast': false}, |
| 171 | 'overrides': { |
| 172 | 'interface': { |
| 173 | 'singleline': {'delimiter': 'semi', 'requireLast': false}, |
| 174 | 'multiline': {'delimiter': 'semi', 'requireLast': true} |
| 175 | }, |
| 176 | 'typeLiteral': { |
| 177 | 'singleline': {'delimiter': 'comma', 'requireLast': false}, |
| 178 | 'multiline': {'delimiter': 'comma', 'requireLast': true} |
| 179 | } |
| 180 | } |
| 181 | } |
| 182 | ], |
Tim van der Lippe | c03ec58 | 2022-01-04 15:20:37 | [diff] [blame] | 183 | '@typescript-eslint/no-floating-promises': [2, {ignoreVoid: true}], |
Songtao Xia | 165a119 | 2021-01-28 14:09:40 | [diff] [blame] | 184 | // func-call-spacing doesn't work well with .ts |
| 185 | 'func-call-spacing': 0, |
| 186 | '@typescript-eslint/func-call-spacing': 2, |
Tim van der Lippe | 0ebbf49 | 2020-12-03 12:13:21 | [diff] [blame] | 187 | |
Jack Franklin | aba8800 | 2020-11-26 16:23:05 | [diff] [blame] | 188 | /** |
| 189 | * Enforce that enum members are explicitly defined: |
| 190 | * const enum Foo { A = 'a' } rather than const enum Foo { A } |
| 191 | */ |
| 192 | '@typescript-eslint/prefer-enum-initializers': 2, |
Jack Franklin | fc49590 | 2020-12-01 17:24:37 | [diff] [blame] | 193 | /** |
| 194 | * Ban non-null assertion operator, e.g.: |
| 195 | * this.foo!.toLowerCase() |
| 196 | */ |
| 197 | '@typescript-eslint/no-non-null-assertion': 2, |
Jack Franklin | 5d89e07 | 2021-05-11 15:30:27 | [diff] [blame] | 198 | '@typescript-eslint/consistent-type-imports': 2, |
Jack Franklin | dd3be2e | 2020-11-27 15:08:54 | [diff] [blame] | 199 | 'rulesdir/const_enum': 2, |
Jack Franklin | 01d09b0 | 2020-12-02 15:15:20 | [diff] [blame] | 200 | 'rulesdir/no_underscored_properties': 2, |
Jack Franklin | 92fe8ff | 2020-12-03 10:51:23 | [diff] [blame] | 201 | 'rulesdir/prefer_readonly_keyword': 2, |
Jack Franklin | 3a80260 | 2022-07-13 08:39:42 | [diff] [blame] | 202 | 'rulesdir/inline_type_imports': 2, |
Tim van der Lippe | 399a956 | 2020-01-16 10:53:28 | [diff] [blame] | 203 | } |
| 204 | }] |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 | [diff] [blame] | 205 | }; |
Jack Franklin | df0e931 | 2022-12-05 10:29:52 | [diff] [blame] | 206 | |
| 207 | // clang-format on |