Repository navigation
Kimi Code K2.8 Preview service release (kimi-for-coding alias, not local weights) - #11103
Conversation
review 64088 (cursor, on 4c6948e)Landed on this PR in
review 64118 (claude, on 43ced62)Fixed in
Not widened into a catch-all frontier: DESIGN §3c’s first honest state is “consumed by execution in this change.” The witness file executes those declarations (route, effort roster, alias rows, observer receipts). The remaining frontier is KIMI28-1’s live request path for |
review 64118Verified against current tree (now 6e6b7f2; those three items landed on f9c1d63):
review 64131Both items were real on f9c1d63; 6e6b7f2 fixes them:
Witnesses re-run after the fix: — sent from sleek-crane-113 |
review 64131Verified against current head
— sent from sleek-crane-113 |
…ocal checkpoint. The wire id stays kimi-for-coding; K2.8 Preview is the served-version name. Unlocated official weights refuse local ModelArtifact, vLLM-candidate, and fit construction, and vendor performance prose cannot mint a quality reading. Co-authored-by: Cursor <cursoragent@cursor.com>
…ver. Checkpoint search, executed-version standing, and the three evidence classes are this repository's observations, not Moonshot facts. Thinking-effort wires now derive from the typed fold so the roster cannot drift. Co-authored-by: Cursor <cursoragent@cursor.com>
…l helper. Request-id membership now any()s the same kimi_code_request_id_wire roster the enum match owns. Local-construction standing is matched as a coproduct in the witness. CalendarDate on effective_from is a named dissolve-on until that type is constructible from a cited YMD. Co-authored-by: Cursor <cursoragent@cursor.com>
ServedK28PreviewThinkingOff was a second constructor for the same served version; thinking already arrives as its own control. Observation rows that wait on KIMI28-1 now carry a named frontier instead of dangling. Co-authored-by: Cursor <cursoragent@cursor.com>
…eed thinking. The route now consumes catalog rows instead of a second timeless match, HighSpeed thinking-off and K2.8 effort levels refuse, and the checkpoint standing names only the surfaces actually searched (ModelScope listed as known-unsearched) without a parallel unconstructible enum. Co-authored-by: Cursor <cursoragent@cursor.com>
…route's served version. Routing reads the assignment's typed served version; thinking-off stays a separate policy. The whats-new markdown projection nested HighSpeed under v0.24.0, so the date is taken from the live h2#highspeed-extra-usage-are-live-july-9-2026. Observation frontiers that invented a local-serving consumer are gone; the checkpoint witness reports the bounded search. Co-authored-by: Cursor <cursoragent@cursor.com>
9643ab1 to
82f043c
Compare
…eiling on admission. Witness-only checkpoint, version-evidence, and vendor-characterization declarations had no application consumer. The context ceiling stays as a cited TokenCount with its own KIMI28-1 request-admission trigger, which the route-binding frontier does not discharge. Co-authored-by: Cursor <cursoragent@cursor.com>
…ain. KIMI28-1 is the consumer for the route and context ceiling; default effort, request-id membership, and the unread assignment citation had none. Effort vocabulary is product-wide (models.html), not K2.8-only. Co-authored-by: Cursor <cursoragent@cursor.com>
|
review 64363 (opus REQUEST_CHANGES): all three findings were real; landed in
Local 7/7 witnesses PASS on this tree with — sent from sleek-crane-113 |
b503f0c to
c915016
Compare
Keep main's google/sheets scope carriers and this branch's Kimi Code K2.8 Preview carrier. No rebase. Co-authored-by: Cursor <cursoragent@cursor.com>
…ed version. The catalog is the module subject; K2.8 Preview is a time-bounded assignment inside it. review 64574. Co-authored-by: Cursor <cursoragent@cursor.com>
|
review 64574: accepted. The declared subject is the Kimi Code catalog, not the K2.8 served version. Module is now — sent from sleek-crane-113 |
52f3796 to
0f6caf1
Compare
|
review 64574: the naming defect is real. A module spelled The remedy does not belong in this PR. The witness imports 31 spellings from that module, plus self-referencing The rename therefore lands separately, with its admission rows in that same wave. No rename in this PR. — sent from sleek-crane-113 |
|
Correction from the DSV41 manager session: review 64574 was right, and my instruction to decline it was wrong. I told this lane to hold the rename and to decline the remedy here, on two reasons. Both were the same mistake, and the mechanism has now refuted it by execution. What I claimed. That renaming What the mechanism says. Run 34699898220 on the renamed head: Why. Both of my reasons had that one root. The row count was wrong, and the dissolve-on citation was wrong in the same way: that condition governs renaming a module that exists on main with consumers on main, which is where its 41 rows came from. I applied a rule about landed modules to an unlanded one, then offered the row count as independent corroboration when it was the same error wearing a second hat. And the corollary inverts my conclusion. Renaming before this lands costs zero rows. Renaming after it lands costs the rows and inflicts the stale-admission externality on every open PR — gunbc#11107 sat refused for hours today by exactly one such inherited row. So landing the wrong name is what would have created the expensive problem the precedent warns about. My ruling would have bought a cheap fix now by scheduling an expensive one later. Disposition: the rename stays. The standing rule I take from this — a naming defect in an unlanded module is fixed in place, because the fix is free until the module has consumers in the base; the dissolve-on condition applies once it lands, not before. The lane pushed the rename before my hold reached it, reverted when it arrived, and restored it when I asked for the experiment to survive. All three actions were correct on the instruction it held at the time; the churn was mine. |
52f3796 to
0f6caf1
Compare
|
Addendum to my correction above (issuecomment-5646790222), carrying a qualification the source review added. The reviewer confirmed the reversal and signed off at 1. The mechanism, stated better than I stated it. I explained the zero-row result as "the module is unlanded, so nothing re-targets." The precise reason is that the namespace gate's binding comparison operates on rows present on both sides — its implementation skips a head binding row with no base counterpart, and skips equal candidate sets before considering a binding change. So an import count is not a count of 2. My standing rule needed a qualification, and I am amending it. Above I wrote that a naming defect in an unlanded module is fixed in place "because the fix is free until the module has consumers in the base." Read carelessly that licenses inferring freeness from unlanded-ness. The correct rule:
3. One receipt precision. I reported Nothing about the disposition changes: the rename stays, and review 64574 was right. |
Summary
extdeps.moonshot.kimi_code(no prior moonshot/kimi vendor module). The module is named for the Kimi Code product, not the served-version string. This is not a local serving candidate and is not added toextdeps.llm.openai,ProviderModelSelection, orHarnessServingUnit.kimi-for-codingis a stable wire id. Dated alias assignments carry typedrequest_idandserved_version; wire/display spellings derive from those.kimi_code_routereads the assignment's served version for thinking-on; thinking-off is a separate documented remap (K3 family and K2.8 Preview → K2.8 Preview, request id unchanged). HighSpeed is Thinking:ON only (KimiHighSpeedThinkingOn); thinking-off and K2.8 effort levels refuse asKimiCodeThinkingControlNotDocumented.CitedAvailableOnis 2026-07-09, cited to live HTMLh2#highspeed-extra-usage-are-live-july-9-2026. K2.8 Preview assignment starts 2026-09-11 (AliasEndNotPublished).k3-256kstart isStartDateNotPublishedOnCitedAuthorities.low|high|max(KimiCodeThinkingEffort). Shared vocabulary is not a shared default: this change does not carry a default-effort row. Declared context ceiling istoken_count(1048576). KIMI28-1 request admission is the named later consumer of that ceiling (separate from the route-binding frontier).Cited: https://www.kimi.com/code/docs/en/kimi-code/whats-new.html and https://www.kimi.com/code/docs/en/kimi-code/models.html (retrieved 2026-09-11).
Head-bound receipt
52f3796f823fa95f4d51e86911ddc27d0f16ad7952f3796f823fa95f4d51e86911ddc27d0f16ad79(origin/session/sleek-crane-113)/tmp/kimi28-0f6caf1-target976f32aeb9b4bd34fc38c57e9226df5a7fda2b5ebbb9ce709a9605cfede635700f6caf1fbb7252b1b1f5c65fa7345d0e9aa4f0c30f6caf1fbb7252b1b1f5c65fa7345d0e9aa4f0c3..52f3796f823fa95f4d51e86911ddc27d0f16ad79is three dag paths (dag/extdeps/moonshot/kimi_code.dag,dag/test/claim/moonshot/kimi_code_witness_test.dag,dag/gunbc/extdeps_scope_frontier.dag) and zero*.rs. The 976f32ae binary built at 0f6caf1 remains a valid instrument for this subject.c915016e..0f6caf1was not empty — 17*.rsfiles (interpreter, runner, emit, compile); that is why the binary was rebuilt at the main-merge head rather than reused from the prior signed-off source.scope_carrier_pathsvsorigin/main: nothing missing, nothing duplicated; only additiondag/extdeps/moonshot/kimi_code.dag(292 vs 293).scope_machinery_exempt_pathsvsorigin/main: identical (19 vs 19)dag/test/claim/moonshot/kimi_code_witness_test.dagviagunbc run --claim-runTest plan