Repository navigation
refactor(replan): replace inline advice with captured review and writeback - #7197
Conversation
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
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
Approval conclusion — APPROVE at 2035dfcdf97d0659c8b4a2f28f58d551050f989d; no blocking finding in this source-level slice.
动机
执行重规划的 Agent,需要先看目标、此前决定和实际结果,再选择下一项工作。
原包把长指导、格式与累计验收混在决策入口;本提案把同一捕获拆成状态读取和写回格式,完整验收及原指导仍可按来源读取。
隔离 File/SQLite 路径能在不重新准入、不新增读回执的情况下读状态、拒绝变更的来源,再由原 writer 接受决定。
本轮只资格化源码级捕获 CLI 交互,不证明安装态、模型选择质量、token 节省或长程净收益;没有部署、模型调用或合并。
剩余缺口:安装态 App/Lark/native Host、真实模型决策与理解、可比长程完整成本和收益仍未验证;较宽原测试集的既有失败仍保留。
改动思路
由既有 TypeScript envelope/context owner 派生只读索引和分步视图,Python 只搬运文件;原始快照及 mutation-time owner 继续承担来源、权限与结算,未增加第二本账或另一次 guard。
当前 PR 的可交付边界是捕获 CLI 的结构化状态读取及原 writer 连续性;更广的模型采用、真实 Host 和可比完整成本沿现有 RFC §11/M3 继续。
| 路径 | 基线 | 当前提交 |
|---|---|---|
| 无捕获,stride 关闭或开启 | 完整原包 | 相同输入下整份 envelope 完全相等,含来源哈希 |
| 有捕获 | 完整内联指导与格式 | 同一来源的 review 状态,再读 writeback;权限和结算相等 |
| 累计验收 | 大段重复上下文 | 全内容哈希索引,完整目标/版本/要求可原样读回 |
| 以前有决定、stride 关闭 | 没有该反馈 | 新增通用反馈,但不启用 stride 决策/结算门禁 |
表格是源码与实际执行的结构对照,不是模型采用或理解的证据。分步读取增加真实 IO,不能只以入口变小宣称净成本下降;既有完整快照仍是指导、示例及遗漏上下文的可达来源。
具体改动
全量 14 files +684/-62,运行时集中在原 typed owner,另有配套 Python IO、三份文档及聚焦测试/公共合成夹具。
关键代码讲解
- replanReadCommands 只对成功、可读的非 settlement-only 捕获构造原路径和原哈希绑定的四种读取。
- replanView 先验证来源及义务关联,再分出状态、原写回、完整验收项和 Explore;相同 ID 的不同版本不被覆盖。
- read_replan_view 读取文件并调用 TypeScript,不借明细读取重新访问 registry、运行 guard 或写读回执。
- projectReplanContext 把既有决定与后来工作回顾开放给通用重规划;观测分类仍来自注册枚举,blocked/unchanged 不冒充先进展。
- project_replan_context 只传递历史及 Todo 事实;来源顺序、作用域与含义继续由 typed owner 决定。
规格是改动前 docs/architecture/rfcs/hierarchical-agent-stride-control-v0.md,固定修订 e3fa61956b6e374b4475162039b7bf9f4282be01;按原文条款映射:7.6 Research scope and evidence-conditioned control — implemented,决定、执行与收益保持分离,原决定引用与 unknown 语义保留;An iterative strategy and engineering boundary — implemented,权限、预算、原 writer 与结算不被读取取代;11. Smallest Useful Implementation Slice — deferred,真实 provider/Host、模型采用与完整成本仍由原 §11/M3 路径资格化。本补丁公开提出调整旧的 off-mode 上下文输出:新增通用反馈,绝不声称所有 off-context 逐字不变;关闭策略不会开启新的 stride 义务。
对主干的风险
最强反例是小包隐藏原要求,或未读状态便沿格式示例机械决定。当前完整捕获和可执行 full_decision 来源仍保留原指导,独立检查确认 staged entry/review/writer 本身不再内联 planning_guidance/decision_value_review,而该正文可从绑定原来源读到;这属于明确的投递顺序变化,不是内容等价或模型遵从证明。四组 immutable base/head 对照保留权限、写回、用户、调度与动作字段;保留相同原始输入后,两种 uncaptured envelope 连来源哈希均完全一致。空历史及普通历史 context 也完全一致;含旧决定的反馈与分类变化是本提案声明的增量。
独立执行了132项 TS、类型检查,以及较宽 Python 集的100通过/20失败;单独分步读取套件9通过,含真实隔离 File/SQLite、缺失 registry 下的只读、来源变更拒绝和原 writer 接受,9项与100有重叠,不相加。原20个失败发生在规范存储初始化前的 Todo 读取,完整诊断/节点/断言在不可变基线和本提交逐字相等。私有诊断改用既有 helper 初始化后,19项通过、2项仍在两版相同的 autonomous_replan_required Host hold 上失败;这不算已验证 native Host 采用,也没有改产品断言或门禁来变绿。
完整 digest-owner 检查仍16通过/1失败,两版唯一相同的 trajectory_input_usage.ts:24 匹配器是范围外既有问题。上述失败原样保留,并以全失败签名、相同夹具/环境、因果路径和另行通过的范围检查归因。精确范围 cqr_a3c66946d838e8ca23f8 为 pass,保留这些 failed/risk 观察;当前 premerge 的6个直接检查和4个选定检查通过,最初 stale-receipt 总失败仍保留。没有查询、轮询或等待远端 CI,未借代码批准豁免其它必需合并门。
语义与 CI 对齐
本次复用注册的进展分类,四个 read-stage 名称只属本地 CLI 闭集;没有科研子串分类、新模型检查点、新强制 pivot 或新 read ACK。配对 RFC 均有变更并实际检查。普通路径完整输出、捕获路径的权限/结算和源码中的要求分别验收,不能只凭字段或签名存在宣称语义保真。
我的整体评价
这是 justified_increment:源码级 CLI 状态读取和 writer 连续性可独立验收,但原 §11/M3 的实际采用与收益仍未闭合。长程控制边界判为 preserved;Agent 交互的状态先行和完整详情可达性判为 improved,没有新增用户确认轮次。由既有 TypeScript envelope/context owner 派生只读索引和分步视图,Python 只搬运文件;原始快照及 mutation-time owner 继续承担来源、权限与结算,未增加第二本账或另一次 guard。 当前 PR 的可交付边界是捕获 CLI 的结构化状态读取及原 writer 连续性;更广的模型采用、真实 Host 和可比完整成本沿现有 RFC §11/M3 继续。
代码量与该局部问题相称,复用原 capture、typed history 和验收 owner,未建立第二数据库;完整持久证据与 legacy/native 路径有实际消费者,应保留。渲染审查不适用:已检查现有 Agent/CLI replan 入口及完整 diff,没有页面、控件或布局变更。未验证范围:安装态 App/Lark/native Host、真实模型决策与理解、可比长程完整成本和收益仍未验证;较宽原测试集的既有失败仍保留。 保留原失败,批准不代表整体测试全绿或合并授权。
English verdict: APPROVE at 2035dfcdf97d0659c8b4a2f28f58d551050f989d for captured CLI state/read/writeback delivery. Identical-input uncaptured envelopes and captured authority/settlement boundaries are verified; generic off-stride feedback is an intentional disclosed change. Focused TypeScript and real File/SQLite staged checks pass. Wider preexisting fixture, native-route expectation and digest-owner failures remain failed with independent base attribution. Installed/model adoption and matched outcome/cost benefit remain unqualified; no merge is authorized.
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
Request changes conclusion — REQUEST_CHANGES at c18abc4d9ebf8fb38a00279bee3c58f828e82256.
动机
执行重规划的 Agent,需要先看目标、原决定和实际结果,再自主选择下一项工作。 旧入口把长指引、预填示例和累计验收混在一起;当前捕获入口先读结构化事实,再读写回约束,原文与完整要求仍按同一来源可达。 当前真实 File/SQLite 读取、原 writer 接受、来源变更拒绝和固定 CLI 路由均通过;但新增分支使必需维护性检查失败。 本次只资格化源码级捕获 CLI 交互;不证明安装采用、真实模型自主选择、token/账单节省或长程净收益。 安装态 App/Lark/native Host、真实模型对减少指引后的采用和理解,以及可比成果/完整成本继续沿原 §11/M3 验收。
改动思路
只读分步投影复用原 owner 合理,但新增读取/错误渲染塞入主调度函数,造成97条语句超过90的必需检查回归,需要局部拆分。 本 PR 保留一次捕获的状态读取、完整详情和原 writer;当前只要求修复新增调度函数债务,模型采用及净收益沿既有验收继续。 现有 typed envelope/context owner 足以派生只读分步视图,Python 保持文件 IO;没有新验收账本或平行决策 owner。 与不改动相比,分步入口让状态与格式各有读取时机;与另建摘要账本或 read ACK 相比,直接索引同一捕获保留了一个真实来源。
| 情况 | 旧行为 | 当前行为与边界 |
|---|---|---|
| 同一输入、未捕获 | 完整 envelope | off/on 两份完整输出逐值相等,包括来源哈希 |
| 同一输入、已捕获 | 内联完整 replan 包 | 先 review 后 writeback;权限、原 writer、用户、调度和欠账约束相同 |
| 实际 source 生成、stride off | 不提供该通用反馈 | 现在提供决定/后续观测,保持收益 unknown;不是所有 off-context 字节相同 |
| 捕获后的指引 | 方法建议及预填示例 | 有意减少建议/示例,保留字段和完整原文入口;不宣称模型更会判断 |
表格是来源与实际执行对照,没有作为模型采用或理解证据。更小的入口增加后续读取 IO,不能据此单独宣称 token、账单或完整成本节省。
具体改动
完整15文件差异与远端补丁均已核对。本次重新评审整份 PR,另外核对2035→c18的指引/示例减少、stride 显式字段和 CLI 路由变化;原2035的批准已 DISMISSED,不能沿用。
先验规范:docs/architecture/rfcs/hierarchical-agent-stride-control-v0.md,固定修订 e3fa61956b6e374b4475162039b7bf9f4282be01。按原条款:7.6 Research scope and evidence-conditioned control 实现决定、随后观测与目标收益分离;An iterative strategy and engineering boundary 保留原权限、预算、来源、writer 和结算 owner;11. Smallest Useful Implementation Slice 仍 deferred,真实 provider/Host、模型采用和完整成本沿原 §11/M3 继续。当前 PR 修改的 paired RFC 是显式提出通用 off-stride 反馈的变化,不能倒过来当成原规范已经要求这项输出的证据。
关键代码讲解
replanReadCommands仅为成功、非 settlement-only 且有原捕获的 replan 生成四种读取。它现在保留原--cli-binargv;环境中另一个安装不能悄悄接管后续 acceptance/Explore 读取。旧捕获保持原有兼容路由。replanView先核验完整来源哈希和 obligation/context,再把累计 outcome 按完整内容索引;同 ID 不同 revision/证据不覆盖,目标与要求数可见,详情完整返回。review 给事实;writeback 删除预填示例,保留原 authoring 限制、合法选项、来源及结算规则,另给完整源文件入口。projectReplanContext复用原因果历史和PROGRESS_RESULT_CLASSES,让通用 followthrough 在 stride off 也可见;blocked/unchanged 计数不再被误叫成功工作,执行与收益继续 unknown。read_replan_view只读文件后交给 typed owner,不访问 registry、不运行 guard、不写读回执。现有 writer 继续验证真正决定与结算。strideControlContract显式列出原合法 basis 和 required fields,取代靠填好的答案示例猜格式;它不替模型选择 grow/switch,也未开启策略。
对主干的风险
P2:新增只读分支触发必需维护性检查
quota.py:516 把读取、异常处理和渲染插入 handle_quota_command,使函数达到97条语句,超过既有90限制。不可变基线同一原检查通过,当前 head 新增这一项未评审债务;76项 Python/132项 typed 通过也不能豁免。请把新增只读处理局部放入现有命令/IO helper,使主调度器回到原门限内;保留同入口、pinned CLI、来源哈希和真实 File/SQLite 行为,不新增协议、放宽限额或要求无关语言迁移。
语义与 CI 对齐
复跑 python examples/control_plane/control-plane-maintainability-ratchet-smoke.py --format json,以及 uv run --extra test python -m pytest -q tests/control_plane/test_replan_staged_read.py。当前标准 premerge 6项直接检查过、19项选中检查17过;除上述 introduced 失败,scheduler-route smoke 也 failed。后者在独立完整基线上同样于 todo_snapshot 产生 canonical_authority_required,发生于 replan 投影之前,属于既有未初始化 fixture;仍保留 failed/独立合并 hold,不要求本 PR 将它冒充新产品缺陷或绕过。
当前独立验证76项 Python(含9项 staged)、132项 TypeScript 和类型检查通过。9项包括真实隔离 File/SQLite × ordinary/envelope、生产生成的完整读取命令、恶意 PATH 中的替代可执行文件、缺失 registry、原始 writer 接受及来源篡改拒绝;读期间原 snapshot/authority 状态不变。1500条真实 append index、21次后续观测、24个累计 outcome 和真实 Explore log 的公共合成夹具验证完整索引/遗漏计数与分项预算,不截断内容。全量公开边界扫描15条路径干净。
另冻结完全相同的原始输入在不可变 base/当前 head 执行4组投影:两份 uncaptured 完整 envelope 精确相等;两份 captured 的差异只在阶段导航、包表示、大小与派生签名,原权限、writer、用户、scheduler、非导航动作及 canonical source hash 相等。未归一化掉控制或哈希。实际生成 source 的通用反馈和 enabled 字段新增单独读差异,不把它们藏入“完全 off 等价”。旧2035的较宽 fixture/native-route 失败继续保留在其原修订;当前没有宣称安装/Host 采用或全面测试绿色。精确质量评审为 fail,未查询或等待远端 CI。
我的整体评价
REQUEST_CHANGES;这是 justified_increment,当前 architecture assessment 为 simplify_now。只读分步投影复用原 owner 合理,但新增读取/错误渲染塞入主调度函数,造成97条语句超过90的必需检查回归,需要局部拆分。 本 PR 保留一次捕获的状态读取、完整详情和原 writer;当前只要求修复新增调度函数债务,模型采用及净收益沿既有验收继续。 代码总体复用原 owner,当前最小修复是局部调度边界,不是拒绝整个结构化反馈方向。
这次减少建议性提示是明确的行为变化:原方法建议和预填示例仍可从原文件查看,但这不等于每次已经投递;原验收、来源、权限、停止与 mutation-time 门禁分别保留。是否改善强 Agent 的自主判断需真实采用/成果证据,不能由完整字段、9项传输测试或更小包体代替。
已检查完整15文件和现有 Agent/CLI replan 入口,没有渲染页面、控件、布局、设置或 Lark 操作变更。 未验证范围:安装态 App/Lark/native Host、真实模型对减少指引后的采用和理解,以及可比成果/完整成本继续沿原 §11/M3 验收。 保留 pre-existing fixture hold、原失败和未知;此评审不授予安装、合并或额外试验权限。
English verdict: REQUEST_CHANGES at c18abc4d9ebf8fb38a00279bee3c58f828e82256. Staged File/SQLite reads, pinned executable routing, source rejection and original-writer continuity pass. The new read branch nevertheless makes the unchanged required maintainability ratchet fail at97 statements versus90, while the immutable base passes. Extract that bounded handling without weakening the gate; keep the separate preexisting scheduler-fixture failure and unqualified installed/model/cost outcomes explicit.
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
|
Addressed the dispatcher finding in The existing capture IO module now owns read/error rendering and the adjacent capture-preparation step. The live dispatcher delegates those operations; the read stays ahead of live quota setup. Exception types, rendered output, exit status, validation order and preallocated-directory handling are unchanged. No thresholds or reviewed exceptions were increased. The unchanged maintainability ratchet fails at the parent (97 statements) and passes after the extraction (90 statements; zero unreviewed findings). The staged-read, capture and host-capture suite reports 34 passed / 13 failed. The independent immutable-parent capture suite reports the same 13 failing test IDs (15 passed); the failures remain disclosed, not reclassified as passes. The staged-read cases cover real isolated File/SQLite, pinned CLI routing, original writer acceptance and changed-source rejection. RFC bilingual validation and diff/public-boundary checks pass. This is an author fix/readback, not a new full-PR approval. Installed-host adoption, autonomous decision quality and complete-cost benefits remain unqualified; no live benchmark was changed by this commit. Standard premerge: all6 direct checks and17/19 selected checks passed initially. The semantic scan initially could not run because TypeScript dev dependencies were absent; after installing the declared dependencies, that unchanged scan passes on both parent and head. The remaining scheduler-route failure reproduces on the immutable parent as |
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
APPROVE at 7e6459411729d3c6ea2e9cd4505f649398b765c9 for the bounded source change. No blocking finding remains in this PR; required pre-existing failures and merge qualification remain separate holds.
动机
执行重规划的 Agent,需要先看目标、原决定和实际结果,再自主选择下一项工作。 旧入口把长指引、预填示例和累计验收混在一起;当前捕获入口先读结构化事实,再读写回约束,原文与完整要求仍按同一来源可达。 真实 File/SQLite 分步读取、原 writer 接受、来源变更拒绝和固定 CLI 路由通过;局部拆分使原调度函数回到既有90条语句门限。 本次只资格化源码级捕获 CLI 交互;不证明安装采用、真实模型自主选择、token/账单节省或长程净收益。 安装态 App/Lark/native Host、真实模型对减少指引后的采用和理解,以及可比成果/完整成本继续沿原 §11/M3 验收。
改动思路
分步只读投影复用原 typed owner,当前 IO/错误渲染提取到既有 capture helper,调度函数97→90且未改限额,职责与实际 caller 相符。 本 PR 交付原捕获的事实读取、完整详情和原 writer 连续性;安装态采用、模型判断及长程净收益继续保留为原验收缺口。 不改动会继续把状态事实和长指引、格式示例混在同一入口;另造摘要数据库或读回执则引入第二个来源。本 PR 使用同一完整捕获,先给事实读取,再按需给原 writer 格式与完整详情。typed envelope/context owner 派生视图,Python 保持文件读取与渲染;没有额外 guard、强制 pivot 或批准流程。
| 路径 | 旧行为 | 当前行为与边界 |
|---|---|---|
| 未捕获,同一输入 | 完整 envelope | off/on 两份完整输出精确相等,含来源哈希 |
| 已捕获 | 内联完整 replan 包 | review→writeback;原权限、writer、调度和欠账相同 |
| 实际 source,stride off | 不提供通用反馈 | 显式增加决定/随后观测,效用仍 unknown |
| 提示与示例 | 提供方法建议和预填答案 | 有意减少建议,保留字段约束和完整源材料入口 |
表格表达已执行的源级对照;不作为模型理解、实际采用或净收益证明。更小入口增加后续读取 IO,也不能单独证明 token 或账单下降。
具体改动
完整15文件与远端/本地 base-to-head 补丁一致;相对 c18 只有两个 Python IO/调度文件变化,另外13个路径 blob、typed 代码及依赖构建均未变,已作明确失效检查。重新评审整份 PR,没有沿用先前2035批准;当前局部修复也没有让已减少的 advisory prose 被称为无损等价。
先验规范为 docs/architecture/rfcs/hierarchical-agent-stride-control-v0.md,固定修订 e3fa61956b6e374b4475162039b7bf9f4282be01。7.6 Research scope and evidence-conditioned control 实现决定、后续观测及收益资格分离;An iterative strategy and engineering boundary 保留原权限、预算、来源、writer 与结算 owner;11. Smallest Useful Implementation Slice 继续 deferred,真实 Host/provider、模型自主采用与匹配完整成本沿原 §11/M3 验收。配套 docs/reference/protocols/turn-envelope-v0.md 同样按该不可变基线读取。当前双语 RFC 对 off-stride 通用反馈的修订是明确提案,不能冒充旧规范已要求的行为。
关键代码讲解
handle_replan_view把捕获读取、原异常集合、格式渲染和退出码放在既有 IO helper;在 live quota setup 前执行。原错误输出和只读边界不变。prepare_quota_capture在新建 capture 前调用原上下文验证;已有预分配目录直接保留,验证次序和异常仍由原 quota 路径处理。它不是新语义 owner。replanReadCommands绑定原显式 CLI 和完整 source hash;恶意 PATH 或删除 registry 不会让另一个安装接管只读。旧捕获保留既有兼容路线。replanView按完整内容索引累计验收,同 ID 的不同 revision/证据不合并丢失;review 先给事实,writeback 保留原字段、合法 basis 与结算限制,完整 capture 另可读。projectReplanContext复用因果历史和已登记结果类,blocked/unchanged 不再被算为成功推进;stride off 的通用反馈不启用 stride 策略,实际执行与效用仍保留 unknown。
对主干的风险
先前97/90的 P2 已有独立反证:在真实 Git worktree 对不可变父版本运行原检查会失败,当前变为90并通过,原始基线完整跟踪3610个 Python 文件且0新债;未增加限额或例外。当前76项 Python(含9项 staged)、132项 TypeScript 和类型检查通过,真实 File/SQLite×ordinary/envelope 覆盖生产读取命令、原 writer、恶意 PATH、缺 registry、来源变更拒绝及读期间原状态不变。
1500条真实 append index、21次后续观测、24个累计 outcome 与真实 Explore log 的公共合成夹具保留完整详情和分项预算;索引不裁剪要求,冲突 revision 不覆盖。独立固定相同输入执行4组公开投影,当前全部完整输出与 c18 精确相同,原 base/head 对照继续成立:两组未捕获完整 envelope 相等;两组捕获仅导航、包表示、大小和派生签名变化,控制/来源哈希等原义保持。实际 source 新增通用反馈另行披露,不宣称全部关闭态字节相同。
语义与 CI 对齐
标准 premerge:6项直接检查通过,19项选中检查18过;必需 scheduler-route fixture 仍在 todo_snapshot 报 canonical_authority_required。同一完整检查在实际不可变基线先于 replan 投影同样失败;完整错误除 event id/时间外相等。另在原始基线、父版本和当前 head 执行相同38项捕获邻接测试,各25过13败;13个完整失败详情只去除 pytest 临时序号、对象内存地址及相同 Python 安装的相对路径深度后相等。失败发生于未初始化 canonical fixture 的早期读取/guard,包含4个随后未能建立连接的超时;没有把它们改称通过或拿相等计数代替归因。
因此代码 APPROVE 与必需检查的 hold 分开:质量收据 cqr_ddfc4176618bdc29145a 仍为 fail;不降低 required、不提高预算,也不授予合并或 dismiss 权限。复跑 python examples/control_plane/control-plane-maintainability-ratchet-smoke.py --format json、uv run --extra test python -m pytest -q tests/control_plane/test_replan_staged_read.py 和原 scheduler-route smoke。15路径公开边界扫描干净;未查询、轮询或等待远端 CI。
证据更正:此前归档目录运行的维护性“通过”缺少 Git 跟踪清单,不能作为有效基线。本次改为实际 clean worktree 并校验 root、不可变 commit 与完整清单,重建了原基线0债、父版本97失败和当前90通过的判据。旧失败记录保留;没有声称旧 fixture 或历史不稳定已被本 PR 修好。
我的整体评价
APPROVE,没有新的阻塞发现。交付是 justified_increment,长程效果在原来源/后续 writer 连续性上 preserved,用户读取交互在已测 CLI 范围 improved;模型自主选择、安装采用和完整成果成本仍不由源码证明。分步只读投影复用原 typed owner,当前 IO/错误渲染提取到既有 capture helper,调度函数97→90且未改限额,职责与实际 caller 相符。 本 PR 交付原捕获的事实读取、完整详情和原 writer 连续性;安装态采用、模型判断及长程净收益继续保留为原验收缺口。 兼容仅为真实独立安装及既有捕获;没有再造协议/控制器,结果类复用原注册 vocabulary,stage 保持本命令局部。建议性提示有意减少,机器字段、原权限、停止和结算义务保留,不称 hard obligation 为可忽略建议。
已检查完整15文件与 CLI/内部 Agent 读取路径,没有渲染页面、控件或布局变更。 未验证范围:安装态 App/Lark/native Host、真实模型对减少指引后的采用和理解,以及可比成果/完整成本继续沿原 §11/M3 验收。 旧 blocking review 的处置与质量/合并资格另行 readback;本结论不包含合并、安装、账户切换或付费执行授权。
English verdict: APPROVE at 7e6459411729d3c6ea2e9cd4505f649398b765c9 for the source-level staged-read slice. The bounded extraction restores the unchanged97-to90 statement gate;76Python,132TypeScript, real File/SQLite read-to-writer paths and four fixed-input comparisons pass. Full immutable-base/head evidence attributes the remaining13capture failures and required scheduler-fixture failure outside this change. Keep those checks failed, CQR and merge readiness on hold, and installed/model/full-cost outcomes open.
…first Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
…t views Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
…first Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
|
This pull request has merge conflicts with Choose the remote for the base repository, not an out-of-date fork. git fetch upstream
git rebase upstream/main
# Resolve each conflict, git add the resolved files, then git rebase --continue.
git push --force-with-lease origin HEADFor a same-repository clone whose Keep the DCO |
Goal And Delivered Outcome
Checkpoint preflight could pass the raw quota packet to the model even when the captured replan reader was available. Replan state, method advice and authoring examples were also repeated in multiple carriers. This replaces that input path with state review followed by existing writeback, and keeps the complete original decision for checkpoint verification.
Before: a captured checkpoint could contain the correct source while the model still received the full inline planning packet. After: the production preflight invokes the CLI capture once, retains the full source for the typed checkpoint selector, and supplies its projected CLI view, including with TurnEnvelope disabled. File/SQLite entrypoint tests assert both sides of that boundary.
Intended base:
main. The long-horizon roadmap's evidence-to-decision-to-execution acceptance remains the frame; transport qualification is not an outcome-improvement claim.Author Declaration
docs/reference/protocols/turn-envelope-v0.mdand the evidence-driven replan boundary indocs/architecture/rfcs/loopx-overall-roadmap-v0.md, at mainc10d8c2b4.Scope And Continuation
This updates the existing replan reader proposal. There is no new planning state machine, read acknowledgement, activation flag, scoring rule or authority owner. Old captures remain readable. New guidance intentionally changes advisory prompting; it is not claimed to preserve model behavior. Original goal requirements, priorities, acceptance, permissions, budgets and stops remain binding. Eligible Todo ordering and agent-proposed rules remain revisable planning choices.
The bounded maintainability change removes the duplicate stride planning-advice builder and reuses one captured observation across CLI/checkpoint consumers. This is complete as a source/transport slice; installation, actual consumer adoption and measured long-horizon benefit remain separate qualification.
Validation
Tested source:
4a53fb7c3; the same changed runtime files are retained at exact heade16bc3359. Final standard premerge ran ate16bc3359. Inputs are synthetic/public fixtures; no model or benchmark evaluation was needed for these source checks.44050e953: stale inline-acceptance assumptions, unmigrated fixtures and the existing DSH bound-replan routing case. They are retained as failures.canonical_authority_required; the same failure reproduces on clean main. Benchmark-sensitive self-merge hold remains.The new local-snapshot allowlist entry reuses the existing read-only
quota.turn_envelope.evaluateowner; it introduces no new domain state or transport-size increase. File/SQLite are real isolated backends. PostgreSQL authority-store behavior is unchanged and was not qualified here.Frontend / Visual Evidence
UI impact: none. This is an agent-facing projection and host-adapter repair through the existing quota CLI and replan interaction. No dashboard or independent user workflow is added.
Boundary Checklist