Skip to content

Fix official provider display for proxy endpoints / 修复代理端点误显示为官方供应商#3571

Merged
esengine merged 1 commit into
main-v2from
codex/fix-official-provider-proxy-effort
Jun 8, 2026
Merged

Fix official provider display for proxy endpoints / 修复代理端点误显示为官方供应商#3571
esengine merged 1 commit into
main-v2from
codex/fix-official-provider-proxy-effort

Conversation

@SivanCola

Copy link
Copy Markdown
Collaborator

Summary

  • require built-in provider cards to match the official endpoint host before labeling them as official access
  • keep proxy endpoints named deepseek or mimo-* visible as custom provider access instead of official presets
  • treat stored effort = "auto" as provider default and harden OpenAI-compatible provider startup against stale auto effort values

Fixes #3561

Testing

  • go test ./internal/config ./internal/provider/openai
  • (cd desktop && go test ./...)
  • pnpm --dir desktop/frontend check:css
  • pnpm --dir desktop/frontend typecheck
  • pnpm --dir desktop/frontend build

@SivanCola SivanCola requested a review from esengine as a code owner June 8, 2026 10:04
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@github-actions github-actions Bot added v2 Go rewrite (1.x) — main-v2 branch, active development desktop Wails desktop app (desktop/**) config Configuration & setup (internal/config) provider Model providers & selection (internal/provider) and removed v2 Go rewrite (1.x) — main-v2 branch, active development labels Jun 8, 2026

@esengine esengine left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified: config/openai/desktop provider tests pass; cleanly resolves the proxy-vs-official classification and the effort=auto 400 from #3561.

@esengine esengine merged commit 9328c48 into main-v2 Jun 8, 2026
10 checks passed
@esengine esengine deleted the codex/fix-official-provider-proxy-effort branch June 8, 2026 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

config Configuration & setup (internal/config) desktop Wails desktop app (desktop/**) provider Model providers & selection (internal/provider)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: 启动错误:openai: provider "deepseek" uses DeepSeek thinking; effort must be high or max

2 participants