From 94e4bce990c888c401725dabaa558b9825564927 Mon Sep 17 00:00:00 2001
From: "PC2\\micha" <remotac@gmail.com>
Date: Sun, 28 Jun 2026 20:07:39 +0800
Subject: [PATCH] fix: kill process tree on Windows via taskkill before SIGTERM
 fallback

On Windows, child.kill('SIGTERM') only terminates the cmd.exe wrapper,
leaving the inner opencode.exe serve running as an orphan. This adds
killProcessTree() which runs taskkill /PID /T /F first, then falls
back to child.kill('SIGTERM') for the close() method.

Fixes #1889
---
 packages/vscode/src/opencode.ts | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/packages/vscode/src/opencode.ts b/packages/vscode/src/opencode.ts
index 540a9aba..95f7f239 100644
--- a/packages/vscode/src/opencode.ts
+++ b/packages/vscode/src/opencode.ts
@@ -133,6 +133,19 @@ function shouldUseWindowsShell(binary: string): boolean {
   return !ext && !trimmed.includes('\\') && !trimmed.includes('/');
 }
 
+function killProcessTree(pid: number | undefined): void {
+  if (!Number.isInteger(pid)) return;
+  if (process.platform === 'win32') {
+    try {
+      spawnSync('taskkill', ['/PID', String(pid), '/T', '/F'], {
+        stdio: 'ignore', timeout: 5000, windowsHide: true,
+      });
+    } catch {
+      // ignore
+    }
+  }
+}
+
 function appendToPath(dir: string) {
   const trimmed = (dir || '').trim();
   if (!trimmed) return;
@@ -695,6 +708,7 @@ async function spawnManagedOpenCodeServer(
   return {
     url,
     close: () => {
+      killProcessTree(child.pid);
       try {
         child.kill('SIGTERM');
       } catch {