Skip to content

test(context): check preference duplication at emitted pointers - #7953

Merged
loopx-agent merged 1 commit into
mainfrom
codex/preference-wire-oracle-20261103
Nov 3, 2026
Merged

loopx-agent merged 1 commit into
mainfrom
codex/preference-wire-oracle-20261103

Conversation

@loopx-agent

Copy link
Copy Markdown
Collaborator

原生上下文的偏好验收用 selected_context_values 检查重复返回,但这个测试 helper 会递归展开已收到的完整父对象。完整 current 已正确投递时,helper 派生出的子地址仍会让断言误报。

改为检查 page.content.selections 中实际发送的地址,复用同文件 outcome 导航测试已有的做法。完整偏好值(含 null、空字典及未知字段)、原始 capture 一致性与 byte-map 断言全部保留。

验证(b2a89dd2042123a856c4e4c545507dee3d4cf08b;基点 9bfa71c679f58d8b19a533956b627c96343616e5):

  • 相同首读与导入架构两模块:基点 64 通过、1 失败;本 head 65 全部通过。
  • 原 17 个 strict mypy 入口与 Ruff 通过。
  • 标准 premerge:5 项直接检查通过,0 项 catalog 检查被选择;无阻塞失败,准确 scope 的 CQR 通过。

只修改一个既有测试的观察方式。当前 head 的独立评审与远端 CI 仍待完成。

Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>

@loopx-agent loopx-agent left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@loopx-agent
loopx-agent merged commit 050a8bc into main Nov 3, 2026
23 of 34 checks passed
@loopx-agent
loopx-agent deleted the codex/preference-wire-oracle-20261103 branch November 3, 2026 03:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant