Task
In tools/mcp/trinity_mcp/cloud_tools.zig, the runTriCloud() function allocates stdout via readToEndAlloc() but never frees it — memory leak on every MCP tool call.
Current Code (around line 86)
const stdout = child.stdout.?.readToEndAlloc(std.heap.page_allocator, MAX_OUTPUT) catch {
return copyToBuf(buf, "Error: Failed to read output");
};
// stdout is NEVER freed!Fix
Add defer std.heap.page_allocator.free(stdout); immediately after the allocation:
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);File
tools/mcp/trinity_mcp/cloud_tools.zig — around line 86
Acceptance
zig build compiles without errorszig fmt passesdefer free present after readToEndAlloc
Task
In
tools/mcp/trinity_mcp/cloud_tools.zig, therunTriCloud()function allocates stdout viareadToEndAlloc()but never frees it — memory leak on every MCP tool call.Current Code (around line 86)
Fix
Add
defer std.heap.page_allocator.free(stdout);immediately after the allocation:File
tools/mcp/trinity_mcp/cloud_tools.zig— around line 86Acceptance
zig buildcompiles without errorszig fmtpassesdeferfree present afterreadToEndAlloc