fix(core): harden PTY resize against native crashes (#27496)

packages/cli/index.ts

@@ -21,9 +21,12 @@ import {
 // Tracking bug: https://github.com/microsoft/node-pty/issues/827
 process.on('uncaughtException', (error) => {
   if (error instanceof Error) {
+    const message = error.message || '';
     const isPtyResizeError =
-      error.message === 'Cannot resize a pty that has already exited';
+      message === 'Cannot resize a pty that has already exited';
-    const isEbadfError = error.message.includes('EBADF');
+    const isEbadfError =
+      message.includes('EBADF') ||
+      (error as { code?: string }).code === 'EBADF';
     const isFromNodePty =
       error.stack?.includes('node-pty') || error.stack?.includes('PtyResize');
 

packages/core/src/services/shellExecutionService.ts

@@ -37,6 +37,7 @@ import {
 } from './sandboxManager.js';
 import type { SandboxConfig } from '../config/config.js';
 import { killProcessGroup } from '../utils/process-utils.js';
+import { isNodeError } from '../utils/errors.js';
 import {
   ExecutionLifecycleService,
   type ExecutionHandle,
@@ -1507,7 +1508,22 @@ export class ShellExecutionService {
     }
 
     const activePty = this.activePtys.get(pid);
-    if (activePty) {
+    if (!activePty) {
+      return;
+    }
+
+    // Skip Windows: process.kill(pid, 0) is heavy and native errors are catchable there.
+    if (process.platform !== 'win32') {
+      try {
+        process.kill(pid, 0);
+      } catch (e) {
+        // Bail only if the process is explicitly confirmed dead (ESRCH).
+        if (isNodeError(e) && e.code === 'ESRCH') {
+          return;
+        }
+      }
+    }
+
     try {
       activePty.ptyProcess.resize(cols, rows);
       activePty.headlessTerminal.resize(cols, rows);
@@ -1530,7 +1546,6 @@ export class ShellExecutionService {
         throw e;
       }
     }
-    }
 
     // Force emit the new state after resize
     if (activePty) {