Coerce non-string transport error payload messages

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

Author: cursoragent

hyperbrowser/transport/error_utils.py

@@ -10,12 +10,10 @@ def extract_error_message(response: httpx.Response, fallback_error: Exception) -
         return response.text or str(fallback_error)
 
     if isinstance(error_data, dict):
-        return (
-            error_data.get("message")
-            or error_data.get("error")
-            or response.text
-            or str(fallback_error)
-        )
+        message = error_data.get("message") or error_data.get("error")
+        if message is not None:
+            return str(message)
+        return response.text or str(fallback_error)
     if isinstance(error_data, str):
         return error_data
-    return response.text or str(fallback_error)
+    return str(response.text or str(fallback_error))

tests/test_transport_response_handling.py

@@ -90,3 +90,28 @@ async def run() -> None:
             await transport.close()
 
     asyncio.run(run())
+
+
+def test_sync_handle_response_with_non_string_message_field_coerces_to_string():
+    transport = SyncTransport(api_key="test-key")
+    try:
+        response = _build_response(500, '{"message":{"detail":"failed"}}')
+
+        with pytest.raises(HyperbrowserError, match="detail"):
+            transport._handle_response(response)
+    finally:
+        transport.close()
+
+
+def test_async_handle_response_with_non_string_message_field_coerces_to_string():
+    async def run() -> None:
+        transport = AsyncTransport(api_key="test-key")
+        try:
+            response = _build_response(500, '{"message":{"detail":"failed"}}')
+
+            with pytest.raises(HyperbrowserError, match="detail"):
+                await transport._handle_response(response)
+        finally:
+            await transport.close()
+
+    asyncio.run(run())