eslint-config-beslogic 2.0.2 → 2.2.1

package/CHANGELOG.md

@@ -1,5 +1,74 @@
 # Changelog
 
+## 2.2.1
+
+- Bumped `eslint-plugin-react` to `^7.33`
+  - Check context objects with `react/display-name`
+  - Added React 18 deprecations
+- Remove accidental recommendation to install `eslint-plugn-react-redux` from non-redux presets in `README.md`
+- Now enforce at least 1 newline after imports even with dprint preset.
+- Turn on `allowImplicit`, `checkForEach`, and `allowVoid` for rule `array-callback-return`
+- Bumped `eslint-plugin-total-functions` to `>=6.0`
+  - which bumps `typescript` dependency to `4.7.3`
+- Bumped `eslint-plugin-testing-library` to `^6.0` which enables a lot more rules by default
+- Bumped `eslint-plugin-unicorn` to `>=49.0`
+  - Re-enabled `prefer-event-target` in Angular preset
+  - Disabled deprecated `unicorn/no-unsafe-regex`
+  - Start checking arguments again for `unicorn/no-useless-undefined`
+    now that it can also autofix `undefined` defaults to optional parameters
+- Re-enabled `unicorn/no-nested-ternary`, which allows a single nested level if parenthesised
+- Bumped `eslint-plugin-sonarjs` to `^1.0`
+  - Updated vulnerable dependencies
+  - They updated their license
+- Bumped `eslint-plugin-simple-import-sort` to `>=12.0`
+- Bumped `eslint-plugin-regexp` to `^2.0`
+  - Reduced dependencies
+  - Removal of deprecated rules: <https://github.com/ota-meshi/eslint-plugin-regexp#removed>
+- Bumped `eslint-plugin-jest` to `^28.0.0`
+  - `jest/no-if` has been replaced by `jest/no-conditional-in-test`
+  - `jest/prefer-importing-jest-globals` is enabled by default, which removes needing a dependency
+    on often outdated `@types/jest` and removes the need for `types: ["jest"]` in `tsconfig.json`
+- Bumped `dprint` to `^0.40.2` to prepare for process plugins
+
+- Bumped `@typescript-eslint/eslint-plugin` to `^7.4`
+  - Re-enabled `@typescript-eslint/no-import-type-side-effects`
+    and set `@typescript-eslint/consistent-type-imports` back to default options
+  - Bumped `eslint` to `^8.56`
+  - which bumps `typescript` dependency to `4.7.5`
+- Prefered `@typescript-eslint/prefer-destructuring` over `autofix/prefer-destructuring`
+- Disabled `@typescript-eslint/key-spacing`, `autofix/func-call-spacing`,
+  and `lines-around-comment` in `dprint` preset
+
+- Patched-out prettier from nx generators
+- Used Search&Replace based package patching:
+  - Removes `patch-package` from dependencies
+  - Removes bundled git patches
+- Speedup check for `eslint-plugin-total-functions` install
+
+- Bumped `eslint-plugin-n` to `>=16.2`. Notably, this:
+  - improves support for ts-node
+  - Updates documentation links
+  - Sets Node 16 as the minimal default w/o having to configure `engines` in `package.json`
+  - Allow for automatic ts mapping detection
+- Re-enabled `n/no-unpublished-*` rules by default as it's now based on package being public/private
+- Set `ignoreTypeImport: true` by default for `n/no-unpublished-import`
+
+## 2.1.0
+
+- Bumped the following minimum versions:
+  - `@angular-eslint/eslint-plugin@>=16.2`
+  - `@angular-eslint/eslint-plugin-template@>=16.2`
+  - `@angular-eslint/template-parser@>=16.2`
+  - `@typescript-eslint/eslint-plugin@^6.2`
+  - `eslint-plugin-jest@^27.2.3`
+  - `eslint-plugin-unused-imports@^3.0`
+  - Assuming `node>=16`, `typescript>=4.5` and `@angular/compiler>=16.2.5`
+  - This should speedup config parsing that had to check for specific versions
+- Made `eslint-plugin-total-functions` optional due to its extra-strictness and it had to be
+  explicitly specified anyway due to version resolution issues
+- Added link to changelog in README
+- Improved detection of installed peer dependencies' version number to be more accurate and faster
+
 ## 2.0.2
 
 - Disabled the following rules in Storybook stories:
