Use imported Cloudflare handler in env API

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

@@ -1,3 +1,4 @@
+export type { RoutesList } from '../../types/astro.js';
 export { App } from './app.js';
 export { BaseApp, type RenderErrorOptions, type RenderOptions } from './base.js';
 export { deserializeManifest, fromRoutingStrategy, toRoutingStrategy } from './common.js';

packages/astro/src/vite-plugin-app/pipeline.ts

@@ -62,8 +62,7 @@ export class AstroServerPipeline extends Pipeline {
 			logger,
 			manifest,
 			settings,
-		}: Pick<AstroServerPipeline, 'loader' | 'logger' | 'manifest' | 'settings'>
-
+		}: Pick<AstroServerPipeline, 'loader' | 'logger' | 'manifest' | 'settings'>,
 	) {
 		const pipeline = new AstroServerPipeline(loader, logger, manifest, settings);
 		pipeline.routesList = manifestData;

packages/integrations/cloudflare/src/entrypoints/image-service.ts

@@ -1,3 +1,4 @@
+/// <reference types="vite/client" />
 import { joinPaths } from '@astrojs/internal-helpers/path';
 import type { ExternalImageService } from 'astro';
 import { baseService } from 'astro/assets';
@@ -25,7 +26,6 @@ const service: ExternalImageService = {
 		}
 
 		const imageEndpoint = joinPaths(
-			// @ts-expect-error Can't recognise import.meta.env
 			import.meta.env.BASE_URL,
 			'/cdn-cgi/image',
 			resizingParams.join(','),

packages/integrations/cloudflare/src/entrypoints/server.ts

@@ -1,24 +1,25 @@
-import type { ExecutionContext, ExportedHandlerFetchHandler } from '@cloudflare/workers-types';
-import type { SSRManifest } from 'astro';
+/// <reference types="vite/client" />
 
-import { App } from 'astro/app';
-import { handle } from '../utils/handler.js';
+import type { ExportedHandler } from '@cloudflare/workers-types';
+import type { RouteInfo, SSRManifest } from 'astro';
+import { App, type BaseApp, createConsoleLogger, DevApp, type RoutesList } from 'astro/app';
+import { type Env, handle } from '../utils/handler.js';
 
-type Env = {
-	[key: string]: unknown;
-	ASSETS: { fetch: (req: Request | string) => Promise<Response> };
-};
+export function createExports(manifest: SSRManifest, routes: RouteInfo[]) {
+	let app: BaseApp;
 
-export function createExports(manifest: SSRManifest) {
-	const app = new App(manifest);
-
-	const fetch = async (
-		request: Parameters<ExportedHandlerFetchHandler>[0],
-		env: Env,
-		context: ExecutionContext,
-	) => {
-		return await handle(manifest, app, request, env, context);
+	if (import.meta.env.DEV) {
+		const routesList: RoutesList = { routes: routes.map((r: RouteInfo) => r.routeData) };
+		const logger = createConsoleLogger('debug');
+		app = new DevApp(manifest, true, logger, routesList);
+	} else {
+		app = new App(manifest);
+	}
+	return {
+		default: {
+			async fetch(request, env, ctx) {
+				return await handle(manifest, app, request, env, ctx);
+			},
+		} satisfies ExportedHandler<Env>,
 	};
-
-	return { default: { fetch } };
 }

packages/integrations/cloudflare/src/utils/handler.ts

@@ -1,18 +1,19 @@
 // @ts-expect-error - It is safe to expect the error here.
 import { env as globalEnv } from 'cloudflare:workers';
 import type {
+	Response as CfResponse,
 	CacheStorage as CloudflareCacheStorage,
 	ExecutionContext,
 	ExportedHandlerFetchHandler,
 } from '@cloudflare/workers-types';
 import type { SSRManifest } from 'astro';
-import type { App } from 'astro/app';
+import type { BaseApp } from 'astro/app';
 import { setGetEnv } from 'astro/env/setup';
 import { createGetEnv } from '../utils/env.js';
 
-type Env = {
+export type Env = {
 	[key: string]: unknown;
-	ASSETS: { fetch: (req: Request | string) => Promise<Response> };
+	ASSETS: { fetch: (req: Request | string) => Promise<CfResponse> };
 };
 
 setGetEnv(createGetEnv(globalEnv as Env));
@@ -36,11 +37,11 @@ declare global {
 
 export async function handle(
 	manifest: SSRManifest,
-	app: App,
+	app: BaseApp,
 	request: Parameters<ExportedHandlerFetchHandler>[0],
 	env: Env,
 	context: ExecutionContext,
-) {
+): Promise<CfResponse> {
 	const { pathname } = new URL(request.url);
 	const bindingName = globalThis.__ASTRO_SESSION_BINDING_NAME;
 	// Assigning the KV binding to globalThis allows unstorage to access it for session storage.
@@ -90,7 +91,7 @@ export async function handle(
 			routeData,
 			locals,
 			prerenderedErrorPageFetch: async (url) => {
-				return env.ASSETS.fetch(url.replace(/\.html$/, ''));
+				return env.ASSETS.fetch(url.replace(/\.html$/, '')) as unknown as Response;
 			},
 		},
 	);
@@ -101,5 +102,5 @@ export async function handle(
 		}
 	}
 
-	return response;
+	return response as unknown as CfResponse;
 }

packages/integrations/cloudflare/test/fixtures/vite-plugin/astro.config.mjs

@@ -5,8 +5,5 @@ import { defineConfig } from 'astro/config';
 
 export default defineConfig({
 	adapter: cloudflare({
-		workerEntryPoint: {
-			path: 'src/worker.ts',
-		}
 	}),
 });

packages/integrations/cloudflare/test/fixtures/vite-plugin/src/dev.ts

@@ -1,5 +1,5 @@
 import { manifest } from "astro:serialized-manifest";
 import { routes } from "astro:routes"
-import { createExports } from "./worker";
+import { createExports } from "@astrojs/cloudflare/entrypoints/server.js";
 
 export default createExports(manifest, routes).default