diff --git a/.changeset/lemon-insects-smash.md b/.changeset/lemon-insects-smash.md new file mode 100644 index 000000000..8e4dd6d7d --- /dev/null +++ b/.changeset/lemon-insects-smash.md @@ -0,0 +1,5 @@ +--- +'astro': patch +--- + +Adding support for config.build.format to the dev server diff --git a/packages/astro/src/vite-plugin-astro-server/index.ts b/packages/astro/src/vite-plugin-astro-server/index.ts index 00461c9bb..b71878079 100644 --- a/packages/astro/src/vite-plugin-astro-server/index.ts +++ b/packages/astro/src/vite-plugin-astro-server/index.ts @@ -191,7 +191,12 @@ async function handleRequest( const devRoot = site ? site.pathname : '/'; const origin = `${viteServer.config.server.https ? 'https' : 'http'}://${req.headers.host}`; const buildingToSSR = isBuildingToSSR(config); - const url = new URL(origin + req.url); + // When file-based build format is used, pages will be built to `/blog.html` + // rather than `/blog/index.html`. The dev server should handle this as well + // to match production deployments. + const url = config.build.format === 'file' + ? new URL(origin + req.url?.replace(/(index)?\.html$/, '')) + : new URL(origin + req.url); const pathname = decodeURI(url.pathname); const rootRelativeUrl = pathname.substring(devRoot.length - 1); if (!buildingToSSR) { diff --git a/packages/astro/test/dev-routing.test.js b/packages/astro/test/dev-routing.test.js index 439ee6988..de765a183 100644 --- a/packages/astro/test/dev-routing.test.js +++ b/packages/astro/test/dev-routing.test.js @@ -247,4 +247,52 @@ describe('Development Routing', () => { expect(body.title).to.equal('data [slug]'); }); }); + + describe('file format routing', () => { + /** @type {import('./test-utils').Fixture} */ + let fixture; + /** @type {import('./test-utils').DevServer} */ + let devServer; + + before(async () => { + fixture = await loadFixture({ + build: { + format: 'file', + }, + root: './fixtures/without-site-config/', + site: 'http://example.com/', + }); + devServer = await fixture.startDevServer(); + }); + + it('200 when loading /index.html', async () => { + const response = await fixture.fetch('/index.html'); + expect(response.status).to.equal(200); + }); + + it('200 when loading /', async () => { + const response = await fixture.fetch('/'); + expect(response.status).to.equal(200); + }); + + it('200 when loading /another.html', async () => { + const response = await fixture.fetch('/another.html'); + expect(response.status).to.equal(200); + }); + + it('200 when loading /another', async () => { + const response = await fixture.fetch('/another'); + expect(response.status).to.equal(200); + }); + + it('200 when loading /1.html', async () => { + const response = await fixture.fetch('/1.html'); + expect(response.status).to.equal(200); + }); + + it('200 when loading /1', async () => { + const response = await fixture.fetch('/1'); + expect(response.status).to.equal(200); + }); + }); });