core: Refactor Node.js Unhandled Rejection Handling to process.emit Interception (#8896)

Author: tobiu

src/Neo.mjs

@@ -1190,15 +1190,19 @@ if (typeof globalThis.addEventListener === 'function') {
             e.preventDefault()
         }
     })
-} else if (typeof process !== 'undefined' && typeof process.on === 'function') {
+} else if (typeof process !== 'undefined' && typeof process.emit === 'function') {
     // Node.js
-    process.on('unhandledRejection', e => {
-        if (e === Neo.isDestroyed) {
-            return
+    // We need to intercept the emit, since test runners like Playwright
+    // will listen to unhandledRejection and fail the test
+    const originalEmit = process.emit;
+
+    process.emit = function(name, data, ...args) {
+        if (name === 'unhandledRejection' && data === Neo.isDestroyed) {
+            return true
         }
 
-        throw e
-    })
+        return originalEmit.apply(this, arguments)
+    }
 }
 
 export default Neo;

test/playwright/setup.mjs

@@ -51,14 +51,4 @@ export function setup(options = {}) {
     Neo.apps ??= {};
     Neo.apps[appConfig.windowId || 1] = finalAppConfig;
     Neo.appsByName = {[finalAppConfig.name]: [finalAppConfig]};
-
-    // Global safeguard: Intercept and suppress Neo.isDestroyed unhandled rejections.
-    // This allows floating promises (like timeouts) to reject safely during test teardown.
-    const originalEmit = process.emit;
-    process.emit = function (name, data, ...args) {
-        if (name === 'unhandledRejection' && data === Symbol.for('Neo.isDestroyed')) {
-            return true;
-        }
-        return originalEmit.apply(this, arguments);
-    };
 }