fixup! repl: remove dependency on domain module

- Mark addUncaughtExceptionCaptureCallback as experimental
- Add changes entry to setUncaughtExceptionCaptureCallback docs
- Use validateFunction instead of manual type check

Author: mcollina

doc/api/process.md

@@ -742,6 +742,8 @@ This feature is not available in [`Worker`][] threads.
 added: REPLACEME
 -->
 
+> Stability: 1 - Experimental
+
 * `fn` {Function}
 
 The `process.addUncaughtExceptionCaptureCallback()` function adds a callback
@@ -4057,6 +4059,11 @@ This implies calling `module.setSourceMapsSupport()` with an option
 
 <!-- YAML
 added: v9.3.0
+changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/61227
+    description: Use `process.addUncaughtExceptionCaptureCallback()` to
+      register multiple callbacks.
 -->
 
 * `fn` {Function|null}

lib/internal/process/execution.js

@@ -18,6 +18,7 @@ const {
     ERR_UNCAUGHT_EXCEPTION_CAPTURE_ALREADY_SET,
   },
 } = require('internal/errors');
+const { validateFunction } = require('internal/validators');
 const { pathToFileURL } = require('internal/url');
 const { exitCodes: { kGenericUserError } } = internalBinding('errors');
 const {
@@ -137,9 +138,7 @@ function setUncaughtExceptionCaptureCallback(fn) {
 // Auxiliary callbacks are called first; if any returns true, the error is handled.
 // Otherwise, the primary callback (if set) is called.
 function addUncaughtExceptionCaptureCallback(fn) {
-  if (typeof fn !== 'function') {
-    throw new ERR_INVALID_ARG_TYPE('fn', 'Function', fn);
-  }
+  validateFunction(fn, 'fn');
   if (exceptionHandlerState.auxiliaryCallbacks.length === 0 &&
       exceptionHandlerState.captureFn === null) {
     exceptionHandlerState.reportFlag =

lib/repl.js

@@ -209,16 +209,16 @@ const kBufferedCommandSymbol = Symbol('bufferedCommand');
 const kLoadingSymbol = Symbol('loading');
 
 function processNewListener(event, listener) {
-  if (event === 'uncaughtException' &&
-      process.domain &&
-      listener.name !== 'domainUncaughtExceptionClear' &&
-      domainSet.has(process.domain)) {
-    // Throw an error so that the event will not be added and the current
-    // domain takes over. That way the user is notified about the error
-    // and the current code evaluation is stopped, just as any other code
-    // that contains an error.
-    throw new ERR_INVALID_REPL_INPUT(
-      'Listeners for `uncaughtException` cannot be used in the REPL');
+  if (event === 'uncaughtException') {
+    const store = replContext.getStore();
+    if (store?.replServer) {
+      // Throw an error so that the event will not be added and the
+      // current REPL handles it. That way the user is notified about
+      // the error and the current code evaluation is stopped, just as
+      // any other code that contains an error.
+      throw new ERR_INVALID_REPL_INPUT(
+        'Listeners for `uncaughtException` cannot be used in the REPL');
+    }
   }
 }
 
@@ -396,24 +396,9 @@ class REPLServer extends Interface {
       // It is possible to introspect the running REPL accessing this variable
       // from inside the REPL. This is useful for anyone working on the REPL.
       module.exports.repl = this;
-    } else if (!addedNewListener) {
-      // Add this listener only once. Otherwise we'd have to add a single
-      // listener to each REPL instance and that could trigger the
-      // `MaxListenersExceededWarning`.
-      process.prependListener('newListener', (event, listener) => {
-        if (event === 'uncaughtException') {
-          const store = replContext.getStore();
-          if (store?.replServer) {
-            // Throw an error so that the event will not be added and the
-            // current REPL handles it. That way the user is notified about
-            // the error and the current code evaluation is stopped, just as
-            // any other code that contains an error.
-            throw new ERR_INVALID_REPL_INPUT(
-              'Listeners for `uncaughtException` cannot be used in the REPL');
-          }
-        }
-      });
-      addedNewListener = true;
+    } else {
+      addProcessNewListener();
+      this.once('exit', removeProcessNewListener);
     }
 
     // Set up exception capture for async error handling