From e4a175d24a7ecb219ec20f92502bec2b86a29709 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Kristo=CC=81fer=20R?= <kristofer@thorlaksson.com>
Date: Tue, 13 May 2025 23:41:18 -0400
Subject: [PATCH] use explicit error set

---
 src/os/hostname.zig | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/src/os/hostname.zig b/src/os/hostname.zig
index fdbe822e3..2cdba5763 100644
--- a/src/os/hostname.zig
+++ b/src/os/hostname.zig
@@ -7,6 +7,14 @@ pub const HostnameParsingError = error{
     NoSpaceLeft,
 };
 
+pub const UrlParsingError = error{
+    HostnameIsNotMacAddress,
+    InvalidFormat,
+    InvalidPort,
+    NoSchemeProvided,
+    UnexpectedCharacter,
+};
+
 fn isUriPathSeparator(c: u8) bool {
     return switch (c) {
         '?', '#' => true,
@@ -39,7 +47,7 @@ fn isValidMacAddress(mac_address: []const u8) bool {
 /// path information for Ghostty's PWD reporting functionality. Takes into account that on macOS
 /// the url passed to this function might have a mac address as its hostname and parses it
 /// correctly.
-pub fn parseUrl(url: []const u8) !std.Uri {
+pub fn parseUrl(url: []const u8) UrlParsingError!std.Uri {
     return std.Uri.parse(url) catch |e| {
         // The mac-address-as-hostname issue is specific to macOS so we just return an error if we
         // hit it on other platforms.
@@ -52,7 +60,7 @@ pub fn parseUrl(url: []const u8) !std.Uri {
         if (e != error.InvalidPort) return e;
 
         const url_without_scheme_start = std.mem.indexOf(u8, url, "://") orelse {
-            return error.InvalidScheme;
+            return error.NoSchemeProvided;
         };
         const scheme = url[0..url_without_scheme_start];
         const url_without_scheme = url[url_without_scheme_start + 3 ..];