Skip to content

fix: 快捷添加模型重定向时错误缓存了模型列表 - #3086

Closed
Bliod-Cook wants to merge 5334 commits into
QuantumNous:mainfrom
Bliod-Cook:fix-quick-select-model-reroute
Closed

fix: 快捷添加模型重定向时错误缓存了模型列表#3086
Bliod-Cook wants to merge 5334 commits into
QuantumNous:mainfrom
Bliod-Cook:fix-quick-select-model-reroute

Conversation

@Bliod-Cook

@Bliod-Cook Bliod-Cook commented Mar 2, 2026

Copy link
Copy Markdown
Contributor

目前的解决方案是每次点按钮都重新获取一次列表

相关历史PR: #2610

Summary by CodeRabbit

Bug Fixes

  • Improved model list validation in channel configuration. Enhanced fetching logic with stricter error handling and better empty state messaging for improved reliability.

Calcium-Ion and others added 30 commits February 4, 2026 00:13
…78addbb73be4739dc54f41acd8b

feat: add useTimeSeconds in error log
…log-column

feat: log search field request_id && conversion_path display
…group-colors

Revert "fix(ui): use distinct color palette for group tags"
…-monitoring

feat(performance): implement system performance monitoring
…bfe886b93606003f6753fcb4e9d

feat: task log show username
…ference

- Add go-i18n library for internationalization
- Create i18n package with translation keys and YAML locale files (zh/en)
- Implement i18n middleware for language detection from user settings and Accept-Language header
- Add Language field to UserSetting DTO
- Update API response helpers with i18n support (ApiErrorI18n, ApiSuccessI18n)
- Migrate hardcoded messages in token, redemption, and user controllers
- Add frontend language preference settings component
- Sync language preference across header selector and user settings
- Auto-restore user language preference on login
fix: default summary = detailed

fix ReasoningContent

fix ReasoningContent

fix ReasoningContent

fix ReasoningContent

Revert "fix ReasoningContent"

This reverts commit 45a88f7.

fix ReasoningContent

fix ReasoningContent
- Change default language fallback to English instead of Chinese
- Add ErrRedeemFailed typed error for model layer translation
- Migrate remaining hardcoded messages in controller/user.go
- Add translation keys: redeem.failed, user.create_default_token_error, common.uuid_duplicate, common.invalid_input
The i18n middleware runs before UserAuth, so user settings weren't
available when language was detected. Now GetLangFromContext checks
user settings first (set by UserAuth) before falling back to the
language set by middleware or Accept-Language header.
Unify Epay subscription response format with top-up flow, and harden frontend error handling to avoid object-to-string issues. Refine subscription plan cards layout to be wider, left-aligned, and visually consistent across breakpoints.
Use dynamic viewport height to prevent sidebar scroll lock in mobile browsers
Harden sidebar scroll container with min-height and momentum scrolling
…-epay

✨ fix: Improve subscription payment handling and card layout consistency
…ll-dvh

🐛 fix: sidebar scroll on mobile dynamic viewport
…reasoning

fix: map Responses reasoning stream to chat completion deltas
…x-override

feat: 支持基于Go Regex规则和全量的请求体透传
Calcium-Ion and others added 27 commits February 28, 2026 17:55
…soning_split

feat: minimax reasoning_split
feat(gemini): implement video generation configuration
…am-override

feat: improve channel override ui/ux
Bumps [axios](https://github.com/axios/axios) from 1.12.0 to 1.13.5.
- [Release notes](https://github.com/axios/axios/releases)
- [Changelog](https://github.com/axios/axios/blob/v1.x/CHANGELOG.md)
- [Commits](axios/axios@v1.12.0...v1.13.5)

---
updated-dependencies:
- dependency-name: axios
  dependency-version: 1.13.5
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
- Refactored the request URL and body construction methods to align with the Veo predictLongRunning endpoint.
- Introduced new data structures for Veo instances and parameters, replacing the previous Gemini video generation configurations.
- Updated the Vertex adaptor to utilize the new Veo request payload format.
…nd_yarn/web/axios-1.13.5

chore(deps): bump axios from 1.12.0 to 1.13.5 in /web
…nd_yarn/electron/multi-227d46b8ec

chore(deps): bump tar and electron-builder in /electron
Bumps [minimatch](https://github.com/isaacs/minimatch) from 3.1.2 to 3.1.5.
- [Changelog](https://github.com/isaacs/minimatch/blob/main/changelog.md)
- [Commits](isaacs/minimatch@v3.1.2...v3.1.5)

---
updated-dependencies:
- dependency-name: minimatch
  dependency-version: 3.1.5
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
…nd_yarn/electron/lodash-4.17.23

build(deps-dev): bump lodash from 4.17.21 to 4.17.23 in /electron
…nd_yarn/electron/minimatch-3.1.5

chore(deps-dev): bump minimatch from 3.1.2 to 3.1.5 in /electron
…nore

fix: preserve explicit zero values in native relay requests
- Introduced a new CCSwitchModal component for managing CCSwitch configurations.
- Updated the TokensPage to include functionality for opening the CCSwitch modal.
- Enhanced the useTokensData hook to handle CCSwitch URLs and trigger the modal.
- Modified chat settings to include a new "CC Switch" entry.
- Updated sidebar logic to skip certain links based on the new configuration.
fix: aws text content blocks must be non-empty
…aws-non-empty-text

Revert "Fix/aws non empty text"
…aws-non-empty-text

Revert "fix: aws text content blocks must be non-empty"
@coderabbitai

coderabbitai Bot commented Mar 2, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e71f5a4 and 16e7585.

📒 Files selected for processing (1)
  • web/src/components/table/channels/modals/EditChannelModal.jsx

Walkthrough

The PR modifies the openModelMappingValueModal function in EditChannelModal.jsx to always fetch upstream models and validate the result before proceeding, replacing conditional re-fetch logic with a stricter single-source fetch-and-validate flow.

Changes

Cohort / File(s) Summary
Model Fetch Logic Update
web/src/components/table/channels/modals/EditChannelModal.jsx
Modified openModelMappingValueModal to always fetch upstream models via fetchUpstreamModelList and validate results (ensuring array and non-empty state). Adds early return on invalid/empty fetch results and displays "暂无模型" message if model list is empty after normalization.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • seefs001

Poem

🐰 A fetch that's true, no guessing twice,
Models validated, oh so nice!
Single source of truth we find,
No empty lists left behind,
Upstream flows with rigid pride! 🌟

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: fixing incorrect model list caching during quick model selection redirection.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ast-grep (0.41.0)
web/src/components/table/channels/modals/EditChannelModal.jsx

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.