fix: spawns hanging (#3192)

1 files changed, 19 insertions, 7 deletions

diff --git a/packages/opencode/src/tool/bash.ts b/packages/opencode/src/tool/bash.ts
index d4e8ce85a..af1610d56 100644
--- a/packages/opencode/src/tool/bash.ts
+++ b/packages/opencode/src/tool/bash.ts
@@ -145,14 +145,26 @@ export const BashTool = Tool.define("bash", {
       })
     }
 
-    const process = spawn(params.command, {
+    const proc = spawn(params.command, {
       shell: true,
       cwd: Instance.directory,
       signal: ctx.abort,
       stdio: ["ignore", "pipe", "pipe"],
       timeout,
+      detached: process.platform !== "win32",
     })
 
+    if (!ctx.abort.aborted) {
+      ctx.abort.addEventListener("abort", () => {
+        if (!proc.pid) return
+        if (process.platform === "win32") {
+          proc.kill()
+          return
+        }
+        process.kill(-proc.pid)
+      })
+    }
+
     let output = ""
 
     // Initialize metadata with empty output
@@ -163,7 +175,7 @@ export const BashTool = Tool.define("bash", {
       },
     })
 
-    process.stdout?.on("data", (chunk) => {
+    proc.stdout?.on("data", (chunk) => {
       output += chunk.toString()
       ctx.metadata({
         metadata: {
@@ -173,7 +185,7 @@ export const BashTool = Tool.define("bash", {
       })
     })
 
-    process.stderr?.on("data", (chunk) => {
+    proc.stderr?.on("data", (chunk) => {
       output += chunk.toString()
       ctx.metadata({
         metadata: {
@@ -184,7 +196,7 @@ export const BashTool = Tool.define("bash", {
     })
 
     await new Promise<void>((resolve) => {
-      process.on("close", () => {
+      proc.on("close", () => {
         resolve()
       })
     })
@@ -192,7 +204,7 @@ export const BashTool = Tool.define("bash", {
     ctx.metadata({
       metadata: {
         output: output,
-        exit: process.exitCode,
+        exit: proc.exitCode,
         description: params.description,
       },
     })
@@ -202,7 +214,7 @@ export const BashTool = Tool.define("bash", {
       output += "\n\n(Output was truncated due to length limit)"
     }
 
-    if (process.signalCode === "SIGTERM" && params.timeout) {
+    if (proc.signalCode === "SIGTERM" && params.timeout) {
       output += `\n\n(Command timed out after ${timeout} ms)`
     }
 
@@ -210,7 +222,7 @@ export const BashTool = Tool.define("bash", {
       title: params.command,
       metadata: {
         output,
-        exit: process.exitCode,
+        exit: proc.exitCode,
         description: params.description,
       },
       output,