refactor: simplify model settings and homepage channel UI#239
Merged
Siri-Ray merged 4 commits intorefactor/openapi-sdk-migrationfrom Mar 18, 2026
Merged
Conversation
- Remove per-model enable/disable toggles from settings page - Add current model selector dropdown at top of providers tab - Auto-select first available model when provider is configured - Simplify sidebar to single list (no enabled/disabled groups) - Homepage: only show "change config" button when no channels connected - Remove model dropdown from homepage (moved to settings) - Remove tab selector from channel manager panel Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
The user name, email, and logout button in the sidebar are unnecessary in the Electron desktop client. Wrap the account section with an isDesktopClient guard so it only renders in the web version. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Tailwind v4 preflight no longer sets cursor: pointer on buttons and other interactive elements. Add the rule back in the base layer so buttons, links, selects, and summary elements show the pointer cursor. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Siri-Ray
approved these changes
Mar 18, 2026
lefarcen
added a commit
that referenced
this pull request
Mar 18, 2026
* refactor: simplify model settings and homepage channel UI - Remove per-model enable/disable toggles from settings page - Add current model selector dropdown at top of providers tab - Auto-select first available model when provider is configured - Simplify sidebar to single list (no enabled/disabled groups) - Homepage: only show "change config" button when no channels connected - Remove model dropdown from homepage (moved to settings) - Remove tab selector from channel manager panel Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * refactor: hide user account section in desktop client The user name, email, and logout button in the sidebar are unnecessary in the Electron desktop client. Wrap the account section with an isDesktopClient guard so it only renders in the web version. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: restore cursor pointer on interactive elements for Tailwind v4 Tailwind v4 preflight no longer sets cursor: pointer on buttons and other interactive elements. Add the rule back in the base layer so buttons, links, selects, and summary elements show the pointer cursor. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * style: fix biome formatting in models.tsx and zh-CN.ts Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
4 tasks
PerishCode
pushed a commit
that referenced
this pull request
Mar 18, 2026
) * refactor: simplify model settings and homepage channel UI (#239) * refactor: simplify model settings and homepage channel UI - Remove per-model enable/disable toggles from settings page - Add current model selector dropdown at top of providers tab - Auto-select first available model when provider is configured - Simplify sidebar to single list (no enabled/disabled groups) - Homepage: only show "change config" button when no channels connected - Remove model dropdown from homepage (moved to settings) - Remove tab selector from channel manager panel Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * refactor: hide user account section in desktop client The user name, email, and logout button in the sidebar are unnecessary in the Electron desktop client. Wrap the account section with an isDesktopClient guard so it only renders in the web version. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: restore cursor pointer on interactive elements for Tailwind v4 Tailwind v4 preflight no longer sets cursor: pointer on buttons and other interactive elements. Add the rule back in the base layer so buttons, links, selects, and summary elements show the pointer cursor. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * style: fix biome formatting in models.tsx and zh-CN.ts Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> * fix(desktop): push config snapshot on cloud disconnect/reconnect After disconnecting from Nexu Official, the gateway config was never updated — it kept trying to route through the stale link provider. After reconnecting, credentials were saved but the gateway was never notified, so models remained unavailable and Feishu bot stayed unresponsive. Add publishPoolConfigSnapshot() calls to both cloudDisconnectRoute and pollCloudForAuthorization, matching the pattern already used by cloudModelsRoute and defaultModelSetRoute. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * feat(desktop): refresh cloud models from Link gateway on startup Cloud model list was only fetched once during initial login and cached in cloud-credentials.json. If the Link gateway added or removed models (e.g. claude-sonnet-4-5 → claude-sonnet-4-6), the desktop stayed stuck on stale model IDs, causing 404 errors on inference. Add refreshCloudModelsOnStartup() that runs once when the API boots: compares cached models with fresh data from Link, updates credentials and pushes a config snapshot if anything changed. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
🤖 Generated with Claude Code