thread-stream icon indicating copy to clipboard operation
thread-stream copied to clipboard

Import Errors Hidden by `real-require` Retry

Open hhanesand opened this issue 7 months ago • 1 comments

In the start function, the error.code === undefined condition in the path which leads to real-require can hide errors which are useful to throw. We had an issue in the file which was imported which resulted in a ReferenceError being thrown. This error has no code value, and therefore the start function tries to require the file. Since the file is a es module, it fails with the ERR_REQUIRE_ESM error. The only way to figure out the real error is to insert a console.log or try to load the module directly.

I am happy to submit a PR. Would one which removes this error.code === undefined condition be accepted? Any suggestions to what kinds of tests should be added?

hhanesand avatar Jul 12 '24 12:07 hhanesand