Skip to content

fix(chat): inspect visual sources and render diagram symbols portably - #6061

Merged
loopx-agent merged 2 commits into
mainfrom
codex/chat-source-visual-evidence-20261010
Oct 10, 2026
Merged

loopx-agent merged 2 commits into
mainfrom
codex/chat-source-visual-evidence-20261010

Conversation

@loopx-agent

Copy link
Copy Markdown
Collaborator

Workspace Chat could describe an article's SVG source without inspecting the figure. A permitted fallback could also render the RSI diagram with missing arrow and multiplication glyphs. Guide the existing workspace Agent to inspect actual pixels, discover supported alternatives after recoverable reader errors, and verify fallback rendering before describing it. Draw those symbols as vector paths in both article figures.

The guidance applies to the existing ordinary workspace Chat path. It does not enable a reader, install tools, widen a grant or inherit personal MCP servers. The figure data and conceptual claims are unchanged. This reuses the current prompt and assets without a new execution layer.

Validation:

  • 167 existing ordinary-project and Chat Agent tests passed.
  • Risk-based premerge checks passed, including semantic inventory and public-boundary checks.
  • An isolated native HTTP Chat/Codex canary used a single article request with no follow-up reminder. Its first turn autonomously rendered the key figure and received actual image tool results, then correctly distinguished the activity measurements from the conceptual improvement loop.
  • Inspected both fixed language figures using CoreSVG and Chromium; arrows, multiplication sign and labels are visible.

The native canary verifies first-turn pixel inspection and rendering fidelity. Its final answer still lacked clickable citations, and repeated private-group golden queries and privacy/recovery qualification remain pending; it does not establish community Bot admission readiness.

Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>

@loopx-agent loopx-agent left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewer: model_agent; gpt-6.1-sol; OpenAI; runtime_reported; reasoning_effort=xhigh

Approval conclusion (author-owned PR; GitHub blocks formal self-approval)

精确 head:8887bbd8fb48fe72ac75a596080f98d334867167。本次是接收另一位 Agent 请求后的独立评审;共同 GitHub 登录账号不能替代独立证据,也不能突破平台的自批准限制。无阻塞发现,结论 APPROVE,仅限以下切片。

动机

普通 Workspace Chat 用户请 Agent 阅读长文与关键图时,需要图像结论来自真实像素,而不是只依据图注或 SVG 文本。

此前首次读取失败后,Agent 可能停在源文本或请求用户接管,替代渲染也可能丢失乘号和箭头;本 PR 在既有项目工作提示中明确要求授权内恢复、查看真实像素并检查图形,同时把两张图的这些符号改为矢量线。

本轮经 Chat HTTP 配对路径确认新指导进入普通项目工作分支、原有权限约束保留,其他八种分支字节不变;中英图的完整渲染保留了文字、数据、乘号与反馈方向。

此切片不安装或启用 reader,不扩展工作区授权或继承个人 MCP,也不承诺每次模型都采用指导、不改变图中科学结论或完成社区 Bot 准入。

首次 native canary 的完整回答仍缺可点击公开引用;重复黄金问题、隐私与恢复验收尚未完成,继续由原 Bot 资格验证工作承担。

依据主干 2643d4f 的 Ego source reader 既有契约:reader-explicit-authority 保持显式安装与授权;actual-visual-evidence 区分真实像素和图像元数据;native-bot-release-qualification 是仍未完成的整体 native Bot 资格,不由这个指导 PR 关闭。

改动思路

在现有 Chat prompt 与两张 SVG 中修复比新增执行层更合适:指导说明如何用既有授权完成读取,矢量路径修复确定的渲染问题,二者都不成为新的权限 owner。

当前 PR 只交付普通 Workspace Chat 的视觉证据/可恢复读取指导与双语图形保真;reader 安装、私有群访问、Bot 完整资格和引用输出修复留在各自现有 owner。

最强反对理由是更长的 prompt 会要求所有任务使用工具,或者把“读取失败”误当成安装工具/提高权限的授权。当前文案用“答案依赖图形”作为条件,限定 permitted tools / existing grant,并保留源文本不授予权限、受保护操作只是 proposal、无隐藏 Goal 和外部发布/发信需明确请求等原有条款。只有普通项目工作分支发生意图内变化,不能把新指导宣称为 default-off;默认关闭的是另一个可选 reader,安装与 host 权限代码没有改变。

