Deduplicate /api prefix when base URL already ends with api

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

Author: cursoragent

hyperbrowser/client/base.py

@@ -78,6 +78,16 @@ def _build_url(self, path: str) -> str:
         if parsed_path.scheme:
             raise HyperbrowserError("path must be a relative API path")
         normalized_path = f"/{stripped_path.lstrip('/')}"
+        parsed_base_url = urlparse(self.config.base_url)
+        base_path = parsed_base_url.path.rstrip("/")
+        base_has_api_suffix = base_path.endswith("/api")
+
         if normalized_path == "/api" or normalized_path.startswith("/api/"):
+            if base_has_api_suffix:
+                deduped_path = normalized_path[len("/api") :]
+                return f"{self.config.base_url}{deduped_path}"
+            return f"{self.config.base_url}{normalized_path}"
+
+        if base_has_api_suffix:
             return f"{self.config.base_url}{normalized_path}"
         return f"{self.config.base_url}/api{normalized_path}"

tests/test_url_building.py

@@ -42,6 +42,35 @@ def test_client_build_url_uses_normalized_base_url():
         client.close()
 
 
+def test_client_build_url_avoids_duplicate_api_when_base_url_already_has_api():
+    client = Hyperbrowser(
+        config=ClientConfig(api_key="test-key", base_url="https://example.local/api")
+    )
+    try:
+        assert client._build_url("/session") == "https://example.local/api/session"
+        assert client._build_url("/api/session") == "https://example.local/api/session"
+    finally:
+        client.close()
+
+
+def test_client_build_url_handles_nested_api_base_paths():
+    client = Hyperbrowser(
+        config=ClientConfig(
+            api_key="test-key", base_url="https://example.local/custom/api"
+        )
+    )
+    try:
+        assert (
+            client._build_url("/session") == "https://example.local/custom/api/session"
+        )
+        assert (
+            client._build_url("/api/session")
+            == "https://example.local/custom/api/session"
+        )
+    finally:
+        client.close()
+
+
 def test_client_build_url_rejects_empty_or_non_string_paths():
     client = Hyperbrowser(config=ClientConfig(api_key="test-key"))
     try: