Skip to content

fix: address PR #159 #181 #154 - reasoning_content in toolloop, Discord MESSAGE_CONTENT intent, minimax config example#1538

Closed
xuwei-xy wants to merge 1 commit intosipeed:mainfrom
xuwei-xy:fix-pr-159-181-154
Closed

fix: address PR #159 #181 #154 - reasoning_content in toolloop, Discord MESSAGE_CONTENT intent, minimax config example#1538
xuwei-xy wants to merge 1 commit intosipeed:mainfrom
xuwei-xy:fix-pr-159-181-154

Conversation

@xuwei-xy
Copy link

Summary

Implements fixes from open PRs #159, #181, #154.

Changes

PR #159 - Preserve reasoning_content for thinking models in toolloop

  • Add \ReasoningContent\ when building assistant messages with tool calls in \pkg/tools/toolloop.go\
  • Fixes multi-turn conversations with thinking models (GLM-Z1, Kimi K2.5) when using subagent/spawn

PR #181 - Enable Discord MESSAGE_CONTENT intent

  • Set \IntentMessageContent\ in \pkg/channels/discord/discord.go\
  • Required to read message content for non-mention messages

PR #154 - Add MiniMax config example

  • Add minimax model example to \config/config.example.json\
  • MiniMax provider was already supported in codebase

Files Modified

  • pkg/tools/toolloop.go
  • pkg/channels/discord/discord.go
  • config/config.example.json

…in toolloop, Discord MESSAGE_CONTENT intent, minimax config example
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


OpenClaw-User seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@sipeed-bot sipeed-bot bot added type: bug Something isn't working go Pull requests that update go code domain: tool domain: channel labels Mar 14, 2026
@wj-xiao wj-xiao closed this Mar 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain: channel domain: tool go Pull requests that update go code type: bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants