Normalize config env validation to HyperbrowserError

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

Author: cursoragent

hyperbrowser/config.py

@@ -1,6 +1,8 @@
 from dataclasses import dataclass
 import os
 
+from .exceptions import HyperbrowserError
+
 
 @dataclass
 class ClientConfig:
@@ -13,7 +15,9 @@ class ClientConfig:
     def from_env(cls) -> "ClientConfig":
         api_key = os.environ.get("HYPERBROWSER_API_KEY")
         if api_key is None:
-            raise ValueError("HYPERBROWSER_API_KEY environment variable is required")
+            raise HyperbrowserError(
+                "HYPERBROWSER_API_KEY environment variable is required"
+            )
 
         base_url = os.environ.get(
             "HYPERBROWSER_BASE_URL", "https://api.hyperbrowser.ai"

tests/test_config.py

@@ -0,0 +1,21 @@
+import pytest
+
+from hyperbrowser.config import ClientConfig
+from hyperbrowser.exceptions import HyperbrowserError
+
+
+def test_client_config_from_env_raises_hyperbrowser_error_without_api_key(monkeypatch):
+    monkeypatch.delenv("HYPERBROWSER_API_KEY", raising=False)
+
+    with pytest.raises(HyperbrowserError, match="HYPERBROWSER_API_KEY"):
+        ClientConfig.from_env()
+
+
+def test_client_config_from_env_reads_api_key_and_base_url(monkeypatch):
+    monkeypatch.setenv("HYPERBROWSER_API_KEY", "test-key")
+    monkeypatch.setenv("HYPERBROWSER_BASE_URL", "https://example.local")
+
+    config = ClientConfig.from_env()
+
+    assert config.api_key == "test-key"
+    assert config.base_url == "https://example.local"