veris-ai 1.14.0__tar.gz → 1.14.1__tar.gz

{veris_ai-1.14.0 → veris_ai-1.14.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: veris-ai
-Version: 1.14.0
+Version: 1.14.1
 Summary: A Python package for Veris AI tools
 Project-URL: Homepage, https://github.com/veris-ai/veris-python-sdk
 Project-URL: Bug Tracker, https://github.com/veris-ai/veris-python-sdk/issues

{veris_ai-1.14.0 → veris_ai-1.14.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "veris-ai"
-version = "1.14.0"
+version = "1.14.1"
 description = "A Python package for Veris AI tools"
 readme = "README.md"
 requires-python = ">=3.11"

{veris_ai-1.14.0 → veris_ai-1.14.1}/src/veris_ai/api_client.py

@@ -20,9 +20,15 @@ class SimulatorAPIClient:
         changes reflected without recreating the singleton.
     """
 
-    def __init__(self) -> None:
+    def __init__(self, timeout: float | None = None, base_url: str | None = None) -> None:
         """Initialize the API client with static timeout configuration."""
-        self.timeout = float(os.getenv("VERIS_MOCK_TIMEOUT", "90.0"))
+        self._timeout = timeout or float(os.getenv("VERIS_MOCK_TIMEOUT", "300.0"))
+        self._base_url = base_url
+
+    @property
+    def base_url(self) -> str:
+        """Get the resolved base URL."""
+        return self._get_base_url()
 
     def _get_base_url(self) -> str:
         """Resolve the base URL from environment.
@@ -33,7 +39,7 @@ class SimulatorAPIClient:
               (do not fall back). This supports tests expecting connection
               failures when an invalid endpoint is provided.
         """
-        return os.getenv("VERIS_API_URL") or "https://simulator.api.veris.ai"
+        return self._base_url or os.getenv("VERIS_API_URL") or "https://simulator.api.veris.ai"
 
     def _build_headers(self) -> dict[str, str] | None:
         """Build headers including OpenTelemetry tracing and API key."""
@@ -55,7 +61,7 @@ class SimulatorAPIClient:
         if not endpoint.startswith(("http://", "https://")):
             raise httpx.ConnectError("Invalid endpoint URL (not absolute): {endpoint}")
 
-        with httpx.Client(timeout=self.timeout) as client:
+        with httpx.Client(timeout=self._timeout) as client:
             response = client.post(endpoint, json=payload, headers=headers)
             response.raise_for_status()
             return response.json() if response.content else None
@@ -73,7 +79,7 @@ class SimulatorAPIClient:
             error_msg = f"Invalid endpoint URL (not absolute): {endpoint}"
             raise httpx.ConnectError(error_msg)
 
-        async with httpx.AsyncClient(timeout=self.timeout) as client:
+        async with httpx.AsyncClient(timeout=self._timeout) as client:
             response = await client.post(endpoint, json=payload, headers=headers)
             response.raise_for_status()
             return response.json() if response.content else None
@@ -81,15 +87,15 @@ class SimulatorAPIClient:
     @property
     def tool_mock_endpoint(self) -> str:
         """Get the tool mock endpoint URL."""
-        return urljoin(self._get_base_url(), "v3/tool_mock")
+        return urljoin(self.base_url, "v3/tool_mock")
 
     def get_log_tool_call_endpoint(self, session_id: str) -> str:
         """Get the log tool call endpoint URL."""
-        return urljoin(self._get_base_url(), f"v3/log_tool_call?session_id={session_id}")
+        return urljoin(self.base_url, f"v3/log_tool_call?session_id={session_id}")
 
     def get_log_tool_response_endpoint(self, session_id: str) -> str:
         """Get the log tool response endpoint URL."""
-        return urljoin(self._get_base_url(), f"v3/log_tool_response?session_id={session_id}")
+        return urljoin(self.base_url, f"v3/log_tool_response?session_id={session_id}")
 
 
 # Global singleton instance
@@ -99,3 +105,9 @@ _api_client = SimulatorAPIClient()
 def get_api_client() -> SimulatorAPIClient:
     """Get the global API client instance."""
     return _api_client
+
+
+def set_api_client_params(base_url: str | None = None, timeout: float | None = None) -> None:
+    """Set the global API client instance for testing purposes."""
+    global _api_client  # noqa: PLW0603
+    _api_client = SimulatorAPIClient(base_url=base_url, timeout=timeout)

{veris_ai-1.14.0 → veris_ai-1.14.1}/src/veris_ai/tool_mock.py

@@ -16,7 +16,7 @@ from typing import (
 
 
 from veris_ai.models import ResponseExpectation, ToolCallOptions
-from veris_ai.api_client import get_api_client
+from veris_ai.api_client import get_api_client, set_api_client_params
 from veris_ai.utils import (
     convert_to_type,
     execute_callback,
@@ -92,6 +92,8 @@ class VerisSDK:
 
             self._set_session_id(token_data["session_id"])
             self._set_thread_id(token_data["thread_id"])
+            if token_data.get("api_url"):
+                set_api_client_params(base_url=token_data["api_url"])
             logger.info(
                 f"Session ID set to {token_data['session_id']}, "
                 f"Thread ID set to {token_data['thread_id']} - mocking enabled"

veris_ai-1.14.1/tests/test_api_client.py

@@ -0,0 +1,140 @@
+"""Tests for SimulatorAPIClient endpoint URL generation and configuration."""
+
+import os
+from unittest.mock import patch
+
+import pytest
+
+from veris_ai.api_client import SimulatorAPIClient, get_api_client, set_api_client_params
+
+
+def test_tool_mock_endpoint_default():
+    """Test that tool_mock_endpoint property returns correct URL with default base_url."""
+    client = SimulatorAPIClient()
+    endpoint = client.tool_mock_endpoint
+
+    # Should use default base URL
+    assert endpoint == "https://simulator.api.veris.ai/v3/tool_mock"
+
+
+def test_tool_mock_endpoint_custom_base_url():
+    """Test that tool_mock_endpoint uses cached base_url from constructor."""
+    client = SimulatorAPIClient(base_url="https://custom.api.com")
+    endpoint = client.tool_mock_endpoint
+
+    assert endpoint == "https://custom.api.com/v3/tool_mock"
+
+
+def test_tool_mock_endpoint_with_env_var():
+    """Test that tool_mock_endpoint respects VERIS_API_URL environment variable."""
+    with patch.dict(os.environ, {"VERIS_API_URL": "https://test.api.veris.ai"}):
+        client = SimulatorAPIClient()
+        endpoint = client.tool_mock_endpoint
+
+        assert endpoint == "https://test.api.veris.ai/v3/tool_mock"
+
+
+def test_log_tool_call_endpoint():
+    """Test get_log_tool_call_endpoint generates correct URL."""
+    client = SimulatorAPIClient(base_url="https://test.api.com")
+    endpoint = client.get_log_tool_call_endpoint("session-123")
+
+    assert endpoint == "https://test.api.com/v3/log_tool_call?session_id=session-123"
+
+
+def test_log_tool_response_endpoint():
+    """Test get_log_tool_response_endpoint generates correct URL."""
+    client = SimulatorAPIClient(base_url="https://test.api.com")
+    endpoint = client.get_log_tool_response_endpoint("session-456")
+
+    assert endpoint == "https://test.api.com/v3/log_tool_response?session_id=session-456"
+
+
+def test_base_url_without_protocol_behavior():
+    """Test that base_url without protocol is treated as relative path by urljoin."""
+    # Note: urljoin treats URLs without protocol as relative paths
+    client = SimulatorAPIClient(base_url="api.example.com")
+    endpoint = client.tool_mock_endpoint
+
+    # urljoin treats "api.example.com" as a relative path, so result is just the path component
+    assert endpoint == "v3/tool_mock"
+
+
+def test_base_url_with_trailing_slash():
+    """Test that base_url with trailing slash is handled correctly."""
+    client = SimulatorAPIClient(base_url="https://test.api.com/")
+    endpoint = client.tool_mock_endpoint
+
+    # urljoin should handle trailing slash correctly
+    assert endpoint == "https://test.api.com/v3/tool_mock"
+
+
+def test_custom_timeout():
+    """Test that custom timeout is set correctly."""
+    client = SimulatorAPIClient(timeout=30.0)
+    assert client._timeout == 30.0
+
+
+def test_default_timeout():
+    """Test that default timeout is used when not specified."""
+    with patch.dict(os.environ, {"VERIS_MOCK_TIMEOUT": "120.0"}):
+        client = SimulatorAPIClient()
+        assert client._timeout == 120.0
+
+
+def test_set_api_client_params():
+    """Test that set_api_client_params reconfigures the global client."""
+    # Get original client
+    original_client = get_api_client()
+    original_endpoint = original_client.tool_mock_endpoint
+
+    # Reconfigure with custom parameters
+    set_api_client_params(base_url="https://custom.test.api", timeout=60.0)
+
+    # Get new client and verify it's been updated
+    new_client = get_api_client()
+    assert new_client.tool_mock_endpoint == "https://custom.test.api/v3/tool_mock"
+    assert new_client._timeout == 60.0
+
+    # Restore original client
+    set_api_client_params()
+
+
+def test_set_api_client_params_partial():
+    """Test that set_api_client_params can set only base_url or timeout."""
+    # Set only base_url
+    set_api_client_params(base_url="https://partial.test")
+    client = get_api_client()
+    assert client.tool_mock_endpoint == "https://partial.test/v3/tool_mock"
+    assert client._timeout == 300  # Should use default
+
+    # Set only timeout
+    set_api_client_params(timeout=45.0)
+    client = get_api_client()
+    assert client._timeout == 45.0
+
+    # Restore defaults
+    set_api_client_params()
+
+
+def test_multiple_endpoint_calls_use_cached_base_url():
+    """Test that multiple endpoint property accesses use the same cached base_url."""
+    client = SimulatorAPIClient(base_url="https://cached.api")
+
+    # Call multiple endpoint methods
+    endpoint1 = client.tool_mock_endpoint
+    endpoint2 = client.get_log_tool_call_endpoint("session-1")
+    endpoint3 = client.get_log_tool_response_endpoint("session-2")
+
+    # All should use the same base URL
+    assert endpoint1.startswith("https://cached.api/")
+    assert endpoint2.startswith("https://cached.api/")
+    assert endpoint3.startswith("https://cached.api/")
+
+
+def test_empty_env_var_uses_default():
+    """Test that empty VERIS_API_URL falls back to default."""
+    with patch.dict(os.environ, {"VERIS_API_URL": ""}, clear=True):
+        client = SimulatorAPIClient()
+        # Empty string should NOT be used, should fall back to default
+        assert client._get_base_url() == "https://simulator.api.veris.ai"

{veris_ai-1.14.0 → veris_ai-1.14.1}/uv.lock

@@ -1571,7 +1571,7 @@ wheels = [
 
 [[package]]
 name = "veris-ai"
-version = "1.13.0"
+version = "1.14.0"
 source = { editable = "." }
 dependencies = [
     { name = "httpx" },