delta/node-new.git/lib/internal/source_map/source_map_cache.js, branch main

lib: add trailing commas to more internal files

2023-02-26T10:34:02+00:00


PR-URL: https://github.com/nodejs/node/pull/46811
Reviewed-By: Moshe Atlow 
Reviewed-By: Debadree Chatterjee 
Reviewed-By: James M Snell 
Reviewed-By: Trivikram Kamat

lib: remove unnecessary ObjectGetValueSafe

2023-01-30T10:54:58+00:00

PR-URL: https://github.com/nodejs/node/pull/46335
Reviewed-By: Luigi Pinca 
Reviewed-By: Colin Ihrig 
Reviewed-By: Antoine du Hamel 
Reviewed-By: Ben Noordhuis 
Reviewed-By: James M Snell

lib: cache parsed source maps to reduce memory footprint

2023-01-22T11:25:19+00:00

This also improves performance to map the stack trace when the
`Error.stack` is accessed.

PR-URL: https://github.com/nodejs/node/pull/46225
Reviewed-By: Yagiz Nizipli 
Reviewed-By: Antoine du Hamel 
Reviewed-By: James M Snell 
Reviewed-By: Ben Noordhuis

tools: add `prefer-proto` rule

2023-01-10T05:38:36+00:00

fixup: add support for `Object.create(null)`

fixup: extend to any 1-argument Object.create call

fixup: add tests
PR-URL: https://github.com/nodejs/node/pull/46083
Reviewed-By: Colin Ihrig 
Reviewed-By: Geoffrey Booth 
Reviewed-By: Yagiz Nizipli 
Reviewed-By: Mohammed Keyvanzadeh 
Reviewed-By: Darshan Sen 
Reviewed-By: Michaël Zasso 
Reviewed-By: Jacob Smith 
Reviewed-By: Antoine du Hamel

lib: lazy-load deps in source_map_cache.js

2022-12-16T11:44:03+00:00

So that the file can be snapshotted.

PR-URL: https://github.com/nodejs/node/pull/45849
Reviewed-By: Geoffrey Booth 
Reviewed-By: Chengzhong Wu

modules: move modules/cjs/helpers.js to modules/helpers.js

2022-12-16T11:43:56+00:00

The helpers are actually shared by the two loaders, so move them
under modules/ directly.

PR-URL: https://github.com/nodejs/node/pull/45849
Reviewed-By: Geoffrey Booth 
Reviewed-By: Chengzhong Wu

bootstrap: lazy load non-essential modules

2022-12-09T22:37:35+00:00

It turns out that even with startup snapshots, there is a non-trivial
overhead for loading internal modules. This patch makes the loading
of the non-essential modules lazy again.

Caveat: we have to make some of the globals lazily-loaded too,
so the WPT runner is updated to test what the state of the global
scope is after the globals are accessed (and replaced with the
loaded value).

PR-URL: https://github.com/nodejs/node/pull/45659
Reviewed-By: Matteo Collina 
Reviewed-By: Yagiz Nizipli 
Reviewed-By: Daeyeon Jeong 
Reviewed-By: Jacob Smith 
Reviewed-By: James M Snell 
Reviewed-By: Rafael Gonzaga 
Reviewed-By: Antoine du Hamel 
Reviewed-By: Minwoo Jung 
Reviewed-By: Tierney Cyren

src,lib: retrieve parsed source map url from v8

2022-10-23T14:37:21+00:00

V8 already parses the source map magic comments. Currently, only scripts
and functions expose the parsed source map URLs. It is unnecessary to
parse the source map magic comments again when the parsed information is
available.

PR-URL: https://github.com/nodejs/node/pull/44798
Reviewed-By: Joyee Cheung 
Reviewed-By: James M Snell 
Reviewed-By: Yagiz Nizipli 
Reviewed-By: Jan Krems

lib: don't match `sourceMappingURL` in strings

2022-09-25T16:11:18+00:00

Prior to this change `sourceMappingURL` in string where being matched
by the RegExp which caused sourcemaps not be loaded when using the
`--enable-source-maps` flag. This commit changes the RegExp to match
the last occurrence.

Fixes: https://github.com/nodejs/node/issues/44654
PR-URL: https://github.com/nodejs/node/pull/44658
Reviewed-By: Chengzhong Wu 
Reviewed-By: Antoine du Hamel

lib: codify findSourceMap return value when not found

2022-08-27T23:13:53+00:00

Return `undefined` when no source map is found for the given filename on
`findSourceMap`.

PR-URL: https://github.com/nodejs/node/pull/44397
Fixes: https://github.com/nodejs/node/issues/44391
Reviewed-By: Luigi Pinca 
Reviewed-By: Antoine du Hamel