Repository navigation
fix(collaboration): deliver original handoff in native Turn context - #7562
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
已授权的本地委派者与只消费标准原生输入的接收者。 此前只读标准输入的宿主缺少原委派;现在原消息和完整简报进入同一工作上下文与捕获,替换消息在启动前被拒绝。 真实 File/SQLite、实际 worker 与 Codex stdin 收到完整长消息;不依赖 bootstrap 也能处理原工作,恢复保持一次执行和结算。
旧 fixture 显式读 DELEGATION.json,因此能够通过,却没有证明标准输入包含原工作。同一 brief 下替换消息仍可启动,可能造成错误执行及再次人工转达。本 PR 只修原生输入,未交付无 Todo 激活、新调度器、真实模型或正式安装。 普通无 Todo 的请求、Turn 与结算身份,正式 App 原消息自然采用、真实模型及持续完整成本仍未验收。
改动思路 / Approach
复用既有来源、Turn 与捕获 owner;只增加原材料投影及实际调用接线,权限与结算仍由原 owner 决定。 当前 PR 交付完整原生材料输入;普通无 Todo 主体与结算、正式 App 采用继续归既有任务。
继续使用 Inbox、peer publication、source policy 和 Delegations 的当前授权校验。TypeScript 校验原 operation、recipient、Todo、workspace、Turn 及认证原消息;Python 读取既有 owner,再将材料交给标准 work_context 和唯一捕获。各宿主继续消费既有输入格式。
没有增加第二个 controller、状态账本或模型 reader。新增内部 locator 不授予权限;新 operation 记录所用输入 profile,历史 absence 保持原路径。已返回 Turn 的恢复只消费原捕获,避免重新读材料与结算锁冲突。
具体改动 / Changes
评审精确 head:36648713f3b3fea855d8a2b0a9548ec7f5efd07c。规范:docs/reference/local-delegation.md @ 0cfbf11。
Source use and recheck:implemented;Acceptance and return:implemented;Disconnect and recovery:implemented;Ordinary Inbox activation boundary:deferred。前三条以原材料输入、既有验收和同捕获恢复实证核对;最后一条由原 request/Turn/settlement 与 C1 工作继续。
关键代码讲解
- Delegations._execute:仅在 fresh 操作自动传 locator;恢复不改读材料。
- delegated_turn_context:读原 peer、认证来源和当前授权,核对实际工作区;配置不进入模型材料。
- delegationTurnWorkContext:用 typed 身份和原消息完全相等来投影原材料;错绑或替换直接拒绝。
已核对全部 11 路径、真实调用链及逐字段/逐条义务:新委派的完整材料自动进入捕获与实际输入;错绑或替换材料在 Host 前拒绝;历史 profile、既有准入/验证/回传与原 Turn 恢复保留。文档如实描述这项新操作默认行为,材料交付不等于模型阅读或执行授权。
独立不可变 main@0cfbf11d 的相同 8 个实际入口用例中,6 个违反原消息/原生输入 oracle、2 个已有拒绝通过,零 skip;失败分别来自缺失输入、仅用 stdin 的宿主无法获得工作、替换消息后仍启动,不是 import 或 unknown-flag 失败。最终 3664871 的非 editable wheel 全部 230 项通过,覆盖真实隔离 File/SQLite、CLI/worker、长中文/换行/引号、实际 Codex stdin、历史记录、错绑、撤权、原材料恢复和单次执行/结算。6 个产品文件与提交、wheel、安装一致,导入审计未逃逸回源码。
最新主干组合的完整 TypeScript 5617 过、75 条可选 provider 条件跳过;typecheck、配置 mypy、Ruff、语义 advisory、双语差分和公开边界检查通过。严格质量回执 cqr_ec2bbd98d68052e57cb8 有效;标准 premerge 的 6 项直接检查与 17 项选定检查全部通过,零失败、零人工 hold。集成前源码结果和中止的旧候选打包运行只保留其版本;历史 CI/额外 strict 失败没有改判为通过。
对主干的风险 / Risks
新增材料与当前来源读取会增加 fresh 操作的输入和 IO;持续模型、费用和整体成本收益未测。历史兼容分支仅服务已存在的 input profile 与实际 generic host;普通 CLI 不带 locator 时不增加此材料,resume 保持原捕获。错误原身份、来源缺失或替换仍拒绝;没有改 Todo、claim、lease、quota 或 source grant 的 authority。原生输入验证使用确定性进程夹具,不能代替正式 App 或真实模型采用。
本次移除了一个未被消费者使用、且已有捕获保护的重复材料 digest;更大的 request-subject/Turn/settlement 改造继续归既有任务与 C1 owner,不在这个修复中另起框架。
我的整体评价 / Verdict
这是可单独验收、可回退的完整原生输入修复;完整父请求仍开放。当前没有验证成立的阻塞发现。该结论是作者的精确版本审查,采用 COMMENTED 记录,不代表 GitHub 正式自批准、独立维护者批准、CI 全部通过、merge readiness 或安装授权。
Approval conclusion (author-owned PR; GitHub blocks formal self-approval): APPROVE.
English verdict: APPROVE the exact-head native-input repair; taskless activation, formal App/model adoption and complete-cost benefit remain open.
Goal And Delivered Outcome
已授权的本地委派虽然保存了完整 brief 和原始消息,原生 Turn 输入此前仍依赖宿主自行发现
DELEGATION.json。只读取标准输入的接收者会缺少原工作材料;相同 brief 下替换原消息也可能继续启动。本修复将经既有 owner 核验的完整 peer brief 和 Chat 原始消息交给共享工作上下文,进入同一不可变 Turn 捕获。新鲜输入在启动前拒绝缺失、替换或错绑材料;恢复使用原捕获。接续已合并的 #7531,独立基于
main@0cfbf11dcea4ba7bc8fb13f06f1c915b63ce8fad。Author Declaration
docs/reference/local-delegation.mdat0cfbf11dcea4ba7bc8fb13f06f1c915b63ce8fad,沿既有 Source use and recheck / Acceptance and return / Disconnect and recovery 契约;controller RFC 的原始材料与单一上下文 owner 方向。delegationTurnWorkContext→ 标准work_context→ immutable capture → actual Host stdinScope And Continuation
当前完成的是原生输入材料修复。Python 读取现有 owner,TypeScript 决定身份一致性;各宿主继续使用共享上下文,无新配置编辑器或恢复说明。内部
--delegation-context-json只是私有 Host locator,不授予执行权限。新 operation 记录输入 profile;既有 generic host 的 bootstrap 文件继续支持。普通无 Todo 的完整闭环及安装态原消息→原对话验收仍由既有工作承接;这次源码修复不宣布它们完成,也不声明长程收益或成本改善。
Validation
36648713f3b3fea855d8a2b0a9548ec7f5efd07c;集成前的489236ec5证据单列。旧 bundle 构建拒绝及标准重建过程保留。早期候选的已中止打包运行不作通过;#7531 由维护者合并后,新增修复已完整移到最新主干。历史额外 strict 失败和旧 CI 失败不改判;本 PR 尚无当前完整 CI 资格,不授予 merge/install。
Frontend / Visual Evidence
Type of Change
LoopX Area
Shared-authority RFC fixture impact
N/A:没有存储 provider、authority schema、路由或 promotion 变更;File/SQLite 实际调用用于验证本修复,未宣称 shared-authority 迁移进展。
Boundary Checklist