agentserver: snapshot is_hosted in AgentConfig.__init__ instead of reading os.environ dynamically

Agent-Logs-Url: https://github.com/Azure/azure-sdk-for-python/sessions/263ad34e-e39a-4de1-aa0d-e0de484300b5

Co-authored-by: ankitbko <3169316+ankitbko@users.noreply.github.com>

Author: Copilot

sdk/agentserver/azure-ai-agentserver-core/azure/ai/agentserver/core/_config.py

@@ -54,6 +54,8 @@ class AgentConfig:
     :param agent_name: Agent name from ``FOUNDRY_AGENT_NAME``.
     :param agent_version: Agent version from ``FOUNDRY_AGENT_VERSION``.
     :param agent_id: Combined identifier (``"name:version"`` or ``"name"`` or ``""``).
+    :param is_hosted: Whether the agent is running in a Foundry-hosted container environment,
+        derived from ``FOUNDRY_HOSTING_ENVIRONMENT``.
     :param project_endpoint: Foundry project endpoint from ``FOUNDRY_PROJECT_ENDPOINT``.
     :param project_id: Foundry project ARM resource ID from ``FOUNDRY_PROJECT_ARM_ID``.
     :param session_id: Default session ID from ``FOUNDRY_AGENT_SESSION_ID``.
@@ -69,6 +71,7 @@ def __init__(
         agent_name: str,
         agent_version: str,
         agent_id: str,
+        is_hosted: bool,
         project_endpoint: str,
         project_id: str,
         session_id: str,
@@ -80,6 +83,7 @@ def __init__(
         self.agent_name = agent_name
         self.agent_version = agent_version
         self.agent_id = agent_id
+        self.is_hosted = is_hosted
         self.project_endpoint = project_endpoint
         self.project_id = project_id
         self.session_id = session_id
@@ -88,18 +92,6 @@ def __init__(
         self.otlp_endpoint = otlp_endpoint
         self.sse_keepalive_interval = sse_keepalive_interval
 
-    @property
-    def is_hosted(self) -> bool:
-        """Whether the agent is running in a Foundry-hosted container environment.
-
-        Returns ``True`` when the platform-injected ``FOUNDRY_HOSTING_ENVIRONMENT``
-        environment variable exists and is non-empty. This variable is set
-        exclusively by the Foundry platform at container startup.
-
-        :rtype: bool
-        """
-        return bool(os.environ.get(_ENV_FOUNDRY_HOSTING_ENVIRONMENT))
-
     @classmethod
     def from_env(cls) -> Self:
         """Create an ``AgentConfig`` by reading all platform environment variables.
@@ -121,6 +113,7 @@ def from_env(cls) -> Self:
             agent_name=agent_name,
             agent_version=agent_version,
             agent_id=agent_id,
+            is_hosted=bool(os.environ.get(_ENV_FOUNDRY_HOSTING_ENVIRONMENT)),
             project_endpoint=os.environ.get(_ENV_FOUNDRY_PROJECT_ENDPOINT, ""),
             project_id=os.environ.get(_ENV_FOUNDRY_PROJECT_ARM_ID, ""),
             session_id=os.environ.get(_ENV_FOUNDRY_AGENT_SESSION_ID, ""),

sdk/agentserver/azure-ai-agentserver-core/tests/test_config.py

@@ -0,0 +1,39 @@
+# ---------------------------------------------------------
+# Copyright (c) Microsoft Corporation. All rights reserved.
+# ---------------------------------------------------------
+"""Unit tests for AgentConfig."""
+import pytest
+
+from azure.ai.agentserver.core._config import AgentConfig
+
+
+class TestAgentConfigIsHosted:
+    """Tests for AgentConfig.is_hosted snapshotting behavior."""
+
+    def test_is_hosted_false_when_env_var_absent(self, monkeypatch: pytest.MonkeyPatch) -> None:
+        """is_hosted is False when FOUNDRY_HOSTING_ENVIRONMENT is not set."""
+        monkeypatch.delenv("FOUNDRY_HOSTING_ENVIRONMENT", raising=False)
+        config = AgentConfig.from_env()
+        assert config.is_hosted is False
+
+    def test_is_hosted_false_when_env_var_empty(self, monkeypatch: pytest.MonkeyPatch) -> None:
+        """is_hosted is False when FOUNDRY_HOSTING_ENVIRONMENT is set to an empty string."""
+        monkeypatch.setenv("FOUNDRY_HOSTING_ENVIRONMENT", "")
+        config = AgentConfig.from_env()
+        assert config.is_hosted is False
+
+    def test_is_hosted_true_when_env_var_set(self, monkeypatch: pytest.MonkeyPatch) -> None:
+        """is_hosted is True when FOUNDRY_HOSTING_ENVIRONMENT is set to a non-empty value."""
+        monkeypatch.setenv("FOUNDRY_HOSTING_ENVIRONMENT", "production")
+        config = AgentConfig.from_env()
+        assert config.is_hosted is True
+
+    def test_is_hosted_snapshotted_at_creation(self, monkeypatch: pytest.MonkeyPatch) -> None:
+        """is_hosted reflects the env var value at creation time, not at access time."""
+        monkeypatch.setenv("FOUNDRY_HOSTING_ENVIRONMENT", "production")
+        config = AgentConfig.from_env()
+        assert config.is_hosted is True
+
+        # Changing the env var after creation must not affect the already-created config.
+        monkeypatch.delenv("FOUNDRY_HOSTING_ENVIRONMENT")
+        assert config.is_hosted is True