Skip to content

fix: 修复 SuperGrok 付费账号多Agent模型空响应问题#406

Merged
chenyme merged 1 commit intochenyme:mainfrom
Huan-zhaojun:fix/supergrok-multiagent
Apr 1, 2026
Merged

fix: 修复 SuperGrok 付费账号多Agent模型空响应问题#406
chenyme merged 1 commit intochenyme:mainfrom
Huan-zhaojun:fix/supergrok-multiagent

Conversation

@Huan-zhaojun
Copy link
Copy Markdown
Contributor

Summary

修复 SuperGrok 付费账号使用多Agent模型(grok-4.20-beta、grok-4.1-mini、grok-4.1-thinking)时返回空响应的问题。根因是 Grok API 已迁移到 modeId 字段,付费账号的多Agent管线需要此字段才能正常初始化;同时 think_closed_once 门控在多 Agent场景下错误丢弃了思维链和正文内容。

Changes

  • 功能新增
  • Bug 修复
  • 重构/清理
  • 文档更新
  • 其他(请说明)

Related Issues

Verification

  • 本地运行验证
  • 单元/集成测试
  • Docker 构建通过
  • 未验证(请说明原因)

验证说明:

  • 付费 SuperGrok SSO + grok-4.20-beta/4.1-mini/4.1-thinking:从 0 tokens → 正常输出(350+ tokens)
  • 免费 SSO 全模型(grok-3/4/4.1 系列):无回归
  • 流式:单个 块 + 连续正文,无中途打断
  • 非流式:内容完整
  • Docker 镜像(ghcr.io)构建并运行验证通过
  • 使用浏览器抓包 + 直接 API 请求 + grok2api 服务三重验证

Breaking Changes

  - 新增 modeId 映射替代旧的 modelName/modelMode 请求格式
    (Grok API 要求付费账号多智能体管线使用 modeId 字段)
  - 移除已废弃的 enable420 标志
  - 修复 think_closed_once 门控导致多智能体思维链 token 被丢弃的问题
  - 使用 messageStepId 将 Agent 思考聚合到单个 <think> 块
  - 丢弃正文输出中途插入的孤儿思考 token(Grok 官网也隐藏这部分)
  - CollectProcessor 添加 fallback token 收集兜底非流式模式
@Huan-zhaojun
Copy link
Copy Markdown
Contributor Author

Huan-zhaojun commented Mar 26, 2026

#339
#382

(本 PR 覆盖了 #382 的修复范围,并额外修复了 StreamProcessor 思维链处理问题)

@imbytecat
Copy link
Copy Markdown

Xnip2026-03-30_18-02-56 用了大佬修复的镜像,但是好像输出的内容格式都有问题,感觉少了一些换行和空格之类的

@Huan-zhaojun
Copy link
Copy Markdown
Contributor Author

用了大佬修复的镜像,但是好像输出的内容格式都有问题,感觉少了一些换行和空格之类的

@imbytecat 收到🫡,我具体看看是什么原因,试着修复一下

shilong20 added a commit to shilong20/grok2api that referenced this pull request Apr 1, 2026
- app_chat.py: 新增 _MODE_ID_MAP,对多Agent模型使用 modeId 替代旧的
  modelName/modelMode(付费号多智能体模式必需)
- chat.py StreamProcessor: 用 _content_started + messageStepId 替代
  think_closed_once,修复多Agent思维链/正文被错误丢弃
- chat.py CollectProcessor: 增加 fallback_tokens 兜底,解决非流式模式下
  modelResponse.message 为空的问题

Made-with: Cursor
@chenyme chenyme merged commit d22a1d5 into chenyme:main Apr 1, 2026
7 of 8 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.

3 participants