Why in node-runer js there is no catch in promise?

[aspirisen]

Here https://github.com/bridgetownrb/node-runner/blob/master/lib/node_runner.js#L3

We have wrapper try-catch block.
But is there any specific reason why Promise.resolve() doesn't have catch clause?

[jaredcwhite]

@aspirisen Honestly not sure! What should happen in a catch clause here? 🤔

[aspirisen]

@jaredcwhite I think it should have same stdout log

process.stdout.write(JSON.stringify(['err', '' + err, err.stack]))

Because func call is not awaited then try catch block doesn't handle the error

With await

With catch clause

[jaredcwhite]

@aspirisen OK, that makes sense to me. Could you make that fix in a PR and I'll merge? TY!