eslint-config-seek 0.0.0-eslint-9-etc-20240810051715 → 0.0.0-eslint-9-etc-20240819093637

package/README.md

@@ -5,7 +5,7 @@
 
 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) or [skuba](https://github.com/seek-oss/skuba).
 
@@ -17,7 +17,7 @@ If you want to use this ESLint configuration in a project not built with sku or
 
 First, install this package, and the necessary peer dependencies listed in this project's [package.json](package.json).
 
-Then create a file named `eslint.config.js` 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:
 
 ```js
 module.exports = require('eslint-config-seek');

package/base.js

@@ -6,6 +6,7 @@ const tsParser = require('@typescript-eslint/parser');
 const jestPlugin = require('eslint-plugin-jest');
 const cypress = require('eslint-plugin-cypress');
 const js = require('@eslint/js');
+const unusedImports = require('eslint-plugin-unused-imports');
 
 const { FlatCompat } = require('@eslint/eslintrc');
 
@@ -64,10 +65,15 @@ const baseRules = {
   'no-label-var': ERROR,
   'no-shadow': ERROR,
   'no-undef-init': ERROR,
-  'no-unused-vars': [
+
+  // Use unused-imports instead of no-unused-vars for autofix support
+  'no-unused-vars': OFF,
+  'unused-imports/no-unused-imports': ERROR,
+  'unused-imports/no-unused-vars': [
     ERROR,
     { argsIgnorePattern: '^_', ignoreRestSiblings: true },
   ],
+
   'handle-callback-err': ERROR,
   'no-new-require': ERROR,
   'no-path-concat': ERROR,
@@ -108,6 +114,7 @@ module.exports = [
   {
     plugins: {
       'import-x': importX,
+      'unused-imports': unusedImports,
     },
 
     languageOptions: {
@@ -149,15 +156,21 @@ module.exports = [
         warnOnUnsupportedTypeScriptVersion: false,
       },
     },
+    plugins: { 'unused-imports': unusedImports },
     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': [
+
+      // Use unused-imports instead of no-unused-vars for autofix support
+      '@typescript-eslint/no-unused-vars': [OFF],
+      'unused-imports/no-unused-imports': ERROR,
+      'unused-imports/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,

package/index.js

@@ -64,17 +64,4 @@ module.exports = [
 
     rules: reactRules,
   },
-  {
-    files: ['**/*.tsx'],
-
-    rules: {
-      '@typescript-eslint/no-unused-vars': [
-        ERROR,
-        {
-          argsIgnorePattern: '^_',
-          ignoreRestSiblings: true,
-        },
-      ],
-    },
-  },
 ];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-seek",
-  "version": "0.0.0-eslint-9-etc-20240810051715",
+  "version": "0.0.0-eslint-9-etc-20240819093637",
   "description": "ESLint configuration used by SEEK",
   "main": "index.js",
   "files": [
@@ -34,6 +34,7 @@
     "eslint-plugin-jest": "^28.7.0",
     "eslint-plugin-react": "^7.35.0",
     "eslint-plugin-react-hooks": "^4.6.2",
+    "eslint-plugin-unused-imports": "^4.1.3",
     "find-root": "^1.1.0",
     "globals": "^15.9.0"
   },
@@ -56,7 +57,7 @@
   },
   "scripts": {
     "release": "changeset publish",
-    "test": "eslint --config eslint.config.js . && eslint --config eslint.config.base.js .",
+    "test": "eslint --config index.js . && eslint --config base.js .",
     "changeset-version": "changeset version && prettier --write ."
   }
 }