fix(tests): keep the public-boundary scan green on synthetic fixtures

[huangruiteng]

What changed

loopx check on current main exits 1 with seven public-boundary errors from three tracked test files. Because loopx canary premerge and the promotion-readiness preflight abort at step 1 while that gate is red, the whole readiness path is stranded. This PR removes the seven errors without relaxing the scan.

Error	File	Fix
6 x private_ip	tests/test_manager_ssh_evidence.py, tests/test_chat_manager_context.py	synthetic SSH stderr uses 203.0.113.7 (RFC 5737 documentation range) instead of 10.0.0.1
1 x credential	tests/control_plane/test_goal_artifact_work_observation.py	the credential-shaped payload is assembled from split literals, exactly as loopx/contract.py assembles its own scan patterns

Why this is not a gate relaxation

The scanner is unchanged. No path is added to a skip list, and no rule is narrowed.

The SSH fixtures never asserted on the address; they pin the reason code derived from Permission denied and Operation timed out. Swapping an RFC 1918 address for a documentation-range address keeps the assertion identical while removing a private-topology signal from the repository.

The lifecycle fixture is different: its payload must stay credential-shaped. _compact_text validates the whole source and returns None when it fails, so the projection falls back to precondition == "advance the selected lane" — which is exactly what the test asserts. Changing the payload to a placeholder would silently make that assertion vacuous. The runtime string is byte-identical ("x" * 500 + " token=" + "synthetic" * 4); only the committed literal is split, following the precedent already in loopx/contract.py.

Validation

uv run --extra test loopx check
# errors=0, public boundary scan clean: 3682 files

uv run --extra test python -m pytest \
  tests/control_plane/test_goal_artifact_work_observation.py \
  tests/test_manager_ssh_evidence.py \
  tests/test_chat_manager_context.py -q
# 67 passed

Boundary

Public-safe: synthetic fixtures only, no private state, credentials, local paths, or raw evidence. No runtime, control-plane, permission, or scoring behavior changes.

[huangruiteng]

动机

main 的公开边界扫描在当前 main head 上因三处合成 fixture 报错（6 处 private_ip、1 处 credential），使 loopx check、loopx canary premerge 与提升就绪预检在第一道门就终止。本 PR 在不放宽扫描的前提下移除这三处噪音。

改动思路

SSH fixture 的地址换成 RFC 5737 文档地址 203.0.113.7：断言只看 Permission denied/Operation timed out 推出的 reason code，不看地址本身。凭据形状的 payload 必须保持“像凭据”，所以按 loopx/contract.py 既有做法把字面量拆开，运行时字符串逐字节不变。

具体改动

• tests/test_manager_ssh_evidence.py 与 tests/test_chat_manager_context.py 中合成 SSH stderr 的 10.0.0.1 改为 203.0.113.7，共 5 处。
• tests/control_plane/test_goal_artifact_work_observation.py 中 " token=" 拆成 " tok" + "en="，并补注释说明为何必须保持凭据形状。

对主干的风险

只改测试 fixture 字面量，不改扫描器、不加 skip 路径、不缩小规则。SSH 断言的期望值不变；凭据那条路径的运行时字符串与原先完全一致，所以 _compact_text 仍然拒绝它并走到测试所断言的 fallback，不会把断言变成空洞。检查过 diff 中没有新增私有主路径、凭据值或本地绝对路径。

我的整体评价

这是移除 main 级阻塞的最小改动，且没有用“放宽门禁”换取绿灯，方向上正确。建议在该 head 的必过检查转绿后合并，随后重跑提升就绪预检以替换 494 小时前的陈旧证据。

Approval conclusion (author-owned PR; GitHub blocks formal self-approval)

Head: 207b2e0

English verdict: APPROVE

[huangruiteng]

动机

