Normalize and validate client config values

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

Author: cursoragent

hyperbrowser/config.py

@@ -13,6 +13,14 @@ class ClientConfig:
     base_url: str = "https://api.hyperbrowser.ai"
     headers: Optional[Dict[str, str]] = None
 
+    def __post_init__(self) -> None:
+        if not isinstance(self.api_key, str):
+            raise HyperbrowserError("api_key must be a string")
+        if not isinstance(self.base_url, str):
+            raise HyperbrowserError("base_url must be a string")
+        self.api_key = self.api_key.strip()
+        self.base_url = self.base_url.strip().rstrip("/")
+
     @classmethod
     def from_env(cls) -> "ClientConfig":
         api_key = os.environ.get("HYPERBROWSER_API_KEY")

tests/test_config.py

@@ -26,3 +26,18 @@ def test_client_config_from_env_reads_api_key_and_base_url(monkeypatch):
 
     assert config.api_key == "test-key"
     assert config.base_url == "https://example.local"
+
+
+def test_client_config_normalizes_whitespace_and_trailing_slash():
+    config = ClientConfig(api_key="  test-key  ", base_url=" https://example.local/ ")
+
+    assert config.api_key == "test-key"
+    assert config.base_url == "https://example.local"
+
+
+def test_client_config_rejects_non_string_values():
+    with pytest.raises(HyperbrowserError, match="api_key must be a string"):
+        ClientConfig(api_key=None)  # type: ignore[arg-type]
+
+    with pytest.raises(HyperbrowserError, match="base_url must be a string"):
+        ClientConfig(api_key="test-key", base_url=None)  # type: ignore[arg-type]