fix: include redirectRoute in environment route filtering

matthewp · matthewp · commit cb95536ef60d · 2026-01-02T11:44:17.000-05:00
diff --git a/packages/astro/src/vite-plugin-pages/pages.ts b/packages/astro/src/vite-plugin-pages/pages.ts
@@ -2,6 +2,7 @@ import type { Plugin as VitePlugin } from 'vite';
 import { DEFAULT_COMPONENTS } from '../core/routing/default.js';
 import { routeIsRedirect } from '../core/routing/index.js';
 import type { RoutesList } from '../types/astro.js';
+import type { RouteData } from '../types/public/internal.js';
 import { VIRTUAL_PAGE_MODULE_ID } from './const.js';
 import { getVirtualModulePageName } from './util.js';
 import { ASTRO_VITE_ENVIRONMENT_NAMES } from '../core/constants.js';
@@ -13,6 +14,23 @@ interface PagesPluginOptions {
 	routesList: RoutesList;
 }
 
+/**
+ * Filters routes for a specific build environment.
+ * Redirects need their target route included so the redirect response can be generated at runtime.
+ */
+function getRoutesForEnvironment(routes: RouteData[], isPrerender: boolean): Set<RouteData> {
+	const result = new Set<RouteData>();
+	for (const route of routes) {
+		if (route.prerender === isPrerender) {
+			result.add(route);
+		}
+		if (route.redirectRoute) {
+			result.add(route.redirectRoute);
+		}
+	}
+	return result;
+}
+
 export function pluginPages({ routesList }: PagesPluginOptions): VitePlugin {
 	return {
 		name: '@astro/plugin-pages',
@@ -50,8 +68,8 @@ export function pluginPages({ routesList }: PagesPluginOptions): VitePlugin {
 				const isPrerender = envName === ASTRO_VITE_ENVIRONMENT_NAMES.prerender;
 				const routes =
 					isSSR || isPrerender
-						? routesList.routes.filter((route) => route.prerender === isPrerender)
-						: routesList.routes;
+						? getRoutesForEnvironment(routesList.routes, isPrerender)
+						: new Set(routesList.routes);
 
 				for (const route of routes) {
 					if (routeIsRedirect(route)) {
