Normalize blank fallback context in request failure messages

Co-authored-by: Shri Sukhani <shrisukhani@users.noreply.github.com>

Author: cursoragent

hyperbrowser/transport/error_utils.py

@@ -77,5 +77,10 @@ def format_request_failure_message(
     effective_method = (
         request_method if request_method != "UNKNOWN" else fallback_method
     )
+    if not isinstance(effective_method, str) or not effective_method.strip():
+        effective_method = "UNKNOWN"
+
     effective_url = request_url if request_url != "unknown URL" else fallback_url
+    if not isinstance(effective_url, str) or not effective_url.strip():
+        effective_url = "unknown URL"
     return f"Request {effective_method} {effective_url} failed"

tests/test_transport_error_utils.py

@@ -92,3 +92,13 @@ def test_format_request_failure_message_prefers_request_context():
     )
 
     assert message == "Request POST https://example.com/actual failed"
+
+
+def test_format_request_failure_message_normalizes_blank_fallback_values():
+    message = format_request_failure_message(
+        httpx.RequestError("network down"),
+        fallback_method="   ",
+        fallback_url="",
+    )
+
+    assert message == "Request UNKNOWN unknown URL failed"