process.env and Bun.env have different results

[miruchigawa]

What version of Bun is running?

1.1.36+ededc168c

What platform is your computer?

Linux 6.1.0-27-cloud-amd64 x86_64 unknown

What steps can reproduce the bug?

This bug was discovered when my RestAPI app displayed the development page. After investigating, we found that process.env.NODE_ENV always returned "development," regardless of what we tried to set—it always produced the same result.

What is the expected behavior?

Bun.env.NODE_ENV // production
process.env.NODE_ENV // production

What do you see instead?

Bun.env.NODE_ENV // production
process.env.NODE_ENV // development

Additional information

No response

[patrickdemers6]

Can you share a minimal reproduction? I'm not seeing this behavior.

// env.js
console.log("bun", Bun.env.NODE_ENV);
console.log("process", process.env.NODE_ENV);

$ NODE_ENV=production bun env.js
bun production
process production

$ NODE_ENV=development bun env.js
bun development
process development

$ bun -v
1.1.36

[miruchigawa]

Maybe this issue comes during build?

***@***:~/Workspaces-Yuna/elysia-waifu$ bun run build
$ bun build src/index.ts --target bun --outdir ./build

  ./index.js 12750.55 KB

[630ms] bundle 811 modules
***@***:~/Workspaces-Yuna/elysia-waifu$ bun start
$ NODE_ENV=production bun build/index.js
Bun production
process development

import { $ } from 'bun';

export async function get_all_versions() {
    const v_bun = Bun.version;
    console.log("Bun", Bun.env.NODE_ENV)
    console.log("process", process.env.NODE_ENV)
    const v_rep = Bun.env.NODE_ENV !== 'production'
        ? 'Devel'
        : await $`git rev-parse --short HEAD`
            .then(r => r.text())
            .catch(_ => 'Unknown');

    return { v_bun, v_rep };
}

[RiskyMH]

Maybe this issue comes during build?

I am pretty sure that's the issue. Bun build inlines process.env variables and as NODE_ENV defaults to development the issue is observed.

The solution should be for bun to not inline NODE_ENV (esp if not even defined), or even better don't inline process.env (#8125).

[miruchigawa]

Okay, this case will be closed.