perf: wrap dynamic import to skip analysis of dynamic imports (#80)

vitejs · Apr 5, 2023 · 4371853 · 4371853
1 parent 14767fc
commit 4371853
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/index.ts b/src/index.ts
@@ -112,7 +112,7 @@ ${result.code}
 
 window.$RefreshReg$ = prevRefreshReg;
 window.$RefreshSig$ = prevRefreshSig;
-import(/* @vite-ignore */ import.meta.url).then((currentExports) => {
+RefreshRuntime.__hmr_import(import.meta.url).then((currentExports) => {
   RefreshRuntime.registerExportsForReactRefresh("${id}", currentExports);
   import.meta.hot.accept((nextExports) => {
     if (!nextExports) return;

diff --git a/src/refresh-runtime.js b/src/refresh-runtime.js
@@ -612,5 +612,9 @@ function predicateOnExport(moduleExports, predicate) {
   return true;
 }
 
+// Hides vite-ignored dynamic import so that Vite can skip analysis if no other
+// dynamic import is present (https://github.com/vitejs/vite/pull/12732)
+export const __hmr_import = (module) => import(/* @vite-ignore */ module);
+
 // For backwards compatibility with @vitejs/plugin-react.
 export default { injectIntoGlobalHook };