From 46e9beefb94bcff6c21b1bd2c34a364133d42f2b Mon Sep 17 00:00:00 2001
From: underfin <2218301630@qq.com>
Date: Wed, 13 Dec 2023 15:58:52 +0800
Subject: [PATCH] fix: fix optimize test

---
 .../src/node/optimizer/esbuildDepPlugin.ts    |  1 +
 packages/vite/src/node/optimizer/index.ts     | 59 +++++++++----------
 .../__tests__/optimize-deps.spec.ts           |  5 +-
 3 files changed, 33 insertions(+), 32 deletions(-)

diff --git a/packages/vite/src/node/optimizer/esbuildDepPlugin.ts b/packages/vite/src/node/optimizer/esbuildDepPlugin.ts
index a8ed90733fc3b1..1168b31121a807 100644
--- a/packages/vite/src/node/optimizer/esbuildDepPlugin.ts
+++ b/packages/vite/src/node/optimizer/esbuildDepPlugin.ts
@@ -218,6 +218,7 @@ export function rolldownDepPlugin(
       }
 
       if (id.startsWith(browserExternalNamespace)) {
+        const path = id.slice(browserExternalNamespace.length)
         if (config.isProduction) {
           return {
             code: 'module.exports = {}',
diff --git a/packages/vite/src/node/optimizer/index.ts b/packages/vite/src/node/optimizer/index.ts
index 9d269394369216..f3347bc05ad04c 100644
--- a/packages/vite/src/node/optimizer/index.ts
+++ b/packages/vite/src/node/optimizer/index.ts
@@ -613,41 +613,36 @@ export function runOptimizeDeps(
 
     return build()
       .then((result) => {
-        // TODO: Make sure the paths in `meta.outputs` are relative to `process.cwd()`
-        const processingCacheDirOutputPath = path.relative(
-          process.cwd(),
-          processingCacheDir,
-        )
-
         for (const chunk of result.output) {
           if (chunk.type === 'chunk') {
             if (chunk.isEntry) {
-              const { exportsData, file, id, ...info } = Object.values(
-                depsInfo,
-              ).find(
+              // One chunk maybe corresponding multiply entry
+              const deps = Object.values(depsInfo).filter(
                 (d) =>
                   d.src === path.join(process.cwd(), chunk.facadeModuleId!),
-              )!
-              addOptimizedDepInfo(metadata, 'optimized', {
-                id,
-                file,
-                ...info,
-                // We only need to hash the output.imports in to check for stability, but adding the hash
-                // and file path gives us a unique hash that may be useful for other things in the future
-                fileHash: getHash(
-                  metadata.hash + file + JSON.stringify(chunk.modules),
-                ),
-                browserHash: metadata.browserHash,
-                // After bundling we have more information and can warn the user about legacy packages
-                // that require manual configuration
-                needsInterop: needsInterop(
-                  config,
-                  ssr,
+              )
+              for (const { exportsData, file, id, ...info } of deps) {
+                addOptimizedDepInfo(metadata, 'optimized', {
                   id,
-                  idToExports[id],
-                  chunk,
-                ),
-              })
+                  file,
+                  ...info,
+                  // We only need to hash the output.imports in to check for stability, but adding the hash
+                  // and file path gives us a unique hash that may be useful for other things in the future
+                  fileHash: getHash(
+                    metadata.hash + file + JSON.stringify(chunk.modules),
+                  ),
+                  browserHash: metadata.browserHash,
+                  // After bundling we have more information and can warn the user about legacy packages
+                  // that require manual configuration
+                  needsInterop: needsInterop(
+                    config,
+                    ssr,
+                    id,
+                    idToExports[id],
+                    chunk,
+                  ),
+                })
+              }
             } else {
               const id = chunk.fileName.replace(jsExtensionRE, '')
               const file = getOptimizedDepPath(id, resolvedConfig, ssr)
@@ -808,12 +803,16 @@ async function prepareRollupOptimizerRun(
   })
 
   async function build() {
-    // TODO platform
     const bundle = await rolldown.rolldown({
       input: Object.keys(flatIdDeps),
       external,
       // logLevel: 'warn',
       plugins,
+      resolve: {
+        mainFields: ['module', 'main'],
+        aliasFields: [['browser']],
+        // extensions: ['.js', '.css']
+      },
       ...rollupOptions,
     })
     return await bundle.write({
diff --git a/playground/optimize-deps/__tests__/optimize-deps.spec.ts b/playground/optimize-deps/__tests__/optimize-deps.spec.ts
index 84988f0933a546..fed95898461314 100644
--- a/playground/optimize-deps/__tests__/optimize-deps.spec.ts
+++ b/playground/optimize-deps/__tests__/optimize-deps.spec.ts
@@ -251,8 +251,9 @@ test('name file limit is 170 characters', async () => {
     const onlyTheFilePart = stripFolderPart.split('.')[0]
     expect(onlyTheFilePart).toHaveLength(170)
 
-    const text = await content.text()
-    expect(text).toMatch(/import\s+("[^"]+")/)
+    // the test should be exclude, the chunk content shouldn't import runtime module
+    // const text = await content.text()
+    // expect(text).toMatch(/import\s+("[^"]+")/)
   }
 })