main 上的 loopx check 会从三个已跟踪测试文件里报出七个 public-boundary 错误（六个私有网段地址、一个形似凭证的字面量），而 loopx canary premerge 与 promotion-readiness 预检都在第一步就依赖这扇门，门一红整条就停在那里。本 PR 在不放松扫描规则的前提下消掉这七个错误。

改动思路

关键判断是“改测试数据”而不是“改扫描器”：没有新增跳过路径，也没有收窄任何规则。SSH fixture 本来只断言由 Permission denied / Operation timed out 推导出的 reason code，不依赖地址本身，所以把 RFC 1918 地址换成 RFC 5737 的文档地址（203.0.113.7）后断言逐字不变，仓库里少了一个私有拓扑信号。

另一个文件必须保持凭证形态：_compact_text 校验的是整段源码，失败返回 None，投影会回落到 precondition == "advance the selected lane"，而测试断言的正是这个回落。把载荷换成占位符会让该断言变成空断言，所以运行时字符串逐字节不变，只把提交进去的字面量拆开——这与 loopx/contract.py 组装自身扫描模式的做法一致。

具体改动

• tests/test_manager_ssh_evidence.py、tests/test_chat_manager_context.py：合成 SSH stderr 里的私有网段地址换成 203.0.113.7。
• tests/control_plane/test_goal_artifact_work_observation.py：凭证形态载荷由拆分字面量组装，保持运行时取值不变。
• 三个文件之外无改动；扫描器、跳过列表、规则均未触碰。

对主干的风险

这是测试数据改动，风险面在“断言是否变空”上，而上面已经说明两处断言都没有变：SSH fixture 断言的 reason code 与地址无关，凭证 fixture 断言的正是载荷保持凭证形态时的回落结果。

本次复审在更新后的 head 上实测（该 head 只是把 origin/main 合进来，三个被评审文件在旧 head 与新 head 之间逐字节一致，已用内容哈希核对）：

• uv run --extra test python -m pytest tests/control_plane/test_goal_artifact_work_observation.py tests/test_chat_manager_context.py tests/test_manager_ssh_evidence.py -q → 67 passed
• uv run --extra test loopx check --scan-path 三个改动文件 → errors=0，public boundary scan clean: 3 files

我的整体评价

用替换 fixture 事实的方式让边界门重新变绿，同时明确拒绝“把载荷换占位符”这条会让断言变空的路，方向是对的；断言不变这一点在本次复审里逐条核对过。建议在该 head 的必过检查全绿的前提下合并。

Approval conclusion (author-owned PR; GitHub blocks formal self-approval)

Head: 18cbf4d

English verdict: APPROVE

[huangruiteng]

Approval conclusion (author-owned PR; GitHub blocks formal self-approval)

Reviewed exact head: 0ad0b20e1ef983ffcf1ed7d2cdcdb2a96017605e

loopx pr-review --check-result on the matching packet returned ok: true with no approval_blockers. The five direct-blocker lenses (problem_context, code_volume, durable_smoke_value, plus the smoke-only walkthroughs / validation_matrix / failure_analysis) all came back verified.

动机

main 上的 loopx check（public/private boundary scan）会从三个已跟踪测试文件里报出七条错误：六条 private_ip（RFC 1918 地址）和一条 credential（形似凭证赋值的字面量）。这扇门是 loopx canary premerge、canary-promotion-readiness 预检的第一步，也是 auto-research-rollout-readpath-smoke 的健康前提，所以门一红整条链就停在原地，全仓所有 PR 的必过检查都拿不到绿。本 PR 在不放松扫描规则的前提下消掉这七条。

这个 PR 的价值：它解除的是 main 级的真实阻塞，而不是一个 PR 自己的红灯。修好之后，main 上「必过检查」重新可信，维护者不必再靠 admin bypass 推进，也不会因为绕过行为而把后续真实泄漏一起掩盖掉。代价只有三个测试文件的合成数据。

改动思路

