fix(web): unify permission mode order and risk colors across settings surfaces - #2125
Conversation
Align the Agent settings default-permission segmented control with the Composer toolbar order, arranging modes from safest (manual) to most permissive (auto).
🦋 Changeset detectedLatest commit: 41eb008 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 |
46165e8 to
4daf75a
Compare
bowenliang123
changed the title
725f3d8 to
4daf75a
Compare
commit: |
The mobile settings sheet still cycled manual → auto → yolo, jumping from the safest mode straight to the most permissive one on a single tap. Align the tap-to-cycle order with the Composer menu and Agent settings (manual → yolo → auto).
Both the desktop status panel and the mobile settings sheet mapped yolo → danger and auto → warning, the inverse of the Composer menu (yolo → warning, auto → danger). Since auto is the most permissive mode, it should carry the danger color everywhere.
wbxl2000
changed the title
|
Codex Review: Didn't find any major issues. 👍 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". |
|
@wbxl2000 Thanks for the review and refinement. This is my very first PR merged to kimi-code. And I do hope kimi-code all the best and will contribute more my effort to it to bring it as first tier coding agent. |
…bileSettingsSheet.vue
2 participants
Related Issue
No linked issue — the problem is explained below.
Problem
In Agent settings, the default permission mode selector showed options in the order
manual → auto → yolo. This is neither ordered by risk/permissiveness nor consistent with the Composer toolbar, which lists them asmanual → yolo → auto.manual: ask for approval on every tool action (safest)yolo: auto-approve tool actions, but still ask questions (middle)auto: fully autonomous, no questions asked (most permissive)Placing
autobetweenmanualandyolomade the risk progression hard to read at a glance.What changed
permissionModesinapps/kimi-web/src/components/settings/SettingsDialog.vuefrom['manual', 'auto', 'yolo']to['manual', 'yolo', 'auto'].Checklist
gen-changesetsskill, or this PR needs no changeset.gen-docsskill, or this PR needs no doc update.Screenshots