Fix #175 by removing system.exit calls in the built-in handlers (#184)

Author: seratch

.changeset/tame-hotels-hope.md

@@ -0,0 +1,5 @@
+---
+'@openai/agents-core': patch
+---
+
+Fix #175 by removing internal system.exit calls

.github/workflows/test.yml

@@ -8,22 +8,27 @@ on:
 jobs:
   test:
     runs-on: ubuntu-latest
+    env:
+      NODE_OPTIONS: "--max_old_space_size=4096"
     strategy:
       matrix:
         # https://nodejs.org/en/about/previous-releases
         node-version: [20, 22, 24]
     steps:
       - name: Checkout repository
         uses: actions/checkout@v4
-      - name: Setup Node.js
-        uses: actions/setup-node@v4
-        with:
-          node-version: ${{ matrix.node-version }}
       - name: Install pnpm
         uses: pnpm/action-setup@v4
         with:
           version: 10.13.1
-          run_install: true
+          run_install: false
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: ${{ matrix.node-version }}
+          cache: 'pnpm'
+      - name: Install dependencies
+        run: pnpm install
       - name: Build all packages
         run: pnpm build
       - name: Run linter

packages/agents-core/src/tracing/provider.ts

@@ -188,19 +188,28 @@ export class TraceProvider {
       // Handle CTRL+C (SIGINT)
       process.on('SIGINT', async () => {
         await cleanup();
-        process.exit(130);
+        if (!hasOtherListenersForSignals('SIGINT')) {
+          // Only when there are no other listeners, exit the process on this SDK side
+          process.exit(130);
+        }
       });
 
       // Handle termination (SIGTERM)
       process.on('SIGTERM', async () => {
         await cleanup();
-        process.exit(0);
+        if (!hasOtherListenersForSignals('SIGTERM')) {
+          // Only when there are no other listeners, exit the process on this SDK side
+          process.exit(0);
+        }
       });
 
       process.on('unhandledRejection', async (reason, promise) => {
         logger.error('Unhandled rejection', reason, promise);
         await cleanup();
-        process.exit(1);
+        if (!hasOtherListenersForEvents('unhandledRejection')) {
+          // Only when there are no other listeners, exit the process on this SDK side
+          process.exit(1);
+        }
       });
     }
   }
@@ -210,6 +219,14 @@ export class TraceProvider {
   }
 }
 
+function hasOtherListenersForSignals(event: 'SIGINT' | 'SIGTERM'): boolean {
+  return process.listeners(event).length > 1;
+}
+
+function hasOtherListenersForEvents(event: 'unhandledRejection'): boolean {
+  return process.listeners(event).length > 1;
+}
+
 let GLOBAL_TRACE_PROVIDER: TraceProvider | undefined = undefined;
 export function getGlobalTraceProvider(): TraceProvider {
   if (!GLOBAL_TRACE_PROVIDER) {