Skip to content

Re-add Kimi K2.5 with fixed API endpoint and improved icon SVG#61

Open
devin-ai-integration[bot] wants to merge 1 commit intomainfrom
devin/1774568779-fix-kimi-endpoint-icon
Open

Re-add Kimi K2.5 with fixed API endpoint and improved icon SVG#61
devin-ai-integration[bot] wants to merge 1 commit intomainfrom
devin/1774568779-fix-kimi-endpoint-icon

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot commented Mar 26, 2026

Summary

Re-adds Kimi K2.5 (Moonshot AI) as a chat model option, applying the two bug fixes originally proposed in PR #42 (which was closed without merging before the feature was removed in PR #46):

  1. API endpoint: api.moonshot.cnapi.moonshot.ai (international endpoint per Moonshot docs)
  2. Icon SVG: Replaced circle-with-diagonal placeholder with a person-in-circle shape

Changes span 7 files across app and server, following the same patterns established by the existing GPT/Gemini/Claude handlers.

Review & Testing Checklist for Human

  • Confirm Kimi K2.5 should be re-added — it was intentionally removed in PR Remove Kimi K2.5 from model options #46. Verify this re-addition is desired.
  • Verify the correct API domain for your deployment regionapi.moonshot.ai is the international endpoint. If targeting China-only users, api.moonshot.cn may be preferred.
  • Verify the model identifier kimi-k2.5 is correct per Moonshot API docs — this was not validated against a live API key.
  • End-to-end test: Set MOONSHOT_API_KEY in server env, select "Kimi K2.5" in the model picker, and verify streaming chat works. This PR was not tested e2e due to lack of API credentials.
  • KimiIcon is a generic placeholder (person-in-circle), not the official Kimi brand icon. Replace if brand accuracy matters.

Notes

  • Server handler uses the same OpenAI-compatible streaming pattern as the existing GPT handler, since Moonshot's API is OpenAI-compatible.
  • The brokenLine recovery logic prepends incomplete JSON fragments to the next chunk before splitting, preserving correct token order (fix from PR Add Kimi K2.5 as a model option #40 review feedback).
  • Requires MOONSHOT_API_KEY environment variable on the server.

Link to Devin session: https://app.devin.ai/sessions/decb6ca81ca64089bd0819d27025eb0a


Open with Devin

Fixes #42: Re-adds Kimi K2.5 model support with two bug fixes applied:
1. API endpoint changed from api.moonshot.cn to api.moonshot.ai (international endpoint)
2. KimiIcon SVG updated to a more recognizable person-in-circle shape

Co-Authored-By: bot_apk <apk@cognition.ai>
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Copy link
Copy Markdown
Contributor Author

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 4 additional findings.

Open in Devin Review

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.

0 participants