Normalize true/false URL sentinels in transport diagnostics

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

Author: cursoragent

README.md

@@ -187,7 +187,7 @@ Polling timeouts and repeated polling failures are surfaced as:
 
 `HyperbrowserPollingError` also covers stalled pagination (no page-batch progress during result collection).
 Transport-level request failures include HTTP method + URL context in error messages.
-URL-like fallback objects are stringified for transport diagnostics, bytes-like fallback methods/URLs are decoded when valid, and missing/malformed/sentinel URL inputs (for example `None`, booleans, invalid bytes, `null`/`undefined`/`nan`, or numeric-like values such as `123`/`1.5`/`1e6`) are normalized to `unknown URL`.
+URL-like fallback objects are stringified for transport diagnostics, bytes-like fallback methods/URLs are decoded when valid, and missing/malformed/sentinel URL inputs (for example `None`, booleans, invalid bytes, `null`/`undefined`/`true`/`false`/`nan`, or numeric-like values such as `123`/`1.5`/`1e6`) are normalized to `unknown URL`.
 
 ```python
 from hyperbrowser import Hyperbrowser

hyperbrowser/transport/error_utils.py

@@ -15,6 +15,8 @@
     "none",
     "null",
     "undefined",
+    "true",
+    "false",
     "nan",
     "inf",
     "+inf",

tests/test_transport_error_utils.py

@@ -332,6 +332,8 @@ def test_format_request_failure_message_normalizes_boolean_fallback_url_values()
         "none",
         "null",
         "undefined",
+        "true",
+        "false",
         "nan",
         "inf",
         "+inf",
@@ -429,6 +431,8 @@ def test_format_generic_request_failure_message_normalizes_boolean_url_values():
         "none",
         "null",
         "undefined",
+        "true",
+        "false",
         "nan",
         "inf",
         "+inf",

tests/test_transport_response_handling.py

@@ -426,6 +426,22 @@ def failing_get(*args, **kwargs):
         transport.close()
 
 
+def test_sync_transport_wraps_unexpected_errors_with_boolean_string_url_fallback():
+    transport = SyncTransport(api_key="test-key")
+    original_get = transport.client.get
+
+    def failing_get(*args, **kwargs):
+        raise RuntimeError("boom")
+
+    transport.client.get = failing_get  # type: ignore[assignment]
+    try:
+        with pytest.raises(HyperbrowserError, match="Request GET unknown URL failed"):
+            transport.get("true")
+    finally:
+        transport.client.get = original_get  # type: ignore[assignment]
+        transport.close()
+
+
 def test_sync_transport_wraps_unexpected_errors_with_numeric_like_string_url_fallback():
     transport = SyncTransport(api_key="test-key")
     original_get = transport.client.get
@@ -548,6 +564,27 @@ async def failing_put(*args, **kwargs):
     asyncio.run(run())
 
 
+def test_async_transport_wraps_unexpected_errors_with_boolean_string_url_fallback():
+    async def run() -> None:
+        transport = AsyncTransport(api_key="test-key")
+        original_put = transport.client.put
+
+        async def failing_put(*args, **kwargs):
+            raise RuntimeError("boom")
+
+        transport.client.put = failing_put  # type: ignore[assignment]
+        try:
+            with pytest.raises(
+                HyperbrowserError, match="Request PUT unknown URL failed"
+            ):
+                await transport.put("false")
+        finally:
+            transport.client.put = original_put  # type: ignore[assignment]
+            await transport.close()
+
+    asyncio.run(run())
+
+
 def test_async_transport_wraps_unexpected_errors_with_numeric_like_string_url_fallback():
     async def run() -> None:
         transport = AsyncTransport(api_key="test-key")
@@ -669,6 +706,22 @@ def failing_get(*args, **kwargs):
         transport.close()
 
 
+def test_sync_transport_request_error_without_request_uses_unknown_url_for_boolean_string_input():
+    transport = SyncTransport(api_key="test-key")
+    original_get = transport.client.get
+
+    def failing_get(*args, **kwargs):
+        raise httpx.RequestError("network down")
+
+    transport.client.get = failing_get  # type: ignore[assignment]
+    try:
+        with pytest.raises(HyperbrowserError, match="Request GET unknown URL failed"):
+            transport.get("true")
+    finally:
+        transport.client.get = original_get  # type: ignore[assignment]
+        transport.close()
+
+
 def test_sync_transport_request_error_without_request_uses_unknown_url_for_numeric_like_string_input():
     transport = SyncTransport(api_key="test-key")
     original_get = transport.client.get
@@ -813,6 +866,27 @@ async def failing_delete(*args, **kwargs):
     asyncio.run(run())
 
 
+def test_async_transport_request_error_without_request_uses_unknown_url_for_boolean_string_input():
+    async def run() -> None:
+        transport = AsyncTransport(api_key="test-key")
+        original_delete = transport.client.delete
+
+        async def failing_delete(*args, **kwargs):
+            raise httpx.RequestError("network down")
+
+        transport.client.delete = failing_delete  # type: ignore[assignment]
+        try:
+            with pytest.raises(
+                HyperbrowserError, match="Request DELETE unknown URL failed"
+            ):
+                await transport.delete("false")
+        finally:
+            transport.client.delete = original_delete  # type: ignore[assignment]
+            await transport.close()
+
+    asyncio.run(run())
+
+
 def test_async_transport_request_error_without_request_uses_unknown_url_for_numeric_like_string_input():
     async def run() -> None:
         transport = AsyncTransport(api_key="test-key")