Skip to content

fix(qoder): preserve user and tool-result images - #192

Merged
caigee-cmd merged 2 commits into
caigee-cmd:mainfrom
gruffwy:fix/qoder-image-preservation
Sep 18, 2026
Merged

caigee-cmd merged 2 commits into
caigee-cmd:mainfrom
gruffwy:fix/qoder-image-preservation

Conversation

@gruffwy

@gruffwy gruffwy commented Sep 18, 2026

Copy link
Copy Markdown

Summary

  • Qoder 的 user 消息原先经过 contentToString,导致 image_url 内容被丢弃。改用现有 normalizeContentForUpstream 保留文本与图片。
  • 工具结果中的 image_url 和 MCP image 数据会提取成 user 图片消息,放在完整、有序的工具结果批次之后;保留调用关联并继续丢弃 orphan 结果。
  • 添加直接图片、工具图片与 orphan 回归,以及中英文 changelog。
  • 独立基于 main,仅修改 worker 图片处理与测试,不包含 custom、namespace 或思考档位改动。

Test plan

  • go test ./...(Linux 容器,独立分支)
  • go vet ./...
  • cd worker && npm test
  • release-notes.py self-test / validate;git diff --check
  • cd frontend && npm run build && npm run lint(未改前端,留给上游 CI)
  • 差异人工检查及凭据模式扫描:无 token、认证数据、原始抓包或主机详情

集成版本已通过 Qoder 直接图片与图片工具结果的识别验证;独立分支使用不含真实凭据的 worker 回归测试。大图、重复图片等未在本 PR 中宣称全面覆盖。

@caigee-cmd
caigee-cmd merged commit 66b2ffb into caigee-cmd:main Sep 18, 2026
6 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.

2 participants