chore: update adapters to better support astro v5

.changeset/small-bikes-decide.md

@@ -0,0 +1,9 @@
+---
+'@astrojs/cloudflare': patch
+'@astrojs/test-utils': patch
+'@astrojs/netlify': patch
+'@astrojs/vercel': patch
+'@astrojs/node': patch
+---
+
+Improves Astro 5 support

packages/cloudflare/package.json

@@ -31,7 +31,6 @@
     "@astrojs/internal-helpers": "0.4.1",
     "@astrojs/underscore-redirects": "^0.4.0-alpha.0",
     "@cloudflare/workers-types": "^4.20241112.0",
-    "@inox-tools/astro-when": "^1.0.1",
     "esbuild": "^0.24.0",
     "estree-walker": "^3.0.3",
     "magic-string": "^0.30.14",
@@ -45,7 +44,7 @@
   },
   "devDependencies": {
     "@astrojs/test-utils": "workspace:*",
-    "astro": "^5.0.0",
+    "astro": "^5.0.6",
     "astro-scripts": "workspace:*",
     "cheerio": "1.0.0",
     "execa": "^8.0.1",

packages/cloudflare/src/entrypoints/middleware.ts

@@ -1,23 +1,12 @@
-import { When, whenAmI } from '@it-astro:when';
 import type { MiddlewareHandler } from 'astro';
 
-const middlewares: Record<When, MiddlewareHandler> = {
-	[When.Client]: () => {
-		throw new Error('Client should not run a middleware!');
-	},
-	[When.DevServer]: (_, next) => next(),
-	[When.Server]: (_, next) => next(),
-	[When.Prerender]: (ctx, next) => {
+export const onRequest: MiddlewareHandler = (context, next) => {
+	if (context.isPrerendered) {
 		// @ts-expect-error
-		if (ctx.locals.runtime === undefined) {
-			// @ts-expect-error
-			ctx.locals.runtime = {
-				env: process.env,
-			};
-		}
-		return next();
-	},
-	[When.StaticBuild]: (_, next) => next(),
-};
+		context.locals.runtime ??= {
+			env: process.env,
+		};
+	}
 
-export const onRequest = middlewares[whenAmI];
+	return next();
+};

packages/cloudflare/src/index.ts

@@ -1,4 +1,10 @@
-import type { AstroConfig, AstroIntegration, IntegrationRouteData } from 'astro';
+import type {
+	AstroConfig,
+	AstroIntegration,
+	HookParameters,
+	IntegrationResolvedRoute,
+	IntegrationRouteData,
+} from 'astro';
 import type { PluginOption } from 'vite';
 
 import { createReadStream } from 'node:fs';
@@ -10,7 +16,6 @@ import {
 	removeLeadingForwardSlash,
 } from '@astrojs/internal-helpers/path';
 import { createRedirectsFromAstroRoutes } from '@astrojs/underscore-redirects';
-import astroWhen from '@inox-tools/astro-when';
 import { AstroError } from 'astro/errors';
 import { defaultClientConditions } from 'vite';
 import { type GetPlatformProxyOptions, getPlatformProxy } from 'wrangler';
@@ -86,13 +91,37 @@ function setProcessEnv(config: AstroConfig, env: Record<string, unknown>) {
 	}
 }
 
+function resolvedRouteToRouteData(
+	assets: HookParameters<'astro:build:done'>['assets'],
+	route: IntegrationResolvedRoute
+): IntegrationRouteData {
+	return {
+		pattern: route.patternRegex,
+		component: route.entrypoint,
+		prerender: route.isPrerendered,
+		route: route.pattern,
+		generate: route.generate,
+		params: route.params,
+		segments: route.segments,
+		type: route.type,
+		pathname: route.pathname,
+		redirect: route.redirect,
+		distURL: assets.get(route.pattern),
+		redirectRoute: route.redirectRoute
+			? resolvedRouteToRouteData(assets, route.redirectRoute)
+			: undefined,
+	};
+}
+
 export default function createIntegration(args?: Options): AstroIntegration {
 	let _config: AstroConfig;
 
 	const cloudflareModulePlugin: PluginOption & CloudflareModulePluginExtra = cloudflareModuleLoader(
 		args?.cloudflareModules ?? true
 	);
 
+	let _routes: IntegrationResolvedRoute[];
+
 	return {
 		name: '@astrojs/cloudflare',
 		hooks: {
@@ -128,7 +157,6 @@ export default function createIntegration(args?: Options): AstroIntegration {
 							},
 						],
 					},
-					integrations: [astroWhen()],
 					image: setImageConfig(args?.imageService ?? 'compile', config.image, command, logger),
 				});
 				if (args?.platformProxy?.configPath) {
@@ -143,6 +171,9 @@ export default function createIntegration(args?: Options): AstroIntegration {
 					order: 'pre',
 				});
 			},
