`parallel/test-tls-invoke-queued` previously used the internal
`_write()` API to hook into the i...

- Communicate the current async stack length through a
typed array field rather than a native ...

Compute the floating point number in JavaScript to avoid having to call
out to the C++ runtime. ...

I noticed that ocsp_request is not being reset in
ClientHelloParser::Reset. I've not been able t...

Assert the server name directly in the `SNICallback`,
since `common.mustCall()` already guarante...

`options.server` only needs to be set when its
contents are actually being inspected.

PR-URL: h...

Refactor and simplify the perf_hooks native internals.

PR-URL: https://github.com/nodejs/node/p...

The `rawPacket` is the current buffer that just parsed. Adding this
buffer to the error object o...

Error occurs while dealing with Tar archives

PR-URL: https://github.com/nodejs/node/pull/17663
...

PR-URL: https://github.com/nodejs/node/pull/17656
Reviewed-By: Colin Ihrig <[email protected]>
R...

PR-URL: https://github.com/nodejs/node/pull/17852
Refs: https://github.com/nodejs/node/pull/1766...

PR-URL: https://github.com/nodejs/node/pull/17852
Refs: https://github.com/nodejs/node/pull/1766...

arrayBufferViews is used by only one function so scope it to that
function (in the common module...

Make setImmediate() immune to `process` global tampering by removing
the dependency on the `proc...

The standard does not throw and has no stack trace.
See https://console.spec.whatwg.org/#assert
...

PR-URL: https://github.com/nodejs/node/pull/17722
Reviewed-By: Jon Moss <[email protected]>
Rev...

PR-URL: https://github.com/nodejs/node/pull/17722
Reviewed-By: Jon Moss <[email protected]>
Rev...

This will also use a proper indentation as a couple of entries
had a extra indentation of two sp...

PR-URL: https://github.com/nodejs/node/pull/17805
Reviewed-By: Michaël Zasso <targos@protonmail....

PR-URL: https://github.com/nodejs/node/pull/17715
Refs: https://github.com/nodejs/node/issues/16...

`common.fail()` no longer exists as its functionality is now in
`assert.fail()`. Replace only tw...

assert.strictEqual accepts 3 arguments, the last of which allows for
user-specified error messag...

Use an interval to keep the event loop open so the test does not exit
before receiving all signa...

SetupHooks is only available via `process.binding('async_wrap')`, so
there's no reason it should...

test-benchmark-fs uses common.tmpDir without first insuring it exists by
calling common.refreshT...

Use the same approach as a previous PR to include the offending line in
the output and underline...

In three fs benchmarks, a temp file is created in the source tree. For
tests, allow the location...

Do not share unnecessary information about nextTick state
between JS & C++, instead only track w...

PR-URL: https://github.com/nodejs/node/pull/17847
Reviewed-By: Joyee Cheung <[email protected]...

There two similar error codes in lib: "ERR_VALUE_OUT_OF_RANGE"
and "ERR_OUT_OF_RANGE". This chan...

PR-URL: https://github.com/nodejs/node/pull/17751
Reviewed-By: Colin Ihrig <[email protected]>
R...

Is safer to use a `process.binding(config)` defined boolean, than to
regex on `process.execArgv`...

The deprecated module was removed via
84a23391f60c0cbde8f78fd805b6fdef6861285f.

PR-URL: https:/...

Throw ERR_SOCKET_CLOSED and ERR_SERVER_NOT_RUNNING
instead of the old-style errors in net.js.

P...

Throw ERR_INVALID_ARG_TYPE when validating arguments passed to
WriteStream.prototype._write.

Re...

Commits are often landed despite failing on one or more CI platforms.
Having a CI status indicat...

This migrates the old style error in _tls_wrap.js to
the new style error ERR_TLS_RENEGOTIATION_D...

PR-URL: https://github.com/nodejs/node/pull/17818
Reviewed-By: Rich Trott <[email protected]>
Rev...

The `auxData` field is not exposed to JavaScript, as DevTools uses it
for its `isDefault` parame...

Use temp directory for open with `O_DSYNC` (which indicates a write may
occur) rather than `fixt...

The change to word boundary was breaking many doc pages. This reverts
the word boundary back to ...

PR-URL: https://github.com/nodejs/node/pull/17829
Reviewed-By: James M Snell <[email protected]>...

Original commit message:
[interpreter] Remove TryInstallOptimizedCode

Removes the interrupt...

PR-URL: https://github.com/nodejs/node/pull/17763
Refs: https://github.com/nodejs/node/issues/17...

PR-URL: https://github.com/nodejs/node/pull/17667
Reviewed-By: Joyee Cheung <[email protected]...

test-fs-fsync makes a copy of a fixture file, but then doesn't do
anything with it and instead o...

PR-URL: https://github.com/nodejs/node/pull/17716
Reviewed-By: James M Snell <[email protected]>...

PR-URL: https://github.com/nodejs/node/pull/17796
Reviewed-By: Anatoli Papirovski <apapirovski@m...

Refs: https://github.com/v8/v8/compare/6.3.292.46...6.3.292.48
PR-URL: https://github.com/nodejs...

PR-URL: https://github.com/nodejs/node/pull/17566
Refs: https://github.com/nodejs/node/issues/17...

Using authentication tags of invalid length does not conform to NIST
standards.

PR-URL: https:/...

PR-URL: https://github.com/nodejs/node/pull/17712
Reviewed-By: Colin Ihrig <[email protected]>
R...

Old style errors are being migrated to internal/errors.js, however, due
to depreciation of _tls_...

Find an invalid file descriptor rather than assuming 42 will be invalid.

PR-URL: https://github...

The test was writing to both REPL input and output but only checking
output. Sending output to b...

`Finish<WriteWrap>` already does the same thing and is
called immediately afterwards anyway.

PR...

PR-URL: https://github.com/nodejs/node/pull/17781
Ref: https://github.com/nodejs/node/pull/17781...

common.firstInvalidFD() is used in only one test. Move it out of the
common module and into the ...

common.getTTYfd() is used in one test only. Move it's definition to that
test and out of the com...

common.projectDir is used in one test, so it's not so common. Remove
from common module to the o...

Because of the potential performance bottlenecks that may be introduced
by `util.inspect()` and ...

PR-URL: https://github.com/nodejs/node/pull/17688
Reviewed-By: Luigi Pinca <[email protected]...

This improves the api descriptions for fs.chown, fs.chmod, and fs.mkdir
along with their *Sync c...

- Use arrow functions, `common.mustCall()`
- Remove redundant `console.log()`s/turn them into as...

PR-URL: https://github.com/nodejs/node/pull/17728
Reviewed-By: Luigi Pinca <[email protected]...

Original commit message:

[inspector] added Runtime.globalLexicalScopeNames method

The ...

Use the V8 inspector protocol, if available, to query the list of
lexically scoped variables (de...

PR-URL: https://github.com/nodejs/node/pull/17730
Reviewed-By: Luigi Pinca <[email protected]...

On Windows setting ADDRCONFIG causes localhost resolution to fail if there are
no network connec...

Using a custom inspect function on the inspected object is
deprecated. Remove the reference from...

util.inspect can actually receive any property and the description
was wrong so far. This fixes ...

The current default formatting is not ideal and this improves
the situation by formatting the ou...

From now on it is possible to use a validation object in throws
instead of the other possibilite...

Reduces duplicated logic.

PR-URL: https://github.com/nodejs/node/pull/17704
Reviewed-By: Anna H...

This makes `Http2Stream`s and `Http2Session`s use actual Timeout
objects in a [kTimeout] symbol ...

This makes `net.Sockets` use actual Timeout objects in a `[kTimeout]`
symbol property, rather th...

This should help keep everything consistent.

PR-URL: https://github.com/nodejs/node/pull/17704
...

PR-URL: https://github.com/nodejs/node/pull/17782
Reviewed-By: Anna Henningsen <[email protected]...

PR-URL: https://github.com/nodejs/node/pull/17735
Reviewed-By: Colin Ihrig <[email protected]>
R...

Add `off` as an alias for `removeListener`

PR-URL: https://github.com/nodejs/node/pull/17156
Re...

PR-URL: https://github.com/nodejs/node/pull/17757
Reviewed-By: Anatoli Papirovski <apapirovski@m...

This should not be important anymore in newer docs. The change is
also documented in the "change...

This lists all function aliases directly below the declared function.

PR-URL: https://github.co...

Fix and refactor the console benchmark. It did not test console
so it got renamed and mainly tes...

PR-URL: https://github.com/nodejs/node/pull/17707
Reviewed-By: Luigi Pinca <[email protected]...

Calling write could throw a maximum call stack size error. To make
sure this is not specific to ...

`OnCallbackPadding` on the native side already clamps
the return value into the right range, so ...

- Only finish outgoing `WriteWrap`s once data has actually been
passed to the underlying socke...

`nghttp2_stream_write_t` was not a necessary redirection layer
and came with the cost of one add...

PR-URL: https://github.com/nodejs/node/pull/17556
Reviewed-By: Rich Trott <[email protected]>
Rev...

Previously the getter would mutate the kDefaultTriggerAsncId value. This
refactor changes the se...

When context is missing the executionAsyncId will be zero. For the
default triggerAsyncId the ze...

rename initTriggerId to defaultTriggerAsyncId such it matches the rest
of our naming.

PR-URL: h...

It was not clear why the error name is actually also tested for when
using a regular expression....

Throw a TypeError in case a error message is provided in the second
argument and a third argumen...

A combination of try catch and common.expectsError is not necessary
as the latter already does e...

The stack frames from .throws and .doesNotThrow got included
even though that was not intended.
...

This commit removes stdlib.h header as it does not seem to be used any
more.

PR-URL: https://gi...

PR-URL: https://github.com/nodejs/node/pull/17734
Reviewed-By: James M Snell <[email protected]>...

PR-URL: https://github.com/nodejs/node/pull/17748
Reviewed-By: Anatoli Papirovski <apapirovski@m...