From 4850ec9ae1dd6b7eb2781fe627a8d647275c58f0 Mon Sep 17 00:00:00 2001 From: zzstoatzz Date: Thu, 2 Apr 2026 21:04:40 -0500 Subject: [PATCH] bump websocket.zig to 4222f98 (httpFallback dispatch), fix test IPC MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit websocket.zig change: plain HTTP requests on the websocket port are now dispatched to Handler.httpFallback() instead of getting a 400 response. this restores health probe / XRPC support on port 3000 that was lost in the 0.16 fork migration. also removes std.debug.print calls from test code — these corrupted the zig 0.16 --listen=- IPC protocol between the build runner and test binary, causing spurious "failed command" output. Co-Authored-By: Claude Opus 4.6 --- build.zig.zon | 6 +++--- src/internal/identity/did_resolver.zig | 11 ++++------- src/internal/identity/handle_resolver.zig | 18 ++++++------------ src/internal/repo/repo_verifier.zig | 6 ++---- 4 files changed, 15 insertions(+), 26 deletions(-) diff --git a/build.zig.zon b/build.zig.zon index f2bf73e..c6e137a 100644 --- a/build.zig.zon +++ b/build.zig.zon @@ -1,12 +1,12 @@ .{ .name = .zat, - .version = "0.3.0-alpha.7", + .version = "0.3.0-alpha.10", .fingerprint = 0x8da9db57ee82fbe4, .minimum_zig_version = "0.16.0", .dependencies = .{ .websocket = .{ - .url = "https://github.com/zzstoatzz/websocket.zig/archive/0261b7d.tar.gz", - .hash = "websocket-0.1.0-ZPISddmnAwDF1Jjj9X9ICO8sAJetd3-rdRUA-vCzYfwF", + .url = "https://github.com/zzstoatzz/websocket.zig/archive/4222f98.tar.gz", + .hash = "websocket-0.1.0-ZPISdQTUAwDJt7jFEbYJhdqRBztn8mHIFh-dNK5dOlxL", }, .@"atproto-interop-tests" = .{ .url = "https://github.com/bluesky-social/atproto-interop-tests/archive/35bb5638ab1e5ce71fb88a0c95953fc557ef1925.tar.gz", diff --git a/src/internal/identity/did_resolver.zig b/src/internal/identity/did_resolver.zig index f328536..a9c2e06 100644 --- a/src/internal/identity/did_resolver.zig +++ b/src/internal/identity/did_resolver.zig @@ -117,10 +117,8 @@ test "resolve did:plc - integration" { defer resolver.deinit(); const did = Did.parse("did:plc:z72i7hdynmk6r22z27h6tvur").?; - var doc = resolver.resolve(did) catch |err| { - // network errors are ok in CI, but compilation must succeed - std.debug.print("network error (expected in CI): {}\n", .{err}); - return; + var doc = resolver.resolve(did) catch { + return; // network error, expected in CI }; defer doc.deinit(); @@ -135,9 +133,8 @@ test "resolve did:plc - leak check (no arena)" { defer resolver.deinit(); const did = Did.parse("did:plc:z72i7hdynmk6r22z27h6tvur").?; - var doc = resolver.resolve(did) catch |err| { - std.debug.print("network error (expected in CI): {}\n", .{err}); - return; + var doc = resolver.resolve(did) catch { + return; // network error, expected in CI }; defer doc.deinit(); diff --git a/src/internal/identity/handle_resolver.zig b/src/internal/identity/handle_resolver.zig index 3340898..38a75c6 100644 --- a/src/internal/identity/handle_resolver.zig +++ b/src/internal/identity/handle_resolver.zig @@ -168,10 +168,8 @@ test "resolve handle (http) - integration" { // resolve a known handle that has .well-known/atproto-did const handle = Handle.parse("jay.bsky.social") orelse return error.InvalidHandle; - const did = resolver.resolveHttp(handle) catch |err| { - // network errors are ok in CI without network access - std.debug.print("network error (expected in some CI): {}\n", .{err}); - return; + const did = resolver.resolveHttp(handle) catch { + return; // network error, expected in CI }; // should be a valid did:plc @@ -187,10 +185,8 @@ test "resolve handle (dns over http) - integration" { defer resolver.deinit(); const handle = Handle.parse("seiso.moe") orelse return error.InvalidHandle; - const did = resolver.resolveDns(handle) catch |err| { - // network errors are ok in CI without network access - std.debug.print("network error (expected in some CI): {}\n", .{err}); - return; + const did = resolver.resolveDns(handle) catch { + return; // network error, expected in CI }; // should be a valid DID @@ -206,10 +202,8 @@ test "resolve handle - integration" { defer resolver.deinit(); const handle = Handle.parse("jay.bsky.social") orelse return error.InvalidHandle; - const did = resolver.resolve(handle) catch |err| { - // network errors are ok in CI without network access - std.debug.print("network error (expected in some CI): {}\n", .{err}); - return; + const did = resolver.resolve(handle) catch { + return; // network error, expected in CI }; // should be a valid DID diff --git a/src/internal/repo/repo_verifier.zig b/src/internal/repo/repo_verifier.zig index bcb6627..7462890 100644 --- a/src/internal/repo/repo_verifier.zig +++ b/src/internal/repo/repo_verifier.zig @@ -476,14 +476,12 @@ test "verify repo - zzstoatzz.io" { var arena = std.heap.ArenaAllocator.init(std.testing.allocator); defer arena.deinit(); - const result = verifyRepo(std.Options.debug_io, arena.allocator(), "zzstoatzz.io") catch |err| { - std.debug.print("network error (expected in CI): {}\n", .{err}); - return; + const result = verifyRepo(std.Options.debug_io, arena.allocator(), "zzstoatzz.io") catch { + return; // network error, expected in CI }; try std.testing.expectEqualStrings("did:plc:xbtmt2zjwlrfegqvch7fboei", result.did); try std.testing.expect(result.record_count > 0); - std.debug.print("verified zzstoatzz.io: {d} records, rev={s}\n", .{ result.record_count, result.commit_rev }); } test "verifyCommitDiff: build tree, serialize partial CAR, verify inversion" { -- 2.51.2