eslint-config-beslogic 2.4.14 → 3.0.0

package/.eslintrc.js

@@ -8,14 +8,14 @@ module.exports = {
   "extends": [
     "./node-js",
     "plugin:eslint-plugin/all",
-    "./dprint",
+    "./stylistic", // For the sake of testing
     "./json-like"
   ],
   "parserOptions": {
     "sourceType": "script"
   },
   "rules": {
-    "max-len": "off",
+    "@stylistic/max-len": "off",
     "max-lines": "off",
     // We're a script project, using the console is fine
     // (same config as in jest)
@@ -35,6 +35,10 @@ module.exports = {
       {
         "case": "kebabCase"
       }
-    ]
+    ],
+
+    // Conflicts with dprint's bracePosition = sameLineUnlessHanging
+    "@stylistic/brace-style": "off",
+    "@stylistic/indent": "off"
   }
 }

package/CHANGELOG.md

@@ -1,5 +1,80 @@
 # Changelog
 
+## Unreleased
+
+### Important migration points
+
+- Remove `eslint-plugin-unused-imports` from your pre-commit configs
+- Uninstall `eslint-plugin-total-functions`
+- Remove the `beslogic/dprint` preset. It no longer exists and is no longer needed
+- If not using dprint, please add the `beslogic/stylistic` preset
+- If you prefered stricter linting, add `beslogic/extra-strict` to your presets
+- If you have ``parserOptions.project` configured, try removing it.
+- If you only have `tsconfig.json` files and no complex `tsconfig.*.json`, you can also try this instead:
+
+  ```js
+  module.exports = {
+    parserOptions: {
+      projectService: {
+        allowDefaultProject: []
+      },
+      // Still needed for plugins that haven't updated to typescript-eslint@8 yet
+      // Namely: eslint-plugin-sonarjs
+      EXPERIMENTAL_useProjectService: true
+    }
+  }
+  ```
+
+`eslint-plugin-sonarjs@2` added a ton of rules, a lot of which duplicates base ESLint rules and other plugins we have. _**Please report those you find**_
+
+### Changes
+
+#### Provided an `extra-strict` preset
+
+- Removed `eslint-plugin-total-functions`
+- Replaced rule `total-functions/no-unsafe-type-assertion` with a configured `no-autofix/@typescript-eslint/consistent-type-assertions` in the new `extra-strict` addon preset that warns on any non-param, non-unknown object cast
+- Disabled `@typescript-eslint/no-invalid-this` in the `extra-strict` addon preset as we assume `"noImplicitThis": true`
+- Moved the following rules to a new `extra-strict` addon preset:
+  - `@typescript-eslint/no-unnecessary-boolean-literal-compare`
+  - `@typescript-eslint/no-unnecessary-condition`
+  - `@typescript-eslint/prefer-nullish-coalescing`
+  - `etc/no-enum`
+  - `no-autofix/@angular-eslint/template/i18n`
+- The `typescript` preset should now work with `"strict": false` in `tsconfig.json`
+- Enabled rule `arrow-body-style` to disallow using "immediate returns" in arrow functions
+- Moved the `etc/no-commented-out-code` rule to the `extra-strict` preset
+
+#### Updated to typescript-eslint v8
+
+- Bumped `typescript-eslint` to `^8.0`
+- Enabled `typescript-eslint`'s [Project Service](https://typescript-eslint.io/blog/announcing-typescript-eslint-v8#project-service). Replacing `settings.parserOptions.project = [...]` and `settings.parserOptions.deprecated__createDefaultProgram = true` in `typescript` and `angular` presets with `settings.parserOptions.projectService = true`
+- Enabled `allowRegExp` for `restrict-template-expressions`
+  - As of `typescript-eslint@8.6`: `Error`, `URL` and `URLSearchParams` are also allowed by default (see <https://typescript-eslint.io/rules/restrict-template-expressions/#allow>)
+- Stopped using `@babel/eslint-parser` by default for Javascript presets. TypeScript projects will use the TypeScript parser from the typescript preset. Scripts don't use Babel. And we don't expect pure-JS projects to use experimental features (if they do, they can configure `@babel/eslint-parser` themselves).
+- Bumped `eslint-plugin-jest` to `^28.7`
+- Removed the `dprint` preset. Those stylistic rules are simply no longer enabled by default
+- Created a new preset `stylistic` to allow re-enabling rules taken care of by dprint
+- Migrated lots of rules from ESLint, typescript-eslint and eslint-plugin-autofix to ESLint Stylistic
+
+#### Various
+
+- Bumped `dprint` to `^0.47.2` to autodiscover `.dprint.jsonc`
+- Bumped the minimal EcmaScript version to 2022 in ESLint and TSConfig settings
+- `.eslintrc.mjs` is now treated the same as `.eslintrc.js` and `.eslintrc.cjs`
+- Bumped `eslint-plugin-no-autofix` to `^2.0`
+- Dropped support for node 17
+- Bumped `eslint-plugin-sonarjs` to `^2.0`
+  - This adds a ton of rules, including some duplicating base ESLint rules and other plugins we have. _**Please report those you find**_
+- Trying out replacing the `eslint-plugin-unused-imports` plugin by `sonarjs/unused-import` rule
+- Bumped `eslint-plugin-import` to `^2.30` which fixes `import/newline-after-import`'s `considerComments`+`exactCount`
+- Allow single-line ternary when they fit on a single line
+- Update the `operator-linebreak` rule to put the operator in front (along with an update to dprint's `operatorPosition`). Both of which default to left-side and we now aggree more with. This also aligns with our Ruff preset
+- Re-enabled `react/jsx-no-leaked-render` in TypeScript (TSX) files
+- Disabled crashing rules from Cartant: `rxjs/no-implicit-any-catch` and `etc/throw-error`. Please upvote:
+  - <https://github.com/cartant/eslint-plugin-etc/issues/47>
+  - <https://github.com/cartant/eslint-plugin-rxjs/issues/122>
+  - <https://github.com/cartant/eslint-plugin-rxjs/issues/132>
+
 ## 2.4.14
 
 - Updated pre-commit.ci documentation

package/README.md

@@ -2,9 +2,7 @@
 
 ## Installation and usage
 
-Run `npm i  --save-dev eslint-config-beslogic`\
-For extra strict TypeScript linting, you can also install [eslint-plugin-total-functions](https://www.npmjs.com/package/eslint-plugin-total-functions): `npm i --save-dev eslint-plugin-total-functions`\
-The installation will be automatically detected and configured.
+Run `npm i  --save-dev eslint-config-beslogic`
 
 Depending on your needs, you will need to also install other peer dependencies for linting.
 
@@ -19,15 +17,7 @@ These are configuration files you'll have to update manually to best work with t
 
 ### `package.json`
 
-Add the following `scripts`:
-
-```json
-"scripts": {
-  "eslint": "eslint ./ --ignore-path .gitignore",
-  "lint": "dprint check && npm run eslint",
-  "lint:fix": "dprint fmt && npm run eslint -- --fix"
-},
-```
+Read the [`package.json`](https://github.com/BesLogic/shared-configs#packagejson-for-dprint-and-eslint) section of our Shared Configs for one-liner shortcut scripts you can use.
 
 ### Base `tsconfig.json`
 
@@ -47,17 +37,15 @@ Extend the following in your base `tsconfig.json` (follow link for more details)
 
 "No Unchecked Indexed Access" makes index accesses even stricter. You may not want it in your project or in your tests. You can disable it with `"compilerOptions": { "noUncheckedIndexedAccess": false }`.
 
-If deploying / bundling with an Angular npm package, also set `angularCompilerOptions": { "strictMetadataEmit": true }`
+If deploying / bundling with an Angular npm package, also set `angularCompilerOptions": { "strictMetadataEmit": true }`.
 
 ### `.vscode/settings.json` and `.vscode/extensions.json`
 
