Skip to content

Add Kimi Code provider - #139

Open
auroter wants to merge 3 commits into
Lazarus-AI:mainfrom
auroter:feat/kimi-provider
Open

Add Kimi Code provider#139
auroter wants to merge 3 commits into
Lazarus-AI:mainfrom
auroter:feat/kimi-provider

Conversation

@auroter

@auroter auroter commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a kimi-code setup preset for Kimi Code membership credentials and the https://api.kimi.com/coding/v1 endpoint
  • offer only the four model IDs verified through the live endpoint: k3-256k, k3, kimi-for-coding, and kimi-for-coding-highspeed
  • use valid high reasoning effort for K3 models and omit unsupported reasoning_effort for the K2.7-backed coding aliases
  • explain Kimi Code/Open Platform key isolation after 401 responses and write provider config files with mode 0600
  • document the setup flow and add catalog, endpoint, setup, permissions, diagnostics, and reasoning regression coverage

Verification

  • live completion returned exactly PONG for all four advertised Kimi Code models
  • the exact committed PR snapshot passes 157 focused provider, setup, fallback, budget, and reasoning tests
  • scoped Ruff lint and format checks passed on the committed snapshot
  • git diff --check passed; no credentials are included in this PR

Scope

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

  • the strict full-suite command currently stops during collection because the existing integration marker is not registered
  • the normal full suite reached 2913 passed, 26 skipped; its 18 failures are existing sandbox tests that require a locally available Docker executable
  • repo-wide Ruff, scoped mypy, and strict docs checks also have existing failures in untouched files

@auroter

auroter commented Aug 11, 2026

Copy link
Copy Markdown
Contributor Author

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.

@whatever whatever left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@auroter auroter changed the title Add Kimi Open Platform and Kimi Code providers Add Kimi Code provider Aug 12, 2026
@auroter

auroter commented Aug 12, 2026

Copy link
Copy Markdown
Contributor Author

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.

@auroter
auroter requested a review from whatever August 12, 2026 01:06
@ehartford

Copy link
Copy Markdown
Contributor

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:

Part Commits Size In the PR description?
Kimi Code provider 77423c3, ed9db4e, 6ba34dc ~250 lines, 12 files ✅ yes
"sourcehunt optimization harness" 95477af +23,142 lines, 146 files ❌ not mentioned

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:

  1. Kimi Code provider — just the three Kimi commits. Scoped, matches the description, has tests, and (nice touch) writes the config 0600. Rebased onto current main, this is easy to land on its own.
  2. Optimization harness — the 95477af commit as a separate PR with a real summary of what it adds and how it was validated.

Heads-up that both will need a rebase — the harness commit touches hunter.py/runner.py/deep_agent, which have changed on main since this branch (hence the current conflict).

Happy to review each once they're split. Thanks!

@ehartford ehartford left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

split gepa to a separate PR

@ehartford
ehartford marked this pull request as draft August 29, 2026 23:43
@auroter
auroter force-pushed the feat/kimi-provider branch from 95477af to 6ba34dc Compare August 30, 2026 03:14
@auroter
auroter force-pushed the feat/kimi-provider branch from 6ba34dc to 4e50c10 Compare August 30, 2026 03:19
@auroter
auroter marked this pull request as ready for review August 31, 2026 18:47
@auroter
auroter requested a review from ehartford August 31, 2026 18:47
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.

3 participants