Conversation
Live verification 260921: k3-256k is the same K3 served under the explicit ceiling id - the same 988-token scaffold and identity answer as bare k3 on the same input. The subscription endpoint lists it alongside kimi-for-coding[-highspeed] and k3, but the opencodex picker and the expected-prices overlay only knew k3 and k3[1m], so usage logged under k3-256k showed as unestimable. - KIMI_CODING_K3_MODELS gains k3-256k, so the picker, context windows (262_144, the advertised ceiling), reasoning ladder and locked-parameter lists all derive it automatically. - expected-prices gains kimi/kimi-code entries at the same KIMI_K3 rate (input 3 / output 15 / cacheRead 0.3), sourced as verified-derived with the live probe note. - Parity and overlay-membership tests updated for the new id. Verified: provider-registry-parity + codex-catalog + usage-cost + model-rename-migration 539 pass, tsc clean. (cherry picked from commit 5910394)
Per maintainer review on #5447: the overlay test only bumped the row count 126->128, so dropping the new k3-256k rows and adding any other two rows would still pass. Name kimi/k3-256k and kimi-code/k3-256k in the mandatory list so the test fails if either row disappears. (cherry picked from commit c0484f4)
Adds a dedicated OAuth preset that routes the Kimi subscription account over the Responses wire, following the same shape as zai-responses and volcengine-plan-responses: - Shares the kimi OAuth account via oauthId, so a user who already logged in with ocx login kimi can add this preset without a second login. - Carries identical model metadata to the Chat preset (kimi-for-coding default, 1M context, adjustable low/high/max ladder, image input). - requiresAdjacentResponsesToolResults is live on this wire (#4726); the Chat preset keeps the flag inert. - Thinking content stays encrypted server-side on Responses, while tool calls and results stay visible; the Chat preset remains the default for plaintext reasoning. Parity tests assert the preset's wire, oauthId reuse, model roster, context window, reasoning ladder, and featured=false. Bracket-suffix stripping and jawcode alias lists pick up the new id automatically. Docs updated in EN and zh-cn. Verified: provider-registry-parity + codex-catalog + model-rename-migration 441 pass, tsc clean, docs build exit 0 (497 pages). (cherry picked from commit 6925faa)
…across locales Address grok-bot review on #5429: - The promptCacheKey comment in kimi-responses now states the actual behavior: only the Chat adapter reads the flag today; the field is kept because the Responses endpoint accepts it (live probe: prompt_cache_key round-trips 200), and enabling the adapter's send is a follow-up. - The EN provider list table gains a kimi-responses row (it previously appeared only in the auth comparison table); the zh-cn table gains the matching row in the same position. - The prompt_cache_key prose in both locales now says the key goes to the Chat Completions request only, with a note that the Responses wire accepts the same field but opencodex does not send it there today. Verified: docs build exit 0 (497 pages), provider-registry-parity 59 pass, tsc clean. (cherry picked from commit 36eef9d)
Keep the shared four-model roster consistent across Chat, key and Responses presets, and regenerate the provider metadata alias from the registry. Connect K3's explicit price namespaces to the current API reference rates, including the default 5-minute cache-write rate. These are comparison estimates, not Code Plan bills or quota. Remove the obsolete K2.7 price for kimi-for-coding, which now serves K2.8 Preview; unknown prices still obey routing policy and an operator modelCosts override remains authoritative. Preserve all five source contributions and their follow-ups as separate -x carries. Add cross-preset price, capability, unknown-cap and override regressions. Close the cost-cap fixture's actual SQLite history handle before removing its home, clear its cached health evidence, and settle pending ACL work without changing timeouts or swallowing cleanup failure. Update the provider and usage contracts and both affected documentation locales. Co-authored-by: panyuanyuan <panyuanyuan@hetao101.com>
…i-coding-roundtrip
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
✅ Deterministic PR hygiene checks passed. |
리뷰 · 우선순위 58 / 80이 PR은 Kimi Coding 쪽 두 갈래(#5447의
메인테이너의 판단이 필요한 지점
너의 추천 방향은 유지하세요. 머지 전에 #5447·#5429를 무효/중복으로 닫을 계획만 잡고, 이 댓글은 grok-bot이 작성했습니다 |
✅ READY
Hygiene✅ Deterministic PR hygiene checks passed. |
|
Superseded by #5609, which is open and not yet merged. |
Summary
Consolidate #5447 and #5429 into one Kimi Coding catalog change. Register the native
k3-256kmodel and add an optionalkimi-responsespreset that shares the existing Kimi OAuth account while keeping Chat as the default. Preserve all five original contributions, including review follow-ups and author attribution.kimi,kimi-code, andkimi-responses; regenerate the metadata alias and verify cross-preset parity and tool-result adjacency.k3-256kconsume different quota.kimi-for-codingnow serves K2.8 Preview, so its obsolete K2.7 price is removed from the existing presets and is not copied to Responses. Its cost remains unknown unless configured throughmodelCosts. Policies that exclude unknown prices or unknown cost under a cap may exclude this alias; allow policies and explicit user prices retain their existing behavior.Pricing/model identity was checked on 2026-09-23 against Kimi API pricing and Kimi Code model configuration. No K2.8 price or actual Code Plan billing equivalence is claimed.
Verification
c86bc00b531dba6f16f15b52954d16be20b2adc1; base devb1b60e880c61fbea12eed326ac43529cc393fb15. The last commit adds only the unchanged screenshot below; runtime, GUI source, docs and tests remain identical to5ebd93d.7e547bf02d8215f1a195a033158cdf0558a7aa8c, the unchanged wrapper over seven explicit provider/catalog/usage/cost-cap/metadata-sync/adjacency files completed 562 passed, 0 failed, 5,616 assertions, 36.10 seconds. Earlier failed runs were retained and are not included in this total. The dev merge to40e468cchanged only nine upstream CI/layout/documentation files; the Kimi contribution files, runtime source and test harness at that point remained byte-identical to the tested head. The subsequent correction below changes documentation totals and brand aliases, not the provider behavior.40e468chead, five explicit CI/workflow/layout files completed 173 passed, 8 skipped, 0 failed, 2,070 assertions, 4.14 seconds. The existing platform/dependency-dependent shell cases were skipped, not counted as passing.40e468cfound two missed integration obligations: translated preset totals and the Responses preset brand alias.5ebd93dupdates the 17 anchored guide/quickstart/structure statements to the registry-derived 97 total / 80 key / 13 OAuth split and wires the existing Kimi brand asset and label. The two failing test files now pass locally: 24 passed, 0 failed, 61 assertions. The production browser build and 497-page docs build also passed after this correction, using existing dependencies.5ebd93dcompleted successfully, including all four Linux shards, gates, docs and aggregate CI. Native/macOS jobs and Windows diagnostic shards were skipped by that run's path policy; this is not a full Windows/macOS test claim.ProviderRail/ProviderIconfixture was independently checked from its saved PNG pixels and matching DOM. All three Kimi marks and names/aliases are visible; the third mark has a complete 20x20 shape matching the first two. Source hashes match the final GUI source. The original composite viewport assertion failed, its exact failed subcondition remains unknown, and it was not rerun. This is bounded synthetic UI evidence, not live-provider or full-app interaction validation.UI evidence
Actual provider rail rows for
kimi,kimi-code, and selectedkimi-responses, using the shared Kimi brand mark. PNG SHA256:5977ce36680a496517cb7b50b9c734d3836d9cdb69694c79b1a708987e77e19b.The two original contributor-owned PRs remain open for maintainer coordination.
Checklist
Co-authored-by: panyuanyuan panyuanyuan@hetao101.com