✅ Deploy Preview for fastidious-cascaron-4ded94 canceled.

@hi-ogawa what do you think? We don't want Vite to bundle it because it adds an import from util as you pointed out in one of your comments. If we don't bundle patched loupe, then this is the only way I found 🤷🏻

Oh, I wasn't aware that node builtins were in optimizeDeps.exclude. So, removing that would be necessary for people using node builtin polyfill explicitly, but what was the reason it has been added previously?

From this comment #4892 (comment), I thought we could simply add vitest > @vitest/utils > loupe to optimizeDeps.include. But, after adding this, I'm seeing Vite's warning locally, so this approach might still have other issues.

Module "util" has been externalized for browser compatibility. Cannot access "util.inspect" in client code. See https://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.

We don't want Vite to bundle it because it adds an import from util as you pointed out in one of your comments.

What do you mean by it adds util?

So, removing that would be necessary for people using node builtin polyfill explicitly, but what was the reason it has been added previously?

We probably forgot to remove it when we provided polyfills ourselves.

What do you mean by it adds util?

It transforms require('util') into import from 'util'. Instead, it should keep it as is.