Skip to content

fix(desktop): preserve curated provider models on refresh#3789

Merged
SivanCola merged 5 commits into
esengine:main-v2from
SivanCola:codex/preserve-curated-model-refresh
Jun 10, 2026
Merged

fix(desktop): preserve curated provider models on refresh#3789
SivanCola merged 5 commits into
esengine:main-v2from
SivanCola:codex/preserve-curated-model-refresh

Conversation

@SivanCola

@SivanCola SivanCola commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Preserve existing curated provider model lists so provider catalog responses do not automatically enter the session model picker.
  • Change Settings > Models > Access so Refresh models opens a discovered-model checklist; users must explicitly save the enabled model list.
  • Filter discovered and saved provider models to chat-capable model IDs so the saved count, provider card chips, and session model picker stay in sync.
  • Add a MiMo-focused regression case covering [Bug]: Windows桌面版会话窗口模型选择重大BUG #3705 and the follow-up in fix(desktop): stop auto-populating empty model lists from provider API #3711, including provider catalog candidates such as mimo-v2-flash / mimo-v2-omni and non-chat asr / tts entries.

Verification

  • pnpm --dir desktop/frontend run typecheck
  • pnpm --dir desktop/frontend run test
  • pnpm --dir desktop/frontend run build
  • go test . -run "Test(FetchProviderModelsFiltersNonChatModels|SaveProviderFiltersNonChatModels|ProviderViewFromEntry_FiltersNonChatModels)" from desktop
  • go test ./internal/config -run "TestIsLikelyChatModel|TestChatModelList|TestModelList"

@SivanCola SivanCola requested a review from esengine as a code owner June 10, 2026 05:33
@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.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@github-actions github-actions Bot added v2 Go rewrite (1.x) — main-v2 branch, active development desktop Wails desktop app (desktop/**) labels Jun 10, 2026
@SivanCola SivanCola enabled auto-merge June 10, 2026 06:53
…ve-conflicts

# Conflicts:
#	desktop/frontend/src/__tests__/provider-model-refresh.test.ts
#	desktop/frontend/src/components/SettingsPanel.tsx
#	desktop/frontend/src/components/WorkspacePanel.tsx

@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.

Solid fix — front+back double-filtering of non-chat models with TS and Go test coverage, and the explicit save-checklist is the right call. Thanks!

@SivanCola SivanCola merged commit 46140a6 into esengine:main-v2 Jun 10, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

desktop Wails desktop app (desktop/**) v2 Go rewrite (1.x) — main-v2 branch, active development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants