From 6061436add0e8a43525608d190a24cad379933d3 Mon Sep 17 00:00:00 2001 From: BenceSzalai Date: Thu, 3 Nov 2022 03:10:39 +0100 Subject: [PATCH] fix: exclude HMR responses from dummy sourcemap generation Compatibility was broken with fix https://github.com/vitejs/vite/pull/9914 but also we don't need and don't want to add dummy sourcemaps to HMR JS requests. --- packages/vite/src/node/server/transformRequest.ts | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/packages/vite/src/node/server/transformRequest.ts b/packages/vite/src/node/server/transformRequest.ts index 252551e4f38a42..6cb541df63c36b 100644 --- a/packages/vite/src/node/server/transformRequest.ts +++ b/packages/vite/src/node/server/transformRequest.ts @@ -253,8 +253,16 @@ async function loadAndTransform( code = transformResult.code! map = transformResult.map - // To enable debugging create a sourcemap for known modified JS files without one: - if (!map && mod.file && mod.type === 'js' && code !== originalCode) { + // To enable IDE debugging add a minimal sourcemap for modified non-HMR JS files without one: + if ( + !map && + mod.file && + mod.type === 'js' && + code !== originalCode && + !code.startsWith( + 'import { createHotContext as __vite__createHotContext } from' + ) + ) { map = new MagicString(code).generateMap({ source: mod.file }) } }