Geoffrey Booth

We shouldn't cache what we don't use. For the purposes of this API the cache can be checked first but if it doesn't have what we need then we read...

cc @nodejs/modules

> I reviewed the src/crypto changes, not the other parts. So between your review and @Qard's, does that cover most or all of it? Along those lines, how much of...

This isn't really a bug. We changed some undocumented internals that `esm` relied on, knowing it would break. Anyone can fork `esm` to update it accordingly; reverting the change would...

> Any update on this? Pull requests are welcome!

> `import { register } from 'node:module'`, and that is _not_ currently synchronous. It is synchronous: https://github.com/nodejs/node/pull/46826

> Do we have to await register() before Module.runMain();? Does register() give us a promise At least according to the docs in the PR, `register` is sync. It does not...

> So it is possible today for require() hooks to call another thread for module resolution and compilation. This would be useful to abstract into its own library for other...

> register(a); await import(b) which loaders handle the import of b? `a` should handle the import of `b`.

FYI there are further changes in the works. The `import { register } from 'module'` mentioned above has landed on `main` in Node, though we’re holding it from being released...