Use mapping inputs for tool wrappers without mutation

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

Author: cursoragent

hyperbrowser/tools/__init__.py

@@ -38,18 +38,26 @@ def _prepare_extract_tool_params(params: Mapping[str, Any]) -> Dict[str, Any]:
     return normalized_params
 
 
+def _to_param_dict(params: Mapping[str, Any]) -> Dict[str, Any]:
+    return dict(params)
+
+
 class WebsiteScrapeTool:
     openai_tool_definition = SCRAPE_TOOL_OPENAI
     anthropic_tool_definition = SCRAPE_TOOL_ANTHROPIC
 
     @staticmethod
-    def runnable(hb: Hyperbrowser, params: dict) -> str:
-        resp = hb.scrape.start_and_wait(params=StartScrapeJobParams(**params))
+    def runnable(hb: Hyperbrowser, params: Mapping[str, Any]) -> str:
+        resp = hb.scrape.start_and_wait(
+            params=StartScrapeJobParams(**_to_param_dict(params))
+        )
         return resp.data.markdown if resp.data and resp.data.markdown else ""
 
     @staticmethod
-    async def async_runnable(hb: AsyncHyperbrowser, params: dict) -> str:
-        resp = await hb.scrape.start_and_wait(params=StartScrapeJobParams(**params))
+    async def async_runnable(hb: AsyncHyperbrowser, params: Mapping[str, Any]) -> str:
+        resp = await hb.scrape.start_and_wait(
+            params=StartScrapeJobParams(**_to_param_dict(params))
+        )
         return resp.data.markdown if resp.data and resp.data.markdown else ""
 
 
@@ -58,13 +66,17 @@ class WebsiteScreenshotTool:
     anthropic_tool_definition = SCREENSHOT_TOOL_ANTHROPIC
 
     @staticmethod
-    def runnable(hb: Hyperbrowser, params: dict) -> str:
-        resp = hb.scrape.start_and_wait(params=StartScrapeJobParams(**params))
+    def runnable(hb: Hyperbrowser, params: Mapping[str, Any]) -> str:
+        resp = hb.scrape.start_and_wait(
+            params=StartScrapeJobParams(**_to_param_dict(params))
+        )
         return resp.data.screenshot if resp.data and resp.data.screenshot else ""
 
     @staticmethod
-    async def async_runnable(hb: AsyncHyperbrowser, params: dict) -> str:
-        resp = await hb.scrape.start_and_wait(params=StartScrapeJobParams(**params))
+    async def async_runnable(hb: AsyncHyperbrowser, params: Mapping[str, Any]) -> str:
+        resp = await hb.scrape.start_and_wait(
+            params=StartScrapeJobParams(**_to_param_dict(params))
+        )
         return resp.data.screenshot if resp.data and resp.data.screenshot else ""
 
 
@@ -73,8 +85,10 @@ class WebsiteCrawlTool:
     anthropic_tool_definition = CRAWL_TOOL_ANTHROPIC
 
     @staticmethod
-    def runnable(hb: Hyperbrowser, params: dict) -> str:
-        resp = hb.crawl.start_and_wait(params=StartCrawlJobParams(**params))
+    def runnable(hb: Hyperbrowser, params: Mapping[str, Any]) -> str:
+        resp = hb.crawl.start_and_wait(
+            params=StartCrawlJobParams(**_to_param_dict(params))
+        )
         markdown = ""
         if resp.data:
             for page in resp.data:
@@ -85,8 +99,10 @@ def runnable(hb: Hyperbrowser, params: dict) -> str:
         return markdown
 
     @staticmethod
-    async def async_runnable(hb: AsyncHyperbrowser, params: dict) -> str:
-        resp = await hb.crawl.start_and_wait(params=StartCrawlJobParams(**params))
+    async def async_runnable(hb: AsyncHyperbrowser, params: Mapping[str, Any]) -> str:
+        resp = await hb.crawl.start_and_wait(
+            params=StartCrawlJobParams(**_to_param_dict(params))
+        )
         markdown = ""
         if resp.data:
             for page in resp.data:
@@ -102,15 +118,15 @@ class WebsiteExtractTool:
     anthropic_tool_definition = EXTRACT_TOOL_ANTHROPIC
 
     @staticmethod
-    def runnable(hb: Hyperbrowser, params: dict) -> str:
+    def runnable(hb: Hyperbrowser, params: Mapping[str, Any]) -> str:
         normalized_params = _prepare_extract_tool_params(params)
         resp = hb.extract.start_and_wait(
             params=StartExtractJobParams(**normalized_params)
         )
         return json.dumps(resp.data) if resp.data else ""
 
     @staticmethod
-    async def async_runnable(hb: AsyncHyperbrowser, params: dict) -> str:
+    async def async_runnable(hb: AsyncHyperbrowser, params: Mapping[str, Any]) -> str:
         normalized_params = _prepare_extract_tool_params(params)
         resp = await hb.extract.start_and_wait(
             params=StartExtractJobParams(**normalized_params)
@@ -123,16 +139,16 @@ class BrowserUseTool:
     anthropic_tool_definition = BROWSER_USE_TOOL_ANTHROPIC
 
     @staticmethod
-    def runnable(hb: Hyperbrowser, params: dict) -> str:
+    def runnable(hb: Hyperbrowser, params: Mapping[str, Any]) -> str:
         resp = hb.agents.browser_use.start_and_wait(
-            params=StartBrowserUseTaskParams(**params)
+            params=StartBrowserUseTaskParams(**_to_param_dict(params))
         )
         return resp.data.final_result if resp.data and resp.data.final_result else ""
 
     @staticmethod
-    async def async_runnable(hb: AsyncHyperbrowser, params: dict) -> str:
+    async def async_runnable(hb: AsyncHyperbrowser, params: Mapping[str, Any]) -> str:
         resp = await hb.agents.browser_use.start_and_wait(
-            params=StartBrowserUseTaskParams(**params)
+            params=StartBrowserUseTaskParams(**_to_param_dict(params))
         )
         return resp.data.final_result if resp.data and resp.data.final_result else ""
 

tests/test_url_building.py

@@ -13,7 +13,10 @@ def test_client_build_url_normalizes_leading_slash():
             client._build_url("/api/session")
             == "https://api.hyperbrowser.ai/api/session"
         )
-        assert client._build_url("api/session") == "https://api.hyperbrowser.ai/api/session"
+        assert (
+            client._build_url("api/session")
+            == "https://api.hyperbrowser.ai/api/session"
+        )
     finally:
         client.close()