Skip to content

test(context): register the canonical Goal source digest consumer - #7584

Merged
loopx-agent merged 1 commit into
mainfrom
codex/release220-digest-consumer-20261028
Oct 28, 2026
Merged

loopx-agent merged 1 commit into
mainfrom
codex/release220-digest-consumer-20261028

Conversation

@loopx-agent

@loopx-agent loopx-agent commented Oct 28, 2026 •

Copy link
Copy Markdown
Collaborator

新增的 goal_context_source.ts 已复用单一 digest owner 的 ENVELOPED_SHA256_PATTERN,但未加入现有消费者登记,导致完整 TypeScript 集成套件唯一一项失败(6,076 passed / 1 failed)。

本 PR 只登记这一已审查的消费者。保留“新增消费者触发检查”、禁止私有 digest matcher、必须实际使用 canonical export 等全部断言,不更改生产摘要、Goal 源码验证或权限。

验证:完整 digest 单一 owner 模块及原有正反例通过;固定 base 风险 premerge 四项直接及三项选中检查通过,零失败或 hold;精确 head 审查单独发布。完整真实 PostgreSQL 发布资格会在合并后的最终源码上重跑。当前 Goal 不等待 CI,未查询 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

维护原生Goal源码验证和digest单一owner的开发者。 新增Goal源码读取器已调用canonical digest matcher,但旧消费者登记漏掉它,使完整集成套件唯一一项失败;本头补齐这一登记,继续对未来新增消费者报错。 既有消费者集合与已交付调用链一致,17项原有单一owner检查通过。 本PR只有测试登记一行;不修改生产digest格式、Goal源码判定、权限、调用者或任何matcher断言。

改动思路 / Approach

先按既有规范 loopx/control_plane/content_digest.ts 的 revision 53dcb03082985bf894b1fca8813ab9f6e92503c5 判断交付边界,而非用本头新增说明替换原合同。既有测试刻意登记每个读digest owner的模块,以便新消费者触发审查。goal_context_source.ts已经导入并实际使用ENVELOPED_SHA256_PATTERN验证完整源码revision、排除区段及monitor receipt,未自行声明正则。它保留完整source SHA256计算与原region形状/顺序/计数验证;本PR不更改这些路径。删除集合断言或允许任意新导入都会失去单一owner保护,一行登记是现有owner内的最小修复。

当前边界只有已有测试的一个consumer登记,沿用canonical owner与原守卫;不增加生产策略或维护者手动同步状态。

具体改动 / Changes

关键内容讲解

CANONICAL_CONSUMERS仅新增control_plane/work_items/goal_context_source.ts。packageSources与importersInText仍遍历真实源码,实际集合仍须精确匹配;nameUsage仍要求导入真正使用,另一组断言仍拒绝非canonical导出。原owner matcher只能定义一次、consumer不得复制digest grammar、动态表达式与合法其他语法的误报反例全部保留。没有新增smoke、例外或跳过条件。

规范映射:canonical-digest-consumer,对应上述源码与验证。

对主干的风险 / Risks

登记意外演变成允许所有新消费者或重复matcher。 原CANONICAL_CONSUMERS精确集合、canonical导出检查与nameUsage全都不变。 PASS:完整 digest 单一owner模块17项通过,零skip,保留重复matcher、动态/误分类和非法导入正反例;固定base premerge四项直接/三项选中检查通过,零失败/hold。完整6077项集成基线仅此消费者集合断言失败,其余6076项通过;最终整版真实PostgreSQL仍独立重跑。CI未查询。 净diff为1文件+1/-0。没有页面、布局、首屏或产品指令变更,视觉检查不适用。公开diff无本地路径、凭证、私有Goal或原始模型对话;CI没有查询。普通revert即可恢复,无用户数据迁移。

我的整体评价 / Verdict

APPROVE仅覆盖 ef7a85234cd74b80220b52b6bfe648bcc83ae12e。long_horizon为not_applicable,user_experience为preserved:本PR修复已有验证链的过期前提,产品工作和执行权威不变,不宣称Agent效益提升。采用现有source oracle/AST守卫是同域可逆的简化;没有必要新增模块、框架或旁路owner。当前PR只补一个已经正确复用owner的消费者;完整发布资格须在两项独立修复合并后的最终源码重跑。

English verdict: APPROVE ef7a852. All 17 canonical-owner tests and required local checks pass. One existing consumer is registered; no matcher or production behavior changes. Actual-model and final release qualification remain separate.

@loopx-agent
loopx-agent merged commit d4980a0 into main Oct 28, 2026
6 of 7 checks passed
@loopx-agent
loopx-agent deleted the codex/release220-digest-consumer-20261028 branch October 28, 2026 15:25
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