@@ -10,6 +79,7 @@
 
 ## 2.0.0
 
-- Created CHANGELOG.md
+- Created `CHANGELOG.md`
 - Changed the maximum line length from `120` to `100`
-- Split language-specific `editor.defaultFormatter` to work around <https://github.com/microsoft/vscode/issues/168411>
+- Split language-specific `editor.defaultFormatter` to work around
+  <https://github.com/microsoft/vscode/issues/168411>

package/README.md

@@ -2,7 +2,9 @@
 
 ## Installation and usage
 
-Run `npm i  --save-dev eslint-config-beslogic eslint-plugin-total-functions`
+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.
 
 Depending on your needs, you will need to also install other peer dependencies for linting.
 
@@ -17,7 +19,7 @@ These are configuration files you'll have to update manually to best work with t
 
 ### `package.json`
 
-Add the following `scripts` and `engines`:
+Add the following `scripts`:
 
 ```json
 "scripts": {
@@ -25,11 +27,6 @@ Add the following `scripts` and `engines`:
   "lint": "dprint check && npm run eslint",
   "lint:fix": "dprint fmt && npm run eslint -- --fix"
 },
-"// node 16": "14 won't install peer deps properly + prefer EventTarget (only available node 16)",
-"engines": {
-  "node": ">=16",
-  "npm": ">=8"
-}
 ```
 
 ### Base `tsconfig.json`
@@ -48,7 +45,7 @@ Extend the following in your base `tsconfig.json` (follow link for more details)
 "extends": "eslint-config-beslogic/tsconfig.4.3.json",
 ```
 
-"No Unchecked Indexed Access" makes index accesses even stricter. You may not want it in your project. You can disable it with `"compilerOptions": { "noUncheckedIndexedAccess": false }`.
+"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 }`
 
@@ -110,10 +107,13 @@ Basic javascript configs, is extended by all other configs except `beslogic/type
 
 ### `beslogic/typescript`
 
-Basic TypeScript-specific configs and `TSX` support, is extended by all other TypeScript exclusive configs. To avoid rules conflicts, it does **not** already extend from `beslogic/javascript`. So you should extend from both in a pure TypeScript project. You may also need to specify `parserOptions.project`. ie: `"parserOptions": { "project": ["tsconfig.json"] },` and `"parser": "@typescript-eslint/parser"`.
+Basic TypeScript-specific configs and `TSX` support, is extended by all other TypeScript exclusive configs. To avoid rules conflicts, it does **not** already extend from `beslogic/javascript`. So you should extend from both in a pure TypeScript project. You may also need to specify `parserOptions.project`. ie: `"parserOptions": { "project": ["tsconfig.json"] },` and `"parser": "@typescript-eslint/parser"` if you reconfigure some rules yourself.
 
 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.
+
 ### `beslogic/node-js`
 
 Configurations for NodeJS backends.
@@ -131,7 +131,7 @@ Configurations for projects using RxJs. Already extends and `beslogic/typescript
 Add the following `devDependencies` to your `package.json`:
 
 ```shell
-npm install --save-dev eslint-plugin-rxjs eslint-plugin-total-functions
+npm install --save-dev eslint-plugin-rxjs
 ```
 
 ### `beslogic/react`
@@ -141,7 +141,7 @@ Configurations for React projects. Comes with `JSX` support.
 Add the following `devDependencies` to your `package.json`:
 
 ```shell
-npm install --save-dev eslint-plugin-jsx-a11y eslint-plugin-react eslint-plugin-react-hooks eslint-plugin-react-redux eslint-plugin-react-prefer-function-component
+npm install --save-dev eslint-plugin-jsx-a11y eslint-plugin-react eslint-plugin-react-hooks eslint-plugin-react-prefer-function-component
 ```
 
 ### `beslogic/react-native`
