Merge output: hybrid and output: static

packages/astro/e2e/custom-client-directives.test.js

@@ -35,12 +35,13 @@ test.describe('Custom Client Directives - build static', () => {
 	testClientDirectivesShared();
 });
 
-test.describe('Custom Client Directives - build server', () => {
+test.describe('Custom Client Directives - build server zzz', () => {
 	let previewServer;
 
 	test.beforeAll(async ({ astro }) => {
 		await astro.build({
 			adapter: testAdapter(),
+			output: 'server',
 		});
 		previewServer = await astro.preview();
 	});

packages/astro/e2e/fixtures/actions-blog/astro.config.mjs

@@ -7,7 +7,7 @@ import node from '@astrojs/node';
 export default defineConfig({
 	site: 'https://example.com',
 	integrations: [db(), react()],
-	output: 'hybrid',
+	output: 'static',
 	adapter: node({
 		mode: 'standalone',
 	}),

packages/astro/e2e/fixtures/actions-react-19/astro.config.mjs

@@ -7,7 +7,7 @@ import node from '@astrojs/node';
 export default defineConfig({
 	site: 'https://example.com',
 	integrations: [db(), react()],
-	output: 'hybrid',
+	output: 'static',
 	adapter: node({
 		mode: 'standalone',
 	}),

packages/astro/e2e/fixtures/server-islands/astro.config.mjs

@@ -6,7 +6,7 @@ import nodejs from '@astrojs/node';
 // https://astro.build/config
 export default defineConfig({
 	base: '/base',
-	output: 'hybrid',
+	output: 'static',
 	adapter: nodejs({ mode: 'standalone' }),
 	integrations: [react(), mdx()],
 	trailingSlash: process.env.TRAILING_SLASH ?? 'always',

packages/astro/e2e/fixtures/view-transitions/astro.config.mjs

@@ -6,7 +6,7 @@ import { defineConfig } from 'astro/config';
 
 // https://astro.build/config
 export default defineConfig({
-	output: 'hybrid',
+	output: 'static',
 	adapter: nodejs({ mode: 'standalone' }),
 	integrations: [react(),vue(),svelte()],
 	redirects: {

packages/astro/package.json

@@ -118,7 +118,8 @@
     "test:e2e:chrome": "playwright test",
     "test:e2e:firefox": "playwright test --config playwright.firefox.config.js",
     "test:types": "tsc --project tsconfig.tests.json",
-    "test:node": "astro-scripts test \"test/**/*.test.js\""
+    "test:node": "astro-scripts test \"test/**/*.test.js\"",
+    "test:units": "astro-scripts test \"test/**/units/**/*.test.js\""
   },
   "dependencies": {
     "@astrojs/compiler": "^2.10.3",

packages/astro/src/actions/integration.ts

@@ -1,6 +1,6 @@
 import { ActionsWithoutServerOutputError } from '../core/errors/errors-data.js';
 import { AstroError } from '../core/errors/errors.js';
-import { isServerLikeOutput, viteID } from '../core/util.js';
+import { viteID } from '../core/util.js';
 import type { AstroSettings } from '../types/astro.js';
 import type { AstroIntegration } from '../types/public/integrations.js';
 import { ACTIONS_TYPES_FILE, VIRTUAL_MODULE_ID } from './consts.js';
@@ -30,7 +30,7 @@ export default function astroIntegrationActionsRouteHandler({
 				});
 			},
 			'astro:config:done': async (params) => {
-				if (!isServerLikeOutput(params.config)) {
+				if (params.buildOutput() === 'static') {
 					const error = new AstroError(ActionsWithoutServerOutputError);
 					error.stack = undefined;
 					throw error;

packages/astro/src/assets/build/generate.ts

@@ -9,7 +9,6 @@ import { AstroError } from '../../core/errors/errors.js';
 import { AstroErrorData } from '../../core/errors/index.js';
 import type { Logger } from '../../core/logger/core.js';
 import { isRemotePath, removeLeadingForwardSlash } from '../../core/path.js';
-import { isServerLikeOutput } from '../../core/util.js';
 import type { MapValue } from '../../type-utils.js';
 import type { AstroConfig } from '../../types/public/config.js';
 import { getConfiguredImageService } from '../internal.js';
@@ -50,7 +49,7 @@ export async function prepareAssetsGenerationEnv(
 	pipeline: BuildPipeline,
 	totalCount: number,
 ): Promise<AssetEnv> {
-	const { config, logger } = pipeline;
+	const { config, logger, settings } = pipeline;
 	let useCache = true;
 	const assetsCacheDir = new URL('assets/', config.cacheDir);
 	const count = { total: totalCount, current: 1 };
@@ -66,8 +65,9 @@ export async function prepareAssetsGenerationEnv(
 		useCache = false;
 	}
 
+	const isServerOutput = settings.buildOutput === 'server';
 	let serverRoot: URL, clientRoot: URL;
-	if (isServerLikeOutput(config)) {
+	if (isServerOutput) {
 		serverRoot = config.build.server;
 		clientRoot = config.build.client;
 	} else {
@@ -77,7 +77,7 @@ export async function prepareAssetsGenerationEnv(
 
 	return {
 		logger,
-		isSSR: isServerLikeOutput(config),
+		isSSR: isServerOutput,
 		count,
 		useCache,
 		assetsCacheDir,

packages/astro/src/assets/endpoint/config.ts

@@ -1,15 +1,27 @@
-import type { AstroSettings } from '../../types/astro.js';
+import { resolveInjectedRoute } from '../../core/routing/manifest/create.js';
+import type { AstroSettings, ManifestData } from '../../types/astro.js';
 
-export function injectImageEndpoint(settings: AstroSettings, mode: 'dev' | 'build') {
+export function injectImageEndpoint(
+	settings: AstroSettings,
+	manifest: ManifestData,
+	mode: 'dev' | 'build',
+	cwd?: string,
+) {
 	const endpointEntrypoint =
 		settings.config.image.endpoint ??
 		(mode === 'dev' ? 'astro/assets/endpoint/node' : 'astro/assets/endpoint/generic');
 
-	settings.injectedRoutes.push({
-		pattern: '/_image',
-		entrypoint: endpointEntrypoint,
+	manifest.routes.push({
+		type: 'endpoint',
+		isIndex: false,
+		route: '/_image',
+		pattern: /^\/_image$/,
+		segments: [[{ content: '_image', dynamic: false, spread: false }]],
+		params: [],
+		component: resolveInjectedRoute(endpointEntrypoint, settings.config.root, cwd).component,
+		generate: () => '',
+		pathname: '/_image',
 		prerender: false,
+		fallbackRoutes: [],
 	});
-
-	return settings;
 }

packages/astro/src/assets/vite-plugin-assets.ts

@@ -10,7 +10,6 @@ import {
 	removeBase,
 	removeQueryString,
 } from '../core/path.js';
-import { isServerLikeOutput } from '../core/util.js';
 import type { AstroPluginOptions, AstroSettings } from '../types/astro.js';
 import { VALID_INPUT_FORMATS, VIRTUAL_MODULE_ID, VIRTUAL_SERVICE_ID } from './consts.js';
 import type { ImageTransform } from './types.js';
@@ -131,7 +130,7 @@ export default function assets({
 					// so that it's tree-shaken away for all platforms that don't need it.
 					export const outDir = /* #__PURE__ */ new URL(${JSON.stringify(
 						new URL(
-							isServerLikeOutput(settings.config)
+							settings.buildOutput === 'server'
 								? settings.config.build.client
 								: settings.config.outDir,
 						),
@@ -222,7 +221,7 @@ export default function assets({
 					if (options?.ssr) {
 						return `export default ${getProxyCode(
 							imageMetadata,
-							isServerLikeOutput(settings.config),
+							settings.buildOutput === 'server',
 						)}`;
 					} else {
 						globalThis.astroAsset.referencedImages.add(imageMetadata.fsPath);

packages/astro/src/content/vite-plugin-content-imports.ts

@@ -8,7 +8,6 @@ import { getProxyCode } from '../assets/utils/proxy.js';
 import { AstroError } from '../core/errors/errors.js';
 import { AstroErrorData } from '../core/errors/index.js';
 import type { Logger } from '../core/logger/core.js';
-import { isServerLikeOutput } from '../core/util.js';
 import type { AstroSettings } from '../types/astro.js';
 import type { AstroConfig } from '../types/public/config.js';
 import type {
@@ -115,7 +114,7 @@ export function astroContentImportPlugin({
 					const code = `
 export const id = ${JSON.stringify(id)};
 export const collection = ${JSON.stringify(collection)};
-export const data = ${stringifyEntryData(data, isServerLikeOutput(settings.config))};
+export const data = ${stringifyEntryData(data, settings.buildOutput === 'server')};
 export const _internal = {
 	type: 'data',
 	filePath: ${JSON.stringify(_internal.filePath)},
@@ -140,7 +139,7 @@ export const _internal = {
 						export const collection = ${JSON.stringify(collection)};
 						export const slug = ${JSON.stringify(slug)};
 						export const body = ${JSON.stringify(body)};
-						export const data = ${stringifyEntryData(data, isServerLikeOutput(settings.config))};
+						export const data = ${stringifyEntryData(data, settings.buildOutput === 'server')};
 						export const _internal = {
 							type: 'content',
 							filePath: ${JSON.stringify(_internal.filePath)},

packages/astro/src/content/vite-plugin-content-virtual-mod.ts

@@ -8,7 +8,6 @@ import type { Plugin } from 'vite';
 import { encodeName } from '../core/build/util.js';
 import { AstroError, AstroErrorData } from '../core/errors/index.js';
 import { appendForwardSlash, removeFileExtension } from '../core/path.js';
-import { isServerLikeOutput } from '../core/util.js';
 import { rootRelativePath } from '../core/viteUtils.js';
 import type { AstroSettings } from '../types/astro.js';
 import type { AstroPluginMetadata } from '../vite-plugin-astro/index.js';
@@ -53,7 +52,7 @@ export function astroContentVirtualModPlugin({
 	fs,
 }: AstroContentVirtualModPluginParams): Plugin {
 	let IS_DEV = false;
-	const IS_SERVER = isServerLikeOutput(settings.config);
+	const IS_SERVER = settings.buildOutput === 'server';
 	const dataStoreFile = new URL(DATA_STORE_FILE, settings.config.cacheDir);
 	return {
 		name: 'astro-content-virtual-mod-plugin',

packages/astro/src/core/build/common.ts

@@ -1,26 +1,27 @@
 import npath from 'node:path';
 import { fileURLToPath, pathToFileURL } from 'node:url';
 import { appendForwardSlash } from '../../core/path.js';
+import type { AstroSettings } from '../../types/astro.js';
 import type { AstroConfig } from '../../types/public/config.js';
 import type { RouteData } from '../../types/public/internal.js';
 
 const STATUS_CODE_PAGES = new Set(['/404', '/500']);
 const FALLBACK_OUT_DIR_NAME = './.astro/';
 
-function getOutRoot(astroConfig: AstroConfig): URL {
-	if (astroConfig.output === 'static') {
-		return new URL('./', astroConfig.outDir);
+function getOutRoot(astroSettings: AstroSettings): URL {
+	if (astroSettings.buildOutput === 'static') {
+		return new URL('./', astroSettings.config.outDir);
 	} else {
-		return new URL('./', astroConfig.build.client);
+		return new URL('./', astroSettings.config.build.client);
 	}
 }
 
 export function getOutFolder(
-	astroConfig: AstroConfig,
+	astroSettings: AstroSettings,
 	pathname: string,
 	routeData: RouteData,
 ): URL {
-	const outRoot = getOutRoot(astroConfig);
+	const outRoot = getOutRoot(astroSettings);
 	const routeType = routeData.type;
 
 	// This is the root folder to write to.
@@ -30,7 +31,7 @@ export function getOutFolder(
 		case 'fallback':
 		case 'page':
 		case 'redirect':
-			switch (astroConfig.build.format) {
+			switch (astroSettings.config.build.format) {
 				case 'directory': {
 					if (STATUS_CODE_PAGES.has(pathname)) {
 						return new URL('.' + appendForwardSlash(npath.dirname(pathname)), outRoot);

packages/astro/src/core/build/generate.ts

@@ -35,7 +35,7 @@ import { callGetStaticPaths } from '../render/route-cache.js';
 import { createRequest } from '../request.js';
 import { matchRoute } from '../routing/match.js';
 import { stringifyParams } from '../routing/params.js';
-import { getOutputFilename, isServerLikeOutput } from '../util.js';
+import { getOutputFilename } from '../util.js';
 import { getOutFile, getOutFolder } from './common.js';
 import { cssOrder, mergeInlineCss } from './internal.js';
 import { BuildPipeline } from './pipeline.js';
@@ -49,12 +49,12 @@ import { getTimeStat, shouldAppendForwardSlash } from './util.js';
 
 export async function generatePages(options: StaticBuildOptions, internals: BuildInternals) {
 	const generatePagesTimer = performance.now();
-	const ssr = isServerLikeOutput(options.settings.config);
+	const ssr = options.settings.buildOutput === 'server';
 	let manifest: SSRManifest;
 	if (ssr) {
 		manifest = await BuildPipeline.retrieveManifest(options, internals);
 	} else {
-		const baseDirectory = getOutputDirectory(options.settings.config);
+		const baseDirectory = getOutputDirectory(options.settings);
 		const renderersEntryUrl = new URL('renderers.mjs', baseDirectory);
 		const renderers = await import(renderersEntryUrl.toString());
 		let middleware: MiddlewareHandler = (_, next) => next();
@@ -138,7 +138,7 @@ export async function generatePages(options: StaticBuildOptions, internals: Buil
 		delete globalThis?.astroAsset?.addStaticImage;
 	}
 
-	await runHookBuildGenerated({ config, logger });
+	await runHookBuildGenerated({ settings: options.settings, logger });
 }
 
 const THRESHOLD_SLOW_RENDER_TIME_MS = 500;
@@ -466,7 +466,7 @@ async function generatePath(
 		body = Buffer.from(await response.arrayBuffer());
 	}
 
-	const outFolder = getOutFolder(config, pathname, route);
+	const outFolder = getOutFolder(pipeline.settings, pathname, route);
 	const outFile = getOutFile(config, outFolder, pathname, route);
 	route.distURL = outFile;
 

packages/astro/src/core/build/index.ts

@@ -19,13 +19,14 @@ import { createNodeLogger } from '../config/logging.js';
 import { createSettings } from '../config/settings.js';
 import { createVite } from '../create-vite.js';
 import { createKey } from '../encryption.js';
+import { AstroError, AstroErrorData } from '../errors/index.js';
 import type { Logger } from '../logger/core.js';
 import { levels, timerMessage } from '../logger/core.js';
 import { apply as applyPolyfill } from '../polyfill.js';
 import { createRouteManifest } from '../routing/index.js';
 import { getServerIslandRouteData } from '../server-islands/endpoint.js';
 import { clearContentLayerCache } from '../sync/index.js';
-import { ensureProcessNodeEnv, isServerLikeOutput } from '../util.js';
+import { ensureProcessNodeEnv } from '../util.js';
 import { collectPagesData } from './page-data.js';
 import { staticBuild, viteBuild } from './static-build.js';
 import type { StaticBuildOptions } from './types.js';
@@ -117,11 +118,16 @@ class AstroBuilder {
 			logger: logger,
 		});
 
-		if (isServerLikeOutput(this.settings.config)) {
-			this.settings = injectImageEndpoint(this.settings, 'build');
+		this.manifest = createRouteManifest({ settings: this.settings }, this.logger);
+
+		if (this.settings.buildOutput === 'server') {
+			injectImageEndpoint(this.settings, this.manifest, 'build');
 		}
 
-		this.manifest = createRouteManifest({ settings: this.settings }, this.logger);
+		// If we're building for the server, we need to ensure that an adapter is installed.
+		if (!this.settings.config.adapter && this.settings.buildOutput === 'server') {
+			throw new AstroError(AstroErrorData.NoAdapterInstalled);
+		}
 
 		const viteConfig = await createVite(
 			{
@@ -211,7 +217,7 @@ class AstroBuilder {
 
 		// You're done! Time to clean up.
 		await runHookBuildDone({
-			config: this.settings.config,
+			settings: this.settings,
 			pages: pageNames,
 			routes: Object.values(allPages)
 				.flat()

packages/astro/src/core/build/page-data.ts

@@ -41,7 +41,7 @@ export function collectPagesData(opts: CollectPagesDataOptions): CollectPagesDat
 				styles: [],
 			};
 
-			if (settings.config.output === 'static') {
+			if (settings.buildOutput === 'static') {
 				const html = `${route.pathname}`.replace(/\/?$/, '/index.html');
 				debug(
 					'build',

packages/astro/src/core/build/pipeline.ts

@@ -16,7 +16,6 @@ import { Pipeline } from '../render/index.js';
 import { createAssetLink, createStylesheetElementSet } from '../render/ssr-element.js';
 import { createDefaultRoutes } from '../routing/default.js';
 import { findRouteToRewrite } from '../routing/rewrite.js';
-import { isServerLikeOutput } from '../util.js';
 import { getOutDirWithinCwd } from './common.js';
 import { type BuildInternals, cssOrder, getPageData, mergeInlineCss } from './internal.js';
 import { ASTRO_PAGE_MODULE_ID, ASTRO_PAGE_RESOLVED_MODULE_ID } from './plugins/plugin-pages.js';
@@ -39,8 +38,7 @@ export class BuildPipeline extends Pipeline {
 	#routesByFilePath: WeakMap<RouteData, string> = new WeakMap<RouteData, string>();
 
 	get outFolder() {
-		const ssr = isServerLikeOutput(this.settings.config);
-		return ssr
+		return this.settings.buildOutput === 'server'
 			? this.settings.config.build.server
 			: getOutDirWithinCwd(this.settings.config.outDir);
 	}
@@ -74,7 +72,7 @@ export class BuildPipeline extends Pipeline {
 			return assetLink;
 		}
 
-		const serverLike = isServerLikeOutput(config);
+		const serverLike = settings.buildOutput === 'server';
 		// We can skip streaming in SSG for performance as writing as strings are faster
 		const streaming = serverLike;
 		super(
@@ -113,8 +111,7 @@ export class BuildPipeline extends Pipeline {
 		staticBuildOptions: StaticBuildOptions,
 		internals: BuildInternals,
 	): Promise<SSRManifest> {
-		const config = staticBuildOptions.settings.config;
-		const baseDirectory = getOutputDirectory(config);
+		const baseDirectory = getOutputDirectory(staticBuildOptions.settings);
 		const manifestEntryUrl = new URL(
 			`${internals.manifestFileName}?time=${Date.now()}`,
 			baseDirectory,