Fixes a bug in how injectRoute parses route patterns on Windows (#3763

) * always remove a leading slash in the `injectRoute` pattern * chore: add changeset
withastro · Jun 29, 2022 · 54cd6b8 · 54cd6b8
1 parent 40be96d
commit 54cd6b8
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/.changeset/seven-rocks-sort.md b/.changeset/seven-rocks-sort.md
@@ -0,0 +1,5 @@
+---
+'astro': patch
+---
+
+Fixes how `injectRoute` parses route patterns on Windows
diff --git a/packages/astro/src/core/routing/manifest/create.ts b/packages/astro/src/core/routing/manifest/create.ts
@@ -7,6 +7,7 @@ import path from 'path';
 import slash from 'slash';
 import { fileURLToPath } from 'url';
 import { warn } from '../../logger/core.js';
+import { removeLeadingForwardSlash } from '../../path.js';
 import { resolvePages } from '../../util.js';
 import { getRouteGenerator } from './generator.js';
 const require = createRequire(import.meta.url);
@@ -294,7 +295,7 @@ export function createRouteManifest(
 		const isDynamic = (str: string) => str?.[0] === '[';
 		const normalize = (str: string) => str?.substring(1, str?.length - 1);
 
-		const segments = name
+		const segments = removeLeadingForwardSlash(name)
 			.split(path.sep)
 			.filter(Boolean)
 			.map((s: string) => {