fix(web): show just the thinking level name in the model pill - #1689
Conversation
🦋 Changeset detectedLatest commit: 2b43b5d 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: adc0427b29
ℹ️ 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".
| "dev:server": "tsx --tsconfig ./tsconfig.dev.json --import ../../build/register-raw-text-loader.mjs ./src/main.ts server run --foreground", | ||
| "dev:kap-server": "tsx --tsconfig ./tsconfig.dev.json --import ../../build/register-raw-text-loader.mjs ./src/main.ts server run --foreground", | ||
| "dev:kap-server:multi": "KIMI_CODE_EXPERIMENTAL_MULTI_SERVER=1 tsx --tsconfig ./tsconfig.dev.json --import ../../build/register-raw-text-loader.mjs ./src/main.ts server run --foreground --port 58628", | ||
| "dev:kap-server:multi": "KIMI_CODE_EXPERIMENTAL_MULTI_SERVER=1 tsx --tsconfig ./tsconfig.dev.json --import ../../build/register-raw-text-loader.mjs ./src/main.ts server run --foreground", |
There was a problem hiding this comment.
When pnpm dev:v2 is started without another server already occupying 58627, this script now falls back to the CLI's default port 58627 (DEFAULT_SERVER_PORT), but the web dev proxy still advertises/switches the multi backend to http://127.0.0.1:58628 in apps/kimi-web/vite.config.ts. That leaves the Sidebar's multi-backend switch pointing at an empty port in this common standalone/debug startup path; preserving --port 58628 keeps the root dev:v2 script aligned with the web preset.
Useful? React with 👍 / 👎.
Drop the "thinking:" / "思考:" prefix from the effort suffix and capitalize the level via effortLabel, matching the segment labels.
adc0427 to
cdc7334
Compare
1 participant
Related Issue
None — small wording tweak, problem explained below.
Problem
The model pill showed the thinking level as
思考: max/thinking: max— verbose, and the level was interpolated raw (lowercase) while the segment control labels are capitalized.What changed
思考:/thinking:prefix from the effort suffix in both locales, so the pill readsCoding Model 0711 Highspeed · Max.effortLabel(same capitalization as the segment labels) instead of the raw stored string.思考/thinkingtag;offstill shows nothing.Checklist
gen-changesetsskill, or this PR needs no changeset.gen-docsskill, or this PR needs no doc update (wording-only tweak).