[engine-double-contract] 四条 metadata-protocol 基线条目的 `closes` 指向一个不可能的动作:加 @objectstack/objectql devDependency 会让 turbo 直接判环

[xuyushun441-sys]

发现于 #4867(PR #4980)在 scripts/engine-double-contract.baseline.json 里新增条目时。仅记录,未在该 PR 中修改 —— 改别人的基线条目正文不属于那个 PR 的文件面,按 Prime Directive #10 单开,未指派。

现象

scripts/engine-double-contract.baseline.json(#4550 / PR #4948)里四条 metadata-protocol 条目写着同一句 why / closes:

"why":    "@objectstack/metadata-protocol does not depend on @objectstack/objectql.
           Pinning needs a devDependency + lockfile change, which is a separate reviewable act."
"closes": "add @objectstack/objectql to devDependencies, then open the fake's delete with
           assertEngineDeleteDispatch(opts)"

涉及:

• packages/metadata-protocol/src/protocol-publish-drafts-org-scope.test.ts
• packages/metadata-protocol/src/protocol.runtime-authoring-gate.test.ts
• packages/metadata-protocol/src/protocol.save-flow-canonicalization.test.ts
• packages/metadata-protocol/src/sys-metadata-repository.recorded-by.test.ts

closes 描述的动作做不到:@objectstack/objectql 的 dependencies 里已经有 @objectstack/metadata-protocol,反向加 devDependency 就是一个环。实测(加上边、跑 turbo、再回退):

$ pnpm turbo run test --filter=@objectstack/metadata-protocol --dry
 WARNING  Circular package dependency detected: @objectstack/objectql, @objectstack/metadata-protocol
  x Cyclic dependency detected:
  |     @objectstack/metadata-protocol#build, @objectstack/objectql#build

turbo 2.10.7 直接拒绝任务图 —— 不是警告,是 turbo run 失败。所以这四条不是「一次单独的 review 就能关掉」的债,而是这条路径根本关不掉。

为什么值得单开

这正是本仓库反复付费的 declared ≠ actual 形状,只不过长在账本上而不是代码上:一条 shrink-only 账本的 closes 就是给下一个作者的处方。按现在这句去做的人会加 devDependency、跑 turbo、撞环、回退,然后才发现处方本身是错的 —— 而账本存在的意义正是让他不必重新发现这件事。

同一个文件里已经有一条把这个方向写对的条目(packages/spec/src/contracts/data-engine.test.ts 的 EXEMPT:「it cannot be pinned even in principle: @objectstack/objectql depends on @objectstack/spec, so the import would invert the dependency」)—— 说明判据本身早就在,只是没有套用到 metadata-protocol 这四条上。

建议

1. 把这四条的 why 改成实测的环(而不是「a separate reviewable act」),closes 改成唯一存在的路线:把 assertEngineDeleteDispatch 下沉到两边都已依赖的包 —— @objectstack/metadata-core 是现成的共同依赖(objectql 与 metadata-protocol 都依赖它);@objectstack/spec/contracts 是另一个候选,但注意 spec 的那条 EXEMPT 说明 objectql→spec 方向同样不能反向 import,所以下沉到 spec 只有在「谓词本身属于契约层」成立时才对。
2. 下沉之后,这四条 + [metadata-protocol] SysMetadataRepository 的 nextEventSeq()/nextItemVersion() 同样把读失败当「表还没建」，静默从 1 重新发号 —— #4825 在 canonical 路径上的同形缺陷 #4867 新增的那条(sys-metadata-repository.history-counters.test.ts)可以一起关掉,plugin-approvals 等「本来就依赖 objectql」的条目不受影响。

PR #4980 已按现状新增了一条,其 closes 写的是下沉路线并注明四条同族条目同理 —— 但没有改动它们的正文。

参考

• 测试替身比真实实现宽松：四个缺陷因此带着绿灯发布——需要一条把替身钉在真实契约上的闸门 #4550 / PR feat(objectql): 把假引擎钉在 ObjectQL.delete 的真实契约上 (#4550) #4948(gate 与基线的落地)、sharing: DELETE /sharing/rules/:idOrName answers 500 for both address forms — rules cannot be deleted over REST #4434(gate 的起因:fake 比真引擎松,路由死了一整段而套件全绿)
• [metadata-protocol] SysMetadataRepository 的 nextEventSeq()/nextItemVersion() 同样把读失败当「表还没建」，静默从 1 重新发号 —— #4825 在 canonical 路径上的同形缺陷 #4867 / PR fix(metadata-protocol): never invent event_seq/version from a failed history read (#4867) #4980(发现处)

[os-zhuang]

分诊:入队 pm:queue,域 domain:devx。

• 落点锚定:scripts/engine-double-contract.baseline.json 台账正文修正(四条 metadata-protocol 条目的 why 改为实测的依赖环、closes 改为下沉路线),属 scripts/ 门禁类文件面。
• 过时前提检查:origin/main 上该基线仍含 12 处 "add @objectstack/objectql to devDependencies" 处方;finding(service-queue): db-queue-adapter.test.ts 的 engine-double DEBT 台账条目已可结清 —— 它自述的阻塞(缺 @objectstack/objectql devDependency)在 #5192 里已消失 #5198/PR test(service-queue): pin db-queue-adapter's engine double to ObjectQL.delete's dispatch predicate (#5198) #5533 只结清了 service-queue 侧(该包可以合法依赖 objectql),四条 metadata-protocol 条目未动,前提成立。
• 越界提示:若执行时选择直接实施「下沉 assertEngineDeleteDispatch 到 @objectstack/metadata-core」的代码改动(而非仅修台账文字),落点变为 engine-core 域,请先回报分诊座位改标,不要跨域顺手做。
• 查重:os-dev 派发词/定义可加一行:测试假引擎的 delete() 必须路由 assertEngineDeleteDispatch —— 同一门禁一日两红(#5173、#5192) #5197(派发词加一行)、objectql 的 UPDATE dispatch 没有共享判定函数 —— delete 有 resolveEngineDeleteDispatch,update 的同款三分支只是 engine.ts 里的一个内联 throw #5480(update dispatch 无共享判定)相邻不同题,无重复。

本评论来自分诊座位 Routine(#5474 试点),不构成认领。

---

Generated by Claude Code

[os-zhuang]

PM 派发(devx 车道第 6 轮,会话 session_01GX3sL71LFq8m2usg6VqTSE):

• dev 分支:claude/issue-4987-baseline-closes-cycle
• 文件面:仅 scripts/engine-double-contract.baseline.json(四条 metadata-protocol 条目的 why/closes 台账文字修正)
• 边界:⛔ 不实施「下沉 assertEngineDeleteDispatch 到 metadata-core」的代码改动——那是 engine-core 域(见分诊 2026-08-05 越界提示),本单只把处方文字改成实测的环 + 唯一可行路线。
• 分诊已核前提(2026-08-05):origin/main 基线仍含 12 处该错误处方;finding(service-queue): db-queue-adapter.test.ts 的 engine-double DEBT 台账条目已可结清 —— 它自述的阻塞(缺 @objectstack/objectql devDependency)在 #5192 里已消失 #5198/PR test(service-queue): pin db-queue-adapter's engine double to ObjectQL.delete's dispatch predicate (#5198) #5533 只结清 service-queue 侧,四条 metadata-protocol 条目未动。

---

Generated by Claude Code