Skip to content

fix(kimi): add kimi-k2.7-code to kimi-coding-cn and moonshot model lists - #53532

Open
wwwqqqzzz wants to merge 1 commit into
NousResearch:mainfrom
wwwqqqzzz:fix/kimi-k2.7-code-missing-cn
Open

fix(kimi): add kimi-k2.7-code to kimi-coding-cn and moonshot model lists#53532
wwwqqqzzz wants to merge 1 commit into
NousResearch:mainfrom
wwwqqqzzz:fix/kimi-k2.7-code-missing-cn

Conversation

@wwwqqqzzz

Copy link
Copy Markdown

Summary

The kimi-k2.7-code model was listed in the kimi-coding provider model list but missing from:

  • kimi-coding-cn in hermes_cli/models.py
  • moonshot in hermes_cli/models.py
  • kimi-coding and kimi-coding-cn in hermes_cli/setup.py

Changes

  • hermes_cli/models.py: Added "kimi-k2.7-code" as first entry in kimi-coding-cn and moonshot provider model lists
  • hermes_cli/setup.py: Added "kimi-k2.7-code" as first entry in kimi-coding and kimi-coding-cn provider model lists

Related

Closes #53520

The kimi-k2.7-code model was listed in kimi-coding provider model list
but missing from kimi-coding-cn and moonshot provider lists in
hermes_cli/models.py, and from both kimi-coding and kimi-coding-cn
lists in hermes_cli/setup.py.

Add it as the first entry in all four lists, matching the existing
kimi-k2.7-code entry in the kimi-coding list.

Closes NousResearch#53520
@alt-glitch alt-glitch added type/feature New feature or request P3 Low — cosmetic, nice to have comp/cli CLI entry point, hermes_cli/, setup wizard provider/kimi Kimi / Moonshot labels Jun 27, 2026

@tonydwb tonydwb left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review Summary

Verdict: Approved

Adds kimi-k2.7-code to the kimi-coding-cn and moonshot model lists, and to the kimi-coding and kimi-coding-cn provider pool setup.

Looks Good

  • Model added to both curated lists and pool setup consistently
  • Correct placement at the top of each list (newest model first)
  • Minimal 4-line change across 2 files

Reviewed by Hermes Agent

@teknium1 teknium1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for tracing the missing curated entries.

Problems

  • hermes_cli/setup.py:102-103 is an unreferenced _DEFAULT_PROVIDER_MODELS duplicate. The active Kimi wizard imports the canonical catalog from hermes_cli.models at hermes_cli/model_setup_flows.py:1902 and selects its endpoint-specific list at line 1936, so those two setup edits have no current runtime effect.
  • The active flow intentionally distinguishes sk-kimi-* Coding Plan keys from legacy Moonshot keys at hermes_cli/model_setup_flows.py:1923-1936. Please verify K2.7 Code is available on the legacy and China endpoints before expanding both corresponding lists.

Suggested changes

  • Drop the unused setup.py edits and add a focused test for the confirmed endpoint-specific catalog behavior.

Automated hermes-sweeper review.

Comment thread hermes_cli/setup.py
"zai": ["glm-5.2", "glm-5.1", "glm-5", "glm-4.7", "glm-4.5", "glm-4.5-flash"],
"kimi-coding": ["kimi-k2.6", "kimi-k2.5", "kimi-k2-thinking", "kimi-k2-turbo-preview"],
"kimi-coding-cn": ["kimi-k2.6", "kimi-k2.5", "kimi-k2-thinking", "kimi-k2-turbo-preview"],
"kimi-coding": ["kimi-k2.7-code", "kimi-k2.6", "kimi-k2.5", "kimi-k2-thinking", "kimi-k2-turbo-preview"],

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

_DEFAULT_PROVIDER_MODELS is currently unreferenced outside this definition; the live Kimi setup flow imports hermes_cli.models._PROVIDER_MODELS instead. Please remove this duplicate-catalog change rather than maintaining a dead second list.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard P3 Low — cosmetic, nice to have provider/kimi Kimi / Moonshot sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

kimi-k2.7-code missing from kimi-coding-cn provider model list

4 participants