fix: preserve unmanaged MCode MCP servers during sync

Author: SaladDay

src-tauri/src/services/mcp.rs

@@ -243,9 +243,11 @@ impl McpService {
         for server in servers.values() {
             if server.apps.is_enabled_for(app) {
                 Self::sync_server_to_app(state, server, app)?;
-            } else {
+            } else if !matches!(app, AppType::Mcode) {
                 Self::remove_server_from_app(state, &server.id, app)?;
             }
+            // MCode's false flag also covers pre-existing, unmanaged servers.
+            // Only explicit disable/delete operations may remove those entries.
         }
 
         Ok(())

src-tauri/tests/mcp_commands.rs

@@ -15,6 +15,41 @@ use support::{
     create_test_state, create_test_state_with_config, ensure_test_home, reset_test_fs, test_mutex,
 };
 
+#[test]
+fn mcode_automatic_sync_preserves_unmanaged_same_name_servers() {
+    let _guard = test_mutex().lock().unwrap();
+    reset_test_fs();
+    let state = create_test_state().unwrap();
+    let path = ensure_test_home().join(".minimax/mcp.json");
+    fs::create_dir_all(path.parent().unwrap()).unwrap();
+    let native = json!({"mcpServers":{"context7":{"command":"native-server","enabled":true}}});
+    fs::write(&path, native.to_string()).unwrap();
+    let server = McpServer {
+        id: "context7".into(),
+        name: "Context7".into(),
+        server: json!({"command":"managed-server"}),
+        apps: McpApps {
+            claude: true,
+            ..Default::default()
+        },
+        description: None,
+        homepage: None,
+        docs: None,
+        tags: vec![],
+    };
+    state.db.save_mcp_server(&server).unwrap();
+    McpService::sync_enabled_for_app(&state, &AppType::Mcode).unwrap();
+    assert_eq!(
+        serde_json::from_str::<serde_json::Value>(&fs::read_to_string(&path).unwrap()).unwrap(),
+        native
+    );
+    McpService::toggle_app(&state, "context7", AppType::Mcode, true).unwrap();
+    McpService::toggle_app(&state, "context7", AppType::Mcode, false).unwrap();
+    let disabled: serde_json::Value =
+        serde_json::from_str(&fs::read_to_string(&path).unwrap()).unwrap();
+    assert!(disabled["mcpServers"].get("context7").is_none());
+}
+
 #[test]
 fn import_default_config_claude_persists_provider() {
     let _guard = test_mutex().lock().expect("acquire test mutex");