Watch

Remove local MCP tool execution logic and migrate to server worker #200

Merged

rwachters merged 6 commits from move-local-mcp-tool-execution-to-server-worker into change-mcp-tool-call-architecture

2026-04-23 19:29:17 +00:00

rwachters commented

2026-04-23 19:28:43 +00:00

(Migrated from github.com)

Description

This pull request removes the deprecated local MCP server API resources and execution logic, centralizing the functionality on the server worker. Key changes include:

API Resource Cleanup: Deprecated resources (e.g., Create, Ids, SetEnabled) were removed from the LocalMCPServerResource.
Execution Logic Removal: Local server execution logic, including the LocalMCPServerProcessManager, MCPClientService, and related models, was deleted.
Server Worker Migration: Transferred local MCP runtime operations to a server-worker model, introducing new server-side APIs and worker communication protocols.
Tool Execution Update: Local MCP tool execution logic is now handled entirely server-side using a new LocalMCPToolCallDispatchService and LocalMCPExecutor.
Error Handling Improvements: Simplified error handling, moving from client-side mediation to server-reported errors.
Dependency Updates: Updated Koin modules and API contracts to reflect the new architecture.
New Features: Added a "draft connection test" flow to validate unsaved configurations and comprehensive unit tests for the new services.

These refactors improve maintainability, scalability, and centralization for MCP tool execution and runtime control.

Checklist

Unit tests added/updated where applicable
Documentation updated for affected APIs
Error handling verified
Dependencies reviewed and updated if needed

### Description This pull request removes the deprecated local MCP server API resources and execution logic, centralizing the functionality on the server worker. Key changes include: - **API Resource Cleanup**: Deprecated resources (e.g., `Create`, `Ids`, `SetEnabled`) were removed from the `LocalMCPServerResource`. - **Execution Logic Removal**: Local server execution logic, including the `LocalMCPServerProcessManager`, `MCPClientService`, and related models, was deleted. - **Server Worker Migration**: Transferred local MCP runtime operations to a server-worker model, introducing new server-side APIs and worker communication protocols. - **Tool Execution Update**: Local MCP tool execution logic is now handled entirely server-side using a new `LocalMCPToolCallDispatchService` and `LocalMCPExecutor`. - **Error Handling Improvements**: Simplified error handling, moving from client-side mediation to server-reported errors. - **Dependency Updates**: Updated Koin modules and API contracts to reflect the new architecture. - **New Features**: Added a "draft connection test" flow to validate unsaved configurations and comprehensive unit tests for the new services. These refactors improve maintainability, scalability, and centralization for MCP tool execution and runtime control. ### Checklist - [ ] Unit tests added/updated where applicable - [ ] Documentation updated for affected APIs - [ ] Error handling verified - [ ] Dependencies reviewed and updated if needed

Rows
Columns