fix: don't mix Vite plugins when spawning temporary Vite server

[userquin]

Changes

• Closes Running Vite server during build breaks plugin assumptions #6364
• Supersedes Do not apply user plugins when spawning temporary Vite server #6366
• Filter plugins using the Astro command
• /cc @bholmesdev
• /cc @natemoo-re

Testing

Added simple test with disparate plugins for static build.

Docs

Bug fix only: maybe we need to add some hints for integrations registering plugins, check for example @vite-pwa/astro integration logic: https://github.com/vite-pwa/astro/blob/main/src/index.ts#L31

[changeset-bot]

🦋 Changeset detected

Latest commit: fdce124

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

[userquin]

We need the command, not the mode.

[userquin]

for preview, integrations should take care, and for vite user plugins we should also filter them before starting Vite preview

[userquin]

don't merge yet, preparing test

[userquin]

We've another problem, when a plugin is included it can be excluded on sync call since we don't have command/action when using apply with function, we need to review it.

You can check this line https://github.com/withastro/astro/blob/main/packages/astro/src/core/build/index.ts#L85, Astro creates vite for build then before run the build Astro creates a new config for sync.

EDIT: seems to be ok now, I'll update the test again, I was using apply function arguments wrong 😬 ...

[userquin]

add else case without plugins?

[natemoo-re]

Great fix and awesome test!

Test failure is unrelated.

[ematipico]

I'm not against the change, but I'd like to see better comments

[bholmesdev]

Added some suggestions too clear up in-line comments. Great fix though, was unaware Vite plugins have an apply utility!