Skip to content

fix: 丢掉空助手消息,并把模型输出预算提到环境变量 - #36

Merged
SATA260 merged 1 commit into
mainfrom
fix/empty-assistant-message
Sep 22, 2026
Merged

SATA260 merged 1 commit into
mainfrom
fix/empty-assistant-message

Conversation

@SATA260

@SATA260 SATA260 commented Sep 22, 2026

Copy link
Copy Markdown
Owner

改了什么 / What changed

  • 正文和 tool_calls 都空的助手消息不再落库,发给模型前也会丢掉;请求被取消时,空流不再当成成功回复。带 tool_calls 的空正文仍保留。
  • 模型预算改为环境变量,默认输出上限从 8192 提高到 65536,避免思考把正文预算耗尽。同时提高输入上限、轮数、工具次数和墙钟,并增加 LLM_THINKING。

怎么验证 / How to verify

  • cd server && go test ./...
  • pnpm test:client
  • 有真实 Key 时:cd server && go test ./pkg/agent/ -count=1 -run TestLiveBlankAssistantWire

相关 Issue / Related issue:Closes #35

Made with Cursor

空正文且没有 tool_calls 的助手消息会被网关拒绝;思考和正文共用的输出上限原先只有 8192,长任务会在正文出现前被截断。

Co-authored-by: Cursor <cursoragent@cursor.com>
@coderabbitai

coderabbitai Bot commented Sep 22, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 9750b786-4750-4c07-b6b0-916a315324b4


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@SATA260
SATA260 merged commit b518b5a into main Sep 22, 2026
3 checks passed
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.

[Bugfix] 修复中断/空流导致落库空助手消息并引发网关参数校验失败的问题

1 participant