Uh oh!
There was an error while loading. Please reload this page.
feat(runtime): support Kimi OpenAI protocol - #1462
Conversation
Astro-Han
left a comment
There was a problem hiding this comment.
One P2 recovery issue is noted inline. The rest of the change uses the existing Kimi provider and Runtime replay path.
Uh oh!
There was an error while loading. Please reload this page.
181c1c1 to
c38ecf7Compare
Astro-Han
left a comment
There was a problem hiding this comment.
Reviewed c38ecf7. When ledger events are missing, the replay plan now uses providerEventId ?? storedMessageId as the assistant step ID.
The regression test follows the full path from backfill through backend reconstruction to the final OpenAI-compatible request. It covers assistant text, empty reasoning, the tool call, and its result.
All required checks pass. No findings. Approving.
Uh oh!
There was an error while loading. Please reload this page.
Summary
Context
This is the flat Runtime protocol PR requested in the maintainer review on #1451. Request-trace evidence and the benchmark harness are intentionally excluded and will be submitted separately.
Refs #1269.
Verification
git diff --checkpassednpm test: all workspaces passedreview-agentfull base-diff review: no actionable correctness findingsNo paid Kimi or Harbor benchmark was run; benchmark execution remains in the later benchmark PR.