delta/node-new.git/lib/assert.js, branch main

assert: deprecate callTracker

2023-04-29T06:57:08+00:00

PR-URL: https://github.com/nodejs/node/pull/47740
Fixes: https://github.com/nodejs/node/issues/47492
Reviewed-By: Erick Wendel 
Reviewed-By: Benjamin Gruenbaum 
Reviewed-By: James M Snell

src: bootstrap prepare stack trace callback in shadow realm

2023-04-04T01:13:37+00:00

Bootstrap per-realm callbacks like `prepare_stack_trace_callback` in
the ShadowRealm. This enables stack trace decoration in the ShadowRealm.

PR-URL: https://github.com/nodejs/node/pull/47107
Reviewed-By: Yagiz Nizipli 
Reviewed-By: Matteo Collina 
Reviewed-By: Joyee Cheung 
Reviewed-By: Colin Ihrig 
Reviewed-By: James M Snell

lib: add trailing commas to all public core modules

2023-02-28T11:10:24+00:00

PR-URL: https://github.com/nodejs/node/pull/46848
Reviewed-By: Debadree Chatterjee 
Reviewed-By: Moshe Atlow 
Reviewed-By: Anatoli Papirovski 
Reviewed-By: Benjamin Gruenbaum 
Reviewed-By: Rafael Gonzaga 
Reviewed-By: Jiawen Geng 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Yongsheng Zhang

assert: fix exception message for assert(0) on try catch block

2023-02-26T18:18:50+00:00

Fixes: https://github.com/nodejs/node/issues/30872
PR-URL: https://github.com/nodejs/node/pull/46760
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Kohei Ueno

assert: remove deprecated getFunction() usage

2023-02-20T10:34:38+00:00

The method is meant to be removed by the V8 team. It is not a
critical functionality that is removed, therefore no alternative
is checked for either.

Refs: https://bugs.chromium.org/p/v8/issues/detail?id=9421

Signed-off-by: Ruben Bridgewater 
PR-URL: https://github.com/nodejs/node/pull/46661
Reviewed-By: Joyee Cheung 
Reviewed-By: Michaël Zasso 
Reviewed-By: Mohammed Keyvanzadeh 
Reviewed-By: Ujjwal Sharma 
Reviewed-By: Colin Ihrig 
Reviewed-By: Luigi Pinca 
Reviewed-By: James M Snell

lib: enforce use of trailing commas for functions

2023-02-14T17:45:16+00:00


PR-URL: https://github.com/nodejs/node/pull/46629
Reviewed-By: Jacob Smith 
Reviewed-By: Geoffrey Booth 
Reviewed-By: Chengzhong Wu 
Reviewed-By: Mohammed Keyvanzadeh

assert: remove `assert.snapshot`

2023-01-07T20:24:24+00:00

PR-URL: https://github.com/nodejs/node/pull/46112
Reviewed-By: Geoffrey Booth 
Reviewed-By: Antoine du Hamel 
Reviewed-By: Luigi Pinca 
Reviewed-By: Tobias Nießen

assert: add assert.Snapshot

2022-08-11T13:07:52+00:00

PR-URL: https://github.com/nodejs/node/pull/44095
Reviewed-By: Benjamin Gruenbaum

src: disambiguate terms used to refer to builtins and addons

2022-08-08T17:36:49+00:00

The term "native module" dates back to some of the oldest code
in the code base. Within the context of Node.js core it usually
refers to modules that are native to Node.js (e.g. fs, http),
but it can cause confusion for people who don't work on this
part of the code base, as "native module" can also refer to
native addons - which is even the case in some of the API
docs and error messages.

This patch tries to make the usage of these terms more consistent.
Now within the context of Node.js core:

- JavaScript scripts that are built-in to Node.js are now referred
  to as "built-in(s)". If they are available as modules,
  they can also be referred to as "built-in module(s)".
- Dynamically-linked shared objects that are loaded into
  the Node.js processes are referred to as "addons".

We will try to avoid using the term "native modules" because it could
be ambiguous.

Changes in this patch:

File names:
- node_native_module.h -> node_builtins.h,
- node_native_module.cc -> node_builtins.cc

C++ binding names:
- `native_module` -> `builtins`

`node::Environment`:
- `native_modules_without_cache` -> `builtins_without_cache`
- `native_modules_with_cache` -> `builtins_with_cache`
- `native_modules_in_snapshot` -> `builtins_in_cache`
- `native_module_require` -> `builtin_module_require`

`node::EnvSerializeInfo`:
- `native_modules` -> `builtins

`node::native_module::NativeModuleLoader`:
- `native_module` namespace -> `builtins` namespace
- `NativeModuleLoader` -> `BuiltinLoader`
- `NativeModuleRecordMap` -> `BuiltinSourceMap`
- `NativeModuleCacheMap` -> `BuiltinCodeCacheMap`
- `ModuleIds` -> `BuiltinIds`
- `ModuleCategories` -> `BuiltinCategories`
- `LoadBuiltinModuleSource` -> `LoadBuiltinSource`

`loader.js`:
- `NativeModule` -> `BuiltinModule` (the `NativeModule` name used in
  `process.moduleLoadList` is kept for compatibility)

And other clarifications in the documentation and comments.

PR-URL: https://github.com/nodejs/node/pull/44135
Fixes: https://github.com/nodejs/node/issues/44036
Reviewed-By: Jacob Smith 
Reviewed-By: Matteo Collina 
Reviewed-By: Michael Dawson 
Reviewed-By: Richard Lau 
Reviewed-By: Jiawen Geng 
Reviewed-By: Chengzhong Wu 
Reviewed-By: Mohammed Keyvanzadeh 
Reviewed-By: Tobias Nießen 
Reviewed-By: Jan Krems

lib: refactor to avoid unsafe regex primordials

2022-06-27T15:16:06+00:00


PR-URL: https://github.com/nodejs/node/pull/43475
Reviewed-By: Geoffrey Booth 
Reviewed-By: Stephen Belanger