fix(hmr): entry mod's importers contains css mod invalidate hmr

[y1d7ng]

Description

fix #3913 #3716

HMR is not working as change entry javascript file or index.html

Additional context

entry file main.js:

import './index.css';

index.css:

@tailwind base;

hmr is not working as change themain.js, the client receive the error update message - hmr update index.css

PostCSS plugin Tailwind JIT register any file as a dependency to a CSS file. Although main.js is an entry file, main.js module's importers includes index.css module. Through the propagateUpdate function, get the wrong result that index.css needs to be updated.

vite/packages/vite/src/node/server/hmr.ts

Line 212 in f623ba3

function propagateUpdate(

---

What is the purpose of this pull request?

• Bug fix
• New Feature
• Documentation update
• Other

Before submitting the PR, please make sure you do the following

• Read the Contributing Guidelines.
• Read the Pull Request Guidelines and follow the Commit Convention.
• Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
• Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
• Ideally, include relevant tests that fail without this PR but pass with it.

[yyx990803]

Thanks @OneNail - the fix is correct, but the way it is implemented can be a bit difficult to read for future maintainers. I refactored it to be more explicit what case it is handling.

[fabio-ivona]

@yyx990803

If I've well understood the related issues, it seems that this fix aims to full reload the page when a tailwindcss app.css change is triggered by a change in an html file (or similar)

this works well quite all the times, but our plugin for livewire hot reload is broken by this (because it triggers a full reload while the client part of the plugin is trying to refresh the single livewire component)

is there any way to make this feature opt-in? or will you accept a PR to allowe to disable this in config

there are cases where a full reload is not needed (if not detrimental) when app.css is updated by tailwind

for reference:

https://github.com/def-studio/vite-livewire-plugin/discussions/12
laravel/vite-plugin#111

[Niputi]

@fabio-ivona please open a new issue

[fabio-ivona]

@Niputi thanks for your repy!

done in #9512