fix(mcp): register process exit handler to dispose instances and prevent orphan processes

Author: sauerdaniel

packages/opencode/src/index.ts

@@ -29,6 +29,7 @@ import { PrCommand } from "./cli/cmd/pr"
 import { SessionCommand } from "./cli/cmd/session"
 import { DbCommand } from "./cli/cmd/db"
 import path from "path"
+import { Instance } from "./project/instance"
 import { Global } from "./global"
 import { JsonMigration } from "./storage/json-migration"
 import { Database } from "./storage/db"
@@ -155,6 +156,19 @@ const cli = yargs(hideBin(process.argv))
   })
   .strict()
 
+// Dispose instances on termination to prevent orphan MCP/LSP processes
+let cleanupDone = false
+async function cleanup(signal: string) {
+  if (cleanupDone) return
+  cleanupDone = true
+  Log.Default.info("cleanup triggered", { signal })
+  try {
+    await Instance.disposeAll()
+  } catch {}
+}
+process.on("SIGINT", () => cleanup("SIGINT").then(() => process.exit(130)))
+process.on("SIGTERM", () => cleanup("SIGTERM").then(() => process.exit(143)))
+
 try {
   await cli.parse()
 } catch (e) {
@@ -195,6 +209,7 @@ try {
   }
   process.exitCode = 1
 } finally {
+  await cleanup("exit")
   // Some subprocesses don't react properly to SIGTERM and similar signals.
   // Most notably, some docker-container-based MCP servers don't handle such signals unless
   // run using `docker run --init`.