Skip to content

feat: add kimi-k2-thinking via canopywave - #1144

Merged
steebchen merged 1 commit into
mainfrom
feat/kimi-k2
Nov 11, 2025
Merged

steebchen merged 1 commit into
mainfrom
feat/kimi-k2

Conversation

@steebchen

@steebchen steebchen commented Nov 11, 2025

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • New Features
    • Added support for the kimi-k2-thinking model through a new provider integration, enabling access to this model with streaming, tool use, and JSON output capabilities.

@coderabbitai

coderabbitai Bot commented Nov 11, 2025

Copy link
Copy Markdown
Contributor

Walkthrough

Added a new provider entry (canopywave) to the "kimi-k2-thinking" moonshot model configuration in packages/models/src/models/moonshot.ts with associated pricing, context size, and capability parameters. No API changes.

Changes

Cohort / File(s) Summary
Moonshot model provider configuration
packages/models/src/models/moonshot.ts
Added canopywave provider to the "kimi-k2-thinking" model with pricing (input: 0.48/1M, output: 2.0/1M), discount 0.75, context size 262144, and features enabled: streaming, tools, JSON output.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

  • Single file modification involving addition of one provider configuration entry with no logic changes or API modifications.

Possibly related PRs

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and accurately describes the main change: adding support for the kimi-k2-thinking model through the canopywave provider.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/kimi-k2

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a3c70ba and 4c093d7.

📒 Files selected for processing (1)
  • packages/models/src/models/moonshot.ts (1 hunks)
🧰 Additional context used
📓 Path-based instructions (2)
**/*.{ts,tsx,js,jsx}

📄 CodeRabbit inference engine (AGENTS.md)

Always use top-level import; never use require() or dynamic import()

Files:

  • packages/models/src/models/moonshot.ts
**/*.{ts,tsx}

📄 CodeRabbit inference engine (CLAUDE.md)

**/*.{ts,tsx}: Never use any or as any in this TypeScript project unless absolutely necessary
Always use top-level import; do not use require or dynamic import()

Files:

  • packages/models/src/models/moonshot.ts
🔇 Additional comments (1)
packages/models/src/models/moonshot.ts (1)

98-110: The review comment contains factual errors and should be disregarded.

The discount field is not unusual—it's a standard optional field in the ProviderModelMapping interface (packages/models/src/models.ts:50) and is used in 70+ provider configurations across the codebase (minimax, alibaba, zai, anthropic, openai, deepseek, google, moonshot). Both discount and maxOutput are optional fields in the schema, so the canopywave provider entry conforms to the interface specification.

Likely an incorrect or invalid review comment.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@steebchen
steebchen added this pull request to the merge queue Nov 11, 2025
github-merge-queue Bot pushed a commit that referenced this pull request Nov 11, 2025
<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **New Features**
* Added support for the kimi-k2-thinking model through a new provider
integration, enabling access to this model with streaming, tool use, and
JSON output capabilities.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
@steebchen
steebchen removed this pull request from the merge queue due to a manual request Nov 11, 2025
@steebchen
steebchen merged commit 97e35e7 into main Nov 11, 2025
13 of 14 checks passed
@steebchen
steebchen deleted the feat/kimi-k2 branch November 11, 2025 08:37
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.

1 participant