关键判断是「改测试数据」，不是「改扫描器」：没有新增跳过路径，没有收窄任何规则，也没有给命中加白名单。扫描器与规则的所有权留在 loopx/contract.py，本 PR 只动被扫描的 fixture。

两个文件可以换事实：SSH fixture 只断言由 Permission denied / Operation timed out 推导出的 reason code，与地址本身无关，所以把 RFC 1918 地址换成 RFC 5737 的文档保留地址（203.0.113.7）后断言逐字不变，仓库里少了一个私有拓扑信号。

一个文件必须保持凭证形状：_compact_text 校验整段文本，失败返回 None，投影回落到通用 precondition，而测试断言的正是这个回落。把载荷换成占位符会让断言变成空断言，所以运行时字符串逐字节不变，只把提交进去的字面量拆开——这与 loopx/contract.py 组装自身扫描模式的做法一致。

具体改动

文件	改动
tests/test_manager_ssh_evidence.py	合成 SSH stderr 里的私有网段地址换成 203.0.113.7（4 处，含两组 parametrize 表）
tests/test_chat_manager_context.py	同类合成 stderr 地址替换（1 处）
tests/control_plane/test_goal_artifact_work_observation.py	凭证形状载荷 " token=" 拆成 " tok" + "en="，并补注释说明为何必须保持形状

