Fix CI import failures and missing is_dev guard

agent_framework.openai causes a circular-import chain under Python 3.12
on Linux: openai.__getattr__ lazily imports agent_framework_openai, which
eventually does `from . import __version__` on the still-initialising
agent_framework package.  Move OpenAIChatClient under TYPE_CHECKING so the
import is never executed at runtime.

Also add the missing is_development_environment() guard to the Azure AI
Foundry extension's add_tool_servers_to_agent, matching the pattern used
by the other three extensions: dev mode skips token exchange and uses ""
as the agentic_app_id for manifest-based discovery.

Author: biswapm

libraries/microsoft-agents-a365-tooling-extensions-agentframework/microsoft_agents_a365/tooling/extensions/agentframework/services/mcp_tool_registration_service.py

@@ -1,15 +1,19 @@
 # Copyright (c) Microsoft Corporation.
 # Licensed under the MIT License.
 
+from __future__ import annotations
+
 import logging
 import uuid
 from datetime import datetime, timezone
-from typing import List, Optional, Sequence
+from typing import TYPE_CHECKING, List, Optional, Sequence
 
 from agent_framework import RawAgent, Message, HistoryProvider, MCPStreamableHTTPTool
-from agent_framework.openai import OpenAIChatClient
 import httpx
 
+if TYPE_CHECKING:
+    from agent_framework.openai import OpenAIChatClient
+
 from microsoft_agents.hosting.core import Authorization, TurnContext
 
 from microsoft_agents_a365.runtime import OperationResult

libraries/microsoft-agents-a365-tooling-extensions-azureaifoundry/microsoft_agents_a365/tooling/extensions/azureaifoundry/services/mcp_tool_registration_service.py

@@ -27,7 +27,10 @@
     McpToolServerConfigurationService,
 )
 from microsoft_agents_a365.tooling.utils.constants import Constants
-from microsoft_agents_a365.tooling.utils.utility import get_mcp_platform_authentication_scope
+from microsoft_agents_a365.tooling.utils.utility import (
+    get_mcp_platform_authentication_scope,
+    is_development_environment,
+)
 
 
 class McpToolRegistrationService:
@@ -99,13 +102,14 @@ async def add_tool_servers_to_agent(
         if project_client is None:
             raise ValueError("project_client cannot be None")
 
-        if not auth_token:
+        is_dev = is_development_environment()
+        if not auth_token and not is_dev:
             scopes = get_mcp_platform_authentication_scope()
             authToken = await auth.exchange_token(context, scopes, auth_handler_name)
             auth_token = authToken.token
 
         try:
-            agentic_app_id = Utility.resolve_agent_identity(context, auth_token)
+            agentic_app_id = "" if is_dev else Utility.resolve_agent_identity(context, auth_token)
             # Get the tool definitions and resources — pass auth context so each server receives
             # its own per-audience Authorization token (V1 = shared ATG, V2 = per-GUID).
             tool_definitions, tool_resources = await self._get_mcp_tool_definitions_and_resources(

tests/tooling/extensions/azureaifoundry/services/test_mcp_tool_registration_service.py

@@ -115,6 +115,7 @@ async def test_list_tool_servers_receives_auth_context(
 
         with (
             patch(f"{_MODULE}.McpTool", return_value=mock_mcp_tool),
+            patch(f"{_MODULE}.is_development_environment", return_value=False),
             patch(f"{_MODULE}.Utility.resolve_agent_identity", return_value="test-aai"),
             patch(f"{_MODULE}.Utility.get_user_agent_header", return_value="AzureAIFoundry/1.0"),
         ):