chore: cleanup

vitejs · Jul 4, 2023 · eecbe97 · eecbe97
1 parent 6b796ce
commit eecbe97
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/packages/vite/src/node/ssr/runtimeClient.ts b/packages/vite/src/node/ssr/runtimeClient.ts
@@ -268,7 +268,7 @@ export class ViteRuntimeClient {
       debugTimer = setTimeout(
         () =>
           warn(
-            `[vite-node] module ${fsPath} takes over 2s to load.\n${getStack()}`,
+            `[vite-runtime-client] module ${fsPath} takes over 2s to load.\n${getStack()}`,
           ),
         2000,
       )
@@ -367,12 +367,14 @@ export class ViteRuntimeClient {
       return exports
     }
 
-    if (transformed == null)
+    if (transformed == null) {
+      const importer = callstack[callstack.length - 2]
       throw new Error(
-        `[vite-runtime-client] Failed to load "${id}" imported from ${
-          callstack[callstack.length - 2]
+        `[vite-runtime-client] Failed to load "${id}"${
+          importer ? `imported from ${importer}` : ''
         }`,
       )
+    }
 
     const modulePath = cleanUrl(moduleId)
     // disambiguate the `<UNIT>:/` on windows: see nodejs/node#31710

diff --git a/packages/vite/src/node/utils/shared.ts b/packages/vite/src/node/utils/shared.ts
@@ -180,7 +180,10 @@ const builtins = new Set([
   'wasi',
 ])
 
+const prefixedBuiltins = new Set(['node:test'])
+
 export function isBuiltin(id: string): boolean {
+  if (prefixedBuiltins.has(id)) return true
   return builtins.has(id.replace(/^node:/, ''))
 }