Skip to content

Commit 1289d10

Browse files
Fix init for sk
1 parent 0082a80 commit 1289d10

1 file changed

Lines changed: 2 additions & 12 deletions

File tree

  • libraries/microsoft-agents-a365-tooling-extensions-semantickernel/microsoft_agents_a365/tooling/extensions/semantickernel/services

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

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -41,20 +41,15 @@ class McpToolRegistrationService:
4141
def __init__(
4242
self,
4343
logger: Optional[logging.Logger] = None,
44-
service_provider: Optional[Any] = None,
45-
mcp_server_configuration_service: Optional[McpToolServerConfigurationService] = None,
4644
):
4745
"""
48-
Initialize the MCP Tool Registration Service.
46+
Initialize the MCP Tool Registration Service for Semantic Kernel.
4947
5048
Args:
5149
logger: Logger instance for logging operations.
52-
service_provider: Service provider for dependency injection.
53-
mcp_server_configuration_service: Service for MCP server configuration.
5450
"""
5551
self._logger = logger or logging.getLogger(self.__class__.__name__)
56-
self._service_provider = service_provider
57-
self._mcp_server_configuration_service = mcp_server_configuration_service
52+
self._mcp_server_configuration_service = McpToolServerConfigurationService(logger=self._logger)
5853

5954
# Store connected plugins to keep them alive
6055
self._connected_plugins = []
@@ -110,11 +105,6 @@ async def add_tool_servers_to_agent(
110105

111106
self._validate_inputs(kernel, agent_user_id, environment_id, auth_token)
112107

113-
if self._mcp_server_configuration_service is None:
114-
raise ValueError(
115-
"MCP server configuration service is required but was not provided during initialization"
116-
)
117-
118108
# Get and process servers
119109
servers = await self._mcp_server_configuration_service.list_tool_servers(
120110
agent_user_id, environment_id, auth_token

0 commit comments

Comments
 (0)