From 319dcadd5dbe2393be2655ecb8abc8bbb6fc8e68 Mon Sep 17 00:00:00 2001 From: sapphi-red Date: Thu, 21 Apr 2022 20:03:38 +0900 Subject: [PATCH] refactor: use preloadMarkerWithQuote --- .../src/node/plugins/importAnalysisBuild.ts | 20 +++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/packages/vite/src/node/plugins/importAnalysisBuild.ts b/packages/vite/src/node/plugins/importAnalysisBuild.ts index 5f320508a22870..d178cfc92fb86f 100644 --- a/packages/vite/src/node/plugins/importAnalysisBuild.ts +++ b/packages/vite/src/node/plugins/importAnalysisBuild.ts @@ -22,7 +22,7 @@ export const preloadMarker = `__VITE_PRELOAD__` export const preloadBaseMarker = `__VITE_PRELOAD_BASE__` const preloadHelperId = 'vite/preload-helper' -const preloadMarkerWithQuote = `"${preloadMarker}"` +const preloadMarkerWithQuote = `"${preloadMarker}"` as const /** * Helper for preloading CSS and direct imports of async chunks in parallel to @@ -328,16 +328,16 @@ export function buildImportAnalysisPlugin(config: ResolvedConfig): Plugin { addDeps(normalizedFile) } - let markPos = code.indexOf(preloadMarker, end) + let markerStartPos = code.indexOf(preloadMarkerWithQuote, end) // fix issue #3051 - if (markPos === -1 && imports.length === 1) { - markPos = code.indexOf(preloadMarker) + if (markerStartPos === -1 && imports.length === 1) { + markerStartPos = code.indexOf(preloadMarkerWithQuote) } - if (markPos > 0) { + if (markerStartPos > 0) { s.overwrite( - markPos - 1, - markPos + preloadMarker.length + 1, + markerStartPos, + markerStartPos + preloadMarkerWithQuote.length, // the dep list includes the main chunk, so only need to // preload when there are actual other deps. deps.size > 1 || @@ -347,7 +347,7 @@ export function buildImportAnalysisPlugin(config: ResolvedConfig): Plugin { : `[]`, { contentOnly: true } ) - rewroteMarkerStartPos.add(markPos - 1) + rewroteMarkerStartPos.add(markerStartPos) } } } @@ -359,7 +359,7 @@ export function buildImportAnalysisPlugin(config: ResolvedConfig): Plugin { if (!rewroteMarkerStartPos.has(markerStartPos)) { s.overwrite( markerStartPos, - markerStartPos + 1 + preloadMarker.length + 1, + markerStartPos + preloadMarkerWithQuote.length, 'void 0', { contentOnly: true } ) @@ -367,7 +367,7 @@ export function buildImportAnalysisPlugin(config: ResolvedConfig): Plugin { markerStartPos = code.indexOf( preloadMarkerWithQuote, - markerStartPos + 1 + preloadMarker.length + 1 + markerStartPos + preloadMarkerWithQuote.length ) } while (markerStartPos >= 0)