Add Kimi Code provider - #139
Conversation
|
Live sanity after the K2.7 fix passed: model=kimi-for-coding, resolved reasoning_effort=None, response=PONG. The focused provider regression suite also passes with 165 tests. |
|
Scope update: this PR now includes only Kimi Code. Live completion returned PONG for every advertised model: k3-256k, k3, kimi-for-coding, and kimi-for-coding-highspeed. Kimi Open Platform and its unverified model IDs were removed. |
|
Thanks for this — the Kimi Code provider itself looks good and I'd be happy to take it. But could you split this PR? Right now it bundles two unrelated things, and the description only covers one of them:
That fourth commit (optimization/GEPA/lair eval modules, the source-window ranker, the FFmpeg reproducers, C fixtures, and the eval datasets) is ~98% of the diff and isn't referenced anywhere in the summary. We can't responsibly review a 23k-line agent-tooling feature as a ride-along under an "add a provider" title — it deserves its own PR with its own description and review. Concretely, please open:
Heads-up that both will need a rebase — the harness commit touches Happy to review each once they're split. Thanks! |
ehartford
left a comment
There was a problem hiding this comment.
split gepa to a separate PR
95477af to
6ba34dc
Compare
6ba34dc to
4e50c10
Compare
Summary
kimi-codesetup preset for Kimi Code membership credentials and thehttps://api.kimi.com/coding/v1endpointk3-256k,k3,kimi-for-coding, andkimi-for-coding-highspeedhighreasoning effort for K3 models and omit unsupportedreasoning_effortfor the K2.7-backed coding aliases0600Verification
PONGfor all four advertised Kimi Code models157focused provider, setup, fallback, budget, and reasoning testsgit diff --checkpassed; no credentials are included in this PRScope
Kimi Open Platform is intentionally not included because its credentials are isolated from Kimi Code and no Open Platform key was available for live validation.
Repository baseline notes
integrationmarker is not registered2913 passed, 26 skipped; its 18 failures are existing sandbox tests that require a locally available Docker executable