mirror of
https://github.com/MoonshotAI/kimi-code.git
synced 2026-08-21 06:35:50 +00:00
e2fe62a5ef
Some checks are pending
CI / lint (push) Waiting to run
CI / test (push) Waiting to run
CI / test-windows (push) Waiting to run
CI / build (push) Waiting to run
CI / typecheck (push) Waiting to run
Nix Build / Check flake.nix workspace sync (push) Waiting to run
Nix Build / nix build .#kimi-code (push) Blocked by required conditions
Release / Release (push) Waiting to run
Release / Deploy docs (push) Blocked by required conditions
Release / Native release artifact (push) Blocked by required conditions
Release / Desktop release artifact (push) Blocked by required conditions
Release / Publish native release assets (push) Blocked by required conditions
* feat(agent-core): guide the model away from repeating denied or failed tool calls - system.md: add a diagnose-before-retrying paragraph next to the existing permission-denial guidance, covering failed tool calls - permission: when the user rejects an approval on the main agent, tell the model not to re-attempt the exact same call (sub agents already had an equivalent hint) * fix(agent-core): close abandoned tool exchanges and dedupe duplicate tool_use ids A turn that dies between a recorded tool.call and its paired tool.result (e.g. a transcript write failure mid-batch) used to leave pendingToolResultIds open forever: every later message was stranded in deferredMessages and user input was silently swallowed. - runOneTurn now defensively closes any dangling tool calls when a turn ends (completed, cancelled, or failed), synthesizing an error result that names the cause, with a warn log and a tool_exchange_abandoned telemetry event - the projector drops assistant tool calls whose id already appeared earlier (first occurrence wins): a duplicate id is wire-invalid on strict providers and not repairable by the strict resend; reported via the existing projection-repair log and telemetry - resume-side closePendingToolResults now logs what it closes (warn for a mid-history gap, info for the routine trailing interruption) * chore: add changesets for tool exchange fixes * fix(agent-core): scope duplicate tool_use id dedup to the strict resend Unconditional dedup regressed providers that emit per-response counter ids (e.g. call_0 in every step) and accept their own duplicates: later tool exchanges silently vanished from the projected history, and a duplicate call's own recorded result was left dangling. - the dedupe pass is now opt-in via dedupeDuplicateToolCalls and enabled only in strictMessages, so the normal projection keeps the history the provider produced - the pass also drops every tool result after the first for an id, so no dangling tool message survives; when the kept call has no result of its own, the surviving one is reattached by the adjacency repair - kosong now classifies the Anthropic "tool_use ids must be unique" 400 as a recoverable request-structure error so it triggers the strict resend |
||
|---|---|---|
| .. | ||
| e2e | ||
| fixtures | ||
| providers | ||
| anthropic-errors.test.ts | ||
| anthropic.test.ts | ||
| capability-providers.test.ts | ||
| capability.test.ts | ||
| catalog.test.ts | ||
| echo-provider.test.ts | ||
| errors.test.ts | ||
| finish-reason.test.ts | ||
| generate.test.ts | ||
| google-genai.test.ts | ||
| kimi-files.test.ts | ||
| kimi.test.ts | ||
| message.test.ts | ||
| openai-common-errors.test.ts | ||
| openai-legacy.test.ts | ||
| openai-responses.test.ts | ||
| provider.test.ts | ||
| regression.test.ts | ||
| scripted-echo-provider.test.ts | ||
| simple-toolset.test.ts | ||
| step.test.ts | ||
| stress.test.ts | ||
| strict-role-alternation.test.ts | ||
| tool-message-conversion.test.ts | ||
| type-safety-negative.ts | ||
| type-safety.test.ts | ||
| typed-tool.test.ts | ||
| usage.test.ts | ||