Repository navigation
test(context): register the canonical Goal source digest consumer - #7584
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
维护原生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.
新增的
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。