test(client): discovery 替身改成真实 /discovery 形状,并本地钉死两处退役拼法

[qq9340100]

Fixes #5787

问题

packages/client/src/client.test.ts 里 connect() 的 discovery 替身拼了两个任何 producer 都没发过的形状:

capabilities: ['metadata', 'data', 'ui'],
endpoints: {}

• endpoints —— 真实 producer(ObjectStackProtocolImplementation.getDiscovery(),packages/metadata-protocol/src/protocol.ts)发的是 routes(ApiRoutesSchema,DiscoverySchema 里必填)。endpoints 只在 dispatcher 那条路径上作为 routes 的逐字副本存在过,两个 discovery 生产者都在线上返回 schema 未声明的顶层字段(scoping / features / endpoints),且 REST 形状永远无法通过 DiscoverySchema #4828 已按 ADR-0049 删除。
• capabilities 是数组 —— DiscoverySchema.capabilities 是对 WellKnownCapabilitiesSchema 的闭合对象,每个词汇键一条 CapabilityDescriptor({ enabled, features?, description? }),SDK 的 client.capabilities 声明为 WellKnownCapabilities,但两个 discovery 生产者填的是互不相交的键集 #5672 ruling A。它从来不是字符串数组;而且它列的三个名字(metadata / data / ui)根本不是 capability 键,是 routes 键。

两处都是惰性的 —— 该用例只断言 expect(fetchMock).toHaveBeenCalled(),不读 body 的任何键 —— 这正是它们连着躲过两轮退役的原因。这是 #5674 在另一个包里的同族兄弟,按「规则的消费半径」而不是按被改的包复查时发现的。

改法

1. 替身按实测的 producer 输出重塑(client.test.ts)

capability map 用 WELL_KNOWN_CAPABILITY_KEYS 构造而不是手写键表 —— 该常量的 schema 注释明确说了它存在的意义就是防止「手列键」这种漂移,手写一份在词汇增长的当天就会悄悄落后。

2. 新增本地 pin packages/client/src/discovery-double-retired-key.test.ts

对齐 packages/rest/src/discovery-double-retired-key.test.ts(#5674)的形状,保持按包,没有提成跨包 repo 级 gate —— issue 正文明确要求不要默认扩大,而且两个包写替身的方式本来就不同:rest mock 的是协议对象(getDiscovery: vi.fn().mockResolvedValue(…)),client mock 的是线(fetch 到 json())。单一跨包扫描器无论如何都得同时带两种捕获形态,失败信息还会指向读者当下并不在的那个包。两条本地 pin,各读各包真实使用的形态,是更小也更好读的安排。要不要合并成 repo 级 gate 是维护者的决定,不该作为本次修复的副作用发生。

pin 的诚实边界(写进了 docblock):

• 读本包替身真实使用的两种形态:fetch mock 的 json: async () = 箭头体,以及对私有 discoveryInfo 字段的直接赋值。第三种写法读不到。
• 字面量带 capabilities 或 routes 键才算 discovery 替身 —— 本包其它响应体(/meta、/data 等)会被捕获然后正确忽略。实测:共捕获 9 个字面量,判定 5 个为 discovery 替身,横跨 2 个文件。
• 只断言两条否定。故意不要求 routes 或完整词汇:隔壁 capability getter 的用例是刻意用残缺和遗留载荷驱动 getter 的(capabilities-vocabulary.test.ts 就是原样重放 SDK 的 client.capabilities 声明为 WellKnownCapabilities,但两个 discovery 生产者填的是互不相交的键集 #5672 之前的 dispatcher map),要求每个替身都合规会把那份覆盖删掉。
• 本文件排除自身扫描,并且是具名排除,而不是听凭 docblock 恰好怎么折行。
• 反空转下限(packages/spec 的「编译期 pin」是失效的:tsconfig 排除了 *.test.ts,vitest 也不做类型检查 #4642 意义上的):扫描器要是悄悄不再匹配,否则会以「扫了个空集」的姿态通过。

验证

pnpm --filter @objectstack/client test —— 20 个测试文件 / 240 个用例全绿(pin 新增 3 条)。
pnpm --filter @objectstack/client typecheck —— tsc --noEmit 加 check:test-typecheck 均通过;新 pin 与 client.test.ts 都不在 test-typecheck-debt.json 里,按该 ledger 的规矩必须零错误,实测零错误(ledger 仍是原来的 3 文件 / 6 错误,未动)。

反向验证(方向为事先预测的「红」,即常规方向): 把退役拼法原样放回 client.test.ts 的替身,pin 的两条否定断言双双转红并点名 client.test.ts,而反空转那条保持绿(证明扫描器仍然匹配到了整个替身population,红不是因为扫了空集):

✓ finds the doubles it claims to police (anti-vacuity)
× no double carries `endpoints` (retired in #4828)
  → expected [ 'client.test.ts' ] to deeply equal []
× no double binds `capabilities` to an array
  → expected [ 'client.test.ts' ] to deeply equal []

随后已还原修复,重跑全绿。

范围

Tests-only,无 changeset(需要 skip-changeset 标签)。没有碰 packages/rest、packages/spec 或任何生产代码;apiName 保持原样 —— 它不是缺陷,是 GetDiscoveryResponse 上已声明的弃用别名,真实 producer 至今与 name 同值双发(protocol 18 移除)。同文件里 meta 面的 mock 未动(#5895 / #5946 刚刚有意重塑过)。

🤖 Generated with Claude Code

https://claude.ai/code/session_01Wbxm29qPKnLf44AbSxizqW

---

Generated by Claude Code

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
objectstack	Ignored		Aug 7, 2026 2:12am

[github-actions]

📓 Docs Drift Check

No hand-written docs reference the 0 changed package(s). ✅