feat(web): add a cache invalidation note to the model switcher#1940
Conversation
🦋 Changeset detectedLatest commit: df764c0 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
commit: |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c0f8c38728
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| </div> | ||
|
|
||
| <div class="md-divider" /> | ||
| <div class="md-cache-note">{{ t('status.cacheNote') }}</div> |
There was a problem hiding this comment.
This note only renders inside the composer dropdown. On mobile, the settings sheet bypasses that dropdown (MobileSettingsSheet.vue:131-133) and opens ModelPicker, whose rows emit select directly (ModelPicker.vue:123-124, 187), while the same sheet changes thinking via setThinkingSegment() (MobileSettingsSheet.vue:121-123), so mobile users can still make both switches that this text warns about without seeing the warning. Please surface status.cacheNote in that path too.
Useful? React with 👍 / 👎.
|
Codex Review: Didn't find any major issues. Breezy! Reviewed commit: ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback". |
Upstream changes: - feat(cli): replace kimi server command tree with kimi web (MoonshotAI#1826) - Removed daemon/server lifecycle (svc/ layer for launchd/systemd/schtasks) - Multi-server instance registry replaces single-writer lock - /web TUI command uses server picker instead of daemon logic - feat(minidb): switch agent-core-v2 query-store to ClusterDb with 16 shards (MoonshotAI#1907) - Multiple kimi processes can share the read model concurrently - feat: scope thinking effort to current session (MoonshotAI#1933) - feat: add thinking_effort to turn telemetry events (MoonshotAI#1916) - fix(agent-core-v2): host fs stat follows symlinks, add explicit lstat (MoonshotAI#1843) - feat(web): add cache invalidation note to model switcher (MoonshotAI#1940) - ci: release packages (MoonshotAI#1868) Conflicts resolved: - 9 modify/delete conflicts: accepted upstream deletion (server/*, svc/*) - 6 content conflicts: merged while preserving fork's i18n t() calls and fork-specific test suites (config edge cases)
1 participant
Related Issue
No linked issue — the problem is explained below.
Problem
The web model switcher lets users change the model or thinking effort mid-session, but nothing indicates that doing so invalidates the existing prompt cache, which can quietly increase token costs.
What changed
width: 0+min-width: 100%so it fills the dropdown width and wraps there naturally without stretching the shrink-to-fit dropdown.Checklist
gen-changesetsskill, or this PR needs no changeset.gen-docsskill, or this PR needs no doc update.