Skip to content

fix(codex): default custom provider wire_api to 'responses' - #293

Open
CZH-THU wants to merge 1 commit into
alibaba:mainfrom
CZH-THU:main
Open

CZH-THU wants to merge 1 commit into
alibaba:mainfrom
CZH-THU:main

Conversation

@CZH-THU

@CZH-THU CZH-THU commented Oct 1, 2026

Copy link
Copy Markdown

Summary

Default codexCustomWireAPI from "chat" to "responses" to align with modern codex behavior and support custom providers that only implement the Responses API.

Problem

When using a custom provider (e.g., tokenswitch, idealab) with a non-default base_url, skill-up generates codex command-line flags like:


\-c model\_providers.skill-up-openai.wire\_api="chat"


This overrides the provider's configured wire_api setting, forcing codex to use /chat/completions. If the upstream endpoint only supports /responses, the request fails or hangs.

Solution

Change codexCustomWireAPI from "chat" to "responses":

// Before
codexCustomWireAPI  = "chat"

// After
codexCustomWireAPI  = "responses"

This aligns with modern codex defaults (which already use "responses") and ensures compatibility with custom providers that implement the Responses API.

Impact

Scenario Before After
Default OpenAI (no base_url) No change No change
Custom provider with Responses API ❌ Fails ✅ Works
Custom provider with Chat API only ✅ Works ❌ Fails

Note: Modern codex defaults to "responses" anyway, so custom providers that only support Chat API are already incompatible with current codex versions.

Testing

Verified with tokenswitch provider (Responses API only):

✅ [1/1] pause-campaign-confirmation: PASS (100.0%)
📋 Results: 1 passed, 0 failed, 0 errors

Related

  • Fixes issue where skill-up eval runs hang or fail with custom providers
  • Aligns with codex's default behavior since v0.80+

Modern codex defaults to Responses API. Align custom provider config
with this default to support endpoints that only implement /responses
(e.g., tokenswitch, idealab).

The legacy 'chat' value caused custom providers to fail when the
upstream endpoint doesn't support /chat/completions.
@CLAassistant

CLAassistant commented Oct 1, 2026 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants