Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bun runtime fails react-mail@>2.0.0 - defaults to it without warning when node isn't installed #1429

Open
codeScourge opened this issue Apr 25, 2024 · 1 comment
Labels
Package: react-email This is the CLI we generally use as just `email` on the temrinal. Type: Bug Confirmed bug

Comments

@codeScourge
Copy link

Describe the Bug

I recently had an issue with React-Mail closing without any error when running with bun (installed on WSL), see reproduction here: https://github.com/codeScourge/react-email. I was able to workaround by downgrading to [email protected]

The error seemed to be fixed by installing Node in WSL, as proposed by gabrielmfern:

I think I've figured it out. When running on WSL you might have not installed Node at all, in that case, it seems like it fails the same you mentioned. It also fails the same way when running with bun --bun dev while having Node installed. So it seems like bun falls back to running with its runtime if Node isn't installed, so basically it runs bun --bun dev. Haven't checked on their code and I couldn't find anything related to this on their docs.

Which package is affected (leave empty if unsure)

No response

Link to the code that reproduces this issue

https://github.com/codeScourge/react-email

To Reproduce

https://github.com/codeScourge/react-email, see ReadME

Expected Behavior

it should start a dev server

What's your node version? (if relevant)

No response

@codeScourge codeScourge added the Type: Bug Confirmed bug label Apr 25, 2024
@gabrielmfern gabrielmfern added the Package: react-email This is the CLI we generally use as just `email` on the temrinal. label Apr 25, 2024
@atomicpages
Copy link

Bun doesn't contain all the dependencies required to run next. For example: bun -b email dev
and bun -b email build relies on TextDecoderStream which isn't supported in bun yet

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Package: react-email This is the CLI we generally use as just `email` on the temrinal. Type: Bug Confirmed bug
Projects
None yet
Development

No branches or pull requests

3 participants