eslint-config-seek 13.1.1 → 14.0.1

package/README.md

@@ -5,35 +5,31 @@
 
 This package includes the shareable ESLint configuration used by [SEEK](https://github.com/seek-oss/).
 
-## Usage in sku Projects
+## Usage in sku and skuba Projects
 
-The easiest way to use this configuration is with [sku](https://github.com/seek-oss/sku), which includes it by default.
+The easiest way to use this configuration is with [sku](https://github.com/seek-oss/sku) or [skuba](https://github.com/seek-oss/skuba).
 
-**You don’t need to install it separately in sku projects.**
+**You don’t need to install it separately in sku and skuba projects.**
 
-## Usage Outside of sku
+## Usage Outside of sku and skuba
 
-If you want to use this ESLint configuration in a project not built with sku, you can install it with following steps.
+If you want to use this ESLint configuration in a project not built with sku or skuba, you can install it with following steps.
 
-First, install this package, ESLint and the necessary plugins listed in this project's [package.json](package.json).
+First, install this package, and the necessary peer dependencies listed in this project's [package.json](package.json).
 
-Then create a file named `.eslintrc` with following contents in the root folder of your project:
+Then create a file named `eslint.config.js` with the following contents in the root folder of your project:
 
-```json
-{
-  "extends": "seek"
-}
+```js
+module.exports = require('eslint-config-seek');
 ```
 
 The default configuration includes support for React projects. For projects that are not based on React, the "base" configuration should be used instead:
 
-```json
-{
-  "extends": "seek/base"
-}
+```js
+module.exports = require('eslint-config-seek/base');
 ```
 
-You can override the settings from `eslint-config-seek` by editing the `.eslintrc` file. Learn more about [configuring ESLint](http://eslint.org/docs/user-guide/configuring) on the ESLint website.
+You can override the settings from `eslint-config-seek` by editing the `eslint.config.js` file. Learn more about [configuring ESLint](https://eslint.org/docs/latest/use/configure/) on the ESLint website.
 
 ## License
 

package/base.js

@@ -1,4 +1,9 @@
-const root = require('find-root')(process.cwd());
+const importX = require('eslint-plugin-import-x');
+const globals = require('globals');
+const jestPlugin = require('eslint-plugin-jest');
+const cypress = require('eslint-plugin-cypress');
+const eslintConfigPrettier = require('eslint-config-prettier');
+const tseslint = require('typescript-eslint');
 
 const OFF = 0;
 const ERROR = 2;
@@ -79,149 +84,159 @@ const baseRules = {
 const { js: jsExtensions, ts: tsExtensions } = require('./extensions');
 const allExtensions = [...jsExtensions, ...tsExtensions];
 
-/** @type {import('eslint').Linter.Config} */
-const baseConfig = {
-  parser: '@babel/eslint-parser',
-  parserOptions: {
-    requireConfigFile: false,
-    sourceType: 'module',
-  },
-  root: true,
-  env: {
+const settings = {
+  'import-x/resolver': {
+    typescript: true,
     node: true,
   },
-  plugins: ['import'],
-  extends: [
-    // this config enables eslint-plugin-import to resolve JavaScript and TypeScript files
-    // https://github.com/import-js/eslint-plugin-import/blob/v2.26.0/config/typescript.js
-    // Some rules provided by eslint-plugin-import e.g. `import/no-duplicates` don't work without it
-    'plugin:import/typescript',
-    'prettier',
-  ],
-  rules: {
-    ...baseRules,
+};
+
+module.exports = [
+  eslintConfigPrettier,
+  {
+    plugins: {
+      'import-x': importX,
+      jest: jestPlugin,
+      cypress,
+      '@typescript-eslint': tseslint.plugin,
+    },
   },
-  overrides: [
-    {
-      // TypeScript config
-      files: [`**/*.{${tsExtensions}}`],
-      parser: '@typescript-eslint/parser',
+  importX.flatConfigs.typescript,
+  {
+    rules: importX.flatConfigs.errors.rules,
+    files: [`**/*.{${jsExtensions}}`],
+  },
+  {
+    rules: importX.flatConfigs.warnings.rules,
+    files: [`**/*.{${jsExtensions}}`],
+  },
+  {
+    languageOptions: {
+      globals: {
+        ...globals.node,
+      },
+
       parserOptions: {
-        // https://github.com/typescript-eslint/typescript-eslint/issues/6544
-        allowAutomaticSingleRunInference: true,
-        ecmaVersion: 2022,
-        project: true,
+        requireConfigFile: false,
+        ecmaVersion: 'latest',
         sourceType: 'module',
+      },
+    },
+    settings,
+    rules: baseRules,
+  },
+  ...[...tseslint.configs.recommended, ...tseslint.configs.stylistic].map(
+    ({ plugins, ...config }) => ({
+      ...config,
+      files: [`**/*.{${tsExtensions}}`],
+    }),
+  ),
+  {
+    files: [`**/*.{${tsExtensions}}`],
+
+    languageOptions: {
+      parser: tseslint.parser,
+
+      parserOptions: {
+        projectService: true,
         warnOnUnsupportedTypeScriptVersion: false,
       },
-      extends: [
-        'plugin:@typescript-eslint/recommended',
-        'plugin:@typescript-eslint/stylistic',
-        'prettier',
+    },
+    settings,
+    rules: {
+      '@typescript-eslint/array-type': [ERROR, { default: 'array-simple' }],
+      '@typescript-eslint/consistent-type-definitions': OFF,
+      '@typescript-eslint/no-unused-expressions': ERROR,
+      '@typescript-eslint/no-unused-vars': [
+        ERROR,
+        { argsIgnorePattern: '^_', ignoreRestSiblings: true },
       ],
-      settings: {
-        // adds comprehensive TypeScript support to eslint-plugin-import
-        // https://github.com/import-js/eslint-import-resolver-typescript
-        'import/resolver': {
-          typescript: {},
-        },
-      },
-      rules: {
-        '@typescript-eslint/array-type': [ERROR, { default: 'array-simple' }],
-        '@typescript-eslint/consistent-type-definitions': OFF,
-        '@typescript-eslint/no-unused-expressions': ERROR,
-        '@typescript-eslint/no-unused-vars': [
-          ERROR,
-          { argsIgnorePattern: '^_', ignoreRestSiblings: true },
-        ],
-        '@typescript-eslint/no-use-before-define': OFF,
-        '@typescript-eslint/no-non-null-assertion': OFF,
-        '@typescript-eslint/ban-ts-comment': OFF,
-        '@typescript-eslint/no-explicit-any': OFF,
-        '@typescript-eslint/explicit-function-return-type': OFF,
-        '@typescript-eslint/no-empty-function': OFF,
-        '@typescript-eslint/no-empty-interface': OFF,
-        '@typescript-eslint/no-inferrable-types': [
-          ERROR,
-          { ignoreParameters: true },
-        ],
-        // prefer TypeScript exhaustiveness checking
-        // https://www.typescriptlang.org/docs/handbook/advanced-types.html#exhaustiveness-checking
-        'default-case': OFF,
-        'arrow-body-style': [ERROR, 'as-needed'],
-        // Use `typescript-eslint`'s no-shadow to avoid false positives with enums
-        // https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-shadow.md
-        'no-shadow': OFF,
-        '@typescript-eslint/no-shadow': ERROR,
+      '@typescript-eslint/no-use-before-define': OFF,
+      '@typescript-eslint/no-non-null-assertion': OFF,
+      '@typescript-eslint/ban-ts-comment': OFF,
+      '@typescript-eslint/no-explicit-any': OFF,
+      '@typescript-eslint/explicit-function-return-type': OFF,
+      '@typescript-eslint/no-empty-function': OFF,
+      '@typescript-eslint/no-empty-interface': OFF,
+      '@typescript-eslint/no-inferrable-types': [
+        ERROR,
+        { ignoreParameters: true },
+      ],
+      // prefer TypeScript exhaustiveness checking
+      // https://www.typescriptlang.org/docs/handbook/advanced-types.html#exhaustiveness-checking
+      'default-case': OFF,
+      'arrow-body-style': [ERROR, 'as-needed'],
+      // Use `typescript-eslint`'s no-shadow to avoid false positives with enums
+      // https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-shadow.md
+      'no-shadow': OFF,
+      '@typescript-eslint/no-shadow': ERROR,
 
-        // These two rules deal with autofixing type imports/exports
-        // https://typescript-eslint.io/rules/consistent-type-imports
-        '@typescript-eslint/consistent-type-imports': [
-          ERROR,
-          { fixStyle: 'inline-type-imports' },
-        ],
-        // https://typescript-eslint.io/rules/consistent-type-exports
-        '@typescript-eslint/consistent-type-exports': [
-          ERROR,
-          { fixMixedExportsWithInlineTypeSpecifier: true },
-        ],
-        // https://typescript-eslint.io/rules/no-import-type-side-effects
-        '@typescript-eslint/no-import-type-side-effects': ERROR,
+      // These two rules deal with autofixing type imports/exports
+      // https://typescript-eslint.io/rules/consistent-type-imports
+      '@typescript-eslint/consistent-type-imports': [
+        ERROR,
+        { fixStyle: 'inline-type-imports' },
+      ],
+      // https://typescript-eslint.io/rules/consistent-type-exports
+      '@typescript-eslint/consistent-type-exports': [
+        ERROR,
+        { fixMixedExportsWithInlineTypeSpecifier: true },
+      ],
+      // https://typescript-eslint.io/rules/no-import-type-side-effects
+      '@typescript-eslint/no-import-type-side-effects': ERROR,
 
-        // This rule deals with merging multiple imports from the same module.
-        // In this case, we want type imports to be inlined when merging with the other imports.
-        // However, there is a pending PR which improves the behaviour of this rule https://github.com/import-js/eslint-plugin-import/pull/2716
-        // https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-duplicates.md#inline-type-imports
-        'import/no-duplicates': [ERROR, { 'prefer-inline': true }],
-      },
+      // This rule deals with merging multiple imports from the same module.
+      // In this case, we want type imports to be inlined when merging with the other imports.
+      // However, there is a pending PR which improves the behaviour of this rule https://github.com/import-js/eslint-plugin-import/pull/2716
+      // https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-duplicates.md#inline-type-imports
+      'import-x/no-duplicates': [ERROR, { 'prefer-inline': true }],
     },
-    {
-      // JavaScript config
-      files: [`**/*.{${jsExtensions}}`],
-      env: {
-        es6: true,
-      },
-      extends: ['plugin:import/errors', 'plugin:import/warnings'],
-      settings: {
-        'import/resolver': {
-          node: {
-            moduleDirectory: [root, 'node_modules'],
-          },
-        },
-      },
-      rules: {
-        'no-undef': ERROR,
-        'no-use-before-define': [ERROR, { functions: false }],
-        'no-unused-expressions': ERROR,
-        'import/no-unresolved': [
-          ERROR,
-          { commonjs: true, amd: true, ignore: ['.svg$', '^file?'] },
-        ],
-        'import/no-duplicates': ERROR,
-      },
+  },
+  {
+    files: [`**/*.{${jsExtensions}}`],
+    languageOptions: {
+      globals: {},
     },
-    {
-      // Jest config
-      files: [
-        `**/__tests__/**/*.{${allExtensions}}`,
-        `**/*.@(spec|test).{${allExtensions}}`,
+    settings,
+    rules: {
+      'no-undef': ERROR,
+      'no-use-before-define': [ERROR, { functions: false }],
+      'no-unused-expressions': ERROR,
+      'import-x/no-unresolved': [
+        ERROR,
+        { commonjs: true, amd: true, ignore: ['.svg$', '^file?'] },
       ],
-      env: {
-        jest: true,
+      'import-x/no-duplicates': ERROR,
+    },
+  },
+  {
+    ...jestPlugin.configs['flat/recommended'],
+    files: [
+      `**/__tests__/**/*.{${allExtensions}}`,
+      `**/*.@(spec|test).{${allExtensions}}`,
+    ],
+  },
+  {
+    files: [
+      `**/__tests__/**/*.{${allExtensions}}`,
+      `**/*.@(spec|test).{${allExtensions}}`,
+    ],
+    languageOptions: {
+      globals: {
+        ...globals.jest,
       },
-      extends: ['plugin:jest/recommended'],
-      plugins: ['jest'],
     },
-    {
-      // Cypress config
-      files: [`**/cypress/**/*.{${allExtensions}}`],
-      extends: ['plugin:cypress/recommended'],
-      env: {
-        'cypress/globals': true,
+  },
+  {
+    ...cypress.configs.recommended,
+    files: [`**/cypress/**/*.{${allExtensions}}`],
+  },
+  {
+    files: [`**/cypress/**/*.{${allExtensions}}`],
+    languageOptions: {
+      globals: {
+        ...cypress.environments.globals.globals,
       },
-      plugins: ['cypress'],
     },
-  ],
-};
-
-module.exports = baseConfig;
+  },
+];

package/index.js

@@ -1,3 +1,10 @@
+const react = require('eslint-plugin-react');
+const reactHooks = require('eslint-plugin-react-hooks');
+const base = require('./base');
+const { fixupPluginRules } = require('@eslint/compat');
+
+const globals = require('globals');
+
 const OFF = 0;
 const ERROR = 2;
 
@@ -16,46 +23,41 @@ const reactRules = {
   ],
 };
 
-/** @type {import('eslint').Linter.Config} */
-const eslintConfig = {
-  env: {
-    browser: true,
-  },
-  settings: {
-    react: {
-      version: 'detect',
+module.exports = [
+  react.configs.flat.recommended,
+  react.configs.flat['jsx-runtime'],
+  ...base,
+  {
+    plugins: {
+      react,
+      'react-hooks': fixupPluginRules(reactHooks),
     },
-  },
-  plugins: ['react', 'react-hooks'],
-  extends: [
-    'plugin:react/recommended',
-    'plugin:react/jsx-runtime',
-    './base.js',
-  ],
-  parserOptions: {
-    babelOptions: {
-      presets: [require.resolve('@babel/preset-react')],
+
+    languageOptions: {
+      globals: globals.browser,
     },
-  },
-  rules: {
-    ...reactRules,
-  },
-  overrides: [
-    {
-      // temporary override until everybody removes the React import
-      files: [`**/*.tsx`],
-      rules: {
-        '@typescript-eslint/no-unused-vars': [
-          ERROR,
-          {
-            argsIgnorePattern: '^_',
-            ignoreRestSiblings: true,
-            varsIgnorePattern: '^React$',
-          },
-        ],
+
+    settings: {
+      react: {
+        version: 'detect',
       },
     },
-  ],
-};
 
-module.exports = eslintConfig;
+    rules: reactRules,
+  },
+  {
+    files: ['**/*.tsx'],
+
+    rules: {
+      // temporary override until everybody removes the React import
+      '@typescript-eslint/no-unused-vars': [
+        ERROR,
+        {
+          argsIgnorePattern: '^_',
+          ignoreRestSiblings: true,
+          varsIgnorePattern: '^React$',
+        },
+      ],
+    },
+  },
+];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-seek",
-  "version": "13.1.1",
+  "version": "14.0.1",
   "description": "ESLint configuration used by SEEK",
   "main": "index.js",
   "files": [
@@ -19,41 +19,37 @@
   },
   "homepage": "https://github.com/seek-oss/eslint-config-seek#readme",
   "dependencies": {
-    "@babel/core": "^7.22.6",
-    "@babel/eslint-parser": "^7.22.6",
-    "@babel/preset-react": "^7.22.5",
-    "eslint-plugin-cypress": "^3.0.0",
-    "@typescript-eslint/eslint-plugin": "^7.2.0",
-    "@typescript-eslint/parser": "^7.2.0",
-    "eslint-config-prettier": "^8.8.0",
-    "eslint-import-resolver-typescript": "^3.6.1",
-    "eslint-plugin-import": "^2.27.5",
-    "eslint-plugin-jest": "^27.9.0",
-    "eslint-plugin-react": "^7.32.2",
-    "eslint-plugin-react-hooks": "^4.6.0",
-    "find-root": "^1.1.0"
+    "@eslint/compat": "^1.1.1",
+    "typescript-eslint": "^8.6.0",
+    "eslint-config-prettier": "^9.1.0",
+    "eslint-import-resolver-typescript": "^3.6.3",
+    "eslint-plugin-cypress": "^3.5.0",
+    "eslint-plugin-import-x": "^4.2.1",
+    "eslint-plugin-jest": "^28.8.0",
+    "eslint-plugin-react": "^7.35.0",
+    "eslint-plugin-react-hooks": "^4.6.2",
+    "globals": "^15.9.0"
   },
   "devDependencies": {
-    "@changesets/cli": "^2.26.2",
-    "@changesets/get-github-info": "^0.5.2",
-    "eslint": "^8.56.0",
-    "prettier": "^2.8.4",
-    "typescript": "~5.4.2"
+    "@changesets/cli": "^2.27.7",
+    "@changesets/get-github-info": "^0.6.0",
+    "eslint": "^9.8.0",
+    "prettier": "^3.3.3",
+    "typescript": "~5.5.4"
   },
   "peerDependencies": {
-    "eslint": ">=8.56.0",
-    "typescript": ">=4.7.5"
+    "eslint": ">=9.9.1",
+    "typescript": ">=5.5.4"
   },
   "engines": {
     "node": ">=18.18.0"
   },
-  "packageManager": "pnpm@8.6.6",
   "volta": {
     "node": "18.18.0"
   },
   "scripts": {
     "release": "changeset publish",
-    "test": "eslint . && eslint --config base.js .",
+    "test": "eslint --config index.js . && eslint --config base.js .",
     "changeset-version": "changeset version && prettier --write ."
   }
 }