Raise explicit client error for blank env base URL

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

Author: cursoragent

hyperbrowser/client/base.py

@@ -48,15 +48,21 @@ def __init__(
                     os.environ.get("HYPERBROWSER_HEADERS")
                 )
             )
+            env_base_url = os.environ.get("HYPERBROWSER_BASE_URL")
+            if base_url is None:
+                if env_base_url is None:
+                    resolved_base_url = "https://api.hyperbrowser.ai"
+                elif not env_base_url.strip():
+                    raise HyperbrowserError(
+                        "HYPERBROWSER_BASE_URL must not be empty when set"
+                    )
+                else:
+                    resolved_base_url = env_base_url
+            else:
+                resolved_base_url = base_url
             config = ClientConfig(
                 api_key=resolved_api_key,
-                base_url=(
-                    base_url
-                    if base_url is not None
-                    else os.environ.get(
-                        "HYPERBROWSER_BASE_URL", "https://api.hyperbrowser.ai"
-                    )
-                ),
+                base_url=resolved_base_url,
                 headers=resolved_headers,
             )
 

tests/test_client_api_key.py

@@ -34,6 +34,26 @@ def test_sync_client_rejects_blank_env_api_key(monkeypatch):
         Hyperbrowser()
 
 
+def test_sync_client_rejects_blank_env_base_url(monkeypatch):
+    monkeypatch.setenv("HYPERBROWSER_API_KEY", "test-key")
+    monkeypatch.setenv("HYPERBROWSER_BASE_URL", "   ")
+
+    with pytest.raises(
+        HyperbrowserError, match="HYPERBROWSER_BASE_URL must not be empty"
+    ):
+        Hyperbrowser()
+
+
+def test_async_client_rejects_blank_env_base_url(monkeypatch):
+    monkeypatch.setenv("HYPERBROWSER_API_KEY", "test-key")
+    monkeypatch.setenv("HYPERBROWSER_BASE_URL", "   ")
+
+    with pytest.raises(
+        HyperbrowserError, match="HYPERBROWSER_BASE_URL must not be empty"
+    ):
+        AsyncHyperbrowser()
+
+
 def test_sync_client_rejects_non_string_api_key():
     with pytest.raises(HyperbrowserError, match="api_key must be a string"):
         Hyperbrowser(api_key=123)  # type: ignore[arg-type]