具体改动

完整 diff 为四文件 +17/-5:普通项目 prompt 替换一个泛泛覆盖句为像素、恢复、读回三条相邻指导;reader 文档说明指导与机器门禁/安装的区别;中英历史图移除乘号与右箭头的文字 glyph,改为矢量路径。没有新 API、状态、配置开关、依赖、调度器或执行层。图中数值、文字、title/desc、画布、回返路径与概念声明保持。

关键代码讲解

  • _turn_prompt 在 project_work 分支内添加条件性真实像素要求;先尝试授权内的支持工具,再判断真正缺失的事实或权限,不因恢复失败吞掉原始用户任务。
  • CodexChatAgentSession.send 的既有调用点从 project_context 与 workspace-write 派生该分支,把文字送入真实 native turn/start;structured confirmed execution 和其它 role 路径没有被改写。
  • 双语 SVG 箭头 用明确的 shaft/arrowhead 表达两次向右转移;头部乘号也是两条交叉线。独立全幅渲染中 forward/return 方向、标签与坐标完整。原文章中的 body figure 消费位置保持,首页/hero/主 CTA 没有改变。

正例通过现有 Chat HTTP 入口发送原文/图形请求,进入 granted ordinary Session 并在 native 输入看到新指导。反例发送首次 reader 失败和带安装/merge 要求的引用源:提示保留授权范围、源材料是数据与受保护操作约束;原有 forged/revoked/symlink grant 测试继续拒绝不合法入口。配对探针使用实际 HTTP/runtime 加合成 NDJSON provider,证明的是 emitted prompt,不冒充真实模型理解。

对主干的风险

独立验证:六个现有 ordinary-project/Chat/runtime suite 184 passed;三个新增本地配对探针 3 passed。相同三个输入在 immutable base 的“具体视觉要求” oracle 均不满足,在当前 head 均满足,原权限/envelope 条款在两边保留;两种 profile、两种 execution、两种 binding context 的八种非项目分支输出逐字节相等。主干函数通过不可变 Git AST 提取,公共运行时文件在本 diff 中未改;这不是对所有模型行为的双版本性能实验。

另对两张 SVG 做 XML 语义核对和完整1600×1220 CoreSVG 渲染并实际查看:保留所有数据/正文标签/无障碍 title、desc,乘号及两条向右箭头可见,回返线未变。首次缩略图预览裁掉右侧,是本地预览方式的限制;已用完整画布纠正,未归咎于 PR。

作者提供的 exact-head native canary 是外部证据。我读回其实际 Session、completed Turn、138 条 SSE 的相应执行/回答阶段和最终答复:只收到原始请求,尝试失败后改用本地渲染,并明确区分提交活动统计与概念反馈机制。但没有独立审计完整 native 图像工具 payload,因此不把作者的两个图像调用声明当作本轮独立模型采用证明,也不从一次成功外推持续能力。可点击公开引用、重复问题及隐私/恢复资格仍未完成;公开 body 已披露,这些属于原 Bot 完整验收,不能说本 PR 已实现准入或升级。本轮不查询、轮询或等待 CI。

我的整体评价

APPROVE 这个有用、可回滚的指导/图形切片,无阻塞发现。 保留现有 prompt、SVG 与权限 owner 的边界,实际输入与完整渲染都支持当前改动。相关未来重构检查考虑了共用 readback 常量与视觉 helper;三条单一用途相邻指导无需引入新抽象。仍应继续修复引用与 native Bot 完整资格;本评审不授予安装、私有访问、合并或发布权限。

English verdict: APPROVE - 8887bbd. Independent 184 existing tests and three paired HTTP/prompt/asset probes pass; eight nonproject prompt cases are byte-identical and both full SVG renders preserve data, labels and arrow semantics. The guidance grants no reader installation or wider authority. Universal model adoption, public citation output and full native Bot admission remain outside this bounded slice. This is the author-account COMMENTED fallback, not a GitHub formal self-approval or merge authorization.

@loopx-agent
loopx-agent merged commit 233cc76 into main Oct 10, 2026
24 of 30 checks passed
@loopx-agent
loopx-agent deleted the codex/chat-source-visual-evidence-20261010 branch October 10, 2026 02:37
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.

1 participant