vite:define can't process std-env

[danielroe]

Describe the bug

std-env contains some environment-agnostic utilities (see source).

The current vite:define plugin can't process this (see compiled library), and there seems to be no way to opt-out of processing a file with the define plugin.

Linked upstream issue: nuxt/nuxt#24793

Reproduction

https://stackblitz.com/edit/vitejs-vite-ndari4

Steps to reproduce

npm run dev (which will run the build command)

System Info

StackBlitz

Used Package Manager

npm

Logs

No response

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
• Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
• The provided reproduction is a minimal reproducible example of the bug.

[stackblitz]

Start a new pull request in StackBlitz Codeflow.

• vitejs-vite-ndari4 (Open reproduction)

[hi-ogawa]

It looks like esbuild's bug. I was testing https://unpkg.com/[email protected]/dist/index.mjs on esbuild playground with define picked from:

vite/packages/vite/src/node/plugins/define.ts

Lines 19 to 26 in 5684fcd

	Object.assign(processEnv, {
	'process.env': `{}`,
	'global.process.env': `{}`,
	'globalThis.process.env': `{}`,
	'process.env.NODE_ENV': JSON.stringify(nodeEnv),
	'global.process.env.NODE_ENV': JSON.stringify(nodeEnv),
	'globalThis.process.env.NODE_ENV': JSON.stringify(nodeEnv),
	})

And I found that applying { define: { "globalThis.process.env": "{}" } } to the source code globalThis.process?.env[o] triggers a panic.

Here are playground links:

• std-env (I ran prettier for easier debugging)
• minimal repro

[hi-ogawa]

I just thought of the quick workaround to replace offending globalThis.process?.env[o] via plugin. This also verifies that the underlying issue is only this exact expression:

https://stackblitz.com/edit/vitejs-vite-k6fmaw?file=vite.config.ts

import { defineConfig } from 'vite';

export default defineConfig({
  plugins: [
    {
      name: 'workaround-esbuild-define-panic',
      enforce: 'pre',
      transform(code) {
        return code.replace(
          'globalThis.process?.env[o]',
          '(globalThis.process?.env)[o]'
        );
      },
    },
  ],
});

[hi-ogawa]

This should be fixed in esbuild v0.19.10 https://github.com/evanw/esbuild/releases/tag/v0.19.10

I confirmed vite build succeeds on stackblitz via package.json "overrides": { "esbuild": "0.19.10" }

https://stackblitz.com/edit/vitejs-vite-21ssdn?file=package.json

[sapphi-red]

@hi-ogawa Thanks for investigating and the fix 💚