Task
In tools/mcp/trinity_mcp/bot/claude_stream.zig line 67, the URI parsing uses catch unreachable:
const uri = std.Uri.parse(api_url) catch unreachable;
This is unsafe because api_url is built dynamically from environment variables and could be malformed.
Fix
Replace with proper error handling that returns an error instead of panicking:
const uri = std.Uri.parse(api_url) catch |err| {
std.log.err("Invalid API URL: {s} — {}", .{ api_url, err });
return error.InvalidUrl;
};File
tools/mcp/trinity_mcp/bot/claude_stream.zig — line 67
Acceptance
zig build compiles without errorszig fmt passes- No
catch unreachable on URI parsing
Task
In
tools/mcp/trinity_mcp/bot/claude_stream.zigline 67, the URI parsing usescatch unreachable:This is unsafe because
api_urlis built dynamically from environment variables and could be malformed.Fix
Replace with proper error handling that returns an error instead of panicking:
File
tools/mcp/trinity_mcp/bot/claude_stream.zig— line 67Acceptance
zig buildcompiles without errorszig fmtpassescatch unreachableon URI parsing