Repository navigation
Limit work context planning to selection facts - #8295
huangruiteng wants to merge 1 commit into
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.
No blocking implementation finding at 850025f68b950ff15cafe07bd4b7570bfa4b6a15. 代码结论 APPROVE;六项预存测试失败及严格质量门保持。
Reviewer: model_agent | gpt-6.1-sol | OpenAI | runtime_reported | reasoning_effort=max
动机 / Motivation
操作者延续既有 Goal/Todo 工作时,工作上下文规划会经过此入口。 过去规划器收到整份决定,连同并不读取的结算、调度与材料;当前 head 只送选择所需事实,最终来源与交给宿主的完整要求仍保留。 规划器在这里决定读哪个任务、是否读 User 待办,并不需要把已经取得的全部材料重新运送一遍。重复传输增加内部处理负担,却不会让它做出更完整的选择。
本次独立对照确认规划结果、完整验收与原任务绑定保持,实际观察到的规划参数显著缩小;没有据此宣称模型用量、总成本或成果提升。 本 PR 只减少既有规划 RPC 的输入,不裁剪最终决定、Goal 要求或必读材料,也不修改权限、开关、调度与结算。
改动思路 / Approach
继续使用同一个工作上下文入口、TypeScript 规划器和 canonical reader。Python 只在过桥之前构造较小的请求,选择、adaptive 主任务优先关系和 User 读取条件仍由原 TypeScript owner 判断。请求只保留它实际读取的字段,最终 packet 没有用这个小请求替换,因此完整来源并未随之缩小。
在既有 Python 传输适配器收窄请求,TypeScript 继续独占选择和 User 读取判断,完整来源仍由原 canonical reader 和最终捕获交付;没有第二份决策或手工同步状态。 当前只交付内部规划参数减量与相同行为,不改变材料交付、权限、产品操作或默认开关。 不应为这处传输优化复制 Python 决策规则,也没有必要重写整条上下文管线。将来规划器新增输入时需要同步检查 carrier 和实际调用方;这是一处明确的共同部署边界,没有新增版本兼容或手工同步状态。
具体改动 / Changes
完整差异为两个文件 +29/-2;生产代码 +14/-1,其余是测试观察及明确选择 raw/off 失败视图。没有新增命令、配置、持久格式或面向用户的操作。
关键代码讲解
attach_work_contextL90 保留原 decided packet,向 planner 传 selected Todo、effective action、adaptive 的 schema/mode/primary id 和 selection_required。后续完整读取与错误投影仍走原路径。planInteractionWorkContextL831 本 PR 未修改它;真实36组输入覆盖 selected/无 selected、adaptive/legacy、选择条件及 settled-skip,完整返回值在不可变基线、head、安装 wheel 和实际 main 合并树上相同。_source_contentL21 仍从 canonical owner 取得完整 Goal 与两 Agent 的累计验收。真实 CLI、记录式 Host、来源篡改拒绝、读取失败和恢复均另行验证,不能由“请求只剩四组字段”替代。
依据 docs/reference/protocols/turn-envelope-v0.md,固定于 32167518b3c2646fd9b6a3a6e0aaf74b258a7b70(原文):Selected work identity 已实现,实际任务和 adaptive 优先关系保持;Complete delivered sources 已实现,完整要求、User 范围及 Host 输入保持;Failed source recovery 已实现,失败仍保留读取义务并阻止依赖工作,恢复后可继续。
对主干的风险 / Risks
最强风险是传输减少同时丢掉主任务选择或原始要求。除了字段观察,本次比较了36组真实 adapter/TS 规划结果;File/SQLite 的完整验收、精确任务体、记录式 Host、篡改拒绝、IO 失败及恢复通过。新增工作、改变原任务、进入等待的六个真实反例在源码、wheel 和当前实际合并树均拒绝,原 capture、已记录 Host、验收与无重复效果断言保持。只在私有验证器中去掉已完整内联错误结果上失效的旧跳转,产品与仓库测试均未改动。
原测试文件必须如实保留失败:head 为8通过/6失败,基线为4通过/10失败。共同六项在未改的负例函数中期待旧 captured_decision,完整 traceback 仅归一 checkout、临时目录、对象地址及行号偏移后逐项相同;实际 raw/off 拒绝已内联完整决定,exit1、quota_action_selection_rejected、not_committed 均成立。额外四项基线失败来自原首个测试的 auto/raw 期待,本 PR 的显式 off 输入使该用例可读;同输入的私有基线对照也通过。没有将这些失败改判通过或豁免严格质量门。
其余验证:10项现有 Host 测试、改动 Python 的 Ruff、标准premerge6项直接/10项选定检查通过。wheel 生产文件与精确 Git head 字节一致,先核验 site-packages 导入;初次从 package 目录调用被真实 workspace guard 正确阻止,随后在独立 worktree 内的空调用目录通过,未关闭该 guard。另对已验证 main 87a153d4d206fdc9e466ae2ed4bf1381006d235a 及发布前更新的 21013c0c113445b71a30f6ce06ab61455b56f1f2 与 head 的真实无冲突合并树分别完成同样36组、完整读回及六个拒绝反例。后者包含选定工作后的 Explore 上下文组合与首读提示改动;这些字段仍由完整 packet 保留,不进入本 PR 的选择 carrier。没有把未集成提案当作已采用。
只观察到一项 IO 失败路径的规划调用:File 参数38,718B、SQLite38,744B降到各995B。这个计量是 RPC 参数 JSON,不是完整 wire、整轮准入、序列耗时或模型 token;作者的整轮比例没有被本次局部测量认证。托管 CI 未查询,真实模型、部署采用与完整成本收益仍未验证。
我的整体评价 / Verdict
现有 adapter 的小范围改动有实际调用方,也保留了原 typed owner;对解决已证实冗余而言,机制和维护成本相称。最有价值的简化已在同一调用点完成,不需要再加抽象或新的请求版本。
long_horizon 与 user_experience 均判断 preserved:完整要求、失败恢复和原任务约束保持,已知意图与授权直接复用,增加的交互轮次为零。没有渲染界面变化,因此 visual_review 不适用;真实模型理解、已安装宿主采用与总成本收益仍未测试。代码批准不能把六项原测试失败或严格资格变成通过,相关 owner 修复并重新资格后仍须核对 readiness。
English verdict: APPROVE for exact head 850025f68b950ff15cafe07bd4b7570bfa4b6a15. The bounded planner transport change preserves independently checked decisions and complete source delivery. Six pre-existing test failures and the strict qualification hold remain; no merge authority or total-cost benefit is claimed.
变更
Quota 工作上下文规划只读取所选 Todo、adaptive 主任务和用户待办读取条件。过去 Python 在两次规划调用中传入整份决策,连同结算、调度和材料。现在调用方只传现有 TypeScript 规划器使用的字段。任务优先级、用户 Todo 可见范围、必读材料和失败恢复仍走各自的原 owner。最终决策继续交付完整来源。
对照
对照
main@32167518b3c2646fd9b6a3a6e0aaf74b258a7b70,验证850025f68b950ff15cafe07bd4b7570bfa4b6a15。相同合成任务、渐进开启、auto 差分,File 和 SQLite 各两次。SQLite 两次规划请求合计约 89,059 → 1,306 B。准入总内部传输 616,367 → 528,614 B(−14.2%),均为 137 RPC。含三次读回和错误摘要重试的序列 1,021,610 → 933,857 B(−8.6%),工具输出均为 44,021 B。这里度量内部传输,模型实际用量和任务收益需要另行验收。验证
captured_decision字段失败,已在未改动基线逐项复现。隔离安装 wheel 的四个规划与验收用例全部通过。