fix(config): support builtin modules and deno (#10457)

* fix(config): skip resolve builtin modules (#10420)

* fix(config): partial deno support (#10446)

Author: bluwy

packages/vite/src/node/config.ts

@@ -25,6 +25,7 @@ import {
   createDebugger,
   createFilter,
   dynamicImport,
+  isBuiltin,
   isExternalUrl,
   isObject,
   lookupFile,
@@ -972,7 +973,11 @@ async function bundleConfigFile(
 
           build.onResolve({ filter: /.*/ }, ({ path: id, importer, kind }) => {
             // externalize bare imports
-            if (id[0] !== '.' && !isAbsolute(id)) {
+            if (id[0] !== '.' && !path.isAbsolute(id) && !isBuiltin(id)) {
+              // partial deno support as `npm:` does not work in `tryNodeResolve`
+              if (id.startsWith('npm:')) {
+                return { external: true }
+              }
               let idFsPath = tryNodeResolve(id, importer, options, false)?.id
               if (idFsPath && (isESM || kind === 'dynamic-import')) {
                 idFsPath = pathToFileURL(idFsPath).href
@@ -1103,7 +1108,3 @@ export function isDepsOptimizerEnabled(
     (command === 'serve' && disabled === 'dev')
   )
 }
-
-function isAbsolute(id: string) {
-  return path.isAbsolute(id) || path.posix.isAbsolute(id)
-}

playground/cli-module/package.json

@@ -8,5 +8,8 @@
     "build": "vite build",
     "debug": "node --inspect-brk ../../packages/vite/bin/vite",
     "serve": "vite preview"
+  },
+  "devDependencies": {
+    "url": "^0.11.0"
   }
 }

playground/cli-module/vite.config.js

@@ -1,5 +1,11 @@
+// eslint-disable-next-line import/no-nodejs-modules
+import { URL } from 'url'
 import { defineConfig } from 'vite'
 
+// make sure bundling works even if `url` refers to the locally installed
+// `url` package instead of the built-in `url` nodejs module
+globalThis.__test_url = URL
+
 export default defineConfig({
   server: {
     host: 'localhost'