fix(serve): replace catch unreachable with try in tri_serve.zig init

ServerStatus.init now returns !ServerStatus instead of ServerStatus,
properly propagating allocation failures instead of using catch unreachable.
Also updated all call sites to use try.

Closes #156

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: Trinity Agent

src/tri/tri_serve.zig

@@ -88,20 +88,13 @@ pub const ServerStatus = struct {
     start_time: i64,
     allocator: std.mem.Allocator,
 
-    pub fn init(allocator: std.mem.Allocator) ServerStatus {
-        const protocols = std.ArrayList([]const u8).initCapacity(allocator, 4) catch return ServerStatus{
-            .running = false,
-            .uptime = 0,
-            .connections = 0,
-            .protocols_active = std.ArrayList([]const u8).initCapacity(allocator, 0) catch unreachable,
-            .start_time = 0,
-            .allocator = allocator,
-        };
+    pub fn init(allocator: std.mem.Allocator) !ServerStatus {
+        const protocols = std.ArrayList([]const u8).initCapacity(allocator, 4);
         return ServerStatus{
             .running = false,
             .uptime = 0,
             .connections = 0,
-            .protocols_active = protocols,
+            .protocols_active = try protocols,
             .start_time = 0,
             .allocator = allocator,
         };
@@ -127,7 +120,7 @@ pub const UnifiedApiServer = struct {
         const server = UnifiedApiServer{
             .allocator = allocator,
             .config = config,
-            .status = ServerStatus.init(allocator),
+            .status = try ServerStatus.init(allocator),
             .registry = api.CommandRegistry.init(allocator),
             .server_socket = null,
         };
@@ -1056,7 +1049,7 @@ test "ServeConfig defaults" {
 }
 
 test "ServerStatus init" {
-    var status = ServerStatus.init(std.testing.allocator);
+    var status = try ServerStatus.init(std.testing.allocator);
     defer status.deinit();
 
     try std.testing.expect(!status.running);