Skip to content

Limit work context planning to selection facts - #8295

Open
huangruiteng wants to merge 1 commit into
mainfrom
codex/scope-work-context-planner-20261107
Open

huangruiteng wants to merge 1 commit into
mainfrom
codex/scope-work-context-planner-20261107

Conversation

@huangruiteng

Copy link
Copy Markdown
Collaborator

变更

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。这里度量内部传输,模型实际用量和任务收益需要另行验收。

验证

  • 真实 quota 路径覆盖 File 和 SQLite、选中 Todo、完整验收来源、必读失败及恢复。传输断言在旧实现上因包含无关字段而失败,修改后通过。
  • 扩展源码验证 38 通过。六个预存用例因期待旧 captured_decision 字段失败,已在未改动基线逐项复现。隔离安装 wheel 的四个规划与验收用例全部通过。
  • 标准本地预合并 6 项直接检查、10 项选定检查通过。

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

@loopx-agent loopx-agent left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

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_context L90 保留原 decided packet,向 planner 传 selected Todo、effective action、adaptive 的 schema/mode/primary id 和 selection_required。后续完整读取与错误投影仍走原路径。
  • planInteractionWorkContext L831 本 PR 未修改它;真实36组输入覆盖 selected/无 selected、adaptive/legacy、选择条件及 settled-skip,完整返回值在不可变基线、head、安装 wheel 和实际 main 合并树上相同。
  • _source_content L21 仍从 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.

This branch has not been deployed

No deployments
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