From e0505412daf2be60f4adde9491332784088522a9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?M=C3=A1ximo=20Mussini?= Date: Thu, 6 Jan 2022 05:31:27 -0300 Subject: [PATCH] fix: use the resolved vite root (#247) * fix: use the resolved vite root Although vite-plugin-svelte needs to infer a root in order to pre-scan svelte dependencies, it should use the root returned in configResolved to ensure compatibility with backend integrations and other plugins that might modify the root. Since vite-plugin-svelte uses `enforce: 'pre'`, it's not viable for other plugins to workaround this by using `enforce: 'pre'`, since the behavior would become order-dependent on how the user applies the plugins, which is error-prone and would result in a bad user experience. * chore: add changeset --- .changeset/popular-tigers-camp.md | 5 +++++ packages/vite-plugin-svelte/src/utils/options.ts | 1 + 2 files changed, 6 insertions(+) create mode 100644 .changeset/popular-tigers-camp.md diff --git a/.changeset/popular-tigers-camp.md b/.changeset/popular-tigers-camp.md new file mode 100644 index 000000000..99b925ea7 --- /dev/null +++ b/.changeset/popular-tigers-camp.md @@ -0,0 +1,5 @@ +--- +'@sveltejs/vite-plugin-svelte': patch +--- + +use the resolved vite root to support backend integrations diff --git a/packages/vite-plugin-svelte/src/utils/options.ts b/packages/vite-plugin-svelte/src/utils/options.ts index 0f1eccfc7..d2806ebaa 100644 --- a/packages/vite-plugin-svelte/src/utils/options.ts +++ b/packages/vite-plugin-svelte/src/utils/options.ts @@ -112,6 +112,7 @@ export function resolveOptions( ...defaultOptions.compilerOptions, ...preResolveOptions.compilerOptions }, + root: viteConfig.root, isProduction: viteConfig.isProduction }; addExtraPreprocessors(merged, viteConfig);