Merge pull request #5 from lingodotdev/fix/internalize-optimize-deps-hack

fix: add optimize deps hach in the plugin

Author: AleksandrSl

cmp/compiler/package.json

@@ -143,7 +143,7 @@
     "tsdown": "^0.16.5",
     "tsx": "^4.19.2",
     "typescript": "^5.9.3",
-    "unplugin": "^2.3.10",
+    "unplugin": "^2.3.11",
     "vitest": "^4.0.13"
   },
   "dependencies": {

cmp/compiler/src/plugin/unplugin.ts

@@ -136,6 +136,15 @@ export const lingoUnplugin = createUnplugin<
     enforce: "pre", // Run before other plugins (especially before React plugin)
 
     vite: {
+      // Vite handles deep merge
+      config() {
+        // Required for custom virtual like modules to be resolved; otherwise they are bundled with raw source code.
+        return {
+          optimizeDeps: {
+            exclude: ["@lingo.dev/compiler"],
+          },
+        };
+      },
       async buildStart() {
         const metadataFilePath = getMetadataPath();
 

cmp/compiler/tests/e2e/shared/development.test.ts

@@ -23,7 +23,7 @@ test.describe.serial("Development Mode", () => {
     let fixture: TestFixture;
     let devServer: DevServer;
 
-    test.beforeAll(async ({}, testInfo) => {
+    test.beforeAll(async () => {
       fixture = await setupFixture({ framework: "next" });
       devServer = await fixture.startDev();
     });
@@ -85,14 +85,7 @@ test.describe.serial("Development Mode", () => {
       console.log("German heading:", germanHeading);
       expect(germanHeading).toBeTruthy();
 
-      // Log if translation system changed the text
-      if (germanHeading !== heading) {
-        console.log("✅ Translation changed the heading");
-      } else {
-        console.log(
-          "⚠️ Heading unchanged (pseudo-translation may not modify all text)",
-        );
-      }
+      expect.soft(germanHeading).not.toEqual(heading);
 
       // Verify translation request was made (optional check)
       console.log("Translation requests:", translationRequests);
@@ -127,7 +120,7 @@ test.describe.serial("Development Mode", () => {
     let fixture: TestFixture;
     let devServer: DevServer;
 
-    test.beforeAll(async ({}) => {
+    test.beforeAll(async () => {
       fixture = await setupFixture({ framework: "vite" });
       devServer = await fixture.startDev();
     });
@@ -171,11 +164,7 @@ test.describe.serial("Development Mode", () => {
       await expect(spanishHeading).toBeVisible();
       const spanishHeadingText = await spanishHeading.textContent();
 
-      if (spanishHeadingText !== initialHeadingText) {
-        console.log("✅ Translation system changed the text");
-      } else {
-        console.log("⚠️ Text unchanged");
-      }
+      expect(spanishHeadingText).not.toEqual(initialHeadingText);
     });
 
     test("should navigate between pages and persist locale", async ({
@@ -197,7 +186,6 @@ test.describe.serial("Development Mode", () => {
       // Verify locale persisted
       const currentLocale = await getCurrentLocale(page);
       expect(currentLocale).toBe(newLocale);
-      console.log(`✅ Locale persisted across navigation to ${newLocale}`);
     });
   });
 });

cmp/demo/vite-react-spa/vite.config.ts

@@ -29,9 +29,6 @@ export default defineConfig({
     viteReact(),
     tailwindcss(),
   ],
-  optimizeDeps: {
-    exclude: ["@lingo.dev/compiler"],
-  },
   resolve: {
     alias: {
       "@": fileURLToPath(new URL("./src", import.meta.url)),