Repository navigation
Skip quota source transport for status-only context hooks - #8290
Conversation
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
loopx-agent
left a comment
There was a problem hiding this comment.
Reviewer: model_agent | gpt-6.1-sol | OpenAI | runtime_reported | reasoning_effort=max
动机 / Motivation
普通 guard 的状态回读不应重复传输完整 governing decision。原校验即使没有上下文、指令或结果字段,也会发送来源并计算后续不会使用的 digest。当前提交省略这类状态结果的来源传输,保留 slot、预算和结果形状检查。本次范围是现有共享校验与重放入口。真实模型采用、完整 Turn 时延和总成本仍未验证。
接受基准是 32167518b3c2646fd9b6a3a6e0aaf74b258a7b70 的 docs/reference/protocols/turn-envelope-v0.md,尤其 Optional memory participation:状态 readback 本身不能启用 Host contribution;真正贡献仍须绑定 governing signature、来源 readback,并保留大小、权限与原结算边界。
改动思路 / Approach
Python 只根据贡献的三个既有字段是否为空选择传输形式,TypeScript 继续拥有校验和来源签名。状态专用结果在原校验入口返回,不成为可重用 contribution;带上下文、指令或结果 schema 的结果继续走原来源绑定。若状态专用请求夹带 source 或贡献,TypeScript 拒绝。无需新增 capability、配置、权限或另一套 digest owner。
具体改动 / Changes
实际合并增量为两个生产文件和两个测试文件、+51/-3。Python 的共享 dispatch 发送状态专用标记或原 source;TypeScript 在完成原 slot、字段和预算检查后,决定是否计算贡献 digest。恢复与投影入口仍重新验证 source、readback 和贡献 digest,无法把状态回读伪装为可重用上下文。
独立 File/SQLite CLI 对照覆盖停用、仅声明可用、启用但无目标、普通 note、显式目标及随后新增的普通任务。两版选择、准入、readback、指引、指令和结果字段相同,guard 均零扣额。SQLite 停用样例仍为 85 次调用,参数 JSON 从 277,607 B 降至 174,414 B;显式目标样例的有效上下文继续保留。隔离安装的 wheel 也跑过同组 SQLite 场景。这里的字节数是固定样例的内部传输观察,不是整轮加速结论。
对主干的风险 / Risks
关键风险是把真正贡献误判为状态,或让未绑定状态进入恢复后的 Host 输入。当前正负检查覆盖三个贡献字段、错误 slot、预算、伪造重用及来源改变;共享代码仍在原异常边界隔离可选能力失败。基线与当前源码各通过 61 项核心上下文测试、213 项 Anti-Entropy 检查及 47 项反馈检查;其余两项反馈路由测试在两版均因原模块缺少 attach_turn_context 失败,相关模块与测试未改动。完整失败已保留,未采用作者的排除过滤。
当前头的 TypeScript 20 项、mypy、工作流范围 Ruff 和类型检查通过;标准本地预检 6 项直接检查和 10 项所选检查通过。对实际主干 df67371c374a5ece11c1f6ceddae11b51e0f0330 的隔离合并树也验证了共享上下文路径。托管 CI 未查询;完整反馈套件的两个既有失败仍保留为本地资格门。
语义与 CI 对齐
状态专用标记是同时交付的本地传输分支,不能激活 Host 输入或 provider 效果。有效贡献、原回执、重放和独立 attestation 仍沿用原 owner。测试通过、内部字节减少和打包运行均不证明真实模型已采用上下文。
我的整体评价 / Verdict
批准当前提交的代码评审。共享入口的传输简化有真实调用对照,关闭和有效贡献两条路径保持各自语义,新增判断仍由已有 typed owner 校验。变化规模与重复传输问题相称。两项基线失败、平台审批及完整成本判断保留,不将局部检查或批准等同合并资格。
English verdict: APPROVE for 9537687bee3de03d9c60644353aec740a837b177. Status-only transport is reduced while registered contributions retain their source and replay checks. Two unchanged routing-test failures and merge qualification remain separate holds.
Change
Validate status-only Turn context results against their registered slots and byte budgets without sending the governing decision through the effect runtime. Context fields, instructions, and result schemas require the existing source binding. Replay still verifies the contribution digest against the current source and readback.
Verification
Base tested
32167518b3c2646fd9b6a3a6e0aaf74b258a7b70. The measured reduction is internal transport on this replay.