fix(config): accept provider-prefixed DeepSeek model IDs (#794)
Allow model IDs like accounts/fireworks/models/deepseek-v4-flash by checking for /deepseek in addition to starts_with(deepseek). Fixes #753 Signed-off-by: wucm667 <stevenwucongmin@gmail.com>
This commit is contained in:
@@ -237,7 +237,7 @@ pub fn normalize_model_name(model: &str) -> Option<String> {
|
||||
}
|
||||
|
||||
let normalized = trimmed.to_ascii_lowercase();
|
||||
if !normalized.starts_with("deepseek") {
|
||||
if !normalized.starts_with("deepseek") && !normalized.contains("/deepseek") {
|
||||
return None;
|
||||
}
|
||||
|
||||
@@ -3438,6 +3438,18 @@ api_key = "old-openrouter-key"
|
||||
assert!(normalize_model_name("").is_none());
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn normalize_model_name_accepts_provider_prefixed_deepseek_ids() {
|
||||
assert_eq!(
|
||||
normalize_model_name("accounts/fireworks/models/deepseek-v4-flash").as_deref(),
|
||||
Some("accounts/fireworks/models/deepseek-v4-flash")
|
||||
);
|
||||
assert_eq!(
|
||||
normalize_model_name("provider/deepseek-ai/deepseek-v4-pro").as_deref(),
|
||||
Some("provider/deepseek-ai/deepseek-v4-pro")
|
||||
);
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn default_context_seams_are_opt_in() {
|
||||
let config = Config::default();
|
||||
|
||||
Reference in New Issue
Block a user