-
Notifications
You must be signed in to change notification settings - Fork 2.3k
fix: migrate new mcp server config #6651
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR implements migration logic for MCP (Model Control Protocol) server configurations to address issue #6646. The migration ensures that users' MCP configuration files are updated with new server configurations when the application is updated.
- Introduces version tracking and migration system using a store to track app and MCP schema versions
- Adds MCP server configuration migration that automatically adds new "exa" server when upgrading from MCP schema version < 1
- Refactors extension installation logic to integrate with the new version-based migration system
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| src-tauri/src/lib.rs | Adds migration orchestration logic with version tracking and calls to migrate extensions and MCP servers |
| src-tauri/src/core/setup.rs | Refactors extension installation and adds new migrate_mcp_servers function for MCP configuration migration |
| src-tauri/src/core/mcp/helpers.rs | Implements helper functions for adding server configurations to MCP config files |
| src-tauri/src/core/mcp/tests.rs | Adds comprehensive test coverage for the new server configuration functionality |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Barecheck - Code coverage reportTotal: 29.67%Your code coverage diff: 0.00% ▴ ✅ All code changes are covered |
Co-authored-by: Copilot <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.



Describe Your Changes
This PR fixes issue #6646 by applying the new MCP server to users' MCP configuration files.
Introduce MCP configuration migration.
Fixes Issues
Self Checklist