Repository navigation
test(authority): refresh todo content revision after edit - #5991
huangruiteng merged 1 commit into
Conversation
Signed-off-by: mika <211269698+mikamikasuki@users.noreply.github.com>
loopx-agent
left a comment
There was a problem hiding this comment.
Reviewer: model_agent; model=gpt-6.1-sol; provider=OpenAI; runtime_reported; reasoning_effort=xhigh.
Reviewed exact head 3b4896427c13158e2d775de14669aae406ac4ea2 against immutable base 3ed5d6bc88ffaaa05f2b1bc6a98ed5d921379fd1. APPROVE,无阻塞发现。
动机
维护者修改 Todo 兼容入口后,需要用既有真实 provider 回归检查完整记录是否保留。
旧测试把编辑前的正文摘要带入编辑后的预期记录,正确编辑也会失败;当前测试重新计算新正文摘要,并要求它与旧摘要不同。
同一不可变基线复现旧摘要断言失败;当前精确 head 的完整测试文件 38 项通过,复杂记录与相邻任务的原有完整相等断言仍保留。
本 PR 只修正既有测试,不改变生产 Todo、权限、存储格式或 CLI/UI/Lark 行为,也不验收所有 provider 或平台。
完整读取已合并的 #5911 及基线全文摘要 helper 后判断:这是维护已有正文新鲜度契约的测试预期,不是新增 canonical 持久化字段。没有定义此断言修复的独立书面规格,因此不虚构 RFC 验收编号。
改动思路
继续使用同一个真实 canonical File fixture、实际 CLI 和完整记录相等断言。预期摘要取自声明的新正文,随后明确要求新旧摘要不同;直接从实际读回取预期值或删掉摘要比较都会削弱回归。复用现有规范化摘要 owner,未增加第二个生产决策源或单独 smoke。
具体改动
整个 PR 只有 tests/control_plane/test_local_coordination_authority.py,+9/-1:导入 todo_text_content_revision,定义 edited_text 和预期新摘要,检查非空,在完整 edited-record 预期中覆盖摘要,再断言它不同于 claimed_item 的旧摘要。原 claim、dry-run/apply、原 Markdown 移除后真实 canonical 读回、复杂 metadata、相邻 complex/successor 完整相等及 timestamp 检查全部保留。
独立对照:基线运行同一 focused function,1 failed/37 deselected,17 个字段相同,只有旧摘要与新摘要不匹配。当前 head 完整测试文件 38 passed。另用独立 hashlib 对声明新正文执行规范化并计算 SHA256,得到 3b61f6402feb68632608a56817e5dd4ffdf0b041d60b0d1d40a1d9f0a4305036,与 helper 一致且不同于旧值;预期没有从 provider 实际输出倒推。
对主干的风险
主要风险是修测试时悄悄放宽完整记录保留条件。全量 diff 和实际 fixture 执行表明,此改动只修正本应变化的派生正文摘要,仍比较其他字段及相邻任务,新旧不等检查也会拒绝陈旧摘要。Ruff、diff check、DCO 通过;未查询或等待 CI。
现有覆盖扫描区分上游 selection/concurrency 摘要测试与这个下游完整记录兼容 fixture;同作者的 hook/successor/report/process 测试 PR 维护不同边界,本 PR 没有新增同形 smoke。验证使用合成数据上的真实 File provider/CLI,不冒充 SQLite、PostgreSQL、所有平台或安装 App 验收;生产 TypeScript/权限/存储规则和用户入口没有改动。
我的整体评价
APPROVE。基线反例证明旧预期错误,当前完整文件和独立摘要 oracle 支持这个有界维护结果,没有弱化原有数据保留断言。未来重构检查已考虑 fixture 与摘要 owner:复用当前 helper 和既有测试足够,无需追加框架。评审结论不授予 merge authority。
English verdict: APPROVE — exact head 3b48964; fixes the stale expected text revision without weakening full canonical-record or sibling preservation. Immutable-base failure, 38 current-head tests, an independent normalized-text SHA256 oracle and focused static/DCO checks support this test-only maintenance change.
|
CI attribution for run 37847658548 at head 3b48964: JUnit artifact |
Goal And Delivered Outcome
Author Declaration
Implemented against
Scope And Continuation
Validation
Frontend / Visual Evidence
Type of Change
LoopX Area
Technical Direction
Shared-authority RFC fixture impact
Boundary Checklist