From e2d658e5fcc34d23e9a7d6c4cf2820e18a24deee Mon Sep 17 00:00:00 2001
From: Jan Jakes <jan.jakes@automattic.com>
Date: Fri, 24 Mar 2023 16:20:39 +0100
Subject: [PATCH] Rewrite .eslintrc.premium.js to new config format

[MAILPOET-5015]
---
 .../js/eslint-config/.eslintrc-premium.js     | 34 ------------
 .../js/eslint-config/eslint-premium.config.js | 54 +++++++++++++++++++
 2 files changed, 54 insertions(+), 34 deletions(-)
 delete mode 100644 packages/js/eslint-config/.eslintrc-premium.js
 create mode 100644 packages/js/eslint-config/eslint-premium.config.js

diff --git a/packages/js/eslint-config/.eslintrc-premium.js b/packages/js/eslint-config/.eslintrc-premium.js
deleted file mode 100644
index 4c4f70b4fc..0000000000
--- a/packages/js/eslint-config/.eslintrc-premium.js
+++ /dev/null
@@ -1,34 +0,0 @@
-module.exports = {
-  extends: ['airbnb', 'plugin:react/jsx-runtime', 'prettier'],
-  env: {
-    browser: true,
-  },
-  parser: '@babel/eslint-parser',
-  parserOptions: {
-    ecmaVersion: 6,
-    ecmaFeatures: {
-      jsx: true,
-    },
-  },
-  plugins: ['react-hooks'],
-  settings: {
-    'import/resolver': 'webpack',
-  },
-  rules: {
-    'class-methods-use-this': 0,
-    // React hooks rules
-    'react-hooks/rules-of-hooks': 'error',
-    'react-hooks/exhaustive-deps': 'warn',
-    'import/prefer-default-export': 0, // we want to stop using default exports and start using named exports
-    // Exceptions
-    'import/extensions': 0, // we wouldn't be able to import jQuery without this line
-    'jsx-a11y/anchor-is-valid': [
-      'error',
-      {
-        components: ['Link'],
-        specialLink: ['to'],
-      },
-    ],
-    'import/no-default-export': 1, // no default exports
-  },
-};
diff --git a/packages/js/eslint-config/eslint-premium.config.js b/packages/js/eslint-config/eslint-premium.config.js
new file mode 100644
index 0000000000..8db3ac8bb3
--- /dev/null
+++ b/packages/js/eslint-config/eslint-premium.config.js
@@ -0,0 +1,54 @@
+const babelParser = require('@babel/eslint-parser');
+const FlatCompat = require('@eslint/eslintrc').FlatCompat;
+const airbnbConfig = require('eslint-config-airbnb');
+const prettierConfig = require('eslint-config-prettier');
+const webpackResolver = require('eslint-import-resolver-webpack');
+const reactJsxRuntimeConfig = require('eslint-plugin-react/configs/jsx-runtime');
+const reactHooksPlugin = require('eslint-plugin-react-hooks');
+const globals = require('globals');
+
+// compat configs
+const compat = new FlatCompat({ baseDirectory: __dirname });
+const airbnbCompatConfig = compat.config(airbnbConfig);
+const prettierCompatConfig = compat.config(prettierConfig);
+
+// React plugin is already defined by airbnb config. This fixes:
+//   TypeError: Key "plugins": Cannot redefine plugin "react"
+delete reactJsxRuntimeConfig.plugins.react;
+
+module.exports = [
+  ...airbnbCompatConfig,
+  reactJsxRuntimeConfig,
+  ...prettierCompatConfig,
+  {
+    languageOptions: {
+      parser: babelParser,
+      globals: {
+        ...globals.browser,
+      },
+    },
+    settings: {
+      'import/resolver': { webpack: webpackResolver },
+    },
+    plugins: {
+      'react-hooks': reactHooksPlugin,
+    },
+    rules: {
+      'class-methods-use-this': 0,
+      // React hooks rules
+      'react-hooks/rules-of-hooks': 'error',
+      'react-hooks/exhaustive-deps': 'warn',
+      'import/prefer-default-export': 0, // we want to stop using default exports and start using named exports
+      // Exceptions
+      'import/extensions': 0, // we wouldn't be able to import jQuery without this line
+      'jsx-a11y/anchor-is-valid': [
+        'error',
+        {
+          components: ['Link'],
+          specialLink: ['to'],
+        },
+      ],
+      'import/no-default-export': 1, // no default exports
+    },
+  },
+];