From e773f84e399e8db2264ac658bf6f0300a899b7c4 Mon Sep 17 00:00:00 2001
From: Roshan Singh <roshanasingh4@gmail.com>
Date: Fri, 16 Jan 2026 15:31:39 +0530
Subject: [PATCH] fix: keep background exec aborts from killing sessions
 (#1000) (thanks @roshanasingh4)

When exec returns early in background mode, the tool-call AbortSignal can fire and previously caused killProcessTree(SIGKILL). Ignore abort after yielding/backgrounding so background sessions keep running.
---
 .../bash-tools.exec.background-abort.test.ts  | 38 +++++++++++++++++++
 src/agents/bash-tools.exec.ts                 |  1 +
 2 files changed, 39 insertions(+)
 create mode 100644 src/agents/bash-tools.exec.background-abort.test.ts

diff --git a/src/agents/bash-tools.exec.background-abort.test.ts b/src/agents/bash-tools.exec.background-abort.test.ts
new file mode 100644
index 000000000..facd98557
--- /dev/null
+++ b/src/agents/bash-tools.exec.background-abort.test.ts
@@ -0,0 +1,38 @@
+import { afterEach, expect, test } from "vitest";
+
+import { createExecTool } from "./bash-tools.exec";
+import { getFinishedSession, getSession, resetProcessRegistryForTests } from "./bash-process-registry";
+import { killProcessTree } from "./shell-utils";
+
+afterEach(() => {
+  resetProcessRegistryForTests();
+});
+
+test("background exec is not killed when tool signal aborts", async () => {
+  const tool = createExecTool({ allowBackground: true, backgroundMs: 0 });
+  const abortController = new AbortController();
+
+  const result = await tool.execute(
+    "toolcall",
+    { command: "node -e \"setTimeout(() => {}, 5000)\"", background: true },
+    abortController.signal,
+  );
+
+  expect(result.details.status).toBe("running");
+  const sessionId = (result.details as { sessionId: string }).sessionId;
+
+  abortController.abort();
+
+  await new Promise((resolve) => setTimeout(resolve, 150));
+
+  const running = getSession(sessionId);
+  const finished = getFinishedSession(sessionId);
+
+  try {
+    expect(finished).toBeUndefined();
+    expect(running?.exited).toBe(false);
+  } finally {
+    const pid = running?.pid;
+    if (pid) killProcessTree(pid);
+  }
+});
diff --git a/src/agents/bash-tools.exec.ts b/src/agents/bash-tools.exec.ts
index efa57a4ab..fb088557c 100644
--- a/src/agents/bash-tools.exec.ts
+++ b/src/agents/bash-tools.exec.ts
@@ -263,6 +263,7 @@ export function createExecTool(
       };
 
       const onAbort = () => {
+        if (yielded || session.backgrounded) return;
         killSession(session);
       };