🦋 Changeset detected

Latest commit: 9181f96

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

Should this close vitejs/vite#9378?

No. The current behaviour still makes no sense. Many things could log info/warnings/errors, and not all of them have access to config.logger. Nor does it make sense that we have to store deferred_warning until configResolved just so that we're able to use config.logger.

However, this new information is useful:

It seems Vite already does this as long as config.logger.warn/warnOnce/error is used.

All we need to do, it seems, is add a config.logger.warn('') inside configResolved, and we will effectively trick Vite into not nuking logs. Updating the PR.

the changes lgtm