Remove cross-platform references from comments and docstrings

- mcp_tool_server_configuration_service.py: replace 'mirrors Node.js behaviour'
  and 'aligns with Node.js' with self-contained descriptions
- utility.py: rephrase is_development_environment() docstring to describe env var
  precedence without mentioning .NET (ASPNETCORE_ENVIRONMENT and DOTNET_ENVIRONMENT
  variable names are kept, only the platform labels are removed)
- test_mcp_server_configuration.py: remove reference to non-existent
  MCP_PLATFORM_APP_ID env var in test section header and docstring; replace with
  accurate guidance pointing to MCP_PLATFORM_AUTHENTICATION_SCOPE /
  MCP_PLATFORM_ENDPOINT as the actual configurable inputs

Author: biswapm

libraries/microsoft-agents-a365-tooling/microsoft_agents_a365/tooling/services/mcp_tool_server_configuration_service.py

@@ -170,8 +170,8 @@ async def list_tool_servers(
             else:
                 # Legacy call without auth context — guard against V2 servers.
                 # V2 servers require per-audience OBO exchange; returning them without a
-                # token would cause silent 401s downstream. Raise early with a clear message
-                # so callers get an actionable migration hint (mirrors Node.js behaviour).
+                # token would cause silent 401s downstream. Raise early with a clear
+                # migration hint so callers know which parameters to add.
                 v2_servers = [s for s in servers if self._is_v2_server(s)]
                 if v2_servers:
                     names = ", ".join(
@@ -226,7 +226,7 @@ def _create_dev_token_acquirer(self) -> TokenAcquirer:
         The CLI (``a365 develop get-token``) writes tokens to the environment before the agent
         starts. Resolution order per server:
 
-        1. ``BEARER_TOKEN_<MCP_SERVER_NAME_UPPER>`` — per-server token (aligns with Node.js)
+        1. ``BEARER_TOKEN_<MCP_SERVER_NAME_UPPER>`` — per-server token
         2. ``BEARER_TOKEN`` — shared fallback token
 
         Tokens are returned **without** a ``Bearer `` prefix. If the env var already contains

libraries/microsoft-agents-a365-tooling/microsoft_agents_a365/tooling/utils/utility.py

@@ -75,13 +75,13 @@ def is_development_environment() -> bool:
     Resolution order (first non-empty value wins):
     1. ``PYTHON_ENVIRONMENT``     — explicit Python SDK variable used in current samples.
     2. ``ENVIRONMENT``            — legacy Python SDK variable (backward compatibility).
-    3. ``ASPNETCORE_ENVIRONMENT`` — .NET / Azure hosting convention.
-    4. ``DOTNET_ENVIRONMENT``     — .NET generic-host convention.
+    3. ``ASPNETCORE_ENVIRONMENT`` — Azure hosting convention.
+    4. ``DOTNET_ENVIRONMENT``     — generic-host convention.
     5. Defaults to ``"Development"`` when none of the above are set.
 
-    ``PYTHON_ENVIRONMENT`` and ``ENVIRONMENT`` are checked before the .NET variables
-    so legacy Python agents that set ``ENVIRONMENT=Production`` are not affected if an
-    unrelated process also sets ``ASPNETCORE_ENVIRONMENT`` (e.g. a .NET sidecar).
+    ``PYTHON_ENVIRONMENT`` and ``ENVIRONMENT`` are checked first so that agents
+    which explicitly set ``ENVIRONMENT=Production`` are not affected if a host
+    process also sets ``ASPNETCORE_ENVIRONMENT``.
 
     Returns:
         bool: True when the resolved environment is "development" (case-insensitive).

tests/tooling/test_mcp_server_configuration.py

@@ -288,20 +288,19 @@ def test_v1_atg_audience_in_uri_form_falls_back_to_atg_scope(self):
         assert resolve_token_scope_for_server(server) == f"{self.PROD_ATG_APP_ID}/.default"
 
     # ------------------------------------------------------------------
-    # V1 test-env — ATG_APP_ID overridden via MCP_PLATFORM_APP_ID
+    # V1 test-env — non-prod ATG audience falls through to V2 scope resolution
     # ------------------------------------------------------------------
 
     def test_v1_test_env_shared_audience_not_treated_as_v2(self):
-        """V1 test env: test audience GUID is a different GUID from prod ATG — treated as V2
-        unless the caller configures ATG_APP_ID to match. This test documents that the SDK
-        uses the prod ATG_APP_ID constant; test environments must set MCP_PLATFORM_APP_ID
-        via the CLI / .env so gateway returns the correct shared audience in discovery."""
+        """Non-prod ATG audience GUID is different from the hardcoded prod ATG_APP_ID constant,
+        so it is classified as V2 and resolved to its own /.default scope. This is intentional:
+        the V2 token exchange works correctly for test app registrations because
+        Tools.ListInvoke.All is pre-consented. Use MCP_PLATFORM_AUTHENTICATION_SCOPE or
+        MCP_PLATFORM_ENDPOINT env vars to point the SDK at a non-prod gateway."""
         from microsoft_agents_a365.tooling.utils.utility import resolve_token_scope_for_server
 
-        # test env audience is a different GUID — with prod ATG_APP_ID hardcoded,
-        # it is treated as V2 (resolved to its own /.default scope).
-        # This is intentional: V2 logic handles it correctly since Tools.ListInvoke.All
-        # is pre-consented on the test app registration.
+        # Non-prod ATG audience — classified as V2 because it does not match
+        # the hardcoded prod ATG_APP_ID constant. Resolved to its own /.default scope.
         server = self._make_server(audience=self.TEST_ATG_APP_ID, scope=None)
         assert resolve_token_scope_for_server(server) == f"{self.TEST_ATG_APP_ID}/.default"