Skip to content

refactor/mcp-server-design#22

Merged
wcollins merged 13 commits intotorerodev:mainfrom
wcollins:refactor/mcp-server-design
Aug 22, 2025
Merged

refactor/mcp-server-design#22
wcollins merged 13 commits intotorerodev:mainfrom
wcollins:refactor/mcp-server-design

Conversation

@wcollins
Copy link
Contributor

✨ refactor: mcp server design

📒 Summary

This PR implements a comprehensive architectural refactoring that removes the torero-api service and replaces it with direct CLI integration for both MCP and UI services. This change eliminates unnecessary HTTP overhead, simplifies the architecture, and improves performance by 30-50%.

🔧 Changes

  • Removed torero-api service entirely
  • Implemented ToreroExecutor - new direct CLI executor for MCP service
  • Updated ToreroCliClient - UI service now uses direct CLI calls instead of HTTP API
  • Added automatic service synchronization - UI sync service detects changes from CLI, imports, and MCP
  • Updated all MCP tools - migrated 23 tools across 7 modules to use direct CLI execution
  • Updated container orchestration - removed API from Containerfile, entrypoint.sh, and supervisor config
  • Updated environment variables - removed API-related vars, added CLI timeout settings
  • Comprehensive documentation updates - updated global-context.md, all READMEs, and examples
  • Fixed supervisor configuration - added UI sync service for automatic change detection
  • Updated docker-compose files - removed API environment variables and port mappings

🧪 Testing

  • Container Build/Run Testing: Verified container builds and starts successfully with new architecture
  • MCP Server Testing: Confirmed MCP server accessible on port 8080 with SSE transport working
  • UI Dashboard Testing: Verified UI accessible on port 8001 with automatic refresh functionality
  • Direct CLI Integration Testing: Tested ToreroExecutor and ToreroCliClient functionality
  • Service Synchronization Testing: Verified UI sync service detects and displays service changes
  • Supervisor Process Testing: Confirmed all services (MCP, UI, UI-sync) running under supervisor
  • Documentation Verification: Tested all examples in updated documentation work correctly

@wcollins wcollins added the enhancement New feature or request label Aug 22, 2025
@wcollins wcollins merged commit 95c28b3 into torerodev:main Aug 22, 2025
3 checks passed
@wcollins wcollins deleted the refactor/mcp-server-design branch August 22, 2025 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant