Results 788 comments of Benjamin Gruenbaum

Maybe just expose the dirent insstead of its name to exclude which would enable: ```js fs.promises.glob("foo/**", { exclude(entry) { return entry.isFile(); }) ```

Can confirm this repros on v20.12.2 but not on v22.0, I recall a fix (by @fhinkel ?) for something in that space a few months ago.

I'm not sure I understand the use case? Shouldn't this be solved from the consumer side that can call `Debugger.resume` (or some such) after attaching with `--inspect-brk`?

> Avoid UTF8 transcoding by directly reading the source code as buffer from disk (this needs to dance with CJS loader monkey patching) FWIW I think loaders/require hooks are probably...

> It looks okay, but I don't want to approve/request changes until a CI is ran. You're welcome to review code changes (as in make recommendations) but note your approval/request-changes...

Maybe we should rename it script runner if this keeps getting happening 😅