-<!-- Keep in sync with https://dev.azure.com/Beslogic/Beslogic%20Architecture%20Reference/_wiki/wikis/Beslogic-Architecture-Reference.wiki/757/VSCode-settings-and-extensions-(.vscode-.code-workspace) -->
-
-Read and copy from [VSCode Settings and Extensions](https://socket.dev/npm/package/eslint-config-beslogic/files/latest/.vscode)
+Read and copy the sections for the languages you use from [BesLogic/shared-configs/.vscode](https://github.com/BesLogic/shared-configs/tree/main/.vscode).
 
-### `dprint.json`
+### `.dprint.jsonc`
 
-If using the [`beslogic/dprint` preset](#beslogicdprint), use the following:
+Extend the following in your base `.dprint.jsonc` (follow link for more details):
 
 ```jsonc
 {
@@ -104,21 +92,26 @@ As you'll easily bust the 250MiB limit on the free tier, omit the following depe
 - `dprint`
 - `@eslint-community/eslint-plugin-eslint-comments`
 - `eslint-import-resolver-typescript`
-- `eslint-plugin-total-functions`
 - `esling-plugin-no-autofix`
 - `eslint-plugin-testing-library`
 - `eslint-plugin-jest-formatting`
 - `eslint-plugin-jest`
 - `eslint-plugin-etc`
 
-If using TypeScript, in your ESLint config, you'll _have to_ use the experimental project service, or you'll run into OOM issues:
-
-```json
-"parser": "@typescript-eslint/parser",
-"parserOptions": {
-  // Experimental, but causes pre-commit to actually find the project and avoids OOM
-  "EXPERIMENTAL_useProjectService": true,
-},
+If using TypeScript and pre-commit.ci, in your ESLint config you'll _have to_ use the project service, or you'll run into OOM issues:
+
+```js
+module.exports = {
+  parser: "@typescript-eslint/parser",
+  parserOptions: {
+    projectService: {
+      allowDefaultProject: []
+    },
+    // Still needed for plugins that haven't updated to typescript-eslint@8 yet
+    // Namely: eslint-plugin-sonarjs
+    EXPERIMENTAL_useProjectService: true
+  }
+}
 ```
 
 ## Patched packages
@@ -159,8 +152,7 @@ Basic TypeScript-specific configs and `TSX` support, is extended by all other Ty
 
 For the linting to be fully effective, please make sure that your base `tsconfig.json` extends `eslint-config-beslogic/tsconfig.X.X.json` (see [Parallel configurations](#parallel-configurations)).
 
-For extra strict TypeScript linting, you can also install [eslint-plugin-total-functions](https://www.npmjs.com/package/eslint-plugin-total-functions): `npm i --save-dev eslint-plugin-total-functions`\
-The installation will be automatically detected and configured.
+If you want extra strict linting, consider adding the [`beslogic/extra-strict` preset](#beslogicextra-strict).
 
 ### `beslogic/node-js`
 
@@ -246,15 +238,21 @@ Add the following `devDependencies` to your `package.json`:
 npm install --save-dev eslint-plugin-storybook
 ```
 
-## Stylistic Presets
+## Stylistic and pedantic Presets
 
 If used, these presets should always come last, in this specific order.
 
-### `beslogic/dprint`
+### `beslogic/extra-strict`
+
+Enables extra-strict rules for `beslogic/typescript` and `beslogic/angular`, and forces that your `tsconfig.json` is strictly configured.
 
-Disables all stylistic rules that are taken care of by [dprint](https://dprint.dev/plugins/typescript/).
+### `beslogic/stylistic`
 
-You must also extend the base configuration in a `dprint.json` file (see [Parallel configurations](#parallel-configurations)).
+Enables stylistic rules that would be taken care of by [dprint](https://dprint.dev/plugins/typescript/). Please read [What About Formatting?](https://typescript-eslint.io/users/what-about-formatting/)
+
+We don't recommend using this preset, and to use dprint instead (see [Parallel configurations](#parallel-configurations)).
+
+But if for some reason, `dprint` isn't an option for you, this preset follows our standards using ESLint.
 
 ### `beslogic/json-like`
 
@@ -263,3 +261,13 @@ Enforces stylistic rules for JS objects to look like valid JSON objects.
 ## Changelog
 
 You can view the changelog under the [Code Tab](https://www.npmjs.com/package/eslint-config-beslogic?activeTab=code), or at <https://socket.dev/npm/package/eslint-config-beslogic/files/latest/CHANGELOG.md>
+
+<!-- (keep this section hidden from public README)
+## Dev testing:
+
+To test updates to ESLint-Config-Beslogic in other existing projects:
+1. At the root of this repo, run `npm i` then `npm run yalc`
+2. In the target project, run `npm i --save-dev yalc` then `yalc add eslint-config-beslogic`
+3. Update the project's .eslintrc config file as needed
+4. Run your linting
+-->

package/TODO-SONARJS_CHECK_IF_DUPLICATED.json

@@ -0,0 +1,175 @@
+// These are the rules I haven't yet validated if they duplicate existing rules
+{
+  "sonarjs/alt-text": "error",
+  "sonarjs/anchor-has-content": "error",
+  "sonarjs/anchor-is-valid": "error",
+  "sonarjs/anchor-precedence": "error",
+  "sonarjs/argument-type": "error",
+  "sonarjs/arguments-order": "error",
+  "sonarjs/array-callback-without-return": "error",
+  "sonarjs/assertions-in-tests": "error",
+  "sonarjs/bitwise-operators": "error",
+  "sonarjs/call-argument-line": "error",
+  "sonarjs/certificate-transparency": "error",
+  "sonarjs/chai-determinate-assertion": "error",
+  "sonarjs/class-name": "error",
+  "sonarjs/code-eval": "error",
+  "sonarjs/comma-or-logical-or-case": "error",
+  "sonarjs/confidential-information-logging": "error",
+  "sonarjs/content-length": "error",
+  "sonarjs/content-security-policy": "error",
+  "sonarjs/cookie-no-httponly": "error",
+  "sonarjs/cors": "error",
+  "sonarjs/csrf": "error",
+  "sonarjs/different-types-comparison": "error",
+  "sonarjs/disabled-auto-escaping": "error",
+  "sonarjs/disabled-resource-integrity": "error",
+  "sonarjs/disabled-timeout": "error",
+  "sonarjs/encryption-secure-mode": "error",
+  "sonarjs/file-permissions": "error",
+  "sonarjs/file-uploads": "error",
+  "sonarjs/fixme-tag": "error",
+  "sonarjs/for-loop-increment-sign": "error",
+  "sonarjs/frame-ancestors": "error",
+  "sonarjs/function-inside-loop": "error",
+  "sonarjs/future-reserved-words": "error",
+  "sonarjs/generator-without-yield": "error",
+  "sonarjs/hashing": "error",
+  "sonarjs/hidden-files": "error",
+  "sonarjs/html-has-lang": "error",
+  "sonarjs/in-operator-type-error": "error",
+  "sonarjs/inconsistent-function-call": "error",
+  "sonarjs/index-of-compare-to-positive-number": "error",
+  "sonarjs/insecure-cookie": "error",
+  "sonarjs/insecure-jwt-token": "error",
+  "sonarjs/inverted-assertion-arguments": "error",
+  "sonarjs/label-has-associated-control": "error",
+  "sonarjs/label-position": "error",
+  "sonarjs/link-with-target-blank": "error",
+  "sonarjs/max-switch-cases": "error",
+  "sonarjs/media-has-caption": "error",
+  "sonarjs/misplaced-loop-counter": "error",
+  "sonarjs/mouse-events-a11y": "error",
+  "sonarjs/new-cap": "error",
+  "sonarjs/new-operator-misuse": "error",
+  "sonarjs/no-accessor-field-mismatch": "error",
+  "sonarjs/no-all-duplicated-branches": "error",
+  "sonarjs/no-alphabetical-sort": "error",
+  "sonarjs/no-angular-bypass-sanitization": "error",
+  "sonarjs/no-array-delete": "error",
+  "sonarjs/no-associative-arrays": "error",
+  "sonarjs/no-async-constructor": "error",
+  "sonarjs/no-base-to-string": "error",
+  "sonarjs/no-case-label-in-switch": "error",
+  "sonarjs/no-clear-text-protocols": "error",
+  "sonarjs/no-code-after-done": "error",
+  "sonarjs/no-collection-size-mischeck": "error",
+  "sonarjs/no-dead-store": "error",
+  "sonarjs/no-duplicate-in-composite": "error",
+  "sonarjs/no-duplicated-branches": "error",
+  "sonarjs/no-element-overwrite": "error",
+  "sonarjs/no-empty-after-reluctant": "error",
+  "sonarjs/no-empty-collection": "error",
+  "sonarjs/no-empty-test-file": "error",
+  "sonarjs/no-equals-in-for-termination": "error",
+  "sonarjs/no-exclusive-tests": "error",
+  "sonarjs/no-extra-arguments": "error",
+  "sonarjs/no-global-this": "error",
+  "sonarjs/no-globals-shadowing": "error",
+  "sonarjs/no-gratuitous-expressions": "error",
+  "sonarjs/no-hardcoded-credentials": "error",
+  "sonarjs/no-hardcoded-ip": "error",
+  "sonarjs/no-identical-conditions": "error",
+  "sonarjs/no-identical-expressions": "error",
+  "sonarjs/no-identical-functions": "error",
+  "sonarjs/no-ignored-exceptions": "error",
+  "sonarjs/no-ignored-return": "error",
+  "sonarjs/no-implicit-global": "error",
+  "sonarjs/no-in-misuse": "error",
+  "sonarjs/no-incomplete-assertions": "error",
+  "sonarjs/no-infinite-loop": "error",
+  "sonarjs/no-internal-api-use": "error",
+  "sonarjs/no-intrusive-permissions": "error",
+  "sonarjs/no-invalid-await": "error",
+  "sonarjs/no-invariant-returns": "error",
+  "sonarjs/no-inverted-boolean-check": "error",
+  "sonarjs/no-ip-forward": "error",
+  "sonarjs/no-labels": "error",
+  "sonarjs/no-literal-call": "error",
+  "sonarjs/no-mime-sniff": "error",
+  "sonarjs/no-misleading-array-reverse": "error",
+  "sonarjs/no-mixed-content": "error",
+  "sonarjs/no-nested-assignment": "error",
+  "sonarjs/no-nested-functions": "error",
+  "sonarjs/no-nested-template-literals": "error",
+  "sonarjs/no-one-iteration-loop": "error",
+  "sonarjs/no-parameter-reassignment": "error",
+  "sonarjs/no-primitive-wrappers": "error",
+  "sonarjs/no-redundant-assignments": "error",
+  "sonarjs/no-redundant-boolean": "error",
+  "sonarjs/no-redundant-jump": "error",
+  "sonarjs/no-redundant-optional": "error",
+  "sonarjs/no-referrer-policy": "error",
+  "sonarjs/no-same-argument-assert": "error",
+  "sonarjs/no-same-line-conditional": "error",
+  "sonarjs/no-selector-parameter": "error",
+  "sonarjs/no-self-compare": "error",
+  "sonarjs/no-self-import": "error",
+  "sonarjs/no-skipped-test": "error",
+  "sonarjs/no-small-switch": "error",
+  "sonarjs/no-table-as-layout": "error",
+  "sonarjs/no-try-promise": "error",
+  "sonarjs/no-undefined-argument": "error",
+  "sonarjs/no-unenclosed-multiline-block": "error",
+  "sonarjs/no-unsafe-unzip": "error",
+  "sonarjs/no-unthrown-error": "error",
+  "sonarjs/no-unused-collection": "error",
+  "sonarjs/no-use-of-empty-return-value": "error",
+  "sonarjs/no-useless-call": "error",
+  "sonarjs/no-useless-constructor": "error",
+  "sonarjs/no-useless-increment": "error",
+  "sonarjs/no-useless-intersection": "error",
+  "sonarjs/no-vue-bypass-sanitization": "error",
+  "sonarjs/no-weak-cipher": "error",
+  "sonarjs/no-weak-keys": "error",
+  "sonarjs/non-existent-operator": "error",
+  "sonarjs/null-dereference": "error",
+  "sonarjs/object-alt-content": "error",
+  "sonarjs/os-command": "error",
+  "sonarjs/post-message": "error",
+  "sonarjs/prefer-default-last": "error",
+  "sonarjs/prefer-for-of": "error",
+  "sonarjs/prefer-function-type": "error",
+  "sonarjs/prefer-namespace-keyword": "error",
+  "sonarjs/prefer-promise-shorthand": "error",
+  "sonarjs/prefer-single-boolean-return": "error",
+  "sonarjs/prefer-type-guard": "error",
+  "sonarjs/prefer-while": "error",
+  "sonarjs/production-debug": "error",
+  "sonarjs/pseudo-random": "error",
+  "sonarjs/public-static-readonly": "error",
+  "sonarjs/publicly-writable-directories": "error",
+  "sonarjs/reduce-initial-value": "error",
+  "sonarjs/redundant-type-aliases": "error",
+  "sonarjs/regex-complexity": "error",
+  "sonarjs/session-regeneration": "error",
+  "sonarjs/sql-queries": "error",
+  "sonarjs/stable-tests": "error",
+  "sonarjs/stateful-regex": "error",
+  "sonarjs/strict-transport-security": "error",
+  "sonarjs/super-invocation": "error",
+  "sonarjs/table-header-reference": "error",
+  "sonarjs/table-header": "error",
+  "sonarjs/test-check-exception": "error",
+  "sonarjs/unnecessary-character-escapes": "error",
+  "sonarjs/unused-named-groups": "error",
+  "sonarjs/unverified-certificate": "error",
+  "sonarjs/unverified-hostname": "error",
+  "sonarjs/updated-const-var": "error",
+  "sonarjs/updated-loop-counter": "error",
+  "sonarjs/use-type-alias": "error",
+  "sonarjs/void-use": "error",
+  "sonarjs/weak-ssl": "error",
+  "sonarjs/x-powered-by": "error",
+  "sonarjs/xml-parser-xxe": "error"
+}

package/angular.js

@@ -35,8 +35,8 @@ const noRestrictedSyntax = [
     "selector":
       "PropertyDefinition[definite=true]:not(:has(Decorator[expression.callee.name=\"Input\"] Property[key.name=\"required\"][value.value=true]))",
     "message":
-      "Forbidden non-null property assertion. https://typescript-eslint.io/rules/no-non-null-assertion . " +
-      "This is only allowed on required component @Input({ required:true }). https://angular.io/api/core/Input#required"
+      "Forbidden non-null property assertion. https://typescript-eslint.io/rules/no-non-null-assertion . "
+      + "This is only allowed on required component @Input({ required:true }). https://angular.io/api/core/Input#required"
   },
   {
     "selector":
@@ -55,6 +55,10 @@ const typeDefinitionOverride = typescriptConfig
   .overrides
   ?.find(override => override.files === "*.d.ts")
 
+const noAutofixRreferArrowFunctionsConfig = javascriptConfig
+  .rules
+  ?.["no-autofix/prefer-arrow/prefer-arrow-functions"]
+  ?.[1]
 const preferArrowFunctionsConfig = javascriptConfig
   .rules
   ?.["prefer-arrow/prefer-arrow-functions"]
@@ -118,53 +122,8 @@ module.exports = {
         // and it was added in 14.2.0. Not enforcing it for now allows a wider range of versions.
         // It's kind of annoying to deal with without autofixes anyway.
         "no-autofix/@angular-eslint/template/attributes-order": "off",
+        // Don't autofix as these should be handled manually, also configured in extra-strict preset
         "@angular-eslint/template/i18n": "off",
-        "no-autofix/@angular-eslint/template/i18n": [
-          // Too many TODO and needs per-project extensions to justify this as "error"
-          hasNoAutofix
-            ? "warn"
-            : "off",
-          {
-            // NOTE: doesn't work for text elements.
-            // See: https://github.com/angular-eslint/angular-eslint/issues/999
-            "boundTextAllowedPattern": "EmPassion",
-            "ignoreTags": [
-              // NOTE: Asked for those to be added in the base library
-              // See: https://github.com/angular-eslint/angular-eslint/issues/997
-              "meta",
-              "link",
-              "title",
-              "mat-drawer",
-              "mat-icon",
-              "ng-container",
-              ":svg:polygon"
-            ],
-            "ignoreAttributes": [
-              // NOTE: Asked for those to be added in the base library
-              // See: https://github.com/angular-eslint/angular-eslint/issues/997
-              "img[ngSrc]",
-              "input[accept]",
-              "cdkColumnDef",
-              "mat-accordion[displayMode]",
-              "mat-dialog-actions[align]",
-              "mat-form-field[appearance]",
-              "mat-form-field[floatLabel]",
-              "matTooltipPosition",
-              "panelClass",
-              // Our extended configs for non-Angular component
-              "data-testid",
-              "ion-input[autocapitalize]",
-              "ion-input[autofocus]",
-              "ion-input[enterkeyhint]",
-              "ion-input[inputmode]",
-              "ion-input[labelPlacement]",
-              "ion-select[interface]",
-              "mtxTooltipClass",
-              "size",
-              "slot"
-            ]
-          }
-        ],
         // This constantly requires wrapping images in an additional div or knowing the exact display size.
         // Doesn't support relative sizing either, not sure if worth even trying to enforce.
         "@angular-eslint/template/prefer-ngsrc": "off",
@@ -187,21 +146,12 @@ module.exports = {
         "plugin:@angular-eslint/template/process-inline-templates"
       ],
       "parserOptions": {
-        // TODO: Try "project": true https://typescript-eslint.io/blog/parser-options-project-true/
-        // This wouldn't work for tsconfig.spec.ts though
-        // https://typescript-eslint.io/blog/parser-options-project-true/#investigating-custom-tsconfig-names
-        // https://github.com/typescript-eslint/typescript-eslint/issues/7383#issuecomment-2073032501
-        // TODO: even better, try: https://typescript-eslint.io/packages/parser/#experimental_useprojectservice
-        // "EXPERIMENTAL_useProjectService": true,
         "project": [
           "tsconfig?(.*).json",
-          "projects/*/tsconfig?(.*).json",
-          "*/tsconfig?(.*).json" // For example: e2e/tsconfig.json
+          "**/tsconfig?(.*).json" // For example: e2e/tsconfig.json
         ],
-        // TODO: Consider use a tsconfig.eslint.json instead
-        // https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/README.md#parseroptionscreatedefaultprogram
-        "deprecated__createDefaultProgram": true,
         // Needed specifically for @typescript-eslint/consistent-type-imports
+        // https://typescript-eslint.io/rules/consistent-type-imports/#caveat-decorators--experimentaldecorators-true--emitdecoratormetadata-true
         "emitDecoratorMetadata": typescriptVersion >= 5
       },
       "rules": {
@@ -361,6 +311,14 @@ module.exports = {
         // https://github.com/bahmutov/eslint-rules/issues/54
         // TODO: consider https://github.com/jonaskello/eslint-plugin-functional/blob/HEAD/docs/rules/prefer-tacit.md
         "extra-rules/potential-point-free": "off",
+        "no-autofix/prefer-arrow/prefer-arrow-functions": [
+          "error",
+          {
+            ...noAutofixRreferArrowFunctionsConfig,
+            // https://github.com/TristonJ/eslint-plugin-prefer-arrow/issues/17
+            "classPropertiesAllowed": false
+          }
+        ],
         "prefer-arrow/prefer-arrow-functions": [
           "error",
           {