fix: skip add plugin when already imported (#70)

packages/migrate-config/src/migrate-config.js

@@ -607,7 +607,11 @@ function createPlugins(plugins, migration) {
 				bindings: ["fixupPluginRules"],
 				added: true,
 			});
-		} else {
+		} else if (
+			!migration.imports
+				.get("@eslint/compat")
+				.bindings.includes("fixupPluginRules")
+		) {
 			migration.imports
 				.get("@eslint/compat")
 				.bindings.push("fixupPluginRules");

packages/migrate-config/tests/fixtures/import-duplicate/.eslintrc.cjs

@@ -0,0 +1,8 @@
+module.exports = {
+  plugins: ["react"],
+  overrides: [
+    {
+      plugins: ["react-hooks"],
+    }
+  ]
+};

packages/migrate-config/tests/fixtures/import-duplicate/expected.cjs

@@ -0,0 +1,17 @@
+const react = require("eslint-plugin-react");
+
+const {
+    fixupPluginRules,
+} = require("@eslint/compat");
+
+const reactHooks = require("eslint-plugin-react-hooks");
+
+module.exports = [{
+    plugins: {
+        react: fixupPluginRules(react),
+    },
+}, {
+    plugins: {
+        "react-hooks": fixupPluginRules(reactHooks),
+    },
+}];

packages/migrate-config/tests/fixtures/import-duplicate/expected.mjs

@@ -0,0 +1,13 @@
+import react from "eslint-plugin-react";
+import { fixupPluginRules } from "@eslint/compat";
+import reactHooks from "eslint-plugin-react-hooks";
+
+export default [{
+    plugins: {
+        react: fixupPluginRules(react),
+    },
+}, {
+    plugins: {
+        "react-hooks": fixupPluginRules(reactHooks),
+    },
+}];

packages/migrate-config/tests/migrate-config-cli.test.js

@@ -25,6 +25,7 @@ const filePaths = [
 	"plugins-dedupe/.eslintrc.yml",
 	"gitignore-simple/.eslintrc.json",
 	"gitignore-complex/.eslintrc.json",
+	"import-duplicate/.eslintrc.cjs",
 ].map(file => `tests/fixtures/${file}`);
 
 //-----------------------------------------------------------------------------