@timobechtel/style 1.14.0 → 2.0.1

package/README.md

@@ -10,7 +10,7 @@ Highly opinionated configuration files for typescript projects. Inspired by [@ve
 ## Usage
 
 ```bash
-npm i -D @timobechtel/style prettier "eslint@^8.57.1" typescript
+npm i -D @timobechtel/style prettier "eslint@^9" typescript
 ```
 
 ### Prettier
@@ -94,7 +94,6 @@ With expo make sure to add `"moduleResolution": "bundler"` to the `compilerOptio
 
   </details>
 
-
 #### Or with React
 
 ```bash
@@ -117,51 +116,44 @@ curl -O https://raw.githubusercontent.com/TimoBechtel/style/refs/heads/main/temp
 ### Eslint
 
 ```bash
-curl -O https://raw.githubusercontent.com/TimoBechtel/style/refs/heads/main/templates/eslint/core/.eslintrc.cjs
-```
-
-#### Fix Parsing errors for config files
-
-You may get a `Parsing error: <FILE> was not found by the project service.` for config files like .eslintrc.cjs when not included in the tsconfig.
-
-To fix, either add to tsconfig or add them to the eslint config:
-
-```diff
-  //...
-  parserOptions: {
-+    projectService: {
-+      allowDefaultProject: ['.eslintrc.cjs'],
-+    },
-    //...
-  },
-  //...
+curl -O https://raw.githubusercontent.com/TimoBechtel/style/refs/heads/main/templates/eslint/core/eslint.config.js
 ```
 
+Note: If your project is not ESM (no `"type": "module"` in `package.json`), rename the file to `eslint.config.mjs`.
 
 <details>
   <summary>Or manually</summary>
 
-  Copy the following to a `.eslintrc.cjs`:
+  Copy the following to an `eslint.config.js`:
 
   ```js
-  const { resolve } = require('node:path');
-
-  const project = resolve(process.cwd(), 'tsconfig.json');
-
-  module.exports = {
-    root: true,
-    extends: [require.resolve('@timobechtel/style/eslint/core.cjs')],
-    parserOptions: {
-      tsconfigRootDir: process.cwd(),
-    },
-    settings: {
-      'import/resolver': {
-        typescript: {
-          project,
+  import path from 'node:path';
+  import { fileURLToPath } from 'node:url';
+  import { defineConfig } from 'eslint/config';
+  import styleCore from '@timobechtel/style/eslint/core.js';
+  import { createTypeScriptImportResolver } from 'eslint-import-resolver-typescript';
+  import { createNodeResolver } from 'eslint-plugin-import-x';
+
+  const __dirname = path.dirname(fileURLToPath(import.meta.url));
+
+  export default defineConfig([
+    ...styleCore,
+    {
+      languageOptions: {
+        parserOptions: {
+          tsconfigRootDir: __dirname,
         },
       },
+      settings: {
+        'import-x/resolver-next': [
+          createTypeScriptImportResolver({
+            project: path.resolve(__dirname, 'tsconfig.json'),
+          }),
+          createNodeResolver(),
+        ],
+      },
     },
-  };
+  ]);
   ```
 
 </details>
@@ -169,28 +161,49 @@ To fix, either add to tsconfig or add them to the eslint config:
 #### React
 
 ```bash
-curl -O https://raw.githubusercontent.com/TimoBechtel/style/refs/heads/main/templates/eslint/react/.eslintrc.cjs
+curl -O https://raw.githubusercontent.com/TimoBechtel/style/refs/heads/main/templates/eslint/react/eslint.config.js
 ```
 
 <details>
   <summary>Or manually</summary>
   
-  Also add `require.resolve('@timobechtel/style/eslint/react.cjs')` to the `extends` array.
+  Also spread `styleReact` from `@timobechtel/style/eslint/react.js`:
+
+  ```js
+  import styleCore from '@timobechtel/style/eslint/core.js';
+  import styleReact from '@timobechtel/style/eslint/react.js';
+  import { defineConfig } from 'eslint/config';
+
+  export default defineConfig([
+    ...styleCore,
+    ...styleReact,
+    // ... your config
+  ]);
+  ```
 
   Example config:
-  <https://raw.githubusercontent.com/TimoBechtel/style/refs/heads/main/templates/eslint/react/.eslintrc.cjs>
+  <https://raw.githubusercontent.com/TimoBechtel/style/refs/heads/main/templates/eslint/react/eslint.config.js>
 </details>
 
+#### Migration from v1.x
+
+If you're upgrading from v1.x, you'll need to:
+
+1. Upgrade to ESLint v9+
+2. Replace `.eslintrc.cjs` with `eslint.config.js`
+3. Update imports to use `.js` extension (e.g., `@timobechtel/style/eslint/core.js`)
+4. Note: Import plugin rules now use `import-x/` prefix instead of `import/`
+
 #### VSCode
 
-Note: You should disable `source.organizeImports` in your VSCode config, as this collides with the `import/order` rule.
+Note: You should disable `source.organizeImports` in your VSCode config, as this collides with the `import-x/order` rule.
 
 Add the following to your VSCode config, e.g. `.vscode/settings.json`
 
 ```json
 {
   "editor.codeActionsOnSave": {
-    // use eslint import/order instead
+    // use eslint import-x/order instead
     "source.sortImports": "never"
   }
 }
@@ -201,7 +214,7 @@ Add the following to your VSCode config, e.g. `.vscode/settings.json`
 This repo also contains a [semantic-release](https://github.com/semantic-release/semantic-release) configuration.
 
 ```bash
-npm i -D semantic-release
+npm i -D semantic-release @semantic-release/changelog @semantic-release/git
 ```
 
 ```bash

package/eslint/core.js

@@ -0,0 +1,86 @@
+import js from '@eslint/js';
+import timobechtelRulesPlugin from '@timobechtel/eslint-plugin-rules';
+import prettierConfig from 'eslint-config-prettier/flat';
+import { createTypeScriptImportResolver } from 'eslint-import-resolver-typescript';
+import { createNodeResolver, importX } from 'eslint-plugin-import-x';
+import unicornPlugin from 'eslint-plugin-unicorn';
+import { defineConfig } from 'eslint/config';
+import globals from 'globals';
+import tseslint from 'typescript-eslint';
+
+import baseRules from './rules/base.js';
+import importRules from './rules/import.js';
+import typescriptRules from './rules/typescript.js';
+import unicornRules from './rules/unicorn.js';
+
+const mergeRules = (configs) =>
+  Object.assign({}, ...configs.map((config) => config?.rules ?? {}));
+
+export default defineConfig([
+  js.configs.recommended,
+  importX.flatConfigs.recommended,
+  importX.flatConfigs.typescript,
+  timobechtelRulesPlugin.configs['flat/all'],
+
+  {
+    languageOptions: {
+      globals: {
+        ...globals.browser,
+        ...globals.node,
+        ...globals.es2021,
+      },
+      ecmaVersion: 2021,
+      sourceType: 'module',
+    },
+    plugins: {
+      unicorn: unicornPlugin,
+    },
+    rules: {
+      ...mergeRules(baseRules),
+      ...mergeRules(importRules),
+      ...mergeRules(unicornRules),
+    },
+    linterOptions: {
+      reportUnusedDisableDirectives: true,
+    },
+    settings: {
+      'import-x/resolver-next': [
+        createTypeScriptImportResolver(),
+        createNodeResolver(),
+      ],
+    },
+  },
+
+  ...tseslint.configs.recommended,
+  ...tseslint.configs.recommendedTypeChecked,
+  ...tseslint.configs.strict,
+  ...tseslint.configs.strictTypeChecked,
+  ...tseslint.configs.stylistic,
+  ...tseslint.configs.stylisticTypeChecked,
+  {
+    languageOptions: {
+      parserOptions: {
+        projectService: true,
+      },
+    },
+  },
+  {
+    files: ['**/*.js'],
+    extends: [tseslint.configs.disableTypeChecked],
+  },
+  {
+    files: ['**/*.ts', '**/*.tsx'],
+    rules: {
+      ...mergeRules(typescriptRules),
+    },
+  },
+
+  {
+    files: ['**/*.test.ts', '**/*.test.tsx', '**/*.spec.ts', '**/*.spec.tsx'],
+    rules: {
+      '@typescript-eslint/ban-ts-comment': 'off',
+    },
+  },
+
+  prettierConfig,
+]);

package/eslint/react.js

@@ -0,0 +1,32 @@
+import prettierConfig from 'eslint-config-prettier/flat';
+import reactPlugin from 'eslint-plugin-react';
+import reactHooksPlugin from 'eslint-plugin-react-hooks';
+import { defineConfig } from 'eslint/config';
+import reactRules from './rules/react.js';
+
+const mergeRules = (configs) =>
+  Object.assign({}, ...configs.map((config) => config?.rules ?? {}));
+
+export default defineConfig([
+  reactPlugin.configs.flat.recommended,
+  reactPlugin.configs.flat['jsx-runtime'],
+
+  {
+    plugins: {
+      react: reactPlugin,
+      'react-hooks': reactHooksPlugin,
+    },
+    settings: {
+      react: {
+        version: 'detect',
+      },
+      linkComponents: ['Link'],
+    },
+    rules: {
+      ...reactHooksPlugin.configs.flat.recommended.rules,
+      ...mergeRules(reactRules),
+    },
+  },
+
+  prettierConfig,
+]);

package/eslint/rules/base.js

@@ -0,0 +1,52 @@
+import { defineConfig } from 'eslint/config';
+
+export default defineConfig({
+  rules: {
+    'prefer-arrow-callback': [
+      'warn',
+      {
+        allowNamedFunctions: true,
+        allowUnboundThis: true,
+      },
+    ],
+    'no-console': [
+      'error',
+      {
+        allow: ['warn', 'error', 'clear', 'info'],
+      },
+    ],
+    'max-depth': ['warn', 4],
+    'no-restricted-globals': ['error', 'event', 'name'],
+    curly: ['warn', 'multi-line'],
+    'default-case-last': 'error',
+    eqeqeq: 'error',
+    'no-alert': 'error',
+    'no-useless-rename': 'warn',
+    'no-var': 'error',
+    'object-shorthand': 'warn',
+    'prefer-const': 'warn',
+    'prefer-rest-params': 'error',
+    'prefer-spread': 'error',
+    'prefer-template': 'warn',
+    'no-promise-executor-return': 'error',
+    'no-unreachable-loop': 'error',
+    'new-cap': ['error', { capIsNew: false }],
+    'new-parens': 'warn',
+    'no-lonely-if': 'warn',
+    'no-unneeded-ternary': 'error',
+    'prefer-object-spread': 'warn',
+    'no-label-var': 'error',
+    'no-undef-init': 'warn',
+    'no-unused-vars': [
+      'error',
+      {
+        args: 'after-used',
+        argsIgnorePattern: '^_',
+        ignoreRestSiblings: false,
+        vars: 'all',
+        varsIgnorePattern: '^_',
+      },
+    ],
+    'no-constant-binary-expression': 'error',
+  },
+});

package/eslint/rules/{css-in-js.cjs → css-in-js.js}

@@ -1,12 +1,10 @@
-// @ts-check
-const { defineConfig } = require('eslint-define-config');
+import { defineConfig } from 'eslint/config';
 
-module.exports = defineConfig({
+export default defineConfig({
   rules: {
     'no-restricted-syntax': [
       'error',
       {
-        // catches common mistakes when writing comments in CSS
         selector:
           'TemplateElement[value.cooked=/\\s+\\u002F\\u002F\\s*/], Literal[value=/\\s+\\u002F\\u002F\\s*/]',
         message: 'Invalid comment syntax. Use `/* */` instead of `//`.',

package/eslint/rules/import.js

@@ -0,0 +1,28 @@
+import { defineConfig } from 'eslint/config';
+
+export default defineConfig({
+  rules: {
+    'import-x/first': 'error',
+    'import-x/newline-after-import': 'warn',
+    'import-x/no-self-import': 'error',
+    'import-x/no-useless-path-segments': ['error'],
+    'import-x/order': [
+      'warn',
+      {
+        groups: [
+          'builtin',
+          'external',
+          'internal',
+          'parent',
+          'sibling',
+          'index',
+        ],
+      },
+    ],
+    'import-x/no-default-export': 'off',
+    'import-x/default': 'off',
+    'import-x/export': 'off',
+    'import-x/namespace': 'off',
+    'import-x/no-unresolved': 'off',
+  },
+});

package/eslint/rules/react.js

@@ -0,0 +1,40 @@
+import { defineConfig } from 'eslint/config';
+
+export default defineConfig({
+  rules: {
+    'react/prop-types': 'off',
+    'react/react-in-jsx-scope': 'off',
+    'react/button-has-type': 'error',
+    'react/function-component-definition': 'warn',
+    'react/hook-use-state': 'warn',
+    'react/jsx-boolean-value': 'warn',
+    'react/jsx-curly-brace-presence': [
+      'warn',
+      {
+        props: 'never',
+        children: 'ignore',
+        propElementValues: 'always',
+      },
+    ],
+    'react/jsx-fragments': 'warn',
+    'react/jsx-no-leaked-render': 'error',
+    'react/jsx-no-target-blank': [
+      'error',
+      {
+        allowReferrer: true,
+      },
+    ],
+    'react/jsx-no-useless-fragment': ['warn', { allowExpressions: true }],
+    'react/jsx-pascal-case': 'warn',
+    'react/jsx-sort-props': [
+      'warn',
+      {
+        callbacksLast: true,
+      },
+    ],
+    'react/no-unstable-nested-components': 'error',
+    'react/self-closing-comp': 'warn',
+    'react-hooks/exhaustive-deps': 'error',
+    'react/destructuring-assignment': ['warn', 'always'],
+  },
+});

package/eslint/rules/typescript.js

@@ -0,0 +1,63 @@
+import { defineConfig } from 'eslint/config';
+import baseRules from './base.js';
+
+const noUnusedVarsConfig = baseRules[0]?.rules?.['no-unused-vars'];
+
+export default defineConfig({
+  rules: {
+    '@typescript-eslint/consistent-type-exports': [
+      'error',
+      { fixMixedExportsWithInlineTypeSpecifier: true },
+    ],
+    '@typescript-eslint/consistent-type-imports': [
+      'error',
+      {
+        prefer: 'type-imports',
+        fixStyle: 'inline-type-imports',
+      },
+    ],
+    '@typescript-eslint/naming-convention': [
+      'error',
+      {
+        format: ['PascalCase'],
+        selector: ['typeLike', 'enumMember'],
+      },
+      {
+        custom: {
+          match: false,
+          regex: '^I[A-Z]|^(Interface|Props|State)$',
+        },
+        format: ['PascalCase'],
+        selector: 'interface',
+      },
+    ],
+    '@typescript-eslint/no-redundant-type-constituents': 'error',
+    '@typescript-eslint/prefer-regexp-exec': 'warn',
+    '@typescript-eslint/require-array-sort-compare': [
+      'error',
+      { ignoreStringArrays: true },
+    ],
+    '@typescript-eslint/switch-exhaustiveness-check': 'error',
+    'default-param-last': 'off',
+    '@typescript-eslint/default-param-last': 'error',
+    'no-loop-func': 'off',
+    '@typescript-eslint/no-loop-func': 'error',
+    'no-unused-vars': 'off',
+    '@typescript-eslint/no-unused-vars': noUnusedVarsConfig,
+    'import-x/default': 'off',
+    'import-x/export': 'off',
+    'import-x/namespace': 'off',
+    'import-x/no-unresolved': 'off',
+    '@typescript-eslint/prefer-nullish-coalescing': 'off',
+    '@typescript-eslint/no-misused-promises': [
+      'error',
+      {
+        checksVoidReturn: false,
+      },
+    ],
+    '@typescript-eslint/consistent-type-definitions': 'off',
+    '@typescript-eslint/method-signature-style': 'off',
+    '@typescript-eslint/consistent-indexed-object-style': 'off',
+    '@typescript-eslint/no-import-type-side-effects': 'error',
+  },
+});

package/eslint/rules/unicorn.js

@@ -0,0 +1,71 @@
+import { defineConfig } from 'eslint/config';
+
+export default defineConfig({
+  rules: {
+    'unicorn/consistent-destructuring': 'warn',
+    'unicorn/consistent-function-scoping': [
+      'warn',
+      {
+        checkArrowFunctions: false,
+      },
+    ],
+    'unicorn/error-message': 'error',
+    'unicorn/escape-case': 'warn',
+    'unicorn/no-empty-file': 'error',
+    'unicorn/no-for-loop': 'warn',
+    'unicorn/no-instanceof-array': 'error',
+    'unicorn/no-invalid-remove-event-listener': 'error',
+    'unicorn/no-object-as-default-parameter': 'error',
+    'unicorn/no-thenable': 'error',
+    'unicorn/no-unnecessary-await': 'error',
+    'unicorn/no-unreadable-iife': 'warn',
+    'unicorn/no-useless-fallback-in-spread': 'warn',
+    'unicorn/no-useless-length-check': 'warn',
+    'unicorn/no-useless-promise-resolve-reject': 'error',
+    'unicorn/no-useless-spread': 'error',
+    'unicorn/no-zero-fractions': 'warn',
+    'unicorn/numeric-separators-style': 'error',
+    'unicorn/prefer-add-event-listener': 'error',
+    'unicorn/prefer-array-find': 'error',
+    'unicorn/prefer-array-flat': 'error',
+    'unicorn/prefer-array-flat-map': 'error',
+    'unicorn/prefer-array-index-of': 'error',
+    'unicorn/prefer-array-some': 'error',
+    'unicorn/prefer-at': 'error',
+    'unicorn/prefer-blob-reading-methods': 'error',
+    'unicorn/prefer-date-now': 'error',
+    'unicorn/prefer-default-parameters': 'warn',
+    'unicorn/prefer-dom-node-append': 'error',
+    'unicorn/prefer-dom-node-dataset': 'error',
+    'unicorn/prefer-dom-node-remove': 'error',
+    'unicorn/prefer-event-target': 'warn',
+    'unicorn/prefer-export-from': [
+      'warn',
+      {
+        ignoreUsedVariables: true,
+      },
+    ],
+    'unicorn/prefer-includes': 'error',
+    'unicorn/prefer-keyboard-event-key': 'error',
+    'unicorn/prefer-logical-operator-over-ternary': 'error',
+    'unicorn/prefer-math-trunc': 'error',
+    'unicorn/prefer-modern-dom-apis': 'error',
+    'unicorn/prefer-modern-math-apis': 'error',
+    'unicorn/prefer-negative-index': 'error',
+    'unicorn/prefer-node-protocol': 'error',
+    'unicorn/prefer-number-properties': ['error', { checkInfinity: false }],
+    'unicorn/prefer-object-from-entries': 'error',
+    'unicorn/prefer-reflect-apply': 'error',
+    'unicorn/prefer-regexp-test': 'error',
+    'unicorn/prefer-set-has': 'error',
+    'unicorn/prefer-set-size': 'error',
+    'unicorn/prefer-spread': 'error',
+    'unicorn/prefer-string-replace-all': 'error',
+    'unicorn/prefer-string-slice': 'error',
+    'unicorn/prefer-string-starts-ends-with': 'error',
+    'unicorn/prefer-ternary': ['warn', 'only-single-line'],
+    'unicorn/prefer-top-level-await': 'error',
+    'unicorn/require-array-join-separator': 'error',
+    'unicorn/switch-case-braces': 'error',
+  },
+});

package/package.json

@@ -1,7 +1,11 @@
 {
   "name": "@timobechtel/style",
-  "version": "1.14.0",
+  "version": "2.0.1",
   "type": "module",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/TimoBechtel/style"
+  },
   "files": [
     "bin",
     "eslint",
@@ -10,30 +14,30 @@
     "tsconfig"
   ],
   "devDependencies": {
-    "eslint": "^8.57.1",
-    "prettier": "^3.4.2",
-    "semantic-release": "^24.2.1",
-    "typescript": "^5.7.2"
+    "@semantic-release/npm": "^13.1.4",
+    "eslint": "^9.39.3",
+    "prettier": "^3.8.1",
+    "semantic-release": "^25.0.3",
+    "typescript": "^5.9.3"
   },
   "peerDependencies": {
-    "eslint": "^8.57.1",
-    "prettier": "^3.4.2",
-    "semantic-release": "^24.2.1",
-    "typescript": "^5.7.2"
+    "eslint": "^9.39.3",
+    "prettier": "^3.8.1",
+    "semantic-release": "^25.0.3",
+    "typescript": "^5.9.3"
   },
   "dependencies": {
+    "@eslint/js": "^9.39.3",
     "@semantic-release/changelog": "^6.0.3",
     "@semantic-release/git": "^10.0.1",
     "@timobechtel/eslint-plugin-rules": "^1.0.0",
-    "@typescript-eslint/eslint-plugin": "^8.19.1",
-    "@typescript-eslint/parser": "^8.19.1",
-    "eslint-config-prettier": "^9.1.0",
-    "eslint-define-config": "^2.1.0",
-    "eslint-import-resolver-typescript": "^3.7.0",
-    "eslint-plugin-import": "^2.31.0",
-    "eslint-plugin-no-template-curly-in-string-fix": "^1.0.4",
-    "eslint-plugin-react": "^7.37.3",
-    "eslint-plugin-react-hooks": "^5.1.0",
-    "eslint-plugin-unicorn": "^56.0.1"
+    "eslint-config-prettier": "^10.1.8",
+    "eslint-import-resolver-typescript": "^4.4.4",
+    "eslint-plugin-import-x": "^4.16.1",
+    "eslint-plugin-react": "^7.37.5",
+    "eslint-plugin-react-hooks": "^7.0.1",
+    "eslint-plugin-unicorn": "^63.0.0",
+    "globals": "^17.3.0",
+    "typescript-eslint": "^8.56.0"
   }
 }

package/eslint/core.cjs

@@ -1,63 +0,0 @@
-// @ts-check
-const { defineConfig } = require('eslint-define-config');
-
-module.exports = defineConfig({
-  extends: [
-    'eslint:recommended',
-    'plugin:import/recommended',
-    'prettier',
-    'plugin:no-template-curly-in-string-fix/recommended',
-    'plugin:@timobechtel/rules/all',
-    require.resolve('./rules/base.cjs'),
-    require.resolve('./rules/import.cjs'),
-    require.resolve('./rules/unicorn.cjs'),
-  ],
-  parser: '@typescript-eslint/parser',
-  plugins: [],
-  env: {
-    es2021: true,
-    node: true,
-    browser: true,
-  },
-  // Report unused `eslint-disable` comments.
-  reportUnusedDisableDirectives: true,
-  // Tell ESLint not to ignore dot-files, which are ignored by default.
-  ignorePatterns: ['!.*.js'],
-  // Global settings used by all overrides.
-  settings: {
-    // Use the Node resolver by default.
-    'import/resolver': { node: {} },
-  },
-  // Global parser options.
-  parserOptions: {
-    ecmaVersion: 2021,
-    sourceType: 'module',
-    projectService: true,
-  },
-  overrides: [
-    {
-      files: ['*.ts?(x)'],
-      extends: [
-        'plugin:@typescript-eslint/recommended',
-        'plugin:@typescript-eslint/recommended-type-checked',
-        'plugin:@typescript-eslint/strict',
-        'plugin:@typescript-eslint/strict-type-checked',
-        'plugin:@typescript-eslint/stylistic',
-        'plugin:@typescript-eslint/stylistic-type-checked',
-        'plugin:import/typescript',
-        'prettier',
-        require.resolve('./rules/typescript.cjs'),
-      ],
-    },
-    {
-      files: ['*.test.ts?(x)', '*.spec.ts?(x)'],
-      rules: {
-        // ts-expect-error makes sense for tests
-        '@typescript-eslint/ban-ts-comment': [
-          'off',
-          { 'ts-expect-error': 'off' },
-        ],
-      },
-    },
-  ],
-});

package/eslint/react.cjs

@@ -1,21 +0,0 @@
-// @ts-check
-const { defineConfig } = require('eslint-define-config');
-
-module.exports = defineConfig({
-  extends: [
-    'plugin:react/recommended',
-    'plugin:react-hooks/recommended',
-    'plugin:import/react',
-    'prettier',
-    require.resolve('./rules/react.cjs'),
-  ],
-  settings: {
-    react: {
-      version: 'detect',
-    },
-    linkComponents: [
-      // Components used as alternatives to <a> for linking, eg. <Link to={ url } />
-      'Link',
-    ],
-  },
-});

package/eslint/rules/base.cjs

@@ -1,84 +0,0 @@
-// @ts-check
-const { defineConfig } = require('eslint-define-config');
-
-module.exports = defineConfig({
-  rules: {
-    // prefer arrow functions for callbacks
-    'prefer-arrow-callback': [
-      'warn',
-      {
-        allowNamedFunctions: true,
-        allowUnboundThis: true,
-      },
-    ],
-    // console logs other than error and warn are only needed for debugging, so they should be removed before merging
-    'no-console': [
-      'error',
-      {
-        allow: [
-          'warn',
-          'error',
-          'clear',
-          // sometimes we really need to log something, even in production, so info is allowed in edge cases
-          'info',
-        ],
-      },
-    ],
-    'max-depth': ['warn', 4],
-    // some globals, like "name" might be similar to the name of a variable, so we prevent them from being used
-    'no-restricted-globals': ['error', 'event', 'name'],
-    // Require curly braces for multiline blocks.
-    curly: ['warn', 'multi-line'],
-    // Require default clauses in switch statements to be last (if used).
-    'default-case-last': 'error',
-    // Require triple equals (`===` and `!==`).
-    eqeqeq: 'error',
-    //  disallow the use of `alert()`
-    'no-alert': 'error',
-    // Disallow renaming import, export, and destructured assignments to the same name.
-    'no-useless-rename': 'warn',
-    //  Require `let` or `const` instead of `var`.
-    'no-var': 'error',
-    // Require object literal shorthand syntax.
-    'object-shorthand': 'warn',
-    // Require default to `const` instead of `let`.
-    'prefer-const': 'warn',
-    // Require rest parameters instead of `arguments`.
-    'prefer-rest-params': 'error',
-    // Require spread syntax instead of `.apply()`.
-    'prefer-spread': 'error',
-    // Require template literals instead of string concatenation.
-    'prefer-template': 'warn',
-    // Disallow returning values from Promise executor functions.
-    'no-promise-executor-return': 'error',
-    // Disallow loops with a body that allows only one iteration.
-    'no-unreachable-loop': 'error',
-    // Require a capital letter for constructors.
-    'new-cap': ['error', { capIsNew: false }],
-    //  Disallow the omission of parentheses when invoking a constructor with no arguments.
-    'new-parens': 'warn',
-    // Disallow if as the only statement in an else block.
-    'no-lonely-if': 'warn',
-    // Disallow ternary operators when simpler alternatives exist.
-    'no-unneeded-ternary': 'error',
-    // Require use of an object spread over Object.assign.
-    'prefer-object-spread': 'warn',
-    // Disallow labels that share a name with a variable.
-    'no-label-var': 'error',
-    // Disallow initializing variables to `undefined`.
-    'no-undef-init': 'warn',
-    // Disallow unused variables.
-    'no-unused-vars': [
-      'error',
-      {
-        args: 'after-used',
-        argsIgnorePattern: '^_',
-        ignoreRestSiblings: false,
-        vars: 'all',
-        varsIgnorePattern: '^_',
-      },
-    ],
-    // Disallow expressions where the operation doesn't affect the value
-    'no-constant-binary-expression': 'error',
-  },
-});

package/eslint/rules/import.cjs

@@ -1,39 +0,0 @@
-// @ts-check
-const { defineConfig } = require('eslint-define-config');
-
-module.exports = defineConfig({
-  rules: {
-    // Disallow non-import statements appearing before import statements.
-    'import/first': 'error',
-    // Require a newline after the last import/require.
-    'import/newline-after-import': 'warn',
-    //  Disallow a module from importing itself.
-    'import/no-self-import': 'error',
-    // Ensures that there are no useless path segments.
-    'import/no-useless-path-segments': ['error'],
-    // Enforce a module import order convention.
-    'import/order': [
-      'warn',
-      {
-        groups: [
-          'builtin', // Node.js built-in modules
-          'external', // Packages
-          'internal', // Aliased modules
-          'parent', // Relative parent
-          'sibling', // Relative sibling
-          'index', // Relative index
-        ],
-      },
-    ],
-    // allow default exports
-    'import/no-default-export': 'off',
-    /**
-     * These are enabled by `import/recommended`, but are better handled by
-     * TypeScript and @typescript-eslint.
-     */
-    'import/default': 'off',
-    'import/export': 'off',
-    'import/namespace': 'off',
-    'import/no-unresolved': 'off',
-  },
-});

package/eslint/rules/react.cjs

@@ -1,90 +0,0 @@
-// @ts-check
-const { defineConfig } = require('eslint-define-config');
-
-module.exports = defineConfig({
-  rules: {
-    // We should prefer TypeScript over prop-types.
-    'react/prop-types': 'off',
-    // Disable requiring React to be imported, as this is no longer required.
-    'react/react-in-jsx-scope': 'off',
-    /**
-     * Require an explicit type when using button elements. Prevents common mistakes where `type="button"` is omitted on `<button>` elements.
-     */
-    'react/button-has-type': 'error',
-    /**
-     * Require consistent function type for function components.
-     */
-    'react/function-component-definition': 'warn',
-    /**
-     * Require destructuring and symmetric naming of `useState` hook value and setter variables.
-     */
-    'react/hook-use-state': 'warn',
-    /**
-     * Require consistent boolean attributes notation in JSX.
-     */
-    'react/jsx-boolean-value': 'warn',
-    /**
-     * Disallow unnecessary curly braces in JSX props and children.
-     */
-    'react/jsx-curly-brace-presence': [
-      'warn',
-      {
-        props: 'never',
-        children: 'ignore',
-        propElementValues: 'always',
-      },
-    ],
-    /**
-     * Require using shorthand form for React fragments, unless required.
-     */
-    'react/jsx-fragments': 'warn',
-    /**
-     * Prevent problematic leaked values from being rendered.
-     */
-    'react/jsx-no-leaked-render': 'error',
-    /**
-     * Prevents usage of unsafe `target='_blank'`.
-     *
-     * This rule is a part of `react/recommended`, but modified to
-     * enable allowReferrer.
-     */
-    'react/jsx-no-target-blank': [
-      'error',
-      {
-        allowReferrer: true,
-      },
-    ],
-    /**
-     * Disallow empty React fragments.
-     */
-    'react/jsx-no-useless-fragment': ['warn', { allowExpressions: true }],
-    /**
-     * Require the use of PascalCase for user-defined JSX components.
-     */
-    'react/jsx-pascal-case': 'warn',
-    /**
-     * Require props to be sorted alphabetically.
-     */
-    'react/jsx-sort-props': [
-      'warn',
-      {
-        // list callbacks after all other props
-        callbacksLast: true,
-      },
-    ],
-    /**
-     * Disallow creating unstable components inside components.
-     */
-    'react/no-unstable-nested-components': 'error',
-    /**
-     * Disallow closing tags for components without children.
-     */
-    'react/self-closing-comp': 'warn',
-    /**
-     * Enforce exhaustive dependencies in `useEffect` and `useCallback` hooks.
-     */
-    'react-hooks/exhaustive-deps': 'error',
-    // prefer destructuring props
-    'react/destructuring-assignment': ['warn', 'always'],
-  },
-});

package/eslint/rules/typescript.cjs

@@ -1,108 +0,0 @@
-const { defineConfig } = require('eslint-define-config');
-const noUnusedVarsConfig = require('./base.cjs').rules['no-unused-vars'];
-
-// @ts-check
-module.exports = defineConfig({
-  rules: {
-    // Require consistent usage of type exports.
-    '@typescript-eslint/consistent-type-exports': [
-      'error',
-      { fixMixedExportsWithInlineTypeSpecifier: true },
-    ],
-    // Require consistent usage of type imports.
-    '@typescript-eslint/consistent-type-imports': [
-      'error',
-      {
-        prefer: 'type-imports',
-        fixStyle: 'inline-type-imports',
-      },
-    ],
-    /**
-     * Require using function property types in method signatures.
-     * These have enhanced typechecking, whereas method signatures do not.
-     */
-    '@typescript-eslint/method-signature-style': 'error',
-    /**
-     * Require consistent naming conventions.
-     * Improves IntelliSense suggestions and avoids name collisions.
-     */
-    '@typescript-eslint/naming-convention': [
-      'error',
-      // Anything type-like should be written in PascalCase.
-      {
-        format: ['PascalCase'],
-        selector: ['typeLike', 'enumMember'],
-      },
-      // Interfaces cannot be prefixed with `I`, or have restricted names.
-      {
-        custom: {
-          match: false,
-          regex: '^I[A-Z]|^(Interface|Props|State)$',
-        },
-        format: ['PascalCase'],
-        selector: 'interface',
-      },
-    ],
-    // Disallow members of unions and intersections that do nothing or override type information.
-    '@typescript-eslint/no-redundant-type-constituents': 'error',
-    // Require using `RegExp.exec()` over `String.match()` for consistency.
-    '@typescript-eslint/prefer-regexp-exec': 'warn',
-    //
-    '@typescript-eslint/require-array-sort-compare': [
-      'error',
-      { ignoreStringArrays: true },
-    ],
-    /**
-     * Require exhaustive checks when using union types in switch statements.
-     * This ensures cases are considered when items are later added to a union.
-     */
-    '@typescript-eslint/switch-exhaustiveness-check': 'error',
-    // Require default parameters to be last.
-    'default-param-last': 'off',
-    '@typescript-eslint/default-param-last': 'error',
-    // Disallow creation of functions within loops.
-    'no-loop-func': 'off',
-    '@typescript-eslint/no-loop-func': 'error',
-    // Disallow unused variables.
-    'no-unused-vars': 'off',
-    '@typescript-eslint/no-unused-vars': noUnusedVarsConfig,
-
-    /**
-     * These are enabled by `import/recommended`, but are better handled by
-     * TypeScript and @typescript-eslint.
-     */
-    'import/default': 'off',
-    'import/export': 'off',
-    'import/namespace': 'off',
-    'import/no-unresolved': 'off',
-
-    // This is disabled as I feel that checking empty strings is a valid use
-    // of `||` over `??`.
-    '@typescript-eslint/prefer-nullish-coalescing': 'off',
-
-    // Disallow Promises in places not designed to handle them.
-    '@typescript-eslint/no-misused-promises': [
-      'error',
-      {
-        // Disabled as I feel that passing a async callback to a function expecting
-        // a void callback is a valid use case.
-        // e.g. fn.on('event', async () => {})
-        // Strictly requiring to return 'undefined' does not have a functional benefit,
-        // but makes the code more verbose.
-        checksVoidReturn: false,
-      },
-    ],
-    // While I'd prefer using types over interfaces, there are a lot of cases
-    // where interfaces are needed, e.g. merging declarations and this rule is too strict.
-    '@typescript-eslint/consistent-type-definitions': 'off',
-    // while using property signatures provides better typechecking,
-    // method signatures provide better hinting in the editor.
-    // (different color for methods vs properties)
-    '@typescript-eslint/method-signature-style': 'off',
-    // There are cases where using the index signature is more descriptive
-    // as the index can be named
-    '@typescript-eslint/consistent-indexed-object-style': 'off',
-    // Enforce using a top-level type qualifier for imports when only types are imported using the inline type qualifier
-    '@typescript-eslint/no-import-type-side-effects': 'error',
-  },
-});

package/eslint/rules/unicorn.cjs

@@ -1,129 +0,0 @@
-// @ts-check
-const { defineConfig } = require('eslint-define-config');
-
-module.exports = defineConfig({
-  plugins: ['unicorn'],
-  rules: {
-    // Use destructured variables over properties.
-    'unicorn/consistent-destructuring': 'warn',
-    // Move function definitions to the highest possible scope.
-    'unicorn/consistent-function-scoping': [
-      'warn',
-      {
-        checkArrowFunctions: false,
-      },
-    ],
-    //  Enforce passing a message value when creating a built-in error.
-    'unicorn/error-message': 'error',
-    //  Require escape sequences to use uppercase values.
-    'unicorn/escape-case': 'warn',
-    // Disallow empty files.
-    'unicorn/no-empty-file': 'error',
-    // Do not use a for loop that can be replaced with a for-of loop.
-    'unicorn/no-for-loop': 'warn',
-    // Require Array.isArray() instead of instanceof Array.
-    'unicorn/no-instanceof-array': 'error',
-    // Prevent calling EventTarget#removeEventListener() with the result of an expression.
-    'unicorn/no-invalid-remove-event-listener': 'error',
-    // Disallow the use of objects as default parameters.
-    'unicorn/no-object-as-default-parameter': 'error',
-    // Disallow then property.
-    'unicorn/no-thenable': 'error',
-    //  Disallow awaiting non-promise values.
-    'unicorn/no-unnecessary-await': 'error',
-    // Disallow unreadable IIFEs.
-    'unicorn/no-unreadable-iife': 'warn',
-    // Disallow useless fallback when spreading in object literals.
-    'unicorn/no-useless-fallback-in-spread': 'warn',
-    // Disallow useless array length check.
-    'unicorn/no-useless-length-check': 'warn',
-    // Disallow returning/yielding Promise.resolve/reject() in async functions or promise callbacks
-    'unicorn/no-useless-promise-resolve-reject': 'error',
-    // Disallow unnecessary spread.
-    'unicorn/no-useless-spread': 'error',
-    // Disallow number literals with zero fractions or dangling dots.
-    'unicorn/no-zero-fractions': 'warn',
-    // Enforce the style of numeric separators by correctly grouping digits.
-    'unicorn/numeric-separators-style': 'error',
-    // Prefer .addEventListener() and .removeEventListener() over on-functions.
-    'unicorn/prefer-add-event-listener': 'error',
-    // Prefer .find(…) and .findLast(…) over the first or last element from .filter(…).
-    'unicorn/prefer-array-find': 'error',
-    // Prefer Array#flat() over legacy techniques to flatten arrays.
-    'unicorn/prefer-array-flat': 'error',
-    // Prefer .flatMap(…) over .map(…).flat().
-    'unicorn/prefer-array-flat-map': 'error',
-    // Prefer Array#{indexOf,lastIndexOf}() over Array#{findIndex,findLastIndex}() when looking for the index of an item.
-    'unicorn/prefer-array-index-of': 'error',
-    // Prefer .some(…) over .filter(…).length check and .{find,findLast}(…).
-    'unicorn/prefer-array-some': 'error',
-    // Prefer .at() method for index access and String#charAt().
-    'unicorn/prefer-at': 'error',
-    // Prefer Blob#arrayBuffer() over FileReader#readAsArrayBuffer(…) and Blob#text() over FileReader#readAsText(…).
-    'unicorn/prefer-blob-reading-methods': 'error',
-    // Prefer Date.now() to get the number of milliseconds since the Unix Epoch.
-    'unicorn/prefer-date-now': 'error',
-    // Prefer default parameters over reassignment.
-    'unicorn/prefer-default-parameters': 'warn',
-    // Prefer Node#append() over Node#appendChild().
-    'unicorn/prefer-dom-node-append': 'error',
-    // Prefer using .dataset on DOM elements over calling attribute methods.
-    'unicorn/prefer-dom-node-dataset': 'error',
-    // Prefer childNode.remove() over parentNode.removeChild(childNode).
-    'unicorn/prefer-dom-node-remove': 'error',
-    // Prefer EventTarget over EventEmitter.
-    'unicorn/prefer-event-target': 'warn',
-    // Prefer export…from when re-exporting.
-    'unicorn/prefer-export-from': [
-      'warn',
-      {
-        ignoreUsedVariables: true,
-      },
-    ],
-    // Prefer .includes() over .indexOf() and Array#some() when checking for existence or non-existence.
-    'unicorn/prefer-includes': 'error',
-    // Prefer KeyboardEvent#key over KeyboardEvent#keyCode.
-    'unicorn/prefer-keyboard-event-key': 'error',
-    // Prefer using a logical operator over a ternary.
-    'unicorn/prefer-logical-operator-over-ternary': 'error',
-    // Enforce the use of Math.trunc instead of bitwise operators.
-    'unicorn/prefer-math-trunc': 'error',
-    // Prefer .before() over .insertBefore(), .replaceWith() over .replaceChild(), prefer one of .before(), .after(), .append() or .prepend() over insertAdjacentText() and insertAdjacentElement().
-    'unicorn/prefer-modern-dom-apis': 'error',
-    // Prefer modern Math APIs over legacy patterns.
-    'unicorn/prefer-modern-math-apis': 'error',
-    // Prefer negative index over .length - index when possible.
-    'unicorn/prefer-negative-index': 'error',
-    // Require using the `node:` protocol when importing Node.js built-in modules.
-    'unicorn/prefer-node-protocol': 'error',
-    // Prefer Number static properties over global ones.
-    'unicorn/prefer-number-properties': ['error', { checkInfinity: false }],
-    // Prefer using Object.fromEntries(…) to transform a list of key-value pairs into an object.
-    'unicorn/prefer-object-from-entries': 'error',
-    // Prefer Reflect.apply() over Function#apply().
-    'unicorn/prefer-reflect-apply': 'error',
-    // using RegExp.test() is faster than string.match()
-    // note: you should not use the global flag /g with RegExp.test() though!
-    'unicorn/prefer-regexp-test': 'error',
-    // Prefer Set#has() over Array#includes() when checking for existence or non-existence.
-    'unicorn/prefer-set-has': 'error',
-    // Prefer using Set#size instead of Array#length.
-    'unicorn/prefer-set-size': 'error',
-    // Prefer the spread operator over Array.from().
-    'unicorn/prefer-spread': 'error',
-    // Prefer String#replaceAll() over regex searches with the global flag.
-    'unicorn/prefer-string-replace-all': 'error',
-    // Prefer String#slice() over String#substr() and String#substring().
-    'unicorn/prefer-string-slice': 'error',
-    // Prefer String#startsWith() & String#endsWith() over RegExp#test().
-    'unicorn/prefer-string-starts-ends-with': 'error',
-    // Prefer ternary expressions over simple if-else statements.
-    'unicorn/prefer-ternary': ['warn', 'only-single-line'],
-    // Prefer top-level await over top-level promises and async function calls.
-    'unicorn/prefer-top-level-await': 'error',
-    // Enforce using the separator argument with Array#join().
-    'unicorn/require-array-join-separator': 'error',
-    // Enforce consistent brace style for case clauses.
-    'unicorn/switch-case-braces': 'error',
-  },
-});