fix: race condition in resolvedPathname

[ascorbic]

Changes

Fixes a race condition in the dev server that could cause TypeError: Missing parameter errors when handling concurrent requests to dynamic routes.

resolvedPathname (the pathname after .html stripping in route matching) was stored as shared instance state on AstroServerApp and DevApp. Concurrent requests could overwrite each other's value, causing a mismatch between the matched route and the pathname used to extract params.

This PR threads resolvedPathname through RenderOptions as a local value per-request instead:

• Adds resolvedPathname to the RenderOptions interface
• AstroServerApp.handleRequest passes it from devMatch to render()
• BaseApp.render() uses it from options when provided, or captures it from devMatch when doing its own route matching
• Removes the createRenderContext overrides that injected the pathname from instance state

Testing

Added a unit test that fires concurrent requests to dynamic routes and verifies params are resolved independently.

Docs

N/A - bug fix only

[changeset-bot]

🦋 Changeset detected

Latest commit: 9f9edc5

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[codspeed-hq]

Merging this PR will not alter performance

✅ 9 untouched benchmarks

---

_{Comparing resolve-path-race (063bbdc) with main (d789452)¹}

Footnotes

1. No successful run was found on main (00e95c4) during the generation of this report, so d789452 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