Fix injectRoute

.changeset/stupid-pants-press.md

@@ -0,0 +1,5 @@
+---
+'astro': patch
+---
+
+Ensure that injected routes from `node_modules` are properly detected

packages/astro/src/@types/astro.ts

@@ -1316,16 +1316,20 @@ export interface AstroUserConfig {
  */
 export type InjectedScriptStage = 'before-hydration' | 'head-inline' | 'page' | 'page-ssr';
 
-/**
- * Resolved Astro Config
- * Config with user settings along with all defaults filled in.
- */
-
 export interface InjectedRoute {
 	pattern: string;
 	entryPoint: string;
 	prerender?: boolean;
 }
+
+export interface ResolvedInjectedRoute extends InjectedRoute {
+	resolvedEntryPoint?: URL;
+}
+
+/**
+ * Resolved Astro Config
+ * Config with user settings along with all defaults filled in.
+ */
 export interface AstroConfig extends z.output<typeof AstroConfigSchema> {
 	// Public:
 	// This is a more detailed type than zod validation gives us.
@@ -1414,6 +1418,7 @@ export interface AstroSettings {
 	config: AstroConfig;
 	adapter: AstroAdapter | undefined;
 	injectedRoutes: InjectedRoute[];
+	resolvedInjectedRoutes: ResolvedInjectedRoute[];
 	pageExtensions: string[];
 	contentEntryTypes: ContentEntryType[];
 	dataEntryTypes: DataEntryType[];

packages/astro/src/core/config/settings.ts

@@ -21,6 +21,7 @@ export function createBaseSettings(config: AstroConfig): AstroSettings {
 
 		adapter: undefined,
 		injectedRoutes: [],
+		resolvedInjectedRoutes: [],
 		pageExtensions: ['.astro', '.html', ...SUPPORTED_MARKDOWN_FILE_EXTENSIONS],
 		contentEntryTypes: [markdownContentEntryType],
 		dataEntryTypes: [

packages/astro/src/core/util.ts

@@ -113,8 +113,9 @@ function isInPagesDir(file: URL, config: AstroConfig): boolean {
 }
 
 function isInjectedRoute(file: URL, settings: AstroSettings) {
-	for (const route of settings.injectedRoutes) {
-		if (file.toString().endsWith(route.entryPoint)) return true;
+	let fileURL = file.toString();
+	for (const route of settings.resolvedInjectedRoutes) {
+		if (route.resolvedEntryPoint && fileURL === route.resolvedEntryPoint.toString()) return true;
 	}
 	return false;
 }

packages/astro/src/vite-plugin-integrations-container/index.ts

@@ -1,4 +1,4 @@
-import type { Plugin as VitePlugin } from 'vite';
+import { normalizePath, type Plugin as VitePlugin } from 'vite';
 import type { AstroSettings } from '../@types/astro.js';
 import type { LogOptions } from '../core/logger/core.js';
 import { runHookServerSetup } from '../integrations/index.js';
@@ -11,10 +11,28 @@ export default function astroIntegrationsContainerPlugin({
 	settings: AstroSettings;
 	logging: LogOptions;
 }): VitePlugin {
+	let hasResolvedInjectedRoutes = false;
 	return {
 		name: 'astro:integration-container',
 		configureServer(server) {
 			runHookServerSetup({ config: settings.config, server, logging });
 		},
+		// Run `load` hook once just to get access to `PluginContext.resolve`
+		// which allows us to resolve each injectedRoute entryPoint to a real file URL on disk
+		async load() {
+			if (hasResolvedInjectedRoutes) return;
+			// Ensure the injectedRoutes are all resolved to their final paths through Rollup
+			settings.resolvedInjectedRoutes = await Promise.all(settings.injectedRoutes.map(async injectedRoute => {
+				let resolvedEntryPoint: URL | undefined;
+				try {
+					const resolvedId = await this.resolve(injectedRoute.entryPoint).then(res => res?.id ?? injectedRoute.entryPoint)
+					const filename = normalizePath(resolvedId);
+					resolvedEntryPoint = new URL(`file://${filename}`);
+				} catch {}
+				return { ...injectedRoute, resolvedEntryPoint }
+			}))
+			hasResolvedInjectedRoutes = true;
+			return;
+		},
 	};
 }