chore(eslint-config-pie): DSW-2294 upgrade eslint to v9

apps/examples/wc-react17/eslint.config.js

@@ -0,0 +1,13 @@
+export default [
+    {
+        root: true,
+        extends: [
+            require.resolve('@justeattakeaway/eslint-config-pie/strict'),
+        ],
+        parserOptions: {
+            ecmaFeatures: {
+                jsx: true,
+            },
+        },
+    },
+];

apps/examples/wc-react18/eslint.config.js

@@ -0,0 +1,12 @@
+export default [
+    {
+        extends: [
+            require.resolve('@justeattakeaway/eslint-config-pie/strict'),
+        ],
+        parserOptions: {
+            ecmaFeatures: {
+                jsx: true,
+            },
+        },
+    },
+];

apps/examples/wc-vue3/.eslintrc.js → apps/examples/wc-vue3/eslint.config.js

@@ -1,4 +1,3 @@
-// eslint-disable-next-line import/no-extraneous-dependencies
 const { vue, vue3 } = require('@justeattakeaway/eslint-config-pie/frameworks');
 
 module.exports = {

commitlint.config.js

@@ -1,4 +1,4 @@
-// eslint-disable-next-line @typescript-eslint/no-var-requires
+// eslint-disable-next-line typescript-eslint/no-var-requires
 const czConfig = require('./.cz-config.js');
 
 module.exports = {

eslint.config.mjs

@@ -0,0 +1,48 @@
+import eslintConfigPie from '@justeattakeaway/eslint-config-pie/strict';
+import eslint from '@eslint/js';
+import tseslint from 'typescript-eslint';
+import importPlugin from 'eslint-plugin-import';
+// import jsonPlugin from 'eslint-plugin-json-format';
+
+export default tseslint.config(
+    eslint.configs.recommended,
+    importPlugin.flatConfigs.recommended,
+    ...tseslint.configs.recommendedTypeChecked,
+    ...eslintConfigPie,
+    {
+        plugins: {
+            import: importPlugin,
+            // json: jsonPlugin,
+        },
+        root: true,
+        settings: {
+            'json/sort-package-json': false,
+        },
+        rules: {
+            'class-methods-use-this': 'off',
+            'import/extensions': 'off',
+            'import/no-unresolved': 'off',
+            'import/no-extraneous-dependencies': 'off',
+            'no-shadow': 'off',
+            'no-use-before-define': 'off',
+            'typescript-eslint/no-var-requires': 'off',
+        },
+        overrides: [
+            // {
+            //     files: ['**/*.ts', '**/*.tsx'],
+            //     parser: '@typescript-eslint/parser',
+            //     parserOptions: {
+            //         project: `${__dirname}/tsconfig.json`,
+            //     },
+            // },
+            {
+                files: ['**/components/**/visual/*.spec.ts'],
+                rules: {
+                    // ForOfStatement is allowed in Visual tests
+                    'no-restricted-syntax': ['error', 'ForInStatement', 'LabeledStatement', 'WithStatement'],
+                    'no-await-in-loop': 'off',
+                },
+            }
+        ],
+    },
+);

package.json

@@ -73,6 +73,7 @@
     "@changesets/cli": "2.26.2",
     "@commitlint/cli": "17.5.1",
     "@commitlint/config-conventional": "17.4.4",
+    "@eslint/js": "9.12.0",
     "@justeat/pie-design-tokens": "6.5.0",
     "@justeattakeaway/browserslist-config-pie": "0.2.0",
     "@justeattakeaway/generator-pie-component": "0.23.0",
@@ -86,10 +87,9 @@
     "@playwright/test": "1.41.0",
     "@rollup/plugin-node-resolve": "15.1.0",
     "@sand4rt/experimental-ct-web": "1.41.0",
+    "@types/eslint__js": "8.42.3",
     "@types/node": "20.4.8",
     "@types/react": "18.3.11",
-    "@typescript-eslint/eslint-plugin": "5.62.0",
-    "@typescript-eslint/parser": "5.62.0",
     "@vitest/coverage-c8": "0.29.8",
     "autoprefixer": "10.4.20",
     "babel-loader": "8.3.0",
@@ -99,10 +99,10 @@
     "cz-customizable": "7.2.1",
     "danger": "11.3.0",
     "dree": "3.4.5",
-    "eslint": "8.57.0",
-    "eslint-plugin-import": "2.27.5",
+    "eslint": "9.12.0",
+    "eslint-plugin-import": "2.31.0",
     "eslint-plugin-json-format": "2.0.1",
-    "eslint-plugin-vitest": "0.3.22",
+    "eslint-plugin-vitest": "0.5.4",
     "glob": "10.3.3",
     "husky": "8.0.3",
     "jsdom": "24.0.0",
@@ -118,7 +118,8 @@
     "stylelint-order": "6.0.4",
     "ts-node": "10.9.1",
     "turbo": "1.10.16",
-    "typescript": "5.4.5",
+    "typescript": "5.6.3",
+    "typescript-eslint": "8.10.0",
     "vite": "5.3.6",
     "vite-plugin-dts": "2.3.0",
     "vitest": "0.29.8",

packages/components/pie-radio-group/src/defs.ts

@@ -1,3 +1,3 @@
 // TODO - please remove the eslint disable comment below when you add props to this interface
-// eslint-disable-next-line @typescript-eslint/no-empty-interface
+// eslint-disable-next-line typescript-eslint/no-empty-interface
 export interface RadioGroupProps {}

packages/components/pie-webc-core/src/mixins/formControl/formControlMixin.ts

@@ -47,7 +47,7 @@ export const FormControlMixin =
                 return this._internals.form;
             }
 
-            // eslint-disable-next-line @typescript-eslint/no-explicit-any
+            // eslint-disable-next-line typescript-eslint/no-explicit-any
             constructor (...args: any[]) {
                 super(...args);
                 this._internals = this.attachInternals();

packages/components/pie-webc-core/src/mixins/types/GenericConstructor.ts

@@ -3,5 +3,5 @@
  * @typedef {new (...args: any[]) => T} Constructor
  * @template T
  */
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
+// eslint-disable-next-line typescript-eslint/no-explicit-any
 export type GenericConstructor<T> = new (...args: any[]) => T;

packages/components/pie-webc-testing/src/helpers/defs.ts

@@ -1,4 +1,4 @@
-/* eslint-disable @typescript-eslint/no-explicit-any */
+/* eslint-disable typescript-eslint/no-explicit-any */
 
 /**
  * This test helper type contains the same keys as the component prop interface.