Prefer response body text for non-JSON HTTP errors

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

Author: cursoragent

hyperbrowser/transport/async_transport.py

@@ -52,7 +52,7 @@ async def _handle_response(self, response: httpx.Response) -> APIResponse:
                 error_data = response.json()
                 message = error_data.get("message") or error_data.get("error") or str(e)
             except Exception:
-                message = str(e)
+                message = response.text or str(e)
             raise HyperbrowserError(
                 message,
                 status_code=response.status_code,

hyperbrowser/transport/sync.py

@@ -40,7 +40,7 @@ def _handle_response(self, response: httpx.Response) -> APIResponse:
                 error_data = response.json()
                 message = error_data.get("message") or error_data.get("error") or str(e)
             except Exception:
-                message = str(e)
+                message = response.text or str(e)
             raise HyperbrowserError(
                 message,
                 status_code=response.status_code,

tests/test_transport_response_handling.py

@@ -47,7 +47,7 @@ def test_sync_handle_response_with_error_and_non_json_body_raises_hyperbrowser_e
     try:
         response = _build_response(500, "server exploded")
 
-        with pytest.raises(HyperbrowserError):
+        with pytest.raises(HyperbrowserError, match="server exploded"):
             transport._handle_response(response)
     finally:
         transport.close()
@@ -59,7 +59,7 @@ async def run() -> None:
         try:
             response = _build_response(500, "server exploded")
 
-            with pytest.raises(HyperbrowserError):
+            with pytest.raises(HyperbrowserError, match="server exploded"):
                 await transport._handle_response(response)
         finally:
             await transport.close()