+			'astro:routes:resolved': ({ routes }) => {
+				_routes = routes;
+			},
 			'astro:config:done': ({ setAdapter, config, buildOutput, logger }) => {
 				if (buildOutput === 'static') {
 					logger.warn(
@@ -165,7 +196,11 @@ export default function createIntegration(args?: Options): AstroIntegration {
 						hybridOutput: 'stable',
 						staticOutput: 'unsupported',
 						i18nDomains: 'experimental',
-						sharpImageService: 'limited',
+						sharpImageService: {
+							support: 'limited',
+							message:
+								'Cloudflare does not support sharp. You can use the `compile` image service to compile images at build time. It will not work for any on-demand rendered images.',
+						},
 						envGetSecret: 'experimental',
 					},
 				});
@@ -258,7 +293,7 @@ export default function createIntegration(args?: Options): AstroIntegration {
 					};
 				}
 			},
-			'astro:build:done': async ({ pages, routes, dir, logger }) => {
+			'astro:build:done': async ({ pages, dir, logger, assets }) => {
 				await cloudflareModulePlugin.afterBuildCompleted(_config);
 				const PLATFORM_FILES = ['_headers', '_redirects', '_routes.json'];
 				if (_config.base !== '/') {
@@ -283,7 +318,7 @@ export default function createIntegration(args?: Options): AstroIntegration {
 					redirectsExists = false;
 				}
 
-				const redirects: IntegrationRouteData['segments'][] = [];
+				const redirects: IntegrationResolvedRoute['segments'][] = [];
 				if (redirectsExists) {
 					const rl = createInterface({
 						input: createReadStream(new URL('./_redirects', _config.outDir)),
@@ -322,7 +357,7 @@ export default function createIntegration(args?: Options): AstroIntegration {
 					await createRoutesFile(
 						_config,
 						logger,
-						routes,
+						_routes,
 						pages,
 						redirects,
 						args?.routes?.extend?.include,
@@ -331,8 +366,10 @@ export default function createIntegration(args?: Options): AstroIntegration {
 				}
 
 				const redirectRoutes: [IntegrationRouteData, string][] = [];
-				for (const route of routes) {
-					if (route.type === 'redirect') redirectRoutes.push([route, '']);
+				for (const route of _routes) {
+					// TODO: Replace workaround after upstream @astrojs/underscore-redirects is changed, to support new IntegrationResolvedRoute type
+					if (route.type === 'redirect')
+						redirectRoutes.push([resolvedRouteToRouteData(assets, route), '']);
 				}
 
 				const trueRedirects = createRedirectsFromAstroRoutes({

packages/cloudflare/src/utils/generate-routes-json.ts

@@ -1,4 +1,9 @@
-import type { AstroConfig, AstroIntegrationLogger, IntegrationRouteData, RoutePart } from 'astro';
+import type {
+	AstroConfig,
+	AstroIntegrationLogger,
+	IntegrationResolvedRoute,
+	RoutePart,
+} from 'astro';
 
 import { existsSync } from 'node:fs';
 import { writeFile } from 'node:fs/promises';
@@ -63,7 +68,7 @@ async function writeRoutesFileToOutDir(
 	}
 }
 
-function segmentsToCfSyntax(segments: IntegrationRouteData['segments'], _config: AstroConfig) {
+function segmentsToCfSyntax(segments: IntegrationResolvedRoute['segments'], _config: AstroConfig) {
 	const pathSegments = [];
 	if (removeLeadingForwardSlash(removeTrailingForwardSlash(_config.base)).length > 0) {
 		pathSegments.push(removeLeadingForwardSlash(removeTrailingForwardSlash(_config.base)));
@@ -163,11 +168,11 @@ class PathTrie {
 export async function createRoutesFile(
 	_config: AstroConfig,
 	logger: AstroIntegrationLogger,
-	routes: IntegrationRouteData[],
+	routes: IntegrationResolvedRoute[],
 	pages: {
 		pathname: string;
 	}[],
-	redirects: IntegrationRouteData['segments'][],
+	redirects: IntegrationResolvedRoute['segments'][],
 	includeExtends:
 		| {
 				pattern: string;
@@ -223,17 +228,17 @@ export async function createRoutesFile(
 	let hasPrerendered404 = false;
 	for (const route of routes) {
 		const convertedPath = segmentsToCfSyntax(route.segments, _config);
-		if (route.pathname === '/404' && route.prerender === true) hasPrerendered404 = true;
+		if (route.pathname === '/404' && route.isPrerendered === true) hasPrerendered404 = true;
 
 		// eslint-disable-next-line @typescript-eslint/switch-exhaustiveness-check
 		switch (route.type) {
 			case 'page':
-				if (route.prerender === false) includePaths.push(convertedPath);
+				if (route.isPrerendered === false) includePaths.push(convertedPath);
 
 				break;
 
 			case 'endpoint':
-				if (route.prerender === false) includePaths.push(convertedPath);
+				if (route.isPrerendered === false) includePaths.push(convertedPath);
 				else excludePaths.push(convertedPath);
 
 				break;

packages/cloudflare/test/fixtures/astro-dev-platform/package.json

@@ -4,7 +4,7 @@
   "private": true,
   "dependencies": {
     "@astrojs/cloudflare": "workspace:*",
-    "astro": "^5.0.0"
+    "astro": "^5.0.6"
   },
   "devDependencies": {
     "wrangler": "^3.84.0"

packages/cloudflare/test/fixtures/astro-env/package.json

@@ -4,7 +4,7 @@
   "private": true,
   "dependencies": {
     "@astrojs/cloudflare": "workspace:*",
-    "astro": "^5.0.0"
+    "astro": "^5.0.6"
   },
   "devDependencies": {
     "wrangler": "^3.84.0"

packages/cloudflare/test/fixtures/compile-image-service/package.json

@@ -4,6 +4,6 @@
   "private": true,
   "dependencies": {
     "@astrojs/cloudflare": "workspace:*",
-    "astro": "^5.0.0"
+    "astro": "^5.0.6"
   }
 }

packages/cloudflare/test/fixtures/external-image-service/package.json

@@ -4,6 +4,6 @@
   "private": true,
   "dependencies": {
     "@astrojs/cloudflare": "workspace:*",
-    "astro": "^5.0.0"
+    "astro": "^5.0.6"
   }
 }

packages/cloudflare/test/fixtures/module-loader/package.json

@@ -4,6 +4,6 @@
   "private": true,
   "dependencies": {
     "@astrojs/cloudflare": "workspace:*",
-    "astro": "^5.0.0"
+    "astro": "^5.0.6"
   }
 }

packages/cloudflare/test/fixtures/no-output/package.json

@@ -4,6 +4,6 @@
   "private": true,
   "dependencies": {
     "@astrojs/cloudflare": "workspace:*",
-    "astro": "^5.0.0"
+    "astro": "^5.0.6"
   }
 }

packages/cloudflare/test/fixtures/routes-json/package.json

@@ -4,6 +4,6 @@
   "private": true,
   "dependencies": {
     "@astrojs/cloudflare": "workspace:*",
-    "astro": "^5.0.0"
+    "astro": "^5.0.6"
   }
 }

packages/cloudflare/test/fixtures/with-solid-js/package.json

@@ -5,7 +5,7 @@
   "dependencies": {
     "@astrojs/cloudflare": "workspace:*",
     "@astrojs/solid-js": "^5.0.0",
-    "astro": "^5.0.0",
+    "astro": "^5.0.6",
     "solid-js": "^1.9.3"
   }
 }

packages/cloudflare/test/fixtures/with-svelte/package.json

@@ -5,7 +5,7 @@
   "dependencies": {
     "@astrojs/cloudflare": "workspace:*",
     "@astrojs/svelte": "^7.0.1",
-    "astro": "^5.0.0",
+    "astro": "^5.0.6",
     "svelte": "^5.6.0"
   }
 }

packages/cloudflare/test/fixtures/with-vue/package.json

@@ -5,7 +5,7 @@
   "dependencies": {
     "@astrojs/cloudflare": "workspace:*",
     "@astrojs/vue": "^5.0.1",
-    "astro": "^5.0.0",
+    "astro": "^5.0.6",
     "vue": "^3.5.13"
   }
 }

packages/cloudflare/test/fixtures/wrangler-preview-platform/package.json

@@ -4,6 +4,6 @@
   "private": true,
   "dependencies": {
     "@astrojs/cloudflare": "workspace:*",
-    "astro": "^5.0.0"
+    "astro": "^5.0.6"
   }
 }