Repository navigation
feat(research): carry joint choice guidance into ordinary work - #8353
loopx-agent wants to merge 2 commits into
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 (author-owned PR; GitHub blocks formal self-approval)
未发现阻断问题。本次独立评审针对完整精确 head df4088fca0607e802f6192f9799e34ed4a46071c,不继承旧头的通过结论。
动机 / Motivation
开启可选 Yardstick 的 Agent,在普通任务首读中收到已关联测量、但尚未声明研究组合。例如,局部指标已经上升,Agent 仍需决定下一笔投入是否值得;旧首读只说明测量范围,已有组合才收到共同选择指导。现在两条路径都能看到有条件的成本与收益提示。已验证的变化是指导到达首读并保留原数据、权限和继续路径;不把提示送达当成模型采纳或成果提升。
不新增评测、设置、调度或决策表单,不宣称模型采用、分数提升或完整成本下降。真实模型是否采纳,以及是否改善成果和完整成本,仍待既有 C13/M3 研究验收。这是一项有实际调用方的 M1 指导交付增量,不能据此宣布更大的研究验收完成。
改动思路 / Approach
沿用普通任务的首读链路:既有配置 owner 决定是否开启,canonical Todo 和问题关联限定证据范围,TypeScript owner 生成上下文,模型保留具体研究选择。未声明组合的首读与已有组合的比较只共享同一段建议;测量筛选、写回和失败恢复仍由原 owner 负责。共同指导留在既有语义 owner,原数据筛选和 effect owner 不动;纯函数复用比新控制器、设置或双份指导更小。本 PR 只交付有条件的指导投影及对照验证;真实模型采用与成果收益留给既有研究验收。
我挑战的主要反例是:反复读到旧测量,就反复被要求比较或评测。最终文字把权衡限定在有意义的证据/probe 边界,并明确无实质变化时继续当前获准工作,这条提示本身不要求额外复盘、测量或决策 payload。within accepted constraints 和未知保持未知也进入了实际输出,不只是文档声明。它没有三选一表格、工具顺序或强制换路线。
具体改动 / Changes
规范依据是 docs/architecture/rfcs/target-yardstick-stride-decisions-v0.md,按变更前不可变版本 a02be126957953b7a61796e3b99fcfe578eb5174 判断。C1 的无新增回合义务、C5 的未知语义、C10 的原权限边界、C12 的关闭隔离已由本次源路径与真实 reader 对照覆盖;C13 的模型采用、其他更优方法及完整成本/成果实验仍明确 deferred,归既有 M3 研究验收。这次同步修改的中英文 RFC 是对边界的公开说明,不能替代这份变更前规范或实际执行证据。
关键代码讲解
workYardstickChoiceGuidance是无参数、无 effect 的共享文本。它保留验收约束、剩余时间、完整实现与验证成本,显式包含搜索、确认和模型/控制开销,并允许有依据地继续。未知不被改写为已验证收益。measuredSearchComparison继续使用精确 profile 和同一候选对;排序、平局、反转提示与候选引用没有变化,只替换共同指导后缀。没有有效比较时仍不生成比较结论。projectExploreTurnContext仍先检查既有 Yardstick 开关、任务归属/状态及问题关联,再给初始测量卡增加指导。原“未选择 outcome/proxy profile”“latest 不等于 current”“best 不等于验收或许可”、省略证据入口和可选附件全部保留。
两份测试覆盖实际输出与条件,中英文 RFC 同步。没有新字段、版本、设置、日志或兼容分支;共享指导提取是同域、行为明确的复用,不是第二个控制器。
对主干的风险 / Risks
最大的风险是模型把建议理解成重复审议,或者新文字越过关闭/归属边界。相同合成输入在 base 与最终 head 的 23 组完整投影对照表明:关闭、未配置、空证据、外部归属、终态、未关联及错误源顺序路径完全相同;其余路径只有指定的两处指导文字改变。增加 40 条无关记录、改变分页/源展示顺序或添加无关注释,也不改变当前任务卡。数据、未知值、权限提示和原当前动作均未因归一化被抹去。
| 验证 | 结果 |
|---|---|
| TypeScript 三个相关语义组 | PASS,39 项 |
| 实际 File/SQLite 首读、关闭、提交前拒绝与已提交附件恢复 | PASS,9 项;原 12000B 首读上限、原 decision/event 和一次扣额断言保留 |
| control-plane typecheck、中英文 RFC、diff/DCO | PASS |
| 模型采用、成果与完整成本收益 | UNVERIFIED,未执行模型实验 |
另一次可选的两文件较宽 QA 被我的 180 秒执行上限中止,没有完整断言总数,因此保留为未完成;没有把它说成通过或预先存在的产品失败。当前必要的不变量由上述独立 9 项真实路径和前后对照覆盖。按本次审阅契约未查询或等待 CI;CI 和合并资格与这份源代码审阅分开。
语义与 CI 对齐
本次复用既有语义 vocabulary,没有新增状态分类或公共协议。新指导是有意的 opt-in 普通回合扩展;原开关和 typed 错误处理、权限、预算、复盘频率与 settlement 义务不变。旧 116294b9f5e3822b6abd70512c2776c698dc42b0 也能通过单元测试却缺少明确的 C1 继续条件,本次已在最终输出中验证修复,未用“绿色测试”替代该语义判断。
我的整体评价 / Verdict
作为有条件指导的交付增量,本 PR 范围适当,long_horizon 和 user_experience 的原连续工作/恢复路径保持;没有新增用户准备、确认轮次或每轮表单。共同指导留在既有语义 owner,原数据筛选和 effect owner 不动;纯函数复用比新控制器、设置或双份指导更小。本 PR 只交付有条件的指导投影及对照验证;真实模型采用与成果收益留给既有研究验收。
受影响的是 Agent CLI/context 文本;检查了相关 App/Lark 消费边界,未发现本 PR 改动渲染页面或布局,因此 visual review 不适用。未验证范围是实际模型理解、采用和收益,不能从 cue、合法 continue 或合成执行推断。远端 head 已复核为上述精确版本;结论只授予本次评审通过,不包含 main 合并、部署、安装或实验执行授权。
English verdict: APPROVE at df4088f. No blocking finding; conditional advisory semantics, default-off/scope parity and real File/SQLite paths verified. Model adoption and outcome benefit remain unqualified; merge authority is separate.
开启 Yardstick 的普通工作回合可能已经收到局部测量,却还没有声明 target–yardstick 组合。原来这条首读路径保留了测量范围和权限说明,但“下一笔研究投入是否值得”的共同选择指导只出现在已有组合的比较里。
现在两条路径复用同一段简短指导:结合预期验收成果收益、剩余时间,以及完整实现、搜索、验证和确认成本,比较继续当前路线、其他机制或组合。仅在有新相关证据或到达有意义的 probe 边界时权衡;无实质变化就继续当前获准工作,不新增复盘、测量或决策记录。未知保持未知,成本包含模型和控制开销,证据支持的路线可以继续。现有测量、缺失值、来源、按需读取及可选记录语义保留;默认关闭、复盘频率、权限、预算和提交协议不变。没有新字段、设置、调用或强制转向。
验证:24 项 TypeScript 语义/负例测试、control-plane typecheck;真实 File/SQLite 首读和执行链原组 36 项通过、4 项可选跳过。原有成本措辞断言暴露一次变化后已恢复原措辞,3 个受影响用例重新通过;最终边界措辞的同 3 个真实首读/执行用例再次通过,原 12KB 首读边界未改;关闭、无测量、非本人任务的完整投影与基线逐字段相同。中英文 RFC 同步,六个文件公开边界检查无错误或警告。
这是普通回合指导的有意扩展;模型是否采纳、是否提高分数或降低完整成本仍未验证。没有新增每轮表单或更大的测量目录。共享指导提取是同域重构;主干合并仍需独立评审与正常授权。