feat(providers): add Kimi (Moonshot AI) provider#5716
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryMedium Risk Overview Registers four models ( Wires Reviewed by Cursor Bugbot for commit 79ed9b8. Configure here. |
Greptile SummaryThis PR adds Kimi as a hosted and BYOK language-model provider. The main changes are:
Confidence Score: 5/5This looks safe to merge.
Important Files Changed
Reviews (2): Last reviewed commit: "fix(providers): preserve reasoning_conte..." | Re-trigger Greptile |
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 79ed9b8. Configure here.
Summary
api.moonshot.ai/v1, OpenAI-compatible)thinking: {type}; forced tool_choice disables thinking on toggle models and downgrades to auto on always-thinking models (Kimi rejects forced tool_choice with thinking enabled — verified live)json_schemastructured output with the sharedenforceStrictSchema()helper;max_completion_tokensoutput cap; image attachments viaimage_urldata URIsKIMI_API_KEY_1..3+ BYOK support incl. settings UI row; wired through registry, types, attachments, tokenization, billing (getHostedModels), and testsType of Change
Testing
Live-tested every wire shape against the real Moonshot API with production keys: chat on all four models, streaming with usage, multi-turn tool loop, forced tool calls, strict json_schema output, thinking toggle, vision parts, and the max_completion_tokens cap. 888 provider/tokenization/byok tests pass; tsc, biome, and check:api-validation clean.
Checklist