Skip to content

fix(hmr): normalize paths when reloading manifest#15937

Merged
ematipico merged 1 commit intomainfrom
fix/hmr-winzozz
Mar 18, 2026
Merged

fix(hmr): normalize paths when reloading manifest#15937
ematipico merged 1 commit intomainfrom
fix/hmr-winzozz

Conversation

@ematipico
Copy link
Copy Markdown
Member

@ematipico ematipico commented Mar 16, 2026

Changes

Closes #15909

In HMR, when checking if we need to reload the manifest, we now normalise paths.

Using Node.js and vite is safe, we're inside a vite plugin.

Testing

I don't have windows and we skip these tests on windows. Will probably use a preview release.

Docs

N/A

@ematipico ematipico added the pr preview Apply this label to a PR to generate a preview release label Mar 16, 2026
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 16, 2026

🦋 Changeset detected

Latest commit: 941b449

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

@github-actions github-actions bot added the pkg: astro Related to the core `astro` package (scope) label Mar 16, 2026
@ematipico ematipico added pr preview Apply this label to a PR to generate a preview release and removed pr preview Apply this label to a PR to generate a preview release labels Mar 16, 2026
@github-actions github-actions bot removed the pr preview Apply this label to a PR to generate a preview release label Mar 16, 2026
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Mar 16, 2026

npm i https://pkg.pr.new/astro@15937

commit: 941b449

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Mar 16, 2026

Merging this PR will not alter performance

✅ 18 untouched benchmarks


Comparing fix/hmr-winzozz (941b449) with main (325901e)1

Open in CodSpeed

Footnotes

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

Copy link
Copy Markdown
Contributor

@Eveeifyeve Eveeifyeve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't have approve permissions, But I do approve of this.

@ematipico ematipico merged commit d236245 into main Mar 18, 2026
28 checks passed
@ematipico ematipico deleted the fix/hmr-winzozz branch March 18, 2026 11:12
@astrobot-houston astrobot-houston mentioned this pull request Mar 18, 2026
dadezzz pushed a commit to dadezzz/ice-notes that referenced this pull request Mar 22, 2026
This PR contains the following updates:

| Package | Change | [Age](https://docs.renovatebot.com/merge-confidence/) | [Confidence](https://docs.renovatebot.com/merge-confidence/) |
|---|---|---|---|
| [astro](https://astro.build) ([source](https://github.com/withastro/astro/tree/HEAD/packages/astro)) | [`6.0.5` → `6.0.6`](https://renovatebot.com/diffs/npm/astro/6.0.5/6.0.6) | ![age](https://developer.mend.io/api/mc/badges/age/npm/astro/6.0.6?slim=true) | ![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/astro/6.0.5/6.0.6?slim=true) |

---

### Release Notes

<details>
<summary>withastro/astro (astro)</summary>

### [`v6.0.6`](https://github.com/withastro/astro/blob/HEAD/packages/astro/CHANGELOG.md#606)

[Compare Source](https://github.com/withastro/astro/compare/astro@6.0.5...astro@6.0.6)

##### Patch Changes

- [#&#8203;15965](withastro/astro#15965) [`2dca307`](withastro/astro@2dca307) Thanks [@&#8203;matthewp](https://github.com/matthewp)! - Fixes client hydration for components imported through Node.js subpath imports (`package.json#imports`, e.g. `#components/*`), for example when using the Cloudflare adapter in development.

- [#&#8203;15770](withastro/astro#15770) [`6102ca2`](withastro/astro@6102ca2) Thanks [@&#8203;jpc-ae](https://github.com/jpc-ae)! - Updates the `create astro` welcome message to highlight the graceful dev/preview server quit command rather than the kill process shortcut

- [#&#8203;15953](withastro/astro#15953) [`7eddf22`](withastro/astro@7eddf22) Thanks [@&#8203;Desel72](https://github.com/Desel72)! - fix(hmr): eagerly recompile on style-only change to prevent stale slots render

- [#&#8203;15916](withastro/astro#15916) [`5201ed4`](withastro/astro@5201ed4) Thanks [@&#8203;trueberryless](https://github.com/trueberryless)! - Fixes `InferLoaderSchema` type inference for content collections defined with a loader that includes a `schema`

- [#&#8203;15864](withastro/astro#15864) [`d3c7de9`](withastro/astro@d3c7de9) Thanks [@&#8203;florian-lefebvre](https://github.com/florian-lefebvre)! - Removes temporary support for Node >=20.19.1 because Stackblitz now uses Node 22 by default

- [#&#8203;15944](withastro/astro#15944) [`a5e1acd`](withastro/astro@a5e1acd) Thanks [@&#8203;fkatsuhiro](https://github.com/fkatsuhiro)! - Fixes SSR dynamic routes with `.html` extension (e.g. `[slug].html.astro`) not working

- [#&#8203;15937](withastro/astro#15937) [`d236245`](withastro/astro@d236245) Thanks [@&#8203;ematipico](https://github.com/ematipico)! - Fixes an issue where HMR didn't correctly work on Windows when adding/changing/deleting routes in `pages/`.

- [#&#8203;15931](withastro/astro#15931) [`98dfb61`](withastro/astro@98dfb61) Thanks [@&#8203;Strernd](https://github.com/Strernd)! - Fix skew protection query params not being applied to island hydration `component-url` and `renderer-url`, and ensure query params are appended safely for asset URLs with existing search/hash parts.

- Updated dependencies \[]:
  - [@&#8203;astrojs/markdown-remark](https://github.com/astrojs/markdown-remark)@&#8203;7.0.1

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My43Ni4yIiwidXBkYXRlZEluVmVyIjoiNDMuNzYuMiIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==-->

Co-authored-by: Renovate Bot <renovate@zarantonello.dev>
Co-committed-by: Renovate Bot <renovate@zarantonello.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: astro Related to the core `astro` package (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Astro v6 Regression - 404s on pages created while dev server is running

3 participants