From cf3a30bef44564a786a5e6763a71e5362f96e30f Mon Sep 17 00:00:00 2001
From: Trinity Agent <trinity-agent@users.noreply.github.com>
Date: Wed, 11 Mar 2026 18:53:12 +0000
Subject: [PATCH] fix(mcp-client): add child.deinit() in spawn error path
 (#166)

When child.spawn() fails, the child process resources were leaking.
Added child.deinit() call before the early return to properly clean up.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 src/tri-api/mcp_client.zig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/tri-api/mcp_client.zig b/src/tri-api/mcp_client.zig
index f22e68bf64..81bf65ed15 100644
--- a/src/tri-api/mcp_client.zig
+++ b/src/tri-api/mcp_client.zig
@@ -57,6 +57,7 @@ pub const McpManager = struct {
 
         child.spawn() catch |err| {
             std.debug.print("[mcp] Failed to spawn {s}: {s}\n", .{ name, @errorName(err) });
+            child.deinit();
             return 0;
         };