klinsmaya/CLIProxyAPI
1
0
Fork 0
mirror of https://gh-proxy.org/https://github.com/klinsmaya/CLIProxyAPI.git synced 2026-11-08 09:26:52 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(codex): require model template for tool search

Browse Source
This commit is contained in:
huahai0202
2026-07-13 19:35:16 +08:00
parent ceaeb75d53
commit e73aad2e0a
2 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
sdk/api/handlers/openai/codex_client_models.go
+5 -3
View File
@@ -192,10 +192,12 @@ func applyCodexClientSearchToolSupport(entry map[string]any, id string, template
return
}
if !templateModel {
entry["supports_search_tool"] = false
return
}
if providersForModel == nil {
if !templateModel {
entry["supports_search_tool"] = false
}
return
}
sdk/api/handlers/openai/codex_client_models_test.go
+4 -6
View File
@@ -175,7 +175,7 @@ func TestCodexClientModelsResponse_DisablesSearchToolForSynthesizedModels(t *tes
}
}
func TestCodexClientModelsResponse_UsesProviderSearchToolSupport(t *testing.T) {
func TestCodexClientModelsResponse_RequiresTemplateAndCodexProvidersForSearchTool(t *testing.T) {
providers := map[string][]string{
"new-codex-model": {"codex"},
"gpt-5.5": {"openai-compatible-deepseek"},
@@ -200,12 +200,10 @@ func TestCodexClientModelsResponse_UsesProviderSearchToolSupport(t *testing.T) {
bySlug[stringModelValue(model, "slug")] = model
}
for _, slug := range []string{"new-codex-model", "gpt-5.6-sol"} {
if got, ok := bySlug[slug]["supports_search_tool"].(bool); !ok || !got {
t.Errorf("%s supports_search_tool = %#v, want true", slug, bySlug[slug]["supports_search_tool"])
}
if got, ok := bySlug["gpt-5.6-sol"]["supports_search_tool"].(bool); !ok || !got {
t.Errorf("gpt-5.6-sol supports_search_tool = %#v, want true", bySlug["gpt-5.6-sol"]["supports_search_tool"])
}
for _, slug := range []string{"gpt-5.5", "gpt-5.4"} {
for _, slug := range []string{"new-codex-model", "gpt-5.5", "gpt-5.4"} {
if got, ok := bySlug[slug]["supports_search_tool"].(bool); !ok || got {
t.Errorf("%s supports_search_tool = %#v, want false", slug, bySlug[slug]["supports_search_tool"])
}