Validate individual extension items during list parsing

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

Author: cursoragent

hyperbrowser/client/managers/extension_utils.py

@@ -15,4 +15,17 @@ def parse_extension_list_response_data(response_data: Any) -> List[ExtensionResp
         raise HyperbrowserError(
             f"Expected list in 'extensions' key but got {type(response_data['extensions'])}"
         )
-    return [ExtensionResponse(**extension) for extension in response_data["extensions"]]
+    parsed_extensions: List[ExtensionResponse] = []
+    for index, extension in enumerate(response_data["extensions"]):
+        if not isinstance(extension, dict):
+            raise HyperbrowserError(
+                f"Expected extension object at index {index} but got {type(extension)}"
+            )
+        try:
+            parsed_extensions.append(ExtensionResponse(**extension))
+        except Exception as exc:
+            raise HyperbrowserError(
+                f"Failed to parse extension at index {index}",
+                original_error=exc,
+            ) from exc
+    return parsed_extensions

tests/test_extension_utils.py

@@ -37,3 +37,22 @@ def test_parse_extension_list_response_data_rejects_missing_extensions_key():
 def test_parse_extension_list_response_data_rejects_non_list_extensions():
     with pytest.raises(HyperbrowserError, match="Expected list in 'extensions' key"):
         parse_extension_list_response_data({"extensions": "not-a-list"})
+
+
+def test_parse_extension_list_response_data_rejects_non_object_extension_items():
+    with pytest.raises(HyperbrowserError, match="Expected extension object at index 0"):
+        parse_extension_list_response_data({"extensions": ["not-an-object"]})
+
+
+def test_parse_extension_list_response_data_wraps_invalid_extension_payloads():
+    with pytest.raises(HyperbrowserError, match="Failed to parse extension at index 0"):
+        parse_extension_list_response_data(
+            {
+                "extensions": [
+                    {
+                        "id": "ext_123",
+                        # missing required fields: name/createdAt/updatedAt
+                    }
+                ]
+            }
+        )