Skip to content

fix(opencode): preserve reasoning_content for Moonshot/Kimi tool-call assistant messages#31995

Open
XenoAmess wants to merge 1 commit into
anomalyco:devfrom
xenoamess-fork:bug-kimi-reasoning_content-missing
Open

fix(opencode): preserve reasoning_content for Moonshot/Kimi tool-call assistant messages#31995
XenoAmess wants to merge 1 commit into
anomalyco:devfrom
xenoamess-fork:bug-kimi-reasoning_content-missing

Conversation

@XenoAmess

@XenoAmess XenoAmess commented Jun 12, 2026

Copy link
Copy Markdown

Issue for this PR

Closes #29619

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

to fix this:

thinking is enabled but reasoning_content is missing in assistant tool call message at index xxx

屏幕截图_20260612_113124

see this

BerriAI/litellm#21672

How did you verify your code works?

we re-ask the failed question and this time it can work.

Screenshots / recordings

image

after the fix it can work.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

If you do not follow this template your PR will be automatically rejected.

@github-actions github-actions Bot added needs:compliance This means the issue will auto-close after 2 hours. needs:issue labels Jun 12, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@github-actions

Copy link
Copy Markdown
Contributor

The following comment was made by an LLM, it may be inaccurate:

Potential Related PR Found:

Additionally, there are related PRs that deal with reasoning/thinking content preservation:

These suggest an ongoing effort to properly handle reasoning content across various LLM providers.

@github-actions github-actions Bot removed needs:compliance This means the issue will auto-close after 2 hours. needs:issue labels Jun 12, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thanks for updating your PR! It now meets our contributing guidelines. 👍

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.

[Kimi K2.6] reasoning_content is missing in assistant tool call messages

1 participant