chore: noExternal esm-env

Author: bluwy

packages/e2e-tests/kit-node/__tests__/kit.spec.ts

@@ -285,7 +285,8 @@ describe('kit-node', () => {
 						'/^svelte\\//', // serialized with toString
 						'e2e-test-dep-svelte-api-only',
 						'svelte-i18n',
-						'esm-env',
+						'esm-env', // first added by svelte-kit
+						'esm-env', // second added by vite-plugin-svelte
 						'@sveltejs/kit'
 					],
 					`ssr.noExternal in ${filename}`

packages/vite-plugin-svelte/src/utils/options.js

@@ -572,6 +572,11 @@ function buildExtraConfigForSvelte(config) {
 	if (!isDepExternaled('svelte', config.ssr?.external ?? [])) {
 		noExternal.push('svelte', /^svelte\//);
 	}
+	// esm-env needs to be bundled by default for the development/production condition
+	// be properly used by svelte
+	if (!isDepExternaled('esm-env', config.ssr?.external ?? [])) {
+		noExternal.push('esm-env');
+	}
 	return { optimizeDeps: { include, exclude }, ssr: { noExternal, external } };
 }