modifying starts_with to contains string matching to address cases where gpt-oss model names from different providers have a prefix like openai/gpt-oss/120b

Andrew Tan · andrewtlw · commit 119ed46f1e04 · 2025-08-28T14:30:43.000+08:00
diff --git a/codex-rs/core/src/model_family.rs b/codex-rs/core/src/model_family.rs
@@ -96,7 +96,7 @@ pub fn find_family_for_model(slug: &str) -> Option<ModelFamily> {
             slug, "gpt-4.1",
             needs_special_apply_patch_instructions: true,
         )
-    } else if slug.starts_with("gpt-oss") {
+    } else if slug.contains("gpt-oss") {
         model_family!(slug, "gpt-oss", apply_patch_tool_type: Some(ApplyPatchToolType::Function))
     } else if slug.starts_with("gpt-4o") {
         simple_model_family!(slug, "gpt-4o")

Original file line number	Diff line number	Diff line change
`@@ -96,7 +96,7 @@ pub fn find_family_for_model(slug: &str) -> Option<ModelFamily> {`
`96`	`96`	`slug, "gpt-4.1",`
`97`	`97`	`needs_special_apply_patch_instructions: true,`
`98`	`98`	`)`
`99`		`- } else if slug.starts_with("gpt-oss") {`
	`99`	`+ } else if slug.contains("gpt-oss") {`
`100`	`100`	`model_family!(slug, "gpt-oss", apply_patch_tool_type: Some(ApplyPatchToolType::Function))`
`101`	`101`	`} else if slug.starts_with("gpt-4o") {`
`102`	`102`	`simple_model_family!(slug, "gpt-4o")`