fix: 🐛 only enable react refresh allowConstantExport for vite

src/rules/__snapshots__/react.spec.ts.snap

@@ -194,7 +194,7 @@ exports[`reactRules > should create react rules 1`] = `
   "react-refresh/only-export-components": [
     "warn",
     {
-      "allowConstantExport": true,
+      "allowConstantExport": undefined,
       "allowExportNames": [],
     },
   ],

src/rules/react.spec.ts

@@ -37,4 +37,17 @@ describe("reactRules", () => {
       ]
     `);
   });
+
+  it("should only enable allowConstantExport for vite", async () => {
+    vi.mocked(isPackageExists).mockImplementation((name) => {
+      return name === "vite";
+    });
+
+    const rules = await reactRules();
+
+    const allowConstantExport =
+      rules["react-refresh/only-export-components"][1].allowConstantExport;
+
+    expect(allowConstantExport).toBeTruthy();
+  });
 });

src/rules/react.ts

@@ -1,6 +1,6 @@
 import type { Rules } from "../types";
 
-import { hasNext } from "../utils/has-dependency";
+import { hasNext, hasVite } from "../utils/has-dependency";
 import { interopDefault } from "../utils/interop-default";
 import { normalizeRuleEntries } from "../utils/normalize-rule-entries";
 
@@ -26,6 +26,7 @@ export const reactRules = async () => {
     interopDefault(import("eslint-plugin-jsx-a11y")),
   ]);
   const isUsingNext = hasNext();
+  const isUsingVite = hasVite();
 
   return {
     ...jsxA11yPlugin.configs.recommended.rules,
@@ -37,7 +38,7 @@ export const reactRules = async () => {
     "react-refresh/only-export-components": [
       "warn",
       {
-        allowConstantExport: true,
+        allowConstantExport: isUsingVite,
         allowExportNames: isUsingNext ? nextAllowedExportNames : [],
       },
     ],

src/utils/has-dependency.ts

@@ -47,3 +47,7 @@ export const hasStorybook = () => {
 export const hasNext = () => {
   return isPackageExists("next");
 };
+
+export const hasVite = () => {
+  return isPackageExists("vite");
+};