github/cc-switch
1
0
Fork 0
mirror of https://github.com/farion1231/cc-switch.git synced 2026-05-23 22:21:00 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(codex): remove entire model_providers table from common config extraction

Browse Source 
Previously only removed base_url from model_providers.* tables. Now removes
the entire model_providers section since all its fields (name, base_url,
wire_api, requires_openai_auth) are provider-specific configuration.

MCP servers configuration remains preserved as it's provider-agnostic.
This commit is contained in:
Jason
2026-01-04 10:07:01 +08:00
parent 63bb673bf2
commit 2c35372ca0
1 changed files with 4 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src-tauri/src/services/provider/mod.rs
+4 -12
View File
@@ -104,8 +104,8 @@ base_url = "http://localhost:8080"
"should remove top-level model"
);
assert!(
!extracted.contains("https://azure.example"),
"should remove model_providers.* base_url"
!extracted.contains("[model_providers"),
"should remove entire model_providers table"
);
assert!(
extracted.contains("http://localhost:8080"),
@@ -463,16 +463,8 @@ impl ProviderService {
// Legacy/alt formats might use a top-level base_url.
root.remove("base_url");
// Remove `base_url` only from `[model_providers.*]` tables (do not touch MCP servers).
if let Some(model_providers) = root.get_mut("model_providers") {
if let Some(model_providers_table) = model_providers.as_table_mut() {
for (_, provider_item) in model_providers_table.iter_mut() {
if let Some(provider_table) = provider_item.as_table_mut() {
provider_table.remove("base_url");
}
}
}
}
// Remove entire model_providers table (provider-specific configuration)
root.remove("model_providers");
// Clean up multiple empty lines (keep at most one blank line).
let mut cleaned = String::new();