fix(browser): correctly overwrite loupe require

vitest-dev · Jul 10, 2024 · 756ac2b · 756ac2b
1 parent 0d534fb
commit 756ac2b
Showing 1 changed file with 1 addition and 6 deletions.
diff --git a/packages/browser/src/node/plugin.ts b/packages/browser/src/node/plugin.ts
@@ -22,8 +22,6 @@ export default (browserServer: BrowserServer, base = '/'): Plugin[] => {
   const distRoot = resolve(pkgRoot, 'dist')
   const project = browserServer.project
 
-  let loupePath: string
-
   return [
     {
       enforce: 'pre',
@@ -228,11 +226,8 @@ export default (browserServer: BrowserServer, base = '/'): Plugin[] => {
           return resolve(distRoot, 'client', id.slice(1))
         }
       },
-      configResolved(config) {
-        loupePath = resolve(config.cacheDir, 'deps/loupe.js')
-      },
       transform(code, id) {
-        if (id.startsWith(loupePath)) {
+        if (id.includes(browserServer.vite.config.cacheDir) && id.includes('loupe.js')) {
           // loupe bundle has a nastry require('util') call that leaves a warning in the console
           const utilRequire = 'nodeUtil = require_util();'
           return code.replace(utilRequire, ' '.repeat(utilRequire.length))