Repository navigation
test(context): check preference duplication at emitted pointers - #7953
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
Approval conclusion (author-owned PR; GitHub blocks formal self-approval)
动机 / Motivation
维护原生上下文首读的开发者,需要区分实际重复发送与测试 helper 为读取完整值而生成的地址。 以前完整偏好父对象正确到达仍触发重复报错;现在只检查 selections 实际发出的地址,完整值另由原断言核对。 同条件首读和导入边界两模块由基线 64 通过、1 失败变为本 head 65 通过;故意注入实际重复子行时三个值形态均在原负断言处拒绝。 本 PR 只修测试 oracle,不改变生产上下文、保留声明、准入或真实模型行为。
改动思路 / Approach
selected_context_values 是值读取 helper,会把完整父对象递归展开成子地址;这些地址不代表另发了一行。当前改动复用同文件已有的做法,从 page.content.selections 取实际 pointer 集合来判断重复。完整内容仍交给独立的原值、capture 不变与 byte-map 断言验证;没有改动生产 reader 或去重逻辑。
具体改动 / Changes
实审 head b2a89dd2042123a856c4e4c545507dee3d4cf08b,基线 9bfa71c679f58d8b19a533956b627c96343616e5;完整差异仅 tests/control_plane/test_context_first_read.py +2/-1。
关键内容讲解
参数化测试第248行继续比较完整 current,涵盖 null、空字典及包含未知字段、转义键、空值/false/0 的对象。249–250行改看实际投递的 rows;252–253行仍核对原 capture、原 decision 和原字节范围。它修正观察对象,没有删掉一个内容条件。
规范依据为 docs/development/good-smokes.md,spec_revision 9bfa71c679f58d8b19a533956b627c96343616e5。Start With The Contract:观察真实读取边界和完整原值;Prefer Semantic Pressure:本轮通过实际 reader 生成页,再仅注入真实重复子行,三个形态均被250行拒绝;Validate The Change:完整相关模块、Ruff 和 premerge 已执行。同版本 docs/reference/required-work-context.md 的 Selected capture details 保留原值、null、原 digest 与不重复父/子选择的合同未变。
对主干的风险 / Risks
PASS:完整首读与导入边界两模块共65项;同命令基线64P/1F,准确失败是 current2 旧 oracle,而不是产品丢值。独立的三项原页正例和三项重复行反例全部符合预期。标准 premerge 的5项直接检查通过,0项catalog被选择;Ruff通过。
已核 #7952 的实际目标是独立整合分支,本 PR 指向 main;同类修复不等于可继承其冻结整合证据。没有新增 smoke/状态/公共协议或生产默认,进一步抽象没有收益。CI 未查询,不作为当前批准前置;此结论不等于合并资格。
我的整体评价 / Verdict
没有阻塞 finding。命名的 oracle 修复目标已完成;long_horizon 和 user_experience 的生产语义保留,维护者能更准确区分内容保真与重复投递。无渲染呈现变化,visual_review 不适用,未验收真实模型采用或长期成本。最强剩余边界是本 PR 不交付生产改动或整合分支整体资格。
English verdict: APPROVE — b2a89dd; the wire-row oracle preserves complete values and rejects deliberate duplicate emissions; 65 tests and native premerge pass.
原生上下文的偏好验收用
selected_context_values检查重复返回,但这个测试 helper 会递归展开已收到的完整父对象。完整current已正确投递时,helper 派生出的子地址仍会让断言误报。改为检查
page.content.selections中实际发送的地址,复用同文件 outcome 导航测试已有的做法。完整偏好值(含 null、空字典及未知字段)、原始 capture 一致性与 byte-map 断言全部保留。验证(
b2a89dd2042123a856c4e4c545507dee3d4cf08b;基点9bfa71c679f58d8b19a533956b627c96343616e5):只修改一个既有测试的观察方式。当前 head 的独立评审与远端 CI 仍待完成。