fix: use correct self-reference import (#180)

Author: sapphi-red

packages/vite/rollup.config.ts

@@ -115,7 +115,6 @@ const nodeConfig = defineConfig({
     constants: path.resolve(__dirname, 'src/node/constants.ts'),
   },
   external: [
-    /^vite\//,
     'fsevents',
     /^rolldown\//,
     /^tsx\//,
@@ -124,6 +123,14 @@ const nodeConfig = defineConfig({
     ...Object.keys(pkg.peerDependencies),
   ],
   plugins: [
+    {
+      name: 'externalize-vite',
+      resolveId(id) {
+        if (id.startsWith('vite/')) {
+          return { id: id.replace(/^vite\//, 'rolldown-vite/'), external: true }
+        }
+      },
+    },
     // Some deps have try...catch require of optional deps, but rollup will
     // generate code that force require them upfront for side effects.
     // Shim them with eval() so rollup can skip these calls.

packages/vite/rollup.dts.config.ts

@@ -15,7 +15,6 @@ const pkg = JSON.parse(
 
 const external = [
   /^node:*/,
-  /^vite\//,
   /^rolldown\//,
   ...Object.keys(pkg.dependencies),
   ...Object.keys(pkg.peerDependencies),
@@ -32,7 +31,18 @@ export default defineConfig({
     format: 'esm',
   },
   external,
-  plugins: [patchTypes(), dts({ respectExternal: true })],
+  plugins: [
+    {
+      name: 'externalize-vite',
+      resolveId(id) {
+        if (id.startsWith('vite/')) {
+          return { id: id.replace(/^vite\//, 'rolldown-vite/'), external: true }
+        }
+      },
+    },
+    patchTypes(),
+    dts({ respectExternal: true }),
+  ],
 })
 
 // Taken from https://stackoverflow.com/a/36328890
@@ -140,7 +150,7 @@ function validateChunkImports(this: PluginContext, chunk: RenderedChunk) {
       !id.startsWith('../') &&
       !id.startsWith('node:') &&
       !id.startsWith('types.d') &&
-      !id.startsWith('vite/') &&
+      !id.startsWith('rolldown-vite/') &&
       // index and moduleRunner have a common chunk "moduleRunnerTransport"
       !id.startsWith('moduleRunnerTransport.d') &&
       !deps.includes(id) &&