diff --git a/cmd/launch/codex.go b/cmd/launch/codex.go index fe48c926..20382269 100644 --- a/cmd/launch/codex.go +++ b/cmd/launch/codex.go @@ -638,7 +638,6 @@ func buildCodexModelEntry(modelName string) map[string]any { "slug": modelName, "display_name": modelName, "context_window": contextWindow, - "apply_patch_tool_type": "function", "shell_type": "default", "visibility": "list", "supported_in_api": true, diff --git a/cmd/launch/codex_test.go b/cmd/launch/codex_test.go index 37e1dac5..7641280c 100644 --- a/cmd/launch/codex_test.go +++ b/cmd/launch/codex_test.go @@ -639,12 +639,15 @@ func TestBuildCodexModelEntryContextWindow(t *testing.T) { } } - requiredKeys := []string{"slug", "display_name", "apply_patch_tool_type", "shell_type"} + requiredKeys := []string{"slug", "display_name", "shell_type"} for _, key := range requiredKeys { if _, ok := entry[key]; !ok { t.Errorf("missing required key %q", key) } } + if _, ok := entry["apply_patch_tool_type"]; ok { + t.Error("apply_patch_tool_type should be omitted so Codex CLI defaults can handle schema changes") + } if _, err := json.Marshal(entry); err != nil { t.Errorf("entry is not JSON serializable: %v", err)