fix(css): remove default import in ssr dev

[bluwy]

Description

Found from withastro/astro#11811

In SSR dev, we still incorrectly support default imports of a CSS file (without queries). This feels like a bug fix, but technically I guess it affects frameworks like vitest that executes in SSR and serve mode?

[stackblitz]

Run & review this pull request in StackBlitz Codeflow.

[hi-ogawa]

I think Vinxi and Remix are relying on default export of ssrLoadModule without ?inline https://github.com/remix-run/remix/blob/c5292edb5997b0818105944b33da7c4cd4567511/packages/remix-dev/vite/styles.ts#L77-L79. Can you check ecosystem ci?

If this was meant to be deprecated already, then maybe it's okay to change though (also it might be just a one liner fix on their end?).

[bluwy]

Yeah it was deprecated in the previous major, and removed in this major (https://vitejs.dev/guide/migration.html#removed-deprecated-apis). Frameworks need to use ?inline to work, which I thought most have needed to migrated, but maybe we didn't check thoroughly 🤔 I'll run ecosystem-ci to check

[bluwy]

/ecosystem-ci run

[vite-ecosystem-ci]

📝 Ran ecosystem CI on b1a7eab: Open

suite	result	latest scheduled
astro	❌ failure	✅ success
nuxt	✅ success	❌ failure
qwik	❌ failure	❌ failure
remix	❌ failure	✅ success
storybook	✅ success	undefined undefined
sveltekit	❌ failure	✅ success
vitest	✅ success	❌ failure

✅ analogjs, histoire, ladle, laravel, marko, previewjs, quasar, rakkas, unocss, vike, vite-plugin-pwa, vite-plugin-react, vite-plugin-react-swc, vite-plugin-svelte, vite-plugin-vue, vite-setup-catalogue, vitepress

[bluwy]

Looks like astro, remix, and sveltekit all fails and likely related to the CSS handling, so this would be breaking then unfortunately. Let's move this to the next major then.