fix(hmr): dev mode reduce unnecessary restart

[btea]

Description

I don't think it's necessary to restart the server to modify the production environment related env in development mode.

Additional context

---

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 PR Title 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.

[stackblitz]

Run & review this pull request in StackBlitz Codeflow.

[ArnaudBarre]

Sorry only checked now.
Could you expose something like getEnvFilesForMode in env.ts that expose this list?
https://github.com/vitejs/vite/blob/main/packages/vite/src/node/env.ts#L21-L26
Then changeEnvShouldRestart can be replaced by getEnvFilesForMode(mode).includes(filename)

[patak-dev]

This is an interesting optimization. My only doubt is if users may expect other .env files they access manually in the config to trigger a server restart. Maybe this is how it should work though and the user needs to manually force a restart for these files.

[ArnaudBarre]

Personally I think this is better to show to the user that .env.prod changes doesn't have impact on the running dev server

[patak-dev]

Personally I think this is better to show to the user that .env.prod changes doesn't have impact on the running dev server

Sounds good to me. I let others also do a review and merge this one