Build edge/worker runtime with webworker ssr target

withastro · May 16, 2023 · 01ef87e · 01ef87e
1 parent d7007a1
commit 01ef87e
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 6 deletions.
diff --git a/.changeset/silver-ties-vanish.md b/.changeset/silver-ties-vanish.md
@@ -0,0 +1,7 @@
+---
+'@astrojs/cloudflare': patch
+'@astrojs/vercel': patch
+'@astrojs/solid-js': patch
+---
+
+Always build edge/worker runtime with Vite `webworker` SSR target
diff --git a/packages/integrations/cloudflare/src/index.ts b/packages/integrations/cloudflare/src/index.ts
@@ -72,8 +72,8 @@ export default function createIntegration(args?: Options): AstroIntegration {
 			},
 			'astro:build:setup': ({ vite, target }) => {
 				if (target === 'server') {
-					vite.resolve = vite.resolve || {};
-					vite.resolve.alias = vite.resolve.alias || {};
+					vite.resolve ||= {};
+					vite.resolve.alias ||= {};
 
 					const aliases = [{ find: 'react-dom/server', replacement: 'react-dom/server.browser' }];
 
@@ -84,8 +84,8 @@ export default function createIntegration(args?: Options): AstroIntegration {
 							(vite.resolve.alias as Record<string, string>)[alias.find] = alias.replacement;
 						}
 					}
-					vite.ssr = vite.ssr || {};
-					vite.ssr.target = vite.ssr.target || 'webworker';
+					vite.ssr ||= {};
+					vite.ssr.target = 'webworker';
 				}
 			},
 			'astro:build:done': async ({ pages }) => {

diff --git a/packages/integrations/solid/src/index.ts b/packages/integrations/solid/src/index.ts
@@ -45,7 +45,6 @@ async function getViteConfiguration(isDev: boolean, astroConfig: AstroConfig) {
 			exclude: ['@astrojs/solid-js/server.js', ...solidPkgsConfig.optimizeDeps.exclude],
 		},
 		ssr: {
-			target: 'node',
 			external: ['babel-preset-solid', ...solidPkgsConfig.ssr.external],
 			noExternal: [...solidPkgsConfig.ssr.noExternal],
 		},

diff --git a/packages/integrations/vercel/src/edge/adapter.ts b/packages/integrations/vercel/src/edge/adapter.ts
@@ -101,7 +101,7 @@ export default function vercelEdge({
 					}
 
 					vite.ssr ||= {};
-					vite.ssr.target ||= 'webworker';
+					vite.ssr.target = 'webworker';
 
 					// Vercel edge runtime is a special webworker-ish environment that supports process.env,
 					// but Vite would replace away `process.env` in webworkers, so we set a define here to prevent it