Fix

withastro · Jan 15, 2025 · ab9d953 · ab9d953
1 parent 9a41e56
commit ab9d953
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 10 deletions.
diff --git a/packages/astro/src/core/routing/manifest/create.ts b/packages/astro/src/core/routing/manifest/create.ts
@@ -724,7 +724,7 @@ export async function createRouteManifest(
 	// Ideally we would only inject the server islands route if server islands are used in the project.
 	// Unfortunately, there is a "circular dependency": to know if server islands are used, we need to run
 	// the build but the build relies on the routes manifest.
-	if (settings.adapter?.adapterFeatures?.buildOutput !== 'static') {
+	if (settings.adapter && settings.adapter.adapterFeatures?.buildOutput !== 'static') {
 		injectServerIslandRoute(settings.config, { routes });
 	}
 	await runHookRoutesResolved({ routes, settings, logger });

diff --git a/packages/astro/test/units/routing/manifest.test.js b/packages/astro/test/units/routing/manifest.test.js
@@ -260,10 +260,6 @@ describe('routing - createRouteManifest', () => {
 		});
 
 		assert.deepEqual(getManifestRoutes(manifest), [
-			{
-				route: '/_server-islands/[name]',
-				type: 'page',
-			},
 			{
 				route: '/_image',
 				type: 'endpoint',
@@ -314,10 +310,7 @@ describe('routing - createRouteManifest', () => {
 		});
 
 		assert.deepEqual(getManifestRoutes(manifest), [
-			{
-				route: '/_server-islands/[name]',
-				type: 'page',
-			},
+
 			{
 				route: '/_image',
 				type: 'endpoint',
@@ -457,7 +450,6 @@ describe('routing - createRouteManifest', () => {
 		});
 
 		assert.deepEqual(getManifestRoutes(manifest), [
-			{ type: 'page', route: '/_server-islands/[name]' },
 			{ type: 'endpoint', route: '/_image' },
 			{ type: 'endpoint', route: '/blog/a-[b].233' },
 			{ type: 'redirect', route: '/posts/a-[b].233' },