Improve extraction of list-based API error messages

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

Author: cursoragent

hyperbrowser/transport/error_utils.py

@@ -8,10 +8,22 @@ def _stringify_error_value(value: Any) -> str:
     if isinstance(value, str):
         return value
     if isinstance(value, dict):
-        for key in ("message", "error", "detail"):
+        for key in ("message", "error", "detail", "msg"):
             nested_value = value.get(key)
             if nested_value is not None:
                 return _stringify_error_value(nested_value)
+    if isinstance(value, (list, tuple)):
+        collected_messages = [
+            item_message
+            for item_message in (_stringify_error_value(item) for item in value)
+            if item_message
+        ]
+        if collected_messages:
+            return (
+                collected_messages[0]
+                if len(collected_messages) == 1
+                else "; ".join(collected_messages)
+            )
     try:
         return json.dumps(value, sort_keys=True)
     except TypeError:

tests/test_transport_response_handling.py

@@ -222,22 +222,36 @@ def test_sync_handle_response_with_dict_without_message_keys_stringifies_payload
         transport.close()
 
 
-def test_async_handle_response_with_non_dict_json_stringifies_payload():
+def test_async_handle_response_with_non_dict_json_uses_first_item_payload():
     async def run() -> None:
         transport = AsyncTransport(api_key="test-key")
         try:
             response = _build_response(500, '[{"code":"UPSTREAM_FAILURE"}]')
 
-            with pytest.raises(
-                HyperbrowserError, match='\\[\\{"code": "UPSTREAM_FAILURE"\\}\\]'
-            ):
+            with pytest.raises(HyperbrowserError, match='"code": "UPSTREAM_FAILURE"'):
                 await transport._handle_response(response)
         finally:
             await transport.close()
 
     asyncio.run(run())
 
 
+def test_sync_handle_response_with_validation_error_detail_list_uses_msg_values():
+    transport = SyncTransport(api_key="test-key")
+    try:
+        response = _build_response(
+            422,
+            '{"detail":[{"msg":"field required"},{"msg":"invalid email address"}]}',
+        )
+
+        with pytest.raises(
+            HyperbrowserError, match="field required; invalid email address"
+        ):
+            transport._handle_response(response)
+    finally:
+        transport.close()
+
+
 def test_sync_transport_post_wraps_request_errors_with_url_context():
     transport = SyncTransport(api_key="test-key")
     original_post = transport.client.post