Fix dev handling

Author: ascorbic

packages/astro/src/vite-plugin-astro-server/base.ts

@@ -30,6 +30,7 @@ export function baseMiddleware(
 
 		if (pathname.startsWith(devRoot)) {
 			req.url = url.replace(devRoot, devRootReplacement);
+			req.url ||= '/';
 			return next();
 		}
 

packages/astro/test/units/routing/manifest.test.js

@@ -37,7 +37,7 @@ function assertRouteRelations(routes, relations) {
 }
 
 describe('routing - createRoutesList', () => {
-	it('using trailingSlash: "never" does not match the index route when it contains a trailing slash', async () => {
+	it('using trailingSlash: "never" matches the index route when it contains a trailing slash', async () => {
 		const fixture = await createFixture({
 			'/src/pages/index.astro': `<h1>test</h1>`,
 		});
@@ -51,8 +51,9 @@ describe('routing - createRoutesList', () => {
 			settings,
 		});
 		const [{ pattern }] = manifest.routes;
-		assert.equal(pattern.test(''), true);
-		assert.equal(pattern.test('/'), false);
+		// Requesting `/` *should* match the index route, because that's what it sees when requesting `/search`.
+		// If they've requested `/search/` this will have been caught by trailingSlashMiddleware
+		assert.equal(pattern.test('/'), true);
 	});
 
 	it('endpoint routes are sorted before page routes', async () => {