From dd92d763239d24ad382654a9cd1191256e46c3a0 Mon Sep 17 00:00:00 2001
From: Trinity Agent <trinity-agent@users.noreply.github.com>
Date: Wed, 11 Mar 2026 19:02:01 +0000
Subject: [PATCH] fix(cloud-tools): add defer free for stdout allocation (#173)

Add defer std.heap.page_allocator.free(stdout) after readToEndAlloc
to fix memory leak in runTriCloud() function that occurred on every
MCP tool call.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 tools/mcp/trinity_mcp/cloud_tools.zig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tools/mcp/trinity_mcp/cloud_tools.zig b/tools/mcp/trinity_mcp/cloud_tools.zig
index 330e820d37..a7f67cd061 100644
--- a/tools/mcp/trinity_mcp/cloud_tools.zig
+++ b/tools/mcp/trinity_mcp/cloud_tools.zig
@@ -70,6 +70,7 @@ fn runTriCloud(buf: *[MAX_OUTPUT]u8, args: []const []const u8) []const u8 {
     const stdout = child.stdout.?.readToEndAlloc(std.heap.page_allocator, MAX_OUTPUT) catch {
         return copyToBuf(buf, "Error: Failed to read output");
     };
+    defer std.heap.page_allocator.free(stdout);
 
     _ = child.wait() catch {};