fix(astro): HMR not work in windows

[Dunqing]

fix: #2987 (comment)

[netlify]

✅ Deploy Preview for unocss ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	4f03ce5
🔍 Latest deploy log	https://app.netlify.com/sites/unocss/deploys/64fd813312fefd0009c62db4
😎 Deploy Preview	https://deploy-preview-3100--unocss.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[userquin]

@Dunqing testing PR on my Windows laptop...

EDIT: since UnoCSS dependencies are linked we need to include vite.server.fs.strict = false in the example Astro config file: I'm getting 403 when resolving astro hmr JS module:

[userquin]

@Dunqing still getting page reload:

[Dunqing]

@Dunqing still getting page reload:

Yes, I am not sure page reload In astro is expected. Can you use vite plugin to test it?

[Dunqing]

@Dunqing testing PR on my Windows laptop...

EDIT: since UnoCSS dependencies are linked we need to include vite.server.fs.strict = false in the example Astro config file: I'm getting 403 when resolving astro hmr JS module:

I tested it in this example, but I am not receiving any error. And looks like the error has nothing to do with the astro integration

[userquin]

I have this on the Astro config file:

[userquin]

@Dunqing adding the vite-plugin-inspect:

This is weird in SSR, there are a few /__uno.css virtual modules, one resolving to /__uno.css and another to the full path:

Then in the client, we have also 2 __uno.css (is there a missmatch? the HMR context is for /__uno.css and the virtual id for update, check also that id is resolved twice or 3 times):

[Dunqing]

@Dunqing adding the vite-plugin-inspect:

This is weird in SSR, there are a few /__uno.css virtual modules, one resolving to /__uno.css and another to the full path:

This full path is to align data-astro-dev-id and data-vite-dev-id.

You can look here

unocss/packages/astro/src/index.ts

Lines 34 to 38 in 92e2f8e

	if (RESOLVED_ID_RE.test(id)) {
	// https://github.com/withastro/astro/blob/087270c61fd5c91ddd37db5c8fd93a8a0ef41f94/packages/astro/src/core/util.ts#L91-L93
	// Align data-astro-dev-id with data-vite-dev-id to fix https://github.com/unocss/unocss/issues/2513
	return this.resolve(join(root, id), importer, { skipSelf: true })
	}

This is strange, and we may need to return the full /__uno.css path in the vite plugin to solve the problem of resolving it twice!

[Dunqing]

If this PR does solve the problem of Windows hmr not working, I think it would be a good idea to merge this PR first to avoid Windows users not being able to use it for a long time. What do you think? @userquin

[Dunqing]

Unocss doesn't work if it put it in the astro integration it only works when a put it as a vite plugin

Did you test it at this PR? The latest version does not include this PR.