Refactor MCP server for server-side persistence and API-backed model #195

Merged

rwachters merged 3 commits from migrate-app-side-LocalMCPServer into change-mcp-tool-call-architecture

2026-04-18 14:54:56 +00:00

rwachters commented

2026-04-18 14:54:48 +00:00

(Migrated from github.com)

Overview

This pull request includes the following changes:

Test Stability:
- Introduced a timeout loop in tests to ensure worker session readiness, reducing potential flakiness.
Local Persistence Removal:
- Removed the local database persistence layer, including all SQLDelight schemas, DAOs, services, and platform-specific implementations.
- Cleaned up related Koin modules on Android, Desktop, and WASM platforms.
Server-Side Persistence Migration:
- Shifted MCP server configuration to server-side management, replacing client-side persistence.
- Established an API-backed model using LocalMCPServerDto for CRUD operations.
- Updated all relevant UI components, services, and viewmodels to align with the new data model.
- Enhanced configuration to introduce workerId and explicit handling of secretEnvironmentVariables.

Checklist

Code/Logic Change

Implementation aligns with contributed feature request or bug report.
Code is modular, well-documented, and adheres to coding standards.

Testing

Unit and integration tests have been added/updated.
Changes have been tested in supported environments.

Documentation

Related documentation has been updated (e.g., README, design documents).
API changes are reflected in relevant references.

Deployment

No deployment blockers identified.

If you have any feedback or modifications for this PR, please let me know!

### Overview This pull request includes the following changes: 1. **Test Stability**: - Introduced a timeout loop in tests to ensure worker session readiness, reducing potential flakiness. 2. **Local Persistence Removal**: - Removed the local database persistence layer, including all SQLDelight schemas, DAOs, services, and platform-specific implementations. - Cleaned up related Koin modules on Android, Desktop, and WASM platforms. 3. **Server-Side Persistence Migration**: - Shifted MCP server configuration to server-side management, replacing client-side persistence. - Established an API-backed model using `LocalMCPServerDto` for CRUD operations. - Updated all relevant UI components, services, and viewmodels to align with the new data model. - Enhanced configuration to introduce `workerId` and explicit handling of `secretEnvironmentVariables`. ### Checklist #### Code/Logic Change - [ ] Implementation aligns with contributed feature request or bug report. - [ ] Code is modular, well-documented, and adheres to coding standards. #### Testing - [ ] Unit and integration tests have been added/updated. - [ ] Changes have been tested in supported environments. #### Documentation - [ ] Related documentation has been updated (e.g., README, design documents). - [ ] API changes are reflected in relevant references. #### Deployment - [ ] No deployment blockers identified. --- If you have any feedback or modifications for this PR, please let me know!