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