fix: skip watchPackageDataPlugin for worker builds (#14762)

vitejs · Oct 26, 2023 · 9babef5 · 9babef5
1 parent 521ca58
commit 9babef5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 10 deletions.
diff --git a/packages/vite/src/node/build.ts b/packages/vite/src/node/build.ts
@@ -893,15 +893,6 @@ export function onRollupWarning(
         return
       }
 
-      // Rollup tracks the build phase slightly earlier before `buildEnd` is called,
-      // so there's a chance we can call `this.addWatchFile` in the invalid phase. Skip for now.
-      if (
-        warning.plugin === 'vite:worker-import-meta-url' &&
-        warning.pluginCode === 'INVALID_ROLLUP_PHASE'
-      ) {
-        return
-      }
-
       if (warningIgnoreList.includes(warning.code!)) {
         return
       }

diff --git a/packages/vite/src/node/plugins/index.ts b/packages/vite/src/node/plugins/index.ts
@@ -32,6 +32,7 @@ export async function resolvePlugins(
   postPlugins: Plugin[],
 ): Promise<Plugin[]> {
   const isBuild = config.command === 'build'
+  const isWorker = config.isWorker
   const buildPlugins = isBuild
     ? await (await import('../build')).resolveBuildPlugins(config)
     : { pre: [], post: [] }
@@ -47,7 +48,7 @@ export async function resolvePlugins(
         ]
       : []),
     isBuild ? metadataPlugin() : null,
-    watchPackageDataPlugin(config.packageCache),
+    !isWorker ? watchPackageDataPlugin(config.packageCache) : null,
     preAliasPlugin(config),
     aliasPlugin({ entries: config.resolve.alias }),
     ...prePlugins,