You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I was building an app where I imported @toruslabs/base-controllers. The @toruslabs/base-controllers have readable-stream as a dev-dependency as listed in the package.json file.
My node version: v18.14.1
Problem
When I ran my code, it threw the below error. Clearly, the error was because it couldn't find any process/ module as there is no such thing.
The error was fixed when I changed all the process/ to process. As I was checking through the readable-stream repository, I found multiple typos in the process module in the repo. This might be breaking a lot of apps and I think this can be easily fixed with a single PR.
Solution
The typo needs to be fixed in these files 👇
I would love to work on the issue, let me know if I missed something.
Thank You
The text was updated successfully, but these errors were encountered:
It's an old trick to tell Node.js (and bundlers that follow the same require() algorithm) to load an npm package instead of a core module.
Might be an issue with Vite specifically. We don't directly test Vite (in dev mode) but we do test Rollup here (which is what Vite uses in production mode):
I was building an app where I imported @toruslabs/base-controllers. The
@toruslabs/base-controllers
havereadable-stream
as a dev-dependency as listed in thepackage.json
file.My node version: v18.14.1
Problem
When I ran my code, it threw the below error. Clearly, the error was because it couldn't find any
process/
module as there is no such thing.The error was fixed when I changed all the
process/
toprocess
. As I was checking through thereadable-stream
repository, I found multiple typos in theprocess
module in the repo. This might be breaking a lot of apps and I think this can be easily fixed with a single PR.Solution
The typo needs to be fixed in these files 👇
I would love to work on the issue, let me know if I missed something.
Thank You
The text was updated successfully, but these errors were encountered: