src: fix some typo errors and rename some variables

doc/api/stream.md

@@ -2001,7 +2001,7 @@ If the loop terminates with a `break`, `return`, or a `throw`, the stream will
 be destroyed. In other terms, iterating over a stream will consume the stream
 fully. The stream will be read in chunks of size equal to the `highWaterMark`
 option. In the code example above, data will be in a single chunk if the file
-has less then 64 KiB of data because no `highWaterMark` option is provided to
+has less than 64 KiB of data because no `highWaterMark` option is provided to
 [`fs.createReadStream()`][].
 
 ##### `readable[Symbol.for('Stream.toAsyncStreamable')]()`

doc/api/vm.md

@@ -2346,7 +2346,7 @@ console.log('OK');
 ```
 
 **Note:** Strictly speaking, in this mode, `node:vm` departs from the letter of
-the ECMAScript specification for [enqueing jobs][], by allowing asynchronous
+the ECMAScript specification for [enqueuing jobs][], by allowing asynchronous
 tasks from different contexts to run in a different order than they were
 enqueued.
 
@@ -2588,7 +2588,7 @@ const { Script, SyntheticModule } = require('node:vm');
 [`vm.runInContext()`]: #vmrunincontextcode-contextifiedobject-options
 [`vm.runInThisContext()`]: #vmruninthiscontextcode-options
 [contextified]: #what-does-it-mean-to-contextify-an-object
-[enqueing jobs]: https://tc39.es/ecma262/#sec-hostenqueuepromisejob
+[enqueuing jobs]: https://tc39.es/ecma262/#sec-hostenqueuepromisejob
 [global object]: https://tc39.es/ecma262/#sec-global-object
 [indirect `eval()` call]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/eval#direct_and_indirect_eval
 [origin]: https://developer.mozilla.org/en-US/docs/Glossary/Origin

test/addons/stringbytes-external-exceed-max/binding.cc

@@ -5,7 +5,7 @@
 #ifdef _AIX
 // AIX allows over-allocation, and will SIGKILL when the allocated pages are
 // used if there is not enough VM. Check for available space until
-// out-of-memory.  Don't allow more then some (large) proportion of it to be
+// out-of-memory.  Don't allow more than some (large) proportion of it to be
 // used for the test strings, so Node & V8 have some space for allocations.
 #include <signal.h>
 #include <sys/vminfo.h>

test/es-module/test-esm-nowarn-exports.mjs

@@ -5,7 +5,7 @@ import { execPath } from 'node:process';
 import { describe, it } from 'node:test';
 
 
-describe('ESM: experiemental warning for import.meta.resolve', { concurrency: !process.env.TEST_PARALLEL }, () => {
+describe('ESM: experimental warning for import.meta.resolve', { concurrency: !process.env.TEST_PARALLEL }, () => {
   it('should not warn when caught', async () => {
     const { code, signal, stderr } = await spawnPromisified(execPath, [
       '--experimental-import-meta-resolve',

test/js-native-api/test_string/test_string.c

@@ -20,7 +20,7 @@ static napi_status validate_and_retrieve_single_string_arg(
 
   NODE_API_ASSERT_STATUS(env,
                          valuetype == napi_string,
-                         "Wrong type of argment. Expects a string.");
+                         "Wrong type of argument. Expects a string.");
 
   return napi_ok;
 }

test/known_issues/test-fs-cp-filter.js

@@ -1,6 +1,6 @@
 'use strict';
 
-// This test will fail because the the implementation does not properly
+// This test will fail because the implementation does not properly
 // handle the case when the `src` or `dest` is a Buffer and the `filter`
 // function is utilized when recursively copying directories.
 // Refs: https://github.com/nodejs/node/issues/58634

test/known_issues/test-fs-readdir-promise-recursive-with-buffer.js

@@ -1,6 +1,6 @@
 'use strict';
 
-// This test will fail because the the implementation does not properly
+// This test will fail because the implementation does not properly
 // handle the case when the path is a Buffer and the function is called
 // in recursive mode.
 

test/known_issues/test-fs-readdir-recursive-with-buffer.js

@@ -1,6 +1,6 @@
 'use strict';
 
-// This test will fail because the the implementation does not properly
+// This test will fail because the implementation does not properly
 // handle the case when the path is a Buffer and the function is called
 // in recursive mode.
 

test/known_issues/test-fs-readdir-sync-recursive-with-buffer.js

@@ -1,6 +1,6 @@
 'use strict';
 
-// This test will fail because the the implementation does not properly
+// This test will fail because the implementation does not properly
 // handle the case when the path is a Buffer and the function is called
 // in recursive mode.
 

test/parallel/test-abortcontroller.js

@@ -13,7 +13,7 @@ const {
 
 const { setTimeout: sleep } = require('timers/promises');
 
-// All of the the tests in this file depend on public-facing Node.js APIs.
+// All of the tests in this file depend on public-facing Node.js APIs.
 // For tests that depend on Node.js internal APIs, please add them to
 // test-abortcontroller-internal.js instead.
 
@@ -93,7 +93,7 @@ test('AbortSignal.abort() creates an already aborted signal', () => {
   assert.ok(signal.aborted);
 });
 
-test('AbortController properties and methods valiate the receiver', () => {
+test('AbortController properties and methods validate the receiver', () => {
   const acSignalGet = Object.getOwnPropertyDescriptor(
     AbortController.prototype,
     'signal'

test/parallel/test-crypto-sec-level.js

@@ -8,7 +8,7 @@ if (!common.hasCrypto)
 const assert = require('assert');
 
 // OpenSSL has a set of security levels which affect what algorithms
-// are available by default. Different OpenSSL veresions have different
+// are available by default. Different OpenSSL versions have different
 // default security levels and we use this value to adjust what a test
 // expects based on the security level. You can read more in
 // https://docs.openssl.org/1.1.1/man3/SSL_CTX_set_security_level/#default-callback-behaviour

test/parallel/test-debugger-break.js

@@ -41,7 +41,7 @@ const cli = startCLI([script]);
   assert.match(
     cli.output,
     /> 3 name = 'Robin';/,
-    'marks the 3nd line');
+    'marks the 3rd line');
 
   await cli.stepCommand('cont');
   assert.ok(

test/parallel/test-domain-uncaught-exception.js

@@ -31,7 +31,7 @@ tests.push({
 });
 
 function test2() {
-  // Throwing from from within a domain that doesn't have an error handler must
+  // Throwing from within a domain that doesn't have an error handler must
   // result in emitting the process' uncaughtException event.
   const d2 = domain.create();
   d2.run(function() {

test/parallel/test-fs-cp-async-socket.mjs

@@ -15,7 +15,7 @@ if (isWindows) {
 
 // See https://github.com/nodejs/node/pull/48409
 if (common.isInsideDirWithUnusualChars) {
-  common.skip('Test is borken in directories with unusual characters');
+  common.skip('Test is broken in directories with unusual characters');
 }
 
 tmpdir.refresh();

test/parallel/test-fs-cp-sync-copy-directory-to-file-error.mjs

@@ -8,7 +8,7 @@ import fixtures from '../common/fixtures.js';
 
 // See https://github.com/nodejs/node/pull/48409
 if (isInsideDirWithUnusualChars) {
-  skip('Test is borken in directories with unusual characters');
+  skip('Test is broken in directories with unusual characters');
 }
 
 tmpdir.refresh();

test/parallel/test-fs-cp-sync-copy-file-to-directory-error.mjs

@@ -10,7 +10,7 @@ tmpdir.refresh();
 
 // See https://github.com/nodejs/node/pull/48409
 if (isInsideDirWithUnusualChars) {
-  skip('Test is borken in directories with unusual characters');
+  skip('Test is broken in directories with unusual characters');
 }
 
 const src = fixtures.path('copy/kitchen-sink/README.md');

test/parallel/test-fs-cp-sync-copy-socket-error.mjs

@@ -14,7 +14,7 @@ if (isWindows) {
 
 // See https://github.com/nodejs/node/pull/48409
 if (common.isInsideDirWithUnusualChars) {
-  common.skip('Test is borken in directories with unusual characters');
+  common.skip('Test is broken in directories with unusual characters');
 }
 
 tmpdir.refresh();

test/parallel/test-fs-cp-sync-parent-symlink-dest-points-to-src-error.mjs

@@ -10,7 +10,7 @@ tmpdir.refresh();
 
 // See https://github.com/nodejs/node/pull/48409
 if (isInsideDirWithUnusualChars) {
-  skip('Test is borken in directories with unusual characters');
+  skip('Test is broken in directories with unusual characters');
 }
 
 const src = nextdir();

test/parallel/test-fs-cp-sync-src-parent-of-dest-error.mjs

@@ -10,7 +10,7 @@ tmpdir.refresh();
 
 // See https://github.com/nodejs/node/pull/48409
 if (isInsideDirWithUnusualChars) {
-  skip('Test is borken in directories with unusual characters');
+  skip('Test is broken in directories with unusual characters');
 }
 
 const src = nextdir('a', tmpdir);

test/parallel/test-fs-glob.mjs

@@ -393,7 +393,7 @@ describe('fsPromises.glob - with file: URL as cwd', function() {
 
 const normalizeDirent = (dirent) => relative(fixtureDir, join(dirent.parentPath, dirent.name));
 // The call to `join()` with only one argument is important, as
-// it ensures that the proper path seperators are applied.
+// it ensures that the proper path separators are applied.
 const normalizePath = (path) => (isAbsolute(path) ? relative(fixtureDir, path) : join(path));
 
 describe('glob - withFileTypes', function() {

test/parallel/test-http-agent-false.js

@@ -35,7 +35,7 @@ const opts = {
 };
 
 // We just want an "error" (no local HTTP server on port 80) or "response"
-// to happen (user happens ot have HTTP server running on port 80).
+// to happen (user happens to have HTTP server running on port 80).
 // As long as the process doesn't crash from a C++ assertion then we're good.
 const req = http.request(opts);
 

test/parallel/test-http-keep-alive-empty-line.mjs

@@ -3,7 +3,7 @@ import assert from 'node:assert';
 import { createServer } from 'node:http';
 import { connect } from 'node:net';
 
-// This test ensures that data like an empty line (`\r\n`) recevied by the
+// This test ensures that data like an empty line (`\r\n`) received by the
 // server after a request, does not reset the keep-alive timeout. See
 // https://github.com/nodejs/node/issues/58140.
 

test/parallel/test-inspector-emit-protocol-event.js

@@ -194,7 +194,7 @@ for (const [domain, events] of Object.entries(EXPECTED_EVENTS)) {
     }
   }
 
-  // Check tht no events are emitted after disabling the domain.
+  // Check that no events are emitted after disabling the domain.
   await session.post('Network.disable');
   session.on('Network.requestWillBeSent', common.mustNotCall());
   inspector.Network.requestWillBeSent({});

test/parallel/test-net-server-blocklist.js

@@ -7,11 +7,11 @@ blockList.addAddress(common.localhostIPv4);
 
 const server = net.createServer({ blockList }, common.mustNotCall());
 server.listen(0, common.localhostIPv4, common.mustCall(() => {
-  const adddress = server.address();
+  const address = server.address();
   const socket = net.connect({
     localAddress: common.localhostIPv4,
-    host: adddress.address,
-    port: adddress.port
+    host: address.address,
+    port: address.port
   });
   socket.on('close', common.mustCall(() => {
     server.close();

test/parallel/test-process-exit-code-validation.js

@@ -110,11 +110,11 @@ if (process.argv[2] === undefined) {
 
   // Check process.exitCode
   for (const arg of invalids) {
-    debug(`invaild code: ${inspect(arg.code)}`);
+    debug(`invalid code: ${inspect(arg.code)}`);
     assert.throws(() => (process.exitCode = arg.code), new RegExp(arg.pattern));
   }
   for (const arg of valids) {
-    debug(`vaild code: ${inspect(arg.code)}`);
+    debug(`valid code: ${inspect(arg.code)}`);
     process.exitCode = arg.code;
   }
 

test/parallel/test-process-threadCpuUsage-main-thread.js

@@ -47,7 +47,7 @@ if (!isSunOS) {
   );
 }
 
-// Test argument validaton
+// Test argument validation
 {
   assert.throws(
     () => process.threadCpuUsage(123),

test/parallel/test-quic-datagram-drop-newest.mjs

@@ -38,7 +38,7 @@ const serverEndpoint = await listen(mustCall(async (serverSession) => {
   alpn: ['quic-test'],
   transportParams: { maxDatagramFrameSize: 1200 },
   ondatagram: mustCall(function(data, early) {
-    // We whould only receive datagrams 1 and 2
+    // We should only receive datagrams 1 and 2
     strictEqual(data.length, 1);
     ok(data[0] === 0 || data[0] === 1);
     ok(!early);

test/parallel/test-repl-colors.js

@@ -7,19 +7,19 @@ const { REPLServer } = require('repl');
 
 let output = '';
 
-const inout = new Duplex({ decodeStrings: false });
-inout._read = function() {
+const duplex = new Duplex({ decodeStrings: false });
+duplex._read = function() {
   this.push('util.inspect("string")\n');
   this.push(null);
 };
-inout._write = function(s, _, cb) {
+duplex._write = function(s, _, cb) {
   output += s;
   cb();
 };
 
-const repl = new REPLServer({ input: inout, output: inout, useColors: true });
-inout.isTTY = true;
-const repl2 = new REPLServer({ input: inout, output: inout });
+const repl = new REPLServer({ input: duplex, output: duplex, useColors: true });
+duplex.isTTY = true;
+const repl2 = new REPLServer({ input: duplex, output: duplex });
 
 process.on('exit', function() {
   // https://github.com/nodejs/node/pull/16485#issuecomment-350428638

test/parallel/test-repl-custom-eval.js

@@ -75,7 +75,7 @@ describe('repl with custom eval', { concurrency: true }, () => {
     delete global.foo;
   });
 
-  it('inherits variables from the global context but does not use it afterwords if `useGlobal` is false', async () => {
+  it('inherits variables from the global context but does not use it afterwards if `useGlobal` is false', async () => {
     global.bar = 'global_bar';
     const context = await new Promise((resolve) => {
       const { replServer } = startNewREPLServer({

test/parallel/test-sqlite-session.js

@@ -232,7 +232,7 @@ suite('conflict resolution', () => {
     const insertSql = 'INSERT INTO data (key, value) VALUES (?, ?)';
     const session = database1.createSession();
     database1.prepare(insertSql).run(1, 'hello');
-    database2.prepare(insertSql).run(2, 'hello');  // database2 already constains hello
+    database2.prepare(insertSql).run(2, 'hello');  // database2 already contains hello
 
     return {
       database2,

test/parallel/test-stream-duplexpair.js

@@ -54,7 +54,7 @@ const { Duplex, duplexPair } = require('stream');
   }));
 }
 
-// Test the case where the the _write never calls [kCallback]
+// Test the case where the _write never calls [kCallback]
 // because a zero-size push doesn't trigger a _read
 {
   const [ serverSide, clientSide ] = duplexPair();

test/parallel/test-timers-interval-promisified.js

@@ -30,7 +30,7 @@ const { setInterval } = timerPromises;
   const iterator = iterable[Symbol.asyncIterator]();
   const promise = iterator.next();
   promise.then((result) => {
-    assert.ok(!result.done, 'iterator was wronly marked as done');
+    assert.ok(!result.done, 'iterator was wrongly marked as done');
     assert.strictEqual(result.value, 'foobar');
     return iterator.return();
   }).then(common.mustCall());
@@ -42,7 +42,7 @@ const { setInterval } = timerPromises;
   const promise = iterator.next();
   promise
     .then((result) => {
-      assert.ok(!result.done, 'iterator was wronly marked as done');
+      assert.ok(!result.done, 'iterator was wrongly marked as done');
       assert.strictEqual(result.value, 'foobar');
       return iterator.next();
     })

test/parallel/test-tls-client-mindhsize.js

@@ -5,7 +5,7 @@ if (!common.hasCrypto)
   common.skip('missing crypto');
 
 // OpenSSL has a set of security levels which affect what algorithms
-// are available by default. Different OpenSSL veresions have different
+// are available by default. Different OpenSSL versions have different
 // default security levels and we use this value to adjust what a test
 // expects based on the security level. You can read more in
 // https://docs.openssl.org/1.1.1/man3/SSL_CTX_set_security_level/#default-callback-behaviour

test/parallel/test-tls-dhe.js

@@ -37,7 +37,7 @@ const {
 } = require('../common/crypto');
 
 // OpenSSL has a set of security levels which affect what algorithms
-// are available by default. Different OpenSSL veresions have different
+// are available by default. Different OpenSSL versions have different
 // default security levels and we use this value to adjust what a test
 // expects based on the security level. You can read more in
 // https://docs.openssl.org/1.1.1/man3/SSL_CTX_set_security_level/#default-callback-behaviour

test/parallel/test-tls-ip-servername-forbidden.js

@@ -8,7 +8,7 @@ if (!common.hasCrypto)
 
 const tls = require('tls');
 
-// Verify that passing an IP address the the servername option
+// Verify that passing an IP address to the servername option
 // throws an error.
 assert.throws(() => tls.connect({
   port: 1234,

test/parallel/test-tls-min-max-version.js

@@ -200,7 +200,7 @@ test(U, U, 'TLSv1_2_method', U, U, 'TLSv1_2_method', 'TLSv1.2');
 test(U, U, 'TLSv1_1_method', U, U, 'TLSv1_1_method', 'TLSv1.1');
 test(U, U, 'TLSv1_method', U, U, 'TLSv1_method', 'TLSv1');
 
-// The default default.
+// The default configuration.
 if (DEFAULT_MIN_VERSION === 'TLSv1.2') {
   test(U, U, 'TLSv1_1_method', U, U, U,
        U, 'ERR_SSL_TLSV1_ALERT_PROTOCOL_VERSION',
@@ -215,7 +215,7 @@ if (DEFAULT_MIN_VERSION === 'TLSv1.2') {
     test(U, U, U, U, U, 'TLSv1_method',
          U, 'ERR_SSL_UNSUPPORTED_PROTOCOL', 'ERR_SSL_WRONG_VERSION_NUMBER');
   } else {
-    // TLS1.3 client hellos are are not understood by TLS1.1 or below.
+    // TLS1.3 client hellos are not understood by TLS1.1 or below.
     test(U, U, U, U, U, 'TLSv1_1_method',
          U, 'ERR_SSL_TLSV1_ALERT_PROTOCOL_VERSION',
          'ERR_SSL_UNSUPPORTED_PROTOCOL');
@@ -237,7 +237,7 @@ if (DEFAULT_MIN_VERSION === 'TLSv1.1') {
     test(U, U, U, U, U, 'TLSv1_method',
          U, 'ERR_SSL_UNSUPPORTED_PROTOCOL', 'ERR_SSL_WRONG_VERSION_NUMBER');
   } else {
-    // TLS1.3 client hellos are are not understood by TLS1.1 or below.
+    // TLS1.3 client hellos are not understood by TLS1.1 or below.
     test(U, U, U, U, U, 'TLSv1_method',
          U, 'ERR_SSL_TLSV1_ALERT_PROTOCOL_VERSION',
          'ERR_SSL_UNSUPPORTED_PROTOCOL');