Issue
The file archon-ui-main/src/services/mcpService.ts appears to be unused in the codebase and duplicates functionality that exists in other services.
Current State
- Not imported anywhere - No components or other services use this file
- Duplicates functionality that is properly separated in:
mcpServerService.ts - Manages Archon's own MCP server
mcpClientService.ts - Handles connections to external MCP servers as a client
- Mixed responsibilities - Contains both server management and client connection methods in one file
Recommendation
Remove mcpService.ts to reduce confusion and maintenance burden. The properly separated services are actively used and provide clearer separation of concerns.
Impact
- No functional impact as the file is not imported anywhere
- Reduces codebase complexity
- Eliminates confusion about which service to use for MCP functionality
This appears to be a legacy file from before the MCP functionality was properly separated into distinct server and client services.
Issue
The file
archon-ui-main/src/services/mcpService.tsappears to be unused in the codebase and duplicates functionality that exists in other services.Current State
mcpServerService.ts- Manages Archon's own MCP servermcpClientService.ts- Handles connections to external MCP servers as a clientRecommendation
Remove
mcpService.tsto reduce confusion and maintenance burden. The properly separated services are actively used and provide clearer separation of concerns.Impact
This appears to be a legacy file from before the MCP functionality was properly separated into distinct server and client services.