[Pkg-javascript-devel] Bug#1008061: Bug#1008061: Bug#1008061: Bug#1008061: node-builtins: Returns a wrong list of builtins module

[Jérémy Lal]

On Mon, Mar 21, 2022 at 8:15 PM Yadd <yadd at debian.org> wrote:

>
>
> Le 21 mars 2022 19:04:24 GMT+01:00, "Jérémy Lal" <kapouer at melix.org> a
> écrit :
> >On Mon, Mar 21, 2022 at 6:57 PM Yadd <yadd at 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://alioth-lists.debian.net/pipermail/pkg-javascript-devel/attachments/20220321/fe483751/attachment.htm>