Avi Drissman | 2497659 | 2022-09-12 15:24:31 | [diff] [blame] | 1 | // Copyright 2017 The Chromium Authors |
dpapad | ff82d51 | 2017-05-25 18:40:15 | [diff] [blame] | 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
| 5 | module.exports = { |
| 6 | 'root': true, |
| 7 | 'env': { |
| 8 | 'browser': true, |
dpapad | a0617309 | 2020-10-17 03:06:16 | [diff] [blame] | 9 | 'es2020': true, |
Christopher Lam | a5f2ad0 | 2018-12-05 10:05:58 | [diff] [blame] | 10 | }, |
| 11 | 'parserOptions': { |
dpapad | a0617309 | 2020-10-17 03:06:16 | [diff] [blame] | 12 | 'ecmaVersion': 2020, |
dpapad | 12b3c47 | 2019-05-29 02:16:05 | [diff] [blame] | 13 | 'sourceType': 'module', |
dpapad | ff82d51 | 2017-05-25 18:40:15 | [diff] [blame] | 14 | }, |
| 15 | 'rules': { |
| 16 | // Enabled checks. |
Dan Beam | 1a0d9dcb | 2019-01-08 09:05:35 | [diff] [blame] | 17 | 'brace-style': ['error', '1tbs'], |
dpapad | f4d85572 | 2022-07-19 01:28:18 | [diff] [blame] | 18 | |
| 19 | // https://google.github.io/styleguide/jsguide.html#features-arrays-trailing-comma |
| 20 | // https://google.github.io/styleguide/jsguide.html#features-objects-use-trailing-comma |
| 21 | 'comma-dangle': ['error', 'always-multiline'], |
| 22 | |
Dan Beam | 1a0d9dcb | 2019-01-08 09:05:35 | [diff] [blame] | 23 | 'curly': ['error', 'multi-line', 'consistent'], |
dpapad | c21c6fb | 2022-04-01 07:12:13 | [diff] [blame] | 24 | 'new-parens': 'error', |
dpapad | b2b0d6cb | 2022-06-27 19:45:11 | [diff] [blame] | 25 | 'no-array-constructor': 'error', |
dpapad | c7570be | 2022-03-28 08:28:23 | [diff] [blame] | 26 | 'no-console': ['error', {allow: ['info', 'warn', 'error', 'assert']}], |
dpapad | b1866df | 2024-04-16 03:59:36 | [diff] [blame] | 27 | 'no-debugger': 'error', |
Dan Beam | b153665 | 2019-02-26 07:36:25 | [diff] [blame] | 28 | 'no-extra-boolean-cast': 'error', |
dpapad | ff82d51 | 2017-05-25 18:40:15 | [diff] [blame] | 29 | 'no-extra-semi': 'error', |
dpapad | 728cff0c | 2017-06-01 01:06:42 | [diff] [blame] | 30 | 'no-new-wrappers': 'error', |
dpapad | d6fe5df | 2023-04-11 16:55:13 | [diff] [blame] | 31 | 'no-restricted-imports': ['error', { |
| 32 | 'paths': [{ |
| 33 | 'name': 'chrome://resources/polymer/v3_0/polymer/polymer_bundled.min.js', |
| 34 | 'importNames': ['Polymer'], |
| 35 | 'message': 'Use PolymerElement instead.', |
| 36 | }, |
| 37 | { |
| 38 | 'name': '//resources/polymer/v3_0/polymer/polymer_bundled.min.js', |
| 39 | 'importNames': ['Polymer'], |
| 40 | 'message': 'Use PolymerElement instead.', |
| 41 | }], |
| 42 | }], |
Christopher Lam | a5f2ad0 | 2018-12-05 10:05:58 | [diff] [blame] | 43 | 'no-restricted-properties': [ |
| 44 | 'error', |
| 45 | { |
dpapad | 7b493a4 | 2019-06-22 02:07:09 | [diff] [blame] | 46 | 'property': '__lookupGetter__', |
| 47 | 'message': 'Use Object.getOwnPropertyDescriptor', |
| 48 | }, |
| 49 | { |
| 50 | 'property': '__lookupSetter__', |
| 51 | 'message': 'Use Object.getOwnPropertyDescriptor', |
| 52 | }, |
| 53 | { |
| 54 | 'property': '__defineGetter__', |
| 55 | 'message': 'Use Object.defineProperty', |
| 56 | }, |
| 57 | { |
| 58 | 'property': '__defineSetter__', |
| 59 | 'message': 'Use Object.defineProperty', |
| 60 | }, |
Dan Beam | 1aa00a12 | 2020-01-16 00:40:41 | [diff] [blame] | 61 | { |
| 62 | 'object': 'cr', |
| 63 | 'property': 'exportPath', |
| 64 | 'message': 'Use ES modules or cr.define() instead', |
| 65 | }, |
dpapad | 2674f928 | 2017-05-31 19:40:39 | [diff] [blame] | 66 | ], |
dpapad | b9eb38f1 | 2023-11-06 21:42:06 | [diff] [blame] | 67 | 'no-restricted-syntax': ['error', { |
| 68 | 'selector': 'CallExpression[callee.object.name=JSON][callee.property.name=parse] > CallExpression[callee.object.name=JSON][callee.property.name=stringify]', |
| 69 | 'message': 'Don\'t use JSON.parse(JSON.stringify(...)) to clone objects. Use structuredClone() instead.', |
dpapad | e4b476da | 2024-01-25 21:56:21 | [diff] [blame] | 70 | }, |
| 71 | { |
| 72 | // https://google.github.io/styleguide/tsguide.html#return-type-only-generics |
| 73 | 'selector': 'TSAsExpression > CallExpression > MemberExpression[property.name=/^querySelector$/]', |
| 74 | 'message': 'Don\'t use \'querySelector(...) as Type\'. Use the type parameter, \'querySelector<Type>(...)\' instead', |
| 75 | }, |
| 76 | { |
| 77 | // https://google.github.io/styleguide/tsguide.html#return-type-only-generics |
dpapad | a5b94b8f | 2024-05-01 14:03:17 | [diff] [blame] | 78 | 'selector': 'TSAsExpression > TSNonNullExpression > CallExpression > MemberExpression[property.name=/^querySelector$/]', |
| 79 | 'message': 'Don\'t use \'querySelector(...)! as Type\'. Use the type parameter, \'querySelector<Type>(...)\', followed by an assertion instead', |
| 80 | }, |
| 81 | { |
| 82 | // https://google.github.io/styleguide/tsguide.html#return-type-only-generics |
dpapad | e4b476da | 2024-01-25 21:56:21 | [diff] [blame] | 83 | 'selector': 'TSAsExpression > CallExpression > MemberExpression[property.name=/^querySelectorAll$/]', |
| 84 | 'message': 'Don\'t use \'querySelectorAll(...) as Type\'. Use the type parameter, \'querySelectorAll<Type>(...)\' instead', |
dpapad | e5ef8675 | 2024-01-31 00:15:19 | [diff] [blame] | 85 | }, |
| 86 | { |
| 87 | // Prevent a common misuse of "!" operator. |
| 88 | "selector": "TSNonNullExpression > CallExpression > MemberExpression[property.name=/^querySelectorAll$/]", |
| 89 | "message": "Remove unnecessary \"!\" non-null operator after querySelectorAll(). It always returns a non-null result", |
dpapad | a21b3e4 | 2024-03-18 23:51:34 | [diff] [blame] | 90 | }, |
| 91 | { |
| 92 | // https://google.github.io/styleguide/jsguide.html#es-module-imports |
| 93 | // 1) Matching only import URLs that have at least one '/' slash, to |
| 94 | // avoid false positives for NodeJS imports like |
| 95 | // `import fs from 'fs';`. |
| 96 | // Using '\u002F' instead of '/' as the suggested workaround for |
| 97 | // https://github.com/eslint/eslint/issues/16555 |
| 98 | // 2) Allowing extensions that have a length between 2-4 characters |
| 99 | // (for example js, css, json) |
| 100 | "selector": "ImportDeclaration[source.value=/^.*\\u002F.*(?<!\\.[a-z]{2}|\\.[a-z]{3}|\\.[a-z]{4})$/]", |
| 101 | "message": "Disallowed extensionless import. Explicitly specify the extension suffix." |
dpapad | b9eb38f1 | 2023-11-06 21:42:06 | [diff] [blame] | 102 | }], |
dpapad | bea253c3 | 2022-04-01 08:52:57 | [diff] [blame] | 103 | 'no-throw-literal': 'error', |
dpapad | 2d023f9 | 2022-03-29 21:11:23 | [diff] [blame] | 104 | 'no-trailing-spaces': 'error', |
Demetrios Papadopoulos | cf870bde | 2019-11-26 19:14:34 | [diff] [blame] | 105 | 'no-var': 'error', |
| 106 | 'prefer-const': 'error', |
dpapad | ab65692 | 2022-04-01 00:01:11 | [diff] [blame] | 107 | 'quotes': ['error', 'single', {allowTemplateLiterals: true}], |
dpapad | 1e511b1 | 2017-05-31 03:31:26 | [diff] [blame] | 108 | 'semi': ['error', 'always'], |
dpapad | 2674f928 | 2017-05-31 19:40:39 | [diff] [blame] | 109 | |
dpapad | d631d17 | 2022-07-20 17:29:01 | [diff] [blame] | 110 | // https://google.github.io/styleguide/jsguide.html#features-one-variable-per-declaration |
| 111 | 'one-var': ['error', { |
| 112 | let: 'never', |
| 113 | const: 'never', |
| 114 | }], |
| 115 | |
dpapad | ff82d51 | 2017-05-25 18:40:15 | [diff] [blame] | 116 | // TODO(dpapad): Add more checks according to our styleguide. |
| 117 | }, |
dpapad | ac97d7b6 | 2021-06-14 07:20:26 | [diff] [blame] | 118 | |
| 119 | 'overrides': [{ |
| 120 | 'files': ['**/*.ts'], |
dpapad | 1c17e50 | 2023-10-04 20:31:12 | [diff] [blame] | 121 | 'parser': './third_party/node/node_modules/@typescript-eslint/parser/dist/index.js', |
dpapad | fa76653e | 2022-02-24 09:08:40 | [diff] [blame] | 122 | 'plugins': [ |
| 123 | '@typescript-eslint', |
| 124 | ], |
| 125 | 'rules': { |
| 126 | 'no-unused-vars': 'off', |
| 127 | '@typescript-eslint/no-unused-vars': [ |
| 128 | 'error', { |
| 129 | argsIgnorePattern: '^_', |
| 130 | varsIgnorePattern: '^_', |
| 131 | } |
dpapad | 5df2304 | 2022-03-21 19:22:37 | [diff] [blame] | 132 | ], |
| 133 | |
dpapad | 39be89f6 | 2023-10-10 20:35:05 | [diff] [blame] | 134 | // https://google.github.io/styleguide/tsguide.html#array-constructor |
| 135 | // Note: The rule below only partially enforces the styleguide, since it |
| 136 | // it does not flag invocations of the constructor with a single |
| 137 | // parameter. |
| 138 | 'no-array-constructor': 'off', |
| 139 | '@typescript-eslint/no-array-constructor': 'error', |
| 140 | |
| 141 | // https://google.github.io/styleguide/tsguide.html#automatic-semicolon-insertion |
dpapad | 5df2304 | 2022-03-21 19:22:37 | [diff] [blame] | 142 | 'semi': 'off', |
| 143 | '@typescript-eslint/semi': ['error'], |
dpapad | a2eca91 | 2022-03-28 09:36:44 | [diff] [blame] | 144 | |
dpapad | 0927916 | 2022-07-07 18:34:06 | [diff] [blame] | 145 | // https://google.github.io/styleguide/tsguide.html#arrayt-type |
| 146 | '@typescript-eslint/array-type': ['error', { |
| 147 | default: 'array-simple', |
| 148 | }], |
| 149 | |
dpapad | d533279 | 2022-07-11 21:15:14 | [diff] [blame] | 150 | // https://google.github.io/styleguide/tsguide.html#type-assertions-syntax |
| 151 | '@typescript-eslint/consistent-type-assertions': ['error', { |
| 152 | assertionStyle: 'as', |
| 153 | }], |
| 154 | |
dpapad | b1a935f | 2022-07-27 16:49:06 | [diff] [blame] | 155 | // https://google.github.io/styleguide/tsguide.html#interfaces-vs-type-aliases |
dpapad | dc88a09 | 2023-10-04 00:19:04 | [diff] [blame] | 156 | '@typescript-eslint/consistent-type-definitions': ['error', 'interface'], |
| 157 | |
rbpotter | 38a7f0c | 2024-02-12 19:01:40 | [diff] [blame] | 158 | // https://google.github.io/styleguide/tsguide.html#import-type |
| 159 | '@typescript-eslint/consistent-type-imports': 'error', |
| 160 | |
dpapad | dc88a09 | 2023-10-04 00:19:04 | [diff] [blame] | 161 | // https://google.github.io/styleguide/tsguide.html#visibility |
| 162 | '@typescript-eslint/explicit-member-accessibility': ['error', { |
| 163 | accessibility: 'no-public', |
| 164 | overrides: { |
| 165 | parameterProperties: 'off', |
| 166 | }, |
| 167 | }], |
dpapad | b1a935f | 2022-07-27 16:49:06 | [diff] [blame] | 168 | |
dpapad | 684f427b | 2022-05-10 05:47:02 | [diff] [blame] | 169 | // https://google.github.io/styleguide/jsguide.html#naming |
dpapad | 50adcde | 2022-05-07 00:05:52 | [diff] [blame] | 170 | '@typescript-eslint/naming-convention': [ |
| 171 | 'error', |
| 172 | { |
dpapad | 684f427b | 2022-05-10 05:47:02 | [diff] [blame] | 173 | selector: ['class', 'interface', 'typeAlias', 'enum', 'typeParameter'], |
dpapad | 978c1dab | 2022-10-19 18:12:09 | [diff] [blame] | 174 | format: ['StrictPascalCase'], |
| 175 | filter: { |
| 176 | regex: '^(' + |
| 177 | // Exclude TypeScript defined interfaces HTMLElementTagNameMap |
| 178 | // and HTMLElementEventMap. |
| 179 | 'HTMLElementTagNameMap|HTMLElementEventMap|' + |
| 180 | // Exclude native DOM types which are always named like HTML<Foo>Element. |
| 181 | 'HTML[A-Za-z]{0,}Element|' + |
| 182 | // Exclude native DOM interfaces. |
| 183 | 'UIEvent|UIEventInit|DOMError|' + |
| 184 | // Exclude the deprecated WebUIListenerBehavior interface. |
| 185 | 'WebUIListenerBehavior)$', |
| 186 | match: false, |
| 187 | }, |
dpapad | 50adcde | 2022-05-07 00:05:52 | [diff] [blame] | 188 | }, |
| 189 | { |
dpapad | 50adcde | 2022-05-07 00:05:52 | [diff] [blame] | 190 | selector: 'enumMember', |
| 191 | format: ['UPPER_CASE'], |
| 192 | }, |
dpapad | 684f427b | 2022-05-10 05:47:02 | [diff] [blame] | 193 | { |
| 194 | selector: 'classMethod', |
dpapad | e9ae77d | 2022-12-12 17:02:00 | [diff] [blame] | 195 | format: ['strictCamelCase'], |
dpapad | b7286e8 | 2022-06-16 01:33:27 | [diff] [blame] | 196 | modifiers: ['public'], |
| 197 | }, |
| 198 | { |
| 199 | selector: 'classMethod', |
dpapad | e9ae77d | 2022-12-12 17:02:00 | [diff] [blame] | 200 | format: ['strictCamelCase'], |
dpapad | b7286e8 | 2022-06-16 01:33:27 | [diff] [blame] | 201 | modifiers: ['private'], |
dpapad | 684f427b | 2022-05-10 05:47:02 | [diff] [blame] | 202 | trailingUnderscore: 'allow', |
dpapad | d5da00a9 | 2023-05-26 17:58:23 | [diff] [blame] | 203 | |
| 204 | // Disallow the 'Tap_' suffix, in favor of 'Click_' in event handlers. |
| 205 | // Note: Unfortunately this ESLint rule does not provide a way to |
| 206 | // customize the error message to better inform developers. |
| 207 | custom: { |
| 208 | regex: '^on[a-zA-Z0-9]+Tap$', |
| 209 | match: false, |
| 210 | }, |
dpapad | 684f427b | 2022-05-10 05:47:02 | [diff] [blame] | 211 | }, |
| 212 | { |
dpapad | e9d1fa7 | 2022-06-09 17:42:50 | [diff] [blame] | 213 | selector: 'classProperty', |
| 214 | format: ['UPPER_CASE'], |
dpapad | 588bb91 | 2022-06-15 05:35:33 | [diff] [blame] | 215 | modifiers: ['private', 'static', 'readonly'], |
| 216 | }, |
| 217 | { |
| 218 | selector: 'classProperty', |
| 219 | format: ['UPPER_CASE'], |
| 220 | modifiers: ['public', 'static', 'readonly'], |
dpapad | e9d1fa7 | 2022-06-09 17:42:50 | [diff] [blame] | 221 | }, |
| 222 | { |
| 223 | selector: 'classProperty', |
| 224 | format: ['camelCase'], |
dpapad | 588bb91 | 2022-06-15 05:35:33 | [diff] [blame] | 225 | modifiers: ['public'], |
| 226 | }, |
| 227 | { |
| 228 | selector: 'classProperty', |
| 229 | format: ['camelCase'], |
| 230 | modifiers: ['private'], |
dpapad | e9d1fa7 | 2022-06-09 17:42:50 | [diff] [blame] | 231 | trailingUnderscore: 'allow', |
| 232 | }, |
| 233 | { |
dpapad | 684f427b | 2022-05-10 05:47:02 | [diff] [blame] | 234 | selector: 'parameter', |
| 235 | format: ['camelCase'], |
| 236 | leadingUnderscore: 'allow', |
| 237 | }, |
dpapad | b17f3beb | 2022-05-10 21:34:13 | [diff] [blame] | 238 | { |
| 239 | selector: 'function', |
| 240 | format: ['camelCase'], |
| 241 | }, |
dpapad | 50adcde | 2022-05-07 00:05:52 | [diff] [blame] | 242 | ], |
| 243 | |
dpapad | 39be89f6 | 2023-10-10 20:35:05 | [diff] [blame] | 244 | // https://google.github.io/styleguide/tsguide.html#member-property-declarations |
dpapad | a2eca91 | 2022-03-28 09:36:44 | [diff] [blame] | 245 | '@typescript-eslint/member-delimiter-style': ['error', { |
| 246 | multiline: { |
| 247 | delimiter: 'comma', |
| 248 | requireLast: true, |
| 249 | }, |
| 250 | singleline: { |
| 251 | delimiter: 'comma', |
| 252 | requireLast: false, |
| 253 | }, |
| 254 | overrides: { |
| 255 | interface: { |
| 256 | multiline: { |
| 257 | delimiter: 'semi', |
| 258 | requireLast: true, |
| 259 | }, |
| 260 | singleline: { |
| 261 | delimiter: 'semi', |
| 262 | requireLast: false, |
| 263 | }, |
| 264 | }, |
| 265 | }, |
dpapad | b5aaf1a | 2022-07-11 18:00:41 | [diff] [blame] | 266 | }], |
| 267 | |
| 268 | // https://google.github.io/styleguide/tsguide.html#wrapper-types |
| 269 | '@typescript-eslint/ban-types': ['error', { |
| 270 | extendDefaults: false, |
| 271 | types: { |
| 272 | String: { |
| 273 | message: 'Use string instead', |
| 274 | fixWith: 'string', |
| 275 | }, |
| 276 | Boolean: { |
| 277 | message: 'Use boolean instead', |
| 278 | fixWith: 'boolean', |
| 279 | }, |
| 280 | Number: { |
| 281 | message: 'Use number instead', |
| 282 | fixWith: 'number', |
| 283 | }, |
| 284 | Symbol: { |
| 285 | message: 'Use symbol instead', |
| 286 | fixWith: 'symbol', |
| 287 | }, |
| 288 | BigInt: { |
| 289 | message: 'Use bigint instead', |
| 290 | fixWith: 'bigint', |
| 291 | }, |
| 292 | } |
| 293 | }], |
dpapad | e9886e4 | 2024-06-13 18:02:25 | [diff] [blame^] | 294 | |
| 295 | // https://google.github.io/styleguide/tsguide.html#ts-ignore |
| 296 | '@typescript-eslint/ban-ts-comment': ['error', {'ts-ignore': true}], |
dpapad | fa76653e | 2022-02-24 09:08:40 | [diff] [blame] | 297 | } |
dpapad | ac97d7b6 | 2021-06-14 07:20:26 | [diff] [blame] | 298 | }] |
dpapad | ff82d51 | 2017-05-25 18:40:15 | [diff] [blame] | 299 | }; |