Skip to content

Update Moonshot(Kimi) configs #8280

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Update Moonshot(Kimi) configs #8280

wants to merge 5 commits into from
+62 −5

Conversation

sjf126
Copy link

@sjf126 sjf126 commented Jul 18, 2025

Checklist

This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

Release note

1. src/renderer/src/config/models.ts:  
   - Kimi models now auto-select tool
   - k2 model pricing configured.  
   - For kimi/moonshot models, `isFunctionCallingModel` now returns true automatically.

2. src/renderer/src/config/providers.ts:  
   Added a link to the Kimi open platform at the top.

3. src/renderer/src/i18n/locales/*.json:  
   Removed an unnecessary sentence that displayed incorrectly in some locales.

4. src/renderer/src/pages/settings/AssistantSettings/AssistantModelSettings.tsx:  
   Set the default temperature for kimi models.

5. Updated the moonshot (kimi) logo.

@YinsenHo YinsenHo requested review from DeJeune and alephpiece and removed request for DeJeune July 18, 2025 11:06
@YinsenHo
Copy link
Collaborator

感谢 Kimi 的同学

@alephpiece
Copy link
Collaborator

需要合并一下最新的 main，不然有一些新变更被意外删除了，感谢🤝

@alephpiece alephpiece mentioned this pull request Jul 19, 2025
Draft
5 tasks
DeJeune
DeJeune approved these changes Jul 19, 2025
View reviewed changes
kangfenmao
kangfenmao reviewed Jul 19, 2025
View reviewed changes
Copy link
Collaborator

@kangfenmao kangfenmao left a comment

Choose a reason for hiding this comment

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

图片大小有点大，可以用 webp 格式，尺寸也可以用更小尺寸

@Pleasurecruise
仅修复 eslint error
4ab5a24 
Improves code readability by reformatting Moonshot provider configuration and related ternary expressions. Indentation and spacing are adjusted for consistency, with no functional changes.
kangfenmao
kangfenmao requested changes Jul 20, 2025
View reviewed changes
src/renderer/src/pages/settings/ProviderSettings/ProviderSetting.tsx
@@ -71,8 +70,19 @@ const ProviderSetting: FC<Props> = ({ providerId }) => {

const providerConfig = PROVIDER_CONFIG[provider.id]
const officialWebsite = providerConfig?.websites?.official
const platformWebsite =
Copy link
Collaborator

Choose a reason for hiding this comment

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

这里建议提取一个函数来处理

src/renderer/src/pages/settings/ProviderSettings/ProviderSetting.tsx
}, [provider.apiHost, provider.id, i18n.language])

// 当语言变化时，更新月之暗面的API地址
useEffect(() => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

这里应该不需要了，切换语言之后再回到这个界面，组件会重新加载

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

@beyondkmp beyondkmp beyondkmp left review comments

@kangfenmao kangfenmao kangfenmao requested changes

@DeJeune DeJeune DeJeune approved these changes

@alephpiece alephpiece Awaiting requested review from alephpiece

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
PM said OK
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@sjf126 @YinsenHo @alephpiece @beyondkmp @kangfenmao @DeJeune @Pleasurecruise