Skip to content

Default Kimi K3 to max reasoning effort#4632

Merged
chrarnoldus merged 1 commit into
mainfrom
feat/kimi-k3-max-reasoning-default
Jul 20, 2026
Merged

Default Kimi K3 to max reasoning effort#4632
chrarnoldus merged 1 commit into
mainfrom
feat/kimi-k3-max-reasoning-default

Conversation

@chrarnoldus

Copy link
Copy Markdown
Contributor

Summary

  • add max, high, and low reasoning variants in that order
  • map kimi-k3 to the new variant set so max is the first/default option
  • keep kimi-k2.7-code on the existing thinking-only setting

Verification

  • pnpm --filter web test -- --runInBand apps/web/src/lib/ai-gateway/custom-llm/order-opencode-variants.test.ts apps/web/src/lib/ai-gateway/providers/direct-byok/opencode-go.test.ts
  • pnpm --filter web typecheck
  • pnpm --filter web lint
  • git diff --check

@chrarnoldus chrarnoldus self-assigned this Jul 20, 2026
@kilo-code-bot

kilo-code-bot Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

The new REASONING_VARIANTS_MAX_HIGH_LOW variant set and its kimi-k3 mapping are self-contained and correctly ordered (max, high, low), 'max' is a valid ReasoningEffortSchema value, and splitting kimi-k3 from kimi-k2.7-code does not affect any other model-matching branch or downstream variant consumer.

Files Reviewed (1 files)
  • apps/web/src/lib/ai-gateway/providers/model-settings.ts

Reviewed by claude-sonnet-5 · Input: 24 · Output: 4.9K · Cached: 474.6K

Review guidance: REVIEW.md from base branch main

@chrarnoldus chrarnoldus reopened this Jul 20, 2026
@chrarnoldus
chrarnoldus enabled auto-merge (squash) July 20, 2026 09:31
@chrarnoldus
chrarnoldus merged commit c99f204 into main Jul 20, 2026
28 checks passed
@chrarnoldus
chrarnoldus deleted the feat/kimi-k3-max-reasoning-default branch July 20, 2026 09:39
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