process: add process.getBuiltinModule(id) #52762
Conversation
|
Review requested:
|
nodejs-github-bot
added
lib / src
|
This is obviously exactly what I asked for (😄) and gets it "done" the quickest, but I'm curious if there are any other opinions about this in general. Is it possible that |
Not quite sure if I am following this but I think |
|
Yeah, I guess it's no different than detecting calls to Just poking holes in my own idea. |
|
We can also suggest in the documentation that users should only consider using this for code paths that can only be run in Node.js (the code example is basically already suggesting that but we can make it clearer). If they want the code depending on the built-ins to be polyfill-able in the browser, they should still use require or import. |
|
There is a TC39 proposal to get original built-in functions, through a new |
|
Just to prove this out to its full extent (and to procrastinate other work I didn't want to do), I polyfilled this API using |
`process.getBuiltinModule(id)` provides a way to load built-in modules
in a globally available function. ES Modules that need to support
other environments can use it to conditionally load a Node.js built-in
when it is run in Node.js, without having to deal with the resolution
error that can be thrown by `import` in a non-Node.js environment or
having to use dynamic `import()` which either turns the module into an
asynchronous module, or turns a synchronous API into an asynchronous
one.
```mjs
if (globalThis.process.getBuiltinModule) {
// Run in Node.js, use the Node.js fs module.
const fs = globalThis.process.getBuiltinModule('fs');
// If `require()` is needed to load user-modules, use
// createRequire()
const module = globalThis.process.getBuiltinModule('module');
const require = module.createRequire(import.meta.url);
const foo = require('foo');
}
```
If `id` specifies a built-in module available in the current Node.js
process, `process.getBuiltinModule(id)` method returns the
corresponding built-in module. If `id` does not correspond to any
built-in module, `undefined` is returned.
`process.getBuiltinModule(id)` accept built-in module IDs that are
recognized by `module.isBuiltin(id)`. Some built-in modules must be
loaded with the `node:` prefix.
The built-in modules returned by `process.getBuiltinModule(id)` are
always the original modules - that is, it's not affected by
`require.cache`.
joyeecheung
force-pushed
the
process-builtin
branch
from
719213c
to
e83917e
Compare
joyeecheung
changed the title
|
Updated the PR a bit:
|
Now it mismatches Alternatively we can add |
|
Fair enough. I still think we should create an alias on |
process: add process.getBuiltinModule(id)
process.getBuiltinModule(id)provides a way to load built-in modulesin a globally available function. ES Modules that need to support
other environments can use it to conditionally load a Node.js built-in
when it is run in Node.js, without having to deal with the resolution
error that can be thrown by
importin a non-Node.js environment orhaving to use dynamic
import()which either turns the module into anasynchronous module, or turns a synchronous API into an asynchronous
one.
If
idspecifies a built-in module available in the current Node.jsprocess,
process.getBuiltinModule(id)method returns thecorresponding built-in module. If
iddoes not correspond to anybuilt-in module,
undefinedis returned.process.getBuiltinModule(id)accept built-in module IDs that arerecognized by
module.isBuiltin(id). Some built-in modules must beloaded with the
node:prefix.The built-in modules returned by
process.getBuiltinModule(id)arealways the original modules - that is, it's not affected by
require.cache.Fixes: #52599