Skip to content

fix(kosong): serialize reasoning-only assistant content#1738

Open
yearth wants to merge 1 commit into
MoonshotAI:mainfrom
yearth:fix/reasoning-only-assistant-content
Open

fix(kosong): serialize reasoning-only assistant content#1738
yearth wants to merge 1 commit into
MoonshotAI:mainfrom
yearth:fix/reasoning-only-assistant-content

Conversation

@yearth

@yearth yearth commented Jul 15, 2026

Copy link
Copy Markdown

Related Issue

Resolve #1404

Problem

See linked issue.

What changed

  • Serialize assistant messages with explicit empty content when they contain only reasoning and no tool calls, while preserving the separate reasoning field.
  • Add regression coverage for interrupted sessions that resume from reasoning-only assistant history.

Checklist

  • I have read the CONTRIBUTING document.
  • I have linked a related issue, or explained the problem above.
  • I have added tests that prove my feature works.
  • Ran gen-changesets skill, or this PR needs no changeset.
  • Ran gen-docs skill, or this PR needs no doc update.

@changeset-bot

changeset-bot Bot commented Jul 15, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 1c36cf3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@moonshot-ai/kimi-code Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

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.

Empty assistant message after terminal crash causes 400 on session resume (DeepSeek)

1 participant