nodejs
diff --git a/Collapse file
‎lib/async_hooks.js‎
Copy file name to clipboardExpand all lines: lib/async_hooks.js
+1-2Lines changed: 1 addition & 2 deletions b/Collapse file
‎lib/async_hooks.js‎
Copy file name to clipboardExpand all lines: lib/async_hooks.js
+1-2Lines changed: 1 addition & 2 deletions
diff --git a/Collapse file
‎lib/internal/async_hooks.js‎
Copy file name to clipboardExpand all lines: lib/internal/async_hooks.js
+2-2Lines changed: 2 additions & 2 deletions b/Collapse file
‎lib/internal/async_hooks.js‎
Copy file name to clipboardExpand all lines: lib/internal/async_hooks.js
+2-2Lines changed: 2 additions & 2 deletions
diff --git a/Collapse file
‎lib/internal/process/task_queues.js‎
Copy file name to clipboardExpand all lines: lib/internal/process/task_queues.js
+2-2Lines changed: 2 additions & 2 deletions b/Collapse file
‎lib/internal/process/task_queues.js‎
Copy file name to clipboardExpand all lines: lib/internal/process/task_queues.js
+2-2Lines changed: 2 additions & 2 deletions
diff --git a/Collapse file
‎src/env.cc‎
Copy file name to clipboardExpand all lines: src/env.cc
+2-3Lines changed: 2 additions & 3 deletions b/Collapse file
‎src/env.cc‎
Copy file name to clipboardExpand all lines: src/env.cc
+2-3Lines changed: 2 additions & 3 deletions
diff --git a/Collapse file
‎test/parallel/test-async-hooks-enabledhooksexits.js‎
Copy file name to clipboard
+18Lines changed: 18 additions & 0 deletions b/Collapse file
‎test/parallel/test-async-hooks-enabledhooksexits.js‎
Copy file name to clipboard
+18Lines changed: 18 additions & 0 deletions
@@ -55,7 +55,6 @@ const {
   emitBefore,
   emitAfter,
   emitDestroy,
-  enabledHooksExist,
   initHooksExist,
   destroyHooksExist,
 } = internal_async_hooks;
@@ -203,7 +202,7 @@ class AsyncResource {
     this[trigger_async_id_symbol] = triggerAsyncId;
 
     if (initHooksExist()) {
-      if (enabledHooksExist() && type.length === 0) {
+      if (type.length === 0) {
         throw new ERR_ASYNC_TYPE(type);
       }
 
 
@@ -480,7 +480,7 @@ function hasHooks(key) {
 }
 
 function enabledHooksExist() {
-  return hasHooks(kCheck);
+  return active_hooks.array.length > 0;
 }
 
 function initHooksExist() {
@@ -562,7 +562,7 @@ function popAsyncContext(asyncId) {
   const stackLength = async_hook_fields[kStackLength];
   if (stackLength === 0) return false;
 
-  if (enabledHooksExist() && async_id_fields[kExecutionAsyncId] !== asyncId) {
+  if (async_hook_fields[kCheck] > 0 && async_id_fields[kExecutionAsyncId] !== asyncId) {
     // Do the same thing as the native code (i.e. crash hard).
     return popAsyncContext_(asyncId);
   }
 
@@ -25,7 +25,7 @@ const {
 
 const {
   getDefaultTriggerAsyncId,
-  getHookArrays,
+  enabledHooksExist,
   newAsyncId,
   initHooksExist,
   emitInit,
@@ -160,7 +160,7 @@ function queueMicrotask(callback) {
   validateFunction(callback, 'callback');
 
   const contextFrame = AsyncContextFrame.current();
-  if (contextFrame || getHookArrays()[0].length > 0) {
+  if (contextFrame || enabledHooksExist()) {
     const asyncResource = new AsyncResource(
       'Microtask',
       defaultMicrotaskResourceOpts,
 
@@ -124,8 +124,7 @@ void AsyncHooks::push_async_context(double async_id,
                                     double trigger_async_id,
                                     Local<Object>* resource) {
   CHECK_IMPLIES(resource != nullptr, !resource->IsEmpty());
-  // Since async_hooks is experimental, do only perform the check
-  // when async_hooks is enabled.
+
   if (fields_[kCheck] > 0) {
     CHECK_GE(async_id, -1);
     CHECK_GE(trigger_async_id, -1);
@@ -1736,7 +1735,7 @@ AsyncHooks::AsyncHooks(Isolate* isolate, const SerializeInfo* info)
     clear_async_id_stack();
 
     // Always perform async_hooks checks, not just when async_hooks is enabled.
-    // TODO(AndreasMadsen): Consider removing this for LTS releases.
+    // Can be disabled via CLI option --no-force-async-hooks-checks
     // See discussion in https://github.com/nodejs/node/pull/15454
     // When removing this, do it by reverting the commit. Otherwise the test
     // and flag changes won't be included.
 
@@ -0,0 +1,18 @@
+// Flags: --expose-internals
+'use strict';
+
+require('../common');
+const assert = require('assert');
+const { createHook } = require('async_hooks');
+const { enabledHooksExist } = require('internal/async_hooks');
+
+assert.strictEqual(enabledHooksExist(), false);
+
+const ah = createHook({});
+assert.strictEqual(enabledHooksExist(), false);
+
+ah.enable();
+assert.strictEqual(enabledHooksExist(), true);
+
+ah.disable();
+assert.strictEqual(enabledHooksExist(), false);
Original file line number	Diff line number	Diff line change
`@@ -480,7 +480,7 @@ function hasHooks(key) {`
`480`	`480`	`}`
`481`	`481`
`482`	`482`	`function enabledHooksExist() {`
`483`		`- return hasHooks(kCheck);`
	`483`	`+ return active_hooks.array.length > 0;`
`484`	`484`	`}`
`485`	`485`
`486`	`486`	`function initHooksExist() {`
`@@ -562,7 +562,7 @@ function popAsyncContext(asyncId) {`
`562`	`562`	`const stackLength = async_hook_fields[kStackLength];`
`563`	`563`	`if (stackLength === 0) return false;`
`564`	`564`
`565`		`- if (enabledHooksExist() && async_id_fields[kExecutionAsyncId] !== asyncId) {`
	`565`	`+ if (async_hook_fields[kCheck] > 0 && async_id_fields[kExecutionAsyncId] !== asyncId) {`
`566`	`566`	`// Do the same thing as the native code (i.e. crash hard).`
`567`	`567`	`return popAsyncContext_(asyncId);`
`568`	`568`	`}`