Skip to content

fix: respect model display_name for kimi-for-coding#2174

Open
tears-mysthrala wants to merge 1 commit into
MoonshotAI:mainfrom
tears-mysthrala:fix-model-display-name-for-kimi-code
Open

fix: respect model display_name for kimi-for-coding#2174
tears-mysthrala wants to merge 1 commit into
MoonshotAI:mainfrom
tears-mysthrala:fix-model-display-name-for-kimi-code

Conversation

@tears-mysthrala
Copy link
Copy Markdown

@tears-mysthrala tears-mysthrala commented May 7, 2026

Related Issue

Resolve #2175

Description

Removes the hardcoded override in model_display_name() that forced "kimi-for-coding" and "kimi-code" to always display as "kimi-for-coding", ignoring the display_name provided by the backend (e.g. "Kimi-k2.6"). The backend already returns the correct display name via the /models endpoint, and it is correctly persisted in the config file — this fix simply allows the UI to actually show it.

Checklist

  • I have read the CONTRIBUTING document.
  • I have linked the related issue, if any.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have run make gen-changelog to update the changelog.
  • I have run make gen-docs to update the user documentation.

Copilot AI review requested due to automatic review settings May 7, 2026 11:05
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes a hardcoded display-name override so UI surfaces the provider-returned display_name (e.g., “Kimi-k2.6”) instead of forcing “kimi-for-coding”, improving model name presentation across the CLI UI.

Changes:

  • Deleted the special-case mapping that normalized ("kimi-for-coding", "kimi-code") to "kimi-for-coding" in model_display_name().
  • Ensures callers that already pass LLMModel.display_name can surface the backend-provided name without being overridden.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

fix: model_display_name ignores display_name for kimi-for-coding

2 participants