fix(#9216): fix style HMR in certain cases

[natemoo-re]

Changes

• Closes PLT-1261
• Adjusts our style HMR logic so that style only changes will only treat the effected style modules as HMR candidates
• This fixes an edge case where updates inside of Astro files referenced by the main page file would cause a full reload because pages/index.astro would remain in the module graph.
• This should close CSS changes do not always correctly update using HMR #9216, which is a report about this specific edge case. As reported, if the style inside of src/layouts/Layout.astro was updated, the whole index page would reload. With this fix, updating the style inside of src/layouts/Layout.astro does not reload the index page but swaps the new styles in seamlessly.

Testing

Tested manually, our existing HMR tests aren't capable of catching this problem

Docs

Bug fix only

[changeset-bot]

🦋 Changeset detected

Latest commit: d6e27e1

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

[natemoo-re]

Would love a @bluwy gut check on this one! The goal is to filter the HMR modules down to just the style that actually changed. We previously only discarded a module if it was known to be irrelevant but this swaps the logic so that we discard everything we don't recognize as relevant.

[bluwy]

Makes sense to me 👍