Exclude ?inline and ?raw from critical styles in dev

[dj-rabel]

• css files with query ?url, ?inline or ?raw shouldn't be injected on SSR bei vite spec
• a combination of these query-params is possible. This is checked as well

Closes: #9844

• Docs (not needed?)
• Tests

Testing Strategy:

• I added a test for the new method, statically checking some possible css urls
• I did not know how to create a test for the whole thing, so I ran a build an copied the file "styles.js" into my regular project and ran the server. To show that, I adjusted the stackblitz from the original Issue (dev server unexpectedly loads CSS when using ?inline imports #9844)
  -> Here it is: https://stackblitz.com/edit/vitejs-vite-qjxcev
  -> And here executing dev without the override as a negative test https://stackblitz.com/edit/vitejs-vite-qjxcev?terminal=dev

[remix-cla-bot]

Hi @dj-rabel,

Welcome, and thank you for contributing to Remix!

Before we consider your pull request, we ask that you sign our Contributor License Agreement (CLA). We require this only once.

You may review the CLA and sign it by adding your name to contributors.yml.

Once the CLA is signed, the CLA Signed label will be added to the pull request.

If you have already signed the CLA and received this response in error, or if you have any questions, please contact us at [email protected].

Thanks!

- The Remix team

[changeset-bot]

🦋 Changeset detected

Latest commit: 7e7c153

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

This PR includes changesets to release 16 packages

Name	Type
@remix-run/dev	Patch
create-remix	Patch
remix	Patch
@remix-run/architect	Patch
@remix-run/cloudflare	Patch
@remix-run/cloudflare-pages	Patch
@remix-run/cloudflare-workers	Patch
@remix-run/css-bundle	Patch
@remix-run/deno	Patch
@remix-run/eslint-config	Patch
@remix-run/express	Patch
@remix-run/node	Patch
@remix-run/react	Patch
@remix-run/serve	Patch
@remix-run/server-runtime	Patch
@remix-run/testing	Patch

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

[remix-cla-bot]

Thank you for signing the Contributor License Agreement. Let's get this merged! 🥳

[markdalgleish]

Thanks for the PR! This is a really great catch.

I've pushed some refactors to make the code a little easier to follow by avoiding a regular expression, and expanded the tests a bit.

[github-actions]

🤖 Hello there,

We just published version 2.12.1-pre.0 which includes this pull request. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

[github-actions]

🤖 Hello there,

We just published version 2.12.1 which includes this pull request. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!