enable unified exec for experiments (#7118)

This commit is contained in:
Ahmed Ibrahim
2025-11-21 13:10:01 -08:00
committed by GitHub
Unverified
parent 8ecaad948b
commit d5f661c91d
3 changed files with 34 additions and 1 deletions
+1 -1
View File
@@ -183,7 +183,7 @@ pub fn find_family_for_model(slug: &str) -> Option<ModelFamily> {
base_instructions: BASE_INSTRUCTIONS.to_string(),
default_reasoning_effort: Some(ReasoningEffort::Medium),
truncation_policy: TruncationPolicy::Bytes(10_000),
shell_type: ConfigShellToolType::ShellCommand,
shell_type: ConfigShellToolType::UnifiedExec,
supports_parallel_tool_calls: true,
)
+18
View File
@@ -1426,6 +1426,24 @@ mod tests {
);
}
#[test]
fn test_exp_5_1_defaults() {
assert_model_tools(
"exp-5.1",
&Features::with_defaults(),
&[
"exec_command",
"write_stdin",
"list_mcp_resources",
"list_mcp_resource_templates",
"read_mcp_resource",
"update_plan",
"apply_patch",
"view_image",
],
);
}
#[test]
fn test_codex_mini_unified_exec_web_search() {
assert_model_tools(
+15
View File
@@ -121,4 +121,19 @@ async fn model_selects_expected_tools() {
],
"gpt-5.1 should expose the apply_patch tool",
);
let exp_tools = collect_tool_identifiers_for_model("exp-5.1").await;
assert_eq!(
exp_tools,
vec![
"exec_command".to_string(),
"write_stdin".to_string(),
"list_mcp_resources".to_string(),
"list_mcp_resource_templates".to_string(),
"read_mcp_resource".to_string(),
"update_plan".to_string(),
"apply_patch".to_string(),
"view_image".to_string()
],
"exp-5.1 should expose the apply_patch tool",
);
}