@@ -151,7 +151,7 @@ Configurations for [React Native](https://reactnative.dev/) projects, including
 Add the following `devDependencies` to your `package.json`:
 
 ```shell
-npm install --save-dev eslint-plugin-react-native eslint-plugin-jsx-a11y eslint-plugin-react eslint-plugin-react-hooks eslint-plugin-react-redux eslint-plugin-react-prefer-function-component
+npm install --save-dev eslint-plugin-react-native eslint-plugin-jsx-a11y eslint-plugin-react eslint-plugin-react-hooks eslint-plugin-react-prefer-function-component
 ```
 
 ### `beslogic/react-redux`
@@ -161,7 +161,7 @@ Configurations for React projects using the Redux bindings. Already extends `bes
 Add the following `devDependencies` to your `package.json`:
 
 ```shell
-npm install --save-dev eslint-plugin-react-redux eslint-plugin-jsx-a11y eslint-plugin-react eslint-plugin-react-hooks eslint-plugin-react-redux eslint-plugin-react-prefer-function-component
+npm install --save-dev eslint-plugin-react-redux eslint-plugin-jsx-a11y eslint-plugin-react eslint-plugin-react-hooks eslint-plugin-react-prefer-function-component
 ```
 
 ### `beslogic/mui`
@@ -171,7 +171,7 @@ npm install --save-dev eslint-plugin-react-redux eslint-plugin-jsx-a11y eslint-p
 Add the following `devDependencies` to your `package.json`:
 
 ```shell
-npm install --save-dev eslint-plugin-mui eslint-plugin-jsx-a11y eslint-plugin-react eslint-plugin-react-hooks eslint-plugin-react-redux eslint-plugin-react-prefer-function-component eslint-plugin-total-functions
+npm install --save-dev eslint-plugin-mui eslint-plugin-jsx-a11y eslint-plugin-react eslint-plugin-react-hooks eslint-plugin-react-prefer-function-component
 ```
 
 ### `beslogic/angular`
@@ -184,7 +184,7 @@ If bundling with an npm package set `"strictMetadataEmit": true` under `angularC
 Add the following `devDependencies` to your `package.json`:
 
 ```shell
-npm install --save-dev @angular-eslint/eslint-plugin @angular-eslint/eslint-plugin-template @angular-eslint/template-parser github:BesLogic/eslint-plugin-rxjs-angular#dist eslint-plugin-angular-file-naming eslint-plugin-total-functions
+npm install --save-dev @angular-eslint/eslint-plugin @angular-eslint/eslint-plugin-template @angular-eslint/template-parser github:BesLogic/eslint-plugin-rxjs-angular#dist eslint-plugin-angular-file-naming
 ```
 
 ### `beslogic/storybook`
@@ -211,3 +211,7 @@ You must also extend the base configuration in a `dprint.json` file (see [Parall
 ### `beslogic/json-like`
 
 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>

package/angular.js

@@ -1,20 +1,10 @@
 // SPDX-License-Identifier: MIT
 // Copyright 2024 Beslogic Inc.
 
-const { getProjectModuleVersion, jestNoRestrictedSyntax } = require("./lib/utils")
+const { getModuleVersion, jestNoRestrictedSyntax } = require("./lib/utils")
 
-let angularEslintPluginVersion = 15
-try {
-  angularEslintPluginVersion = getProjectModuleVersion("@angular-eslint/eslint-plugin")
-} catch (error) {
-  console.warn(error)
-}
-let typescriptVersion = 4.3
-try {
-  typescriptVersion = getProjectModuleVersion("typescript")
-} catch (error) {
-  console.warn(error)
-}
+const angularEslintVersion = getModuleVersion("@angular-eslint/eslint-plugin", 16.2)
+const typescriptVersion = getModuleVersion("typescript", 4.7)
 
 const javascriptConfig = require("./javascript")
 const typescriptConfig = require("./typescript")
@@ -28,33 +18,29 @@ const noRestrictedSyntax = [
     "message":
       "Angular hooks need to be functions defined on the prototype: https://github.com/angular/angular/issues/7270#issuecomment-201137617"
   },
+  // TODO: Possibly implement that ourselves?
+  // Upstream request: https://github.com/MelvinVermeer/eslint-plugin-no-relative-import-paths/issues/30
   {
     "selector": "ImportDeclaration[source.value=/\\.\\.\\u002F?$/]",
     "message": "Don't use imports that ends in \"..\" or \"../\""
   },
   // From typescript configs, but allow on @Input()
-  angularEslintPluginVersion >= 16
-    ? {
-      "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"
-    }
-    : {
-      "selector":
-        "PropertyDefinition[definite=true]:not(:has(Decorator[expression.callee.name=\"Input\"]))",
-      "message":
-        "Forbidden non-null property assertion. https://typescript-eslint.io/rules/no-non-null-assertion . " +
-        "This is only allowed on component @Input() restricted by its selector."
-    }
+  {
+    "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"
+  }
 ]
 
-const typeDefinitionOverride = typescriptConfig.overrides?.find(override =>
-  override.files === "*.d.ts"
-)
+const typeDefinitionOverride = typescriptConfig
+  .overrides
+  ?.find(override => override.files === "*.d.ts")
 
-const preferArrowFunctionsConfig = javascriptConfig.rules?.["prefer-arrow/prefer-arrow-functions"]
+const preferArrowFunctionsConfig = javascriptConfig
+  .rules
+  ?.["prefer-arrow/prefer-arrow-functions"]
   ?.[1]
 const preventAbbreviationsConfig = {
   ...javascriptConfig.rules?.["unicorn/prevent-abbreviations"]?.[1]
@@ -107,7 +93,7 @@ module.exports = {
         ],
         // Autofix for this rule was broken: https://github.com/angular-eslint/angular-eslint/pull/1602
         // somewhat fixed in 16.1, but not proper: https://github.com/angular-eslint/angular-eslint/issues/1197
-        "@angular-eslint/template/attributes-order": angularEslintPluginVersion >= 17.1
+        "@angular-eslint/template/attributes-order": angularEslintVersion >= 17.1
           ? "error"
           : "off",
         // and it was added in 14.2.0. Not enforcing it for now allows a wider range of versions.
@@ -158,9 +144,6 @@ module.exports = {
             ]
           }
         ],
-        // "@angular-eslint/template/prefer-ngsrc": angularEslintPluginVersion >= 16.2
-        //   ? "error"
-        //   : "off",
         // 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",
@@ -183,6 +166,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",
@@ -190,9 +179,9 @@ module.exports = {
         ],
         // TODO: Consider use a tsconfig.eslint.json instead
         // https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/README.md#parseroptionscreatedefaultprogram
-        "createDefaultProgram": true,
+        "deprecated__createDefaultProgram": true,
         // Needed specifically for @typescript-eslint/consistent-type-imports
-        "emitDecoratorMetadata": true
+        "emitDecoratorMetadata": typescriptVersion >= 5
       },
       "rules": {
         /*
@@ -289,32 +278,8 @@ module.exports = {
         ],
         // Incompatible with Angular's dependecy injection
         "@typescript-eslint/no-invalid-this": "off",
-        // Both redundant with Angular's dependecy injection
-        "@typescript-eslint/no-parameter-properties": "off",
+        // Redundant with Angular's dependecy injection
         "@typescript-eslint/parameter-properties": "off",
-
-        /*
-         * These two configs are meant to work around the following issue wit Decorators:
-         * https://github.com/typescript-eslint/typescript-eslint/issues/5468#issuecomment-1906180432
-         * Once the following PR is merged, we can check for that specific @typescript-eslint version
-         * to only apply the workaround before it.
-         * https://github.com/typescript-eslint/typescript-eslint/pull/8335
-         */
-        // This only relates to TypeScript 5.0's --verbatimModuleSyntax,
-        // by not enforcing this rule yet, we are not loosing much compared to what we had before.
-        // We have to disable it for the "fixStyle: 'inline-type-imports'" workaround to be allowed at all time
-        "@typescript-eslint/no-import-type-side-effects": "off",
-        "@typescript-eslint/consistent-type-imports": [
-          "error",
-          typescriptVersion < 4.5
-            ? {}
-            : {
-              // Sets the autofix to use `import { type A } from '...'` which works around the above mentionned issue
-              // This only affects autofixing to help devs not aware of this edge case, both styles are always allowed.
-              "fixStyle": "inline-type-imports" // Only available since typescript 4.5
-            }
-        ],
-
         "unicorn/no-keyword-prefix": [
           "error",
           {
@@ -325,8 +290,6 @@ module.exports = {
             ]
           }
         ],
-        // False positives with Angular's EventEmitter: https://github.com/sindresorhus/eslint-plugin-unicorn/issues/1853
-        "unicorn/prefer-event-target": "off",
         // Angular is ESM
         "unicorn/prefer-module": "error",
         // This is not supported in Angular's Vite builder https://github.com/angular/angular-cli/issues/26507

package/dprint.js

@@ -177,11 +177,13 @@ module.exports = {
         "computed-property-spacing": "off",
         "autofix/computed-property-spacing": "off",
         "func-call-spacing": "off",
+        "autofix/func-call-spacing": "off",
         "@typescript-eslint/func-call-spacing": "off",
         "generator-star-spacing": "off",
         "autofix/generator-star-spacing": "off",
         "key-spacing": "off",
         "autofix/key-spacing": "off",
+        "@typescript-eslint/key-spacing": "off",
         "keyword-spacing": "off",
         "autofix/keyword-spacing": "off",
         "@typescript-eslint/keyword-spacing": "off",
@@ -224,8 +226,9 @@ module.exports = {
         /*
          * Unconfigurable or unknown linked configuration
          */
-        "@typescript-eslint/lines-around-comment": "off",
+        "lines-around-comment": "off",
         "autofix/lines-around-comment": "off",
+        "@typescript-eslint/lines-around-comment": "off",
         "no-extra-parens": "off",
         "autofix/no-extra-parens": "off",
         "@typescript-eslint/no-extra-parens": "off",
@@ -236,8 +239,7 @@ module.exports = {
         "new-parens": "off",
         "autofix/new-parens": "off",
         "no-floating-decimal": "off",
-        "autofix/no-floating-decimal": "off",
-        "import/newline-after-import": "off"
+        "autofix/no-floating-decimal": "off"
       }
     }
   ]

package/javascript.js

@@ -69,7 +69,7 @@ module.exports = {
   ],
   "extends": [
     "eslint:recommended",
-    "plugin:sonarjs/recommended",
+    "plugin:sonarjs/recommended-legacy",
     "plugin:unicorn/all",
     "plugin:regexp/all",
     "plugin:import/recommended",
@@ -90,6 +90,7 @@ module.exports = {
   "ignorePatterns": ["node_modules/"],
   // Useful when using experimental features and code transpiled by Babel
   "parser": "@babel/eslint-parser",
+  "reportUnusedDisableDirectives": true,
   "overrides": [
     {
       "files": [
@@ -430,9 +431,18 @@ module.exports = {
       { "considerQueryString": true }
     ],
     "import/no-empty-named-blocks": "error",
+    // exactCount has issues with require, at least that's taken care of by dprint:
+    // https://github.com/import-js/eslint-plugin-import/issues/2905
+    // With https://github.com/dprint/dprint-plugin-typescript/issues/394 we could disable this
+    // rule entirely in dprint preset.
     "import/newline-after-import": [
       "error",
-      { "considerComments": true }
+      {
+        /* eslint-disable-next-line etc/no-commented-out-code
+        -- see above comment, bump minimal eslint-plugin-import when this is fixed and enforced */
+        // "exactCount": false,
+        "considerComments": true
+      }
     ],
     "import/no-named-as-default": "error",
     "import/no-named-as-default-member": "error",
@@ -469,6 +479,9 @@ module.exports = {
         "case": "camelCase"
       }
     ],
+    // Only works on 3 node imports by default, and can't configure a default for all imports.
+    // Leave it up to the programmer.
+    "unicorn/import-style": "off",
     "unicorn/no-array-reduce": [
       "error",
       {
@@ -486,16 +499,10 @@ module.exports = {
         ]
       }
     ],
-    "unicorn/no-nested-ternary": "off",
     "unicorn/no-null": "off",
-    // Not as bad in frontend environment
-    "unicorn/no-unsafe-regex": "warn",
-    "unicorn/no-useless-undefined": [
-      "error",
-      {
-        "checkArguments": false
-      }
-    ],
+    // This rule has been deprecated in favor of eslint-plugin-regexp
+    "unicorn/no-unsafe-regex": "off",
+    "unicorn/no-useless-undefined": "error",
     "unicorn/numeric-separators-style": "error",
     "unicorn/prefer-add-event-listener": "error",
     "unicorn/prefer-array-find": [
@@ -569,18 +576,18 @@ module.exports = {
       {
         "groups": [
           // Side effect imports.
-          ["^\\u0000"],
+          [String.raw`^\u0000`],
           // Node.js builtins prefixed with `node:`.
           ["^node:"],
           // Packages.
           // Things that start with a letter (or digit or underscore), or `@` followed by a letter.
-          ["^@?\\w"],
+          [String.raw`^@?\w`],
           // Absolute imports and other imports such as Vue-style `@/foo`.
           // Anything not matched in another group.
           ["^"],
           // Relative imports.
           // Anything that starts with a dot or src/
-          ["^(\\.|src/)"]
+          [String.raw`^(\.|src/)`]
         ]
       }
     ],
@@ -623,6 +630,8 @@ module.exports = {
     ],
     // I don't think this is that much more readable imo. Especially for regexp beginners
     "regexp/prefer-named-capture-group": "off",
+    // TODO: Re-enable once VSCode has syntax highlighting for the v flag
+    "regexp/require-unicode-sets-regexp": "off",
     // Obsoleted by eslint-plugin-regexp
     "no-invalid-regexp": "off",
     "prefer-named-capture-group": "off",
@@ -633,7 +642,10 @@ module.exports = {
      * Excludes rules taken care of by eslint-plugin-autofix
      */
     "accessor-pairs": "error",
-    "array-callback-return": "error",
+    "array-callback-return": [
+      "error",
+      { "allowImplicit": true, "checkForEach": true, "allowVoid": true }
+    ],
     "block-scoped-var": "error",
     "camelcase": "error",
     "class-methods-use-this": "error",
@@ -828,7 +840,8 @@ module.exports = {
         ],
         "patterns": [
           {
-            // no-realtive-import paths doesn't catch ./..
+            // no-relative-import paths doesn't catch ./..
+            // TODO: Consider fixing ourselves
             // https://github.com/MelvinVermeer/eslint-plugin-no-relative-import-paths/issues/30
             "group": ["./../"],
             "message": "Relative parent imports are not allowed."

package/jest.js

@@ -1,5 +1,8 @@
 // SPDX-License-Identifier: MIT
 // Copyright 2024 Beslogic Inc.
+const { getModuleVersion } = require("./lib/utils")
+
+const jestGlobalsVersion = getModuleVersion("@jest/globals", 29)
 
 /** @type {import("eslint").Linter.Config} */
 module.exports = {
@@ -67,6 +70,12 @@ module.exports = {
         "onlyFunctionsWithExpectInLoop": true
       }
     ],
+    // https://github.com/jestjs/jest/issues/15084
+    // Since this will require a major jest version bump anyway,
+    // I feel confortable not waiting for our own major version bump to add this
+    ...(jestGlobalsVersion < 30
+      ? { "jest/prefer-importing-jest-globals": "off" }
+      : {}),
     // Top-level describe is often used for component name, which is capitalized
     "jest/prefer-lowercase-title": [
       "error",

package/json-like.js

@@ -29,6 +29,7 @@ module.exports = {
       "error",
       "double"
     ],
-    "autofix/quote-props": "error"
+    "autofix/quote-props": "error",
+    "unicorn/prefer-string-raw": "off"
   }
 }