src: move process.binding('async_wrap') internal

This commit makes the async_wrap builtin an internal builtin, and
changes usage of the builtin from using process.binding('async_wrap')
to use internalBinding instead.

Refs: #22160

PR-URL: #22469
Refs: #22160
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

Author: danbev

lib/internal/async_hooks.js

@@ -4,7 +4,8 @@ const {
   ERR_ASYNC_TYPE,
   ERR_INVALID_ASYNC_ID
 } = require('internal/errors').codes;
-const async_wrap = process.binding('async_wrap');
+const { internalBinding } = require('internal/bootstrap/loaders');
+const async_wrap = internalBinding('async_wrap');
 /* async_hook_fields is a Uint32Array wrapping the uint32_t array of
  * Environment::AsyncHooks::fields_[]. Each index tracks the number of active
  * hooks for each type.

lib/internal/bootstrap/node.js

@@ -356,7 +356,8 @@
         'crypto',
         'contextify',
         'tcp_wrap',
-        'tls_wrap']);
+        'tls_wrap',
+        'async_wrap']);
     process.binding = function binding(name) {
       return internalBindingWhitelist.has(name) ?
         internalBinding(name) :

lib/internal/crypto/pbkdf2.js

@@ -1,8 +1,8 @@
 'use strict';
 
-const { AsyncWrap, Providers } = process.binding('async_wrap');
-const { Buffer } = require('buffer');
 const { internalBinding } = require('internal/bootstrap/loaders');
+const { AsyncWrap, Providers } = internalBinding('async_wrap');
+const { Buffer } = require('buffer');
 const { INT_MAX, pbkdf2: _pbkdf2 } = internalBinding('crypto');
 const { validateInt32 } = require('internal/validators');
 const {

lib/internal/crypto/random.js

@@ -1,8 +1,8 @@
 'use strict';
 
-const { AsyncWrap, Providers } = process.binding('async_wrap');
-const { Buffer, kMaxLength } = require('buffer');
 const { internalBinding } = require('internal/bootstrap/loaders');
+const { AsyncWrap, Providers } = internalBinding('async_wrap');
+const { Buffer, kMaxLength } = require('buffer');
 const { randomBytes: _randomBytes } = internalBinding('crypto');
 const {
   ERR_INVALID_ARG_TYPE,

lib/internal/crypto/scrypt.js

@@ -1,8 +1,8 @@
 'use strict';
 
-const { AsyncWrap, Providers } = process.binding('async_wrap');
-const { Buffer } = require('buffer');
 const { internalBinding } = require('internal/bootstrap/loaders');
+const { AsyncWrap, Providers } = internalBinding('async_wrap');
+const { Buffer } = require('buffer');
 const { INT_MAX, scrypt: _scrypt } = internalBinding('crypto');
 const { validateInt32 } = require('internal/validators');
 const {

lib/internal/trace_events_async_hooks.js

@@ -2,7 +2,7 @@
 
 const { internalBinding } = require('internal/bootstrap/loaders');
 const { trace } = internalBinding('trace_events');
-const async_wrap = process.binding('async_wrap');
+const async_wrap = internalBinding('async_wrap');
 const async_hooks = require('async_hooks');
 const { SafeMap, SafeSet } = require('internal/safe_globals');
 

src/async_wrap.cc

@@ -765,4 +765,4 @@ Local<Object> AsyncWrap::GetOwner(Environment* env, Local<Object> obj) {
 
 }  // namespace node
 
-NODE_BUILTIN_MODULE_CONTEXT_AWARE(async_wrap, node::AsyncWrap::Initialize)
+NODE_MODULE_CONTEXT_AWARE_INTERNAL(async_wrap, node::AsyncWrap::Initialize)

test/common/index.js

@@ -79,7 +79,8 @@ if (process.env.NODE_TEST_WITH_ASYNC_HOOKS) {
   const destroydIdsList = {};
   const destroyListList = {};
   const initHandles = {};
-  const async_wrap = process.binding('async_wrap');
+  const { internalBinding } = require('internal/test/binding');
+  const async_wrap = internalBinding('async_wrap');
 
   process.on('exit', () => {
     // iterate through handles to make sure nothing crashes

test/parallel/test-async-wrap-destroyid.js

@@ -1,7 +1,9 @@
+// Flags: --expose-internals
 'use strict';
 
 const common = require('../common');
-const async_wrap = process.binding('async_wrap');
+const { internalBinding } = require('internal/test/binding');
+const async_wrap = internalBinding('async_wrap');
 const assert = require('assert');
 const async_hooks = require('async_hooks');
 const RUNS = 5;

test/pseudo-tty/test-async-wrap-getasyncid-tty.js

@@ -1,11 +1,13 @@
+// Flags: --expose-internals --no-warnings
 'use strict';
 
 // see also test/sequential/test-async-wrap-getasyncid.js
 
 const common = require('../common');
 const assert = require('assert');
 const tty_wrap = process.binding('tty_wrap');
-const { TTYWRAP } = process.binding('async_wrap').Providers;
+const { internalBinding } = require('internal/test/binding');
+const { TTYWRAP } = internalBinding('async_wrap').Providers;
 const providers = { TTYWRAP };
 
 // Make sure that the TTYWRAP Provider is tested.

test/sequential/test-async-wrap-getasyncid.js

@@ -7,7 +7,7 @@ const assert = require('assert');
 const fs = require('fs');
 const fsPromises = fs.promises;
 const net = require('net');
-const providers = Object.assign({}, process.binding('async_wrap').Providers);
+const providers = Object.assign({}, internalBinding('async_wrap').Providers);
 const fixtures = require('../common/fixtures');
 const tmpdir = require('../common/tmpdir');
 const { getSystemErrorName } = require('util');

test/sequential/test-inspector-async-call-stack.js

@@ -1,10 +1,12 @@
+// Flags: --expose-internals
 'use strict';
 const common = require('../common');
 common.skipIfInspectorDisabled();
 common.skipIf32Bits();
 
 const assert = require('assert');
-const async_wrap = process.binding('async_wrap');
+const { internalBinding } = require('internal/test/binding');
+const async_wrap = internalBinding('async_wrap');
 const { kTotals } = async_wrap.constants;
 const inspector = require('inspector');