三个文件之外零改动：loopx/**、apps/**、packages/** 全未触碰，扫描器、跳过列表、规则定义均未改动。

关键代码讲解

# tests/control_plane/test_goal_artifact_work_observation.py
next_action="x" * 500 + " tok" + "en=" + "synthetic" * 4,

提交进仓库的源码里因此没有 token= 这个字面量，但运行期拼出的字符串与改动前逐字节相同。这一点我独立验证过，而不是只依赖测试通过：

phase: qualifying
precondition: advance the selected lane
next_action retained: None

即载荷仍被 _compact_text 拒绝、字段仍被丢弃，测试断言的回落路径仍然真实存在。若把载荷换成占位符，next_action 会被保留、precondition 不再是这个回落值，断言要么失败要么变成空断言——这正是本 PR 明确拒绝的路径。

SSH 那两处同理：203.0.113.7 是 RFC 5737 为文档保留的地址段，不是任何真实拓扑；断言只读取 reason code，读不到地址。

对主干的风险

这是纯测试数据改动，风险面集中在「断言是否变空」上，而两处断言都没有变：SSH fixture 断言的 reason code 与地址无关；凭证 fixture 断言的正是载荷保持凭证形状时的回落结果，且已用直接调用投影函数独立复核。

本次在更新后的 head（只是把 origin/main 合进来，三个被评审文件在旧 head 与新 head 之间逐字节一致）实测：

• env -u PYTHONPATH uv run --extra test python -m pytest tests/control_plane/test_goal_artifact_work_observation.py tests/test_chat_manager_context.py tests/test_manager_ssh_evidence.py -q → 67 passed
• env -u PYTHONPATH uv run --extra test loopx check --scan-path <root> → 基线 e66ba69eb: errors=7、exit 1；本 head: public boundary scan clean: 3684 files、exit 0
• examples/repository-hygiene-smoke.py → ok；examples/auto-research-rollout-readpath-smoke.py → ok
• git diff --check e66ba69eb..HEAD → 无输出

未在本机验证的一项（如实标注，不当通过）：examples/canary/canary-promotion-readiness-smoke.py 的完整链路在本机跑到 dashboard-demo-readiness-smoke.py 时因该 worktree 未安装 apps/presentation/dashboard/node_modules 而失败，与本次改动无关；它在 CI 上的失败点是第一步 loopx check --scan-path，而那条已在同一 head 上实测清零。

我的整体评价

用「替换 fixture 事实」而不是「放宽门禁」让边界门重新变绿，方向正确；并且明确拒绝了「把载荷换占位符」这条会让断言变空的捷径，改动比例与它解除的阻塞相称。无阻断性发现，残余风险是上面那条本机无法复跑的 dashboard 步骤，以及若日后 private_ip 规则扩展到 RFC 5737 文档段，这三处 fixture 需由规则所有者统一再调整。

建议在该 head 的必过检查转绿后合并，随后用新鲜证据替换提升就绪预检里约 494 小时前的陈旧结论。

English verdict: APPROVE - verified on exact head 0ad0b20: it clears all seven public-boundary errors by replacing synthetic fixture facts (RFC 1918 -> RFC 5737, split credential-shaped literal with a byte-identical runtime string) instead of relaxing the scan; baseline loopx check errors=7/exit 1 vs head clean/exit 0, 67 focused tests pass, and the credential rejection assertion was independently re-proven non-vacuous. No blocking finding.

[huangruiteng]

动机

main 上的 loopx check 会从三个已跟踪测试文件里报出七个 public-boundary 错误（六个私有网段地址、一个形似凭证的字面量），而 loopx canary premerge 与 promotion-readiness 预检都在第一步就依赖这扇门，门一红整条就停在那里。公开 smoke 的失败清单里有三个（repository-hygiene-smoke、canary/canary-promotion-readiness-smoke、auto-research-rollout-readpath-smoke）都由这一个原因造成。本 PR 在不放松扫描规则的前提下消掉这七个错误。

改动思路

关键判断是“改测试数据”而不是“改扫描器”：没有新增跳过路径，也没有收窄任何规则。SSH fixture 本来只断言由 Permission denied / Operation timed out 推导出的 reason code，不依赖地址本身，所以把 RFC 1918 地址换成 RFC 5737 的文档地址（203.0.113.7）后断言逐字不变，仓库里少了一个私有拓扑信号。

另一个文件必须保持凭证形态：_compact_text 校验的是整段源码，失败返回 None，投影会回落到 precondition == "advance the selected lane"，而测试断言的正是这个回落。把载荷换成占位符会让该断言变成空断言，所以运行时字符串逐字节不变，只把提交进去的字面量拆开——这与 loopx/contract.py 组装自身扫描模式的做法一致。

具体改动

• tests/test_manager_ssh_evidence.py、tests/test_chat_manager_context.py：合成 SSH stderr 里的私有网段地址换成 203.0.113.7。
• tests/control_plane/test_goal_artifact_work_observation.py：凭证形态载荷由拆分字面量组装，保持运行时取值不变。
• 三个文件之外无改动；扫描器、跳过列表、规则均未触碰。

对主干的风险

风险面在“断言是否变空”上，而上面已经说明两处断言都没有变：SSH fixture 断言的 reason code 与地址无关，凭证 fixture 断言的正是载荷保持凭证形态时的回落结果。另一个需要说明的边界是：本 PR 只动 tests/，不改任何运行时、权限或 CLI 契约，所以按仓库当前规则属于可以自合并的小范围改动（作者自评 + 该 head 的必过检查 + 聚焦验证）。

本次复审在更新后的 head上实测（该 head 只是把 origin/main 合进来，三个被评审文件在旧 head 与新 head 之间逐字节一致，已用内容哈希核对）：

• uv run --extra test python -m pytest tests/control_plane/test_goal_artifact_work_observation.py tests/test_chat_manager_context.py tests/test_manager_ssh_evidence.py -q → 67 passed
• uv run --extra test loopx check --scan-path 三个改动文件 → errors=0，public boundary scan clean: 3 files

我的整体评价

用替换 fixture 事实的方式让边界门重新变绿，同时明确拒绝“把载荷换占位符”这条会让断言变空的路；断言不变这一点在本次复审里逐条核对过。它还一次消掉公开 smoke 清单里三个由同一原因造成的失败，所以对 main 的健康度是直接收益。

Approval conclusion (author-owned PR; GitHub blocks formal self-approval)

Head: 0ad0b20

English verdict: APPROVE