Require concrete list crawl responses in tool wrappers

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

Author: cursoragent

hyperbrowser/tools/__init__.py

@@ -426,7 +426,7 @@ def _read_crawl_page_field(page: Any, *, field_name: str, page_index: int) -> An
 def _render_crawl_markdown_output(response_data: Any) -> str:
     if response_data is None:
         return ""
-    if not isinstance(response_data, list):
+    if type(response_data) is not list:
         raise HyperbrowserError("crawl tool response data must be a list")
     try:
         crawl_pages = list(response_data)

tests/test_tools_response_handling.py

@@ -838,19 +838,19 @@ def test_crawl_tool_uses_unknown_url_for_blank_page_urls():
     assert "page body" in output
 
 
-def test_crawl_tool_wraps_response_iteration_failures():
+def test_crawl_tool_rejects_list_subclass_response_data():
     class _BrokenList(list):
         def __iter__(self):
             raise RuntimeError("cannot iterate pages")
 
     client = _SyncCrawlClient(_Response(data=_BrokenList([SimpleNamespace()])))
 
     with pytest.raises(
-        HyperbrowserError, match="Failed to iterate crawl tool response data"
+        HyperbrowserError, match="crawl tool response data must be a list"
     ) as exc_info:
         WebsiteCrawlTool.runnable(client, {"url": "https://example.com"})
 
-    assert exc_info.value.original_error is not None
+    assert exc_info.value.original_error is None
 
 
 def test_browser_use_tool_rejects_non_string_final_result():
@@ -1154,6 +1154,24 @@ async def run() -> None:
     asyncio.run(run())
 
 
+def test_async_crawl_tool_rejects_list_subclass_response_data():
+    class _BrokenList(list):
+        def __iter__(self):
+            raise RuntimeError("cannot iterate pages")
+
+    async def run() -> None:
+        client = _AsyncCrawlClient(_Response(data=_BrokenList([SimpleNamespace()])))
+        with pytest.raises(
+            HyperbrowserError, match="crawl tool response data must be a list"
+        ) as exc_info:
+            await WebsiteCrawlTool.async_runnable(
+                client, {"url": "https://example.com"}
+            )
+        assert exc_info.value.original_error is None
+
+    asyncio.run(run())
+
+
 def test_async_browser_use_tool_rejects_non_string_final_result():
     async def run() -> None:
         client = _AsyncBrowserUseClient(