Repository navigation
docs(controller): define intelligent control mediation and full-cycle cost - #7517
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 · max
Approval conclusion (author-owned PR; GitHub blocks formal self-approval)
动机 / Motivation
实现或评审 controller/executor 架构、需要决定控制事务由谁判断的贡献者。 原文只概括 controller 读契约和提交操作;本次明确任务选择、连贯交接、必要材料与结果解释四项判断,并给出失败恢复和小任务反例。 双语 RFC 与路线图说明智能中介和完整成本的判断边界,索引账本数量与现有十八对文件一致。 本 PR 不实现双会话,不切换 writer、安装宿主或启动模型实验,也不把设计澄清当运行时收益。 C1/C2 的真实入口切换、原 quota 对统一单会话验收,以及后续双会话实际采用与完整成本对照仍未由本 PR 交付。
改动思路 / Approach
这次有价值的增量是明确“由谁花注意力作判断”,避免把 controller 理解为一次确定性封装或把 lossless codec 当成语义压缩。确定性的 composition 组合领域事实,codec 精确恢复 full/delta,controller 才按任务判断后继工作、材料和结果;Core 仍拥有合法状态迁移。既有 RFC 就是合适的 owner,增加第二份 schema、模型状态库或按关键词选择相关材料都会使恢复与权限更难判断,原文现明确排除这些路线。
导出产物有效但打包检查失败的例子把判断接到后继工作:保留有用修复和原未结算身份,交接有证据的打包纠正,再由独立验证和当前 Core 回执判断验收。若反证触及共同要求,先改任务并交付该要求;controller 丢失则恢复原 capture/session/attempt,不能让 executor 接管 Core。独立授权工作继续沿原路线,未加逐工具请示或全局停工。
具体改动 / Changes
评审准确 head:e1d2f0421516cecade81d16dbabbac03e3c587fc,base:b69021498ef7483b1d2b190bc3f4c7bb6675c882,完整六文件 +123/-4。独立参照是修改前已完整阅读的 docs/architecture/rfcs/controller-executor-unified-contract-v0.md,固定修订 b69021498ef7483b1d2b190bc3f4c7bb6675c882。这是依据既有接受边界判断新说明,并非用本 PR 重写后的条文证明自己已交付运行时。
关键内容讲解
- 1. Decision summary:implemented 于设计澄清层。可选双会话的 controller 仍是唯一面向 Core 的模型客户端;新增“智能中介”没有新增执行权限。原 quota 与统一单会话先验收的次序完整保留。
- 5. Proposed architecture:implemented 于文档层。新段落 的四行表分别对应选工作、连贯交接、必要材料及结果解释。原始材料可达、摘要不是完整阅读证明、executor 的局部工具自主和 Core 的 admission/claim/lease/budget 分工都保留。中英文对 controller 失败、共同要求矛盾、旧权限不能缓存等限定一致。
- 9. Validation and acceptance:implemented 为验收设计补充,未声称通过。新增 intelligent mediation 行要求实际决策后果,不能用少查询/短交接验收;成本段把 controller 推理、材料交接、验证、恢复、executor 和人工工作放进完整比较。实际重复消费仍计成本,同一用量回执转发则不重复计数;未知账单/缓存与 workload 差异保留。它明确不替代前置的 original-quota 对 unified single-session gate,与既有研究 measurement model 第 6 节的完整成本口径一致。
- 11. Normative delivery plan and refactor tasks:运行时实现 deferred,仍由既有 C1/C2 和原 #7430 等工作承担,没有另造路线图。两份 overall roadmap 链接新设计段落;两个 status 索引从 16 修为 18,与现有 ledger 的 36 份语言文件、十八对条目相符。
对主干的风险 / Risks
最强风险是新名称被解读为“第二个模型天然便宜”或“压缩后可丢约束”。新说明恰好把这些当可反驳假设:小任务、快速变化共同约束、频繁 controller 恢复都可能更贵;材料不明先补材料,权限/取消/收尾仍查当前 owner;executor 摘要不能重建失去的义务。用量回执的去重也没有免掉两会话真正重复处理的输入。仅自动双语文件存在检查不能发现漏译这些限定,因此我逐段比较了四行表、三类操作、失败例子及成本与前置 gate。
PASS:准确 base/head 双语 RFC diff、完整 docs-governance、十八对账本数量和新路由锚点核对、公开边界检查;标准 premerge 4 direct +14 selected 自动检查全部通过,零失败、零人工 hold。未查询或等待 CI,也未执行模型、benchmark 或运行时部署测试;这些不属于六文档变更的当前实现声明。最初误指不存在的治理脚本是评审命令错误,已使用仓库实际治理入口验证,未归因作者。
语义与 CI 对齐
结论为 aligned / reuse_existing:智能中介沿已有 controller 会话与 composition/codec/Core owner 补充说明,未新增 Enum、协议字段、安装指令或泛化权限。新增规范性验收描述属于后续实现应证明的设计要求;本 PR 的本地自动检查不能替代实际 executor 决策、原结果结算和已接受成果。
准确范围的 strict change-quality 回执 cqr_94e0a92dd5b3930e9f37 已记录并读回 valid;safe-fix 未应用,未修改作者源码。
我的整体评价 / Verdict
这是 justified_increment。long_horizon=preserved、user_experience=preserved:保留原累计要求、真实工作链、局部自主及既有 Goal Chat/settings/readback 入口;没有把 executor 变成控制面文书操作者。六个 Markdown 源文件未改产品呈现或交互,因此 visual_review=not_applicable;这里没有可用来宣称已安装的 UI 证据。
体量与设计歧义相称,复用已有 RFC 和路线图优于新增框架;无需为减少文档行数另起抽象。C1/C2 实现与真实 final model input、原 quota 对统一单会话验证、后续三组同条件完整成本/成果比较仍是具体未交付边界。未发现本切片阻断问题,同账号下以 COMMENTED 承载批准结论;这不是 formal approval、运行时 qualification 或 merge/安装/实验授权。
English verdict: APPROVE — e1d2f0421516cecade81d16dbabbac03e3c587fc: coherent bilingual design clarification with preserved authority, recovery and prior single-session gates; runtime adoption and full-cost benefit remain unqualified.
The controller/executor RFC assigned the controller session reading and typed Core operations, but left its intelligent mediation implicit. This revision explains which task, material and result decisions the session should absorb, separately from deterministic composition and exact delta encoding.
The English/Chinese RFC and roadmap now distinguish those responsibilities, illustrate a failed-check decision, preserve Core authority and original recovery/settlement identity, and define counterexamples and complete-cycle cost attribution. No overhead threshold, mandatory token limit, runtime/default change or experiment is introduced. The generated bilingual RFC index also corrects an existing ledger count from 16 to 18.
Validation on
e1d2f0421(baseb69021498): documentation governance, staged/exact-head bilingual checks and diff checks passed. Public-boundary scanning read all six files cleanly; the combinedloopx checkstill reports an unrelated unmigrated Goal authority error and is not reported green. Runtime tests were not run for this documentation-only change.This is an independently reviewable design refinement; it does not depend on #7430's unfinished runtime cutover or qualify C1/C2 implementation, installed adoption or net benefit. Reverting this commit restores the prior design text. The related refactor pass reused the existing ownership and measurement sections; no second protocol or parallel roadmap was added.