On Mon, Mar 21, 2022 at 8:15 PM Yadd <y...@debian.org> wrote:
>
>
> Le 21 mars 2022 19:04:24 GMT+01:00, "Jérémy Lal" <kapo...@melix.org> a
> écrit :
> >On Mon, Mar 21, 2022 at 6:57 PM Yadd <y...@debian.org> wrote:
> >
> >>
> >>
> >> On 21/03/2022 18:15, Yadd wrote:
> >> > Package: node-builtins
> >> > Version: 4.0.0-1
> >> > Severity: important
> >> >
> >> > Hi,
> >> >
> >> > node-builtins returns a static modules list which is up-to date for
> >> > Node.js < 11. It should be patched to use node-builtin-modules result
> >> > for node.js >= 12
> >> >
> >>
> >> Comparison:
> >>
> >> $ node
> >> > require("builtins")()
> >> [
> >> 'assert', 'buffer', 'child_process',
> >> 'cluster', 'console', 'constants',
> >> 'crypto', 'dgram', 'dns',
> >> 'domain', 'events', 'fs',
> >> 'http', 'https', 'module',
> >> 'net', 'os', 'path',
> >> 'punycode', 'querystring', 'readline',
> >> 'repl', 'stream', 'string_decoder',
> >> 'sys', 'timers', 'tls',
> >> 'tty', 'url', 'util',
> >> 'vm', 'zlib', 'v8',
> >> 'process', 'inspector', 'async_hooks',
> >> 'http2', 'perf_hooks', 'trace_events',
> >> 'worker_threads'
> >> ]
> >> > require("builtin-modules")
> >> [
> >> 'assert', 'async_hooks', 'buffer',
> >> 'child_process', 'cluster', 'console',
> >> 'constants', 'crypto', 'dgram',
> >> 'dns', 'domain', 'events',
> >> 'fs', 'http', 'http2',
> >> 'https', 'inspector', 'module',
> >> 'net', 'os', 'path',
> >> 'perf_hooks', 'process', 'punycode',
> >> 'querystring', 'readline', 'repl',
> >> 'stream', 'string_decoder', 'timers',
> >> 'tls', 'trace_events', 'tty',
> >> 'url', 'util', 'v8',
> >> 'vm', 'wasi', 'worker_threads',
> >> 'zlib'
> >> ]
> >>
> >
> >Good catch,
> >do you want me to do it ?
>
> >Jérémy
>
> Yes, thanks !
Suggested approach would break the module API:
require('builtins')({ version: '6.0.0' })
require('builtins')({ version: '6.0.0', experimental: true })
it's supposed to be queryable.
I'd rather fix the list.
Jérémy
