fix: compatible with Win32 when enable onDemand compile

alibaba · Aug 16, 2023 · 1129e80 · 1129e80
1 parent b588f0a
commit 1129e80
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/.changeset/empty-penguins-give.md b/.changeset/empty-penguins-give.md
@@ -0,0 +1,5 @@
+---
+'@ice/app': patch
+---
+
+fix: compatible with Win32 when enable onDemand request
diff --git a/packages/ice/src/service/Runner.ts b/packages/ice/src/service/Runner.ts
@@ -6,6 +6,7 @@ import vm from 'vm';
 import { isNodeBuiltin } from 'mlly';
 import consola from 'consola';
 import lodash from '@ice/bundles/compiled/lodash/index.js';
+import dynamicImport from '../utils/dynamicImport.js';
 
 export interface ModuleResult {
   code?: string;
@@ -244,7 +245,9 @@ class Runner {
   }
 
   async interopedImport(id: string) {
-    const importedModule = await import(id);
+    // If dependency is pre bundled, it will returned as an absolute path,
+    // use dynamic import to get the module otherwise it will cause error of URL scheme in Win32.
+    const importedModule = path.isAbsolute(id) ? await dynamicImport(id) : await import(id);
     if (!this.shouldInterop(id, importedModule)) {
       return importedModule;
     }