sms settings 的 provider options 表与 SMS transports 之间没有契约测试 —— mail 有,sms 没有(#5094 同形,目前两端恰好一致) #5773

Activity

1. baozhoutao

   added

   finding

   on Aug 6, 2026
2. claude

   added

   domain:services

   on Aug 6, 2026
3. claude commented on Aug 6, 2026

   

   claudebot

   on Aug 6, 2026 – with Claude

   Contributor

   More actions

   分诊(发现分诊轮,#4949 纪律):持有(留 finding),补域标签 domain:services。

   • 域锚定:两端都在 services —— manifest 侧 packages/services/service-settings/src/manifests/sms.manifest.ts,transport 侧 packages/services/service-sms/src/transports/index.ts;按域表 packages/services/* ⇒ domain:services。(mail 的对照物 mail-manifest-providers.contract.test.ts 住在 plugin-email,按域表同属 services 行,所以无论测试最终落哪一边,车道都不变 —— 这也是不必为「测试住哪个包」另开决策卡的原因。)
   • 为什么还留着(不晋级):① 今日两端集合恰好相等(log / aliyun / twilio),无用户可达缺陷,正文自陈是休眠的漂移风险;② 更硬的一条 —— 这道契约测试的原料 SMS_TRANSPORT_PROVIDERS 具名导出尚未在 main 上,它由 PR fix(cli,service-sms)!: 启动期拒绝表外的 OS_SMS_PROVIDER,而不是静默降级成 LogSmsTransport (#5713) #5771(CLI 启动直读 process.env.OS_SMS_PROVIDER / OS_EMAIL_PROVIDER,完全绕过 settings 的 options 表 —— #5204 的闸门看不到这条路 #5713,cli 车道在飞,本轮复核 state=open、未合并) 引入。现在派发会与在飞 PR 争同一批文件。
   • 重启条件:PR fix(cli,service-sms)!: 启动期拒绝表外的 OS_SMS_PROVIDER,而不是静默降级成 LogSmsTransport (#5713) #5771 合入 main 后晋级 pm:queue。晋级时,正文那个未决点(测试住 service-sms 需新增对 service-settings 的依赖、还是住 service-settings 反向 import service-sms,取决于依赖方向是否成环)作为实施内第一步判定即可,不另开决策卡 —— 它是可由代码判定的技术问题,不是产品拍板。
   • 关联已核:plugin-email: SendGrid / Amazon SES 设置项同样后端无实现 —— #5087 的同形缺口 #5094(mail 侧同形漂移的原始案例,以及那道双向契约测试的来源)、service-settings: select 型 specifier 的 options 在保存期完全不校验 —— 声明的枚举不被强制 #5131(options 表在写入路径上的执行)、CLI 启动直读 process.env.OS_SMS_PROVIDER / OS_EMAIL_PROVIDER,完全绕过 settings 的 options 表 —— #5204 的闸门看不到这条路 #5713 / PR fix(cli,service-sms)!: 启动期拒绝表外的 OS_SMS_PROVIDER,而不是静默降级成 LogSmsTransport (#5713) #5771(不同的面:构造期 env 拒收,不是 settings 下拉框)。

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

   ---

   Generated by Claude Code
4. os-zhuang commented on Aug 6, 2026

   

   os-zhuang

   on Aug 6, 2026 – with Claude

   Contributor

   More actions

   分诊(发现分诊轮):重启条件已触发 ⇒ 晋级 pm:queue(域标签维持 domain:services)

   上一轮(2026-08-06 04:58Z)判持有,记的重启条件是「PR #5771 合入后 —— 该 PR 提供的具名导出是这道测试的原料」。本轮核验该条件已满足:

   • PR fix(cli,service-sms)!: 启动期拒绝表外的 OS_SMS_PROVIDER,而不是静默降级成 LogSmsTransport (#5713) #5771 已 MERGED,merged_at = 2026-08-06 05:02:03Z(base f205c32)。
   • origin/main(1624f4a)上原料到位:SMS_TRANSPORT_PROVIDERS = ['log','aliyun','twilio'] as const(packages/services/service-sms/src/transports/index.ts:27)、isSmsTransportProvider()(:38),并经 packages/services/service-sms/src/index.ts:13-14 出包;SmsProviderTag 已从数组派生(不再是第二份字面量)。
   • 另一端仍是正文描述的形状:packages/services/service-settings/src/manifests/sms.manifest.ts:24 的 provider 仍是带 options 表的 select,sms.manifest.test.ts 仍只断言 schema/权限/密钥字段,没有任何测试比对取值集合。两端「恰好一致」的休眠漂移风险原样成立。

   落点与待决取舍(留给派发词,不是拍板项): 双向比对 sms.manifest.ts 的 options 与 SMS_TRANSPORT_PROVIDERS;测试住哪一侧(service-sms 新增对 service-settings 的依赖——需先确认不成环;或 service-settings 反向 import service-sms)由 dev 在实读依赖图后定,mail 侧的既有形是 packages/plugins/plugin-email/src/mail-manifest-providers.contract.test.ts(住在消费侧插件里),可作方向参考。

   查重:三仓 open issue 与 PR 各搜一遍,无影子单;#5713/#5094/#5131 均为相关面而非重复。

   ℹ️ 提示:domain:services 座位自 2026-08-06 05:15Z 起按维护者指示暂停(见 #4604 该行),本单入队后需等座位恢复才会被选批。

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

   ---

   Generated by Claude Code
5. os-zhuang

   added

   pm:queue

   and removed

   finding

   on Aug 6, 2026
6. hotlong

   added

   pm:dispatched

   on Aug 6, 2026
7. hotlong

   self-assigned this

   on Aug 6, 2026
8. hotlong commented on Aug 6, 2026

   

   hotlong

   on Aug 6, 2026 – with Claude

   Contributor

   More actions

   认领:PM 循环第 3 轮(services 车道)
   会话:session_015a5qkLzpGXhLL2F5gvJ7dD
   分支:claude/issue-5773-sms-manifest-provider-contract
   Worktree:objectstack-issue-5773
   域:domain:services
   文件面:新契约测试一份(住 packages/services/service-sms 或 packages/services/service-settings,依赖方向 dev 实读后定,mail 先例住消费侧)+ sms.manifest.ts 如需加指向注释;⛔ 不改 provider 词表本身
   串行约束已清:PR #5771(原料 SMS_TRANSPORT_PROVIDERS)已合于 05:02Z,今日 de770bf 触过 service-sms,基于合并后 origin/main;#2814(将增 sms settings 键、同 manifest 文件)后轮串行;#5712 服务半边 blocked(#5933),同包无在飞冲突

   ---

   Generated by Claude Code
9. hotlong commented on Aug 6, 2026

   

   hotlong

   on Aug 6, 2026 – with Claude

   Contributor

   More actions

   验收:ACCEPT → PR #5982(tests-only;CI 绿后转 ready 入队,本座位跟到 MERGED)。

   • 双向契约测试照 mail 先例落地且更进一步:⊆/⊇ 两方向独立成 it 各自点名漂移值;第二层断言正确的 transport 类(非「没抛异常」);isSmsTransportProvider(CLI 启动直读 process.env.OS_SMS_PROVIDER / OS_EMAIL_PROVIDER,完全绕过 settings 的 options 表 —— #5204 的闸门看不到这条路 #5713 CLI 拒收判据)与下拉框锁同词表;默认值 log 在表内且可构建。
   • 探针 B 的预测落空是本单最有价值的产出:照抄 mail 先例的三行 toBeInstanceOf 被实测证明是「词表第三份拷贝」反模式(删项重复报红、加项静默不覆盖)→ 改为 PROVIDER_FIXTURES 表 + 键集 ≡ SMS_TRANSPORT_PROVIDERS 覆盖断言,且刻意用运行时断言避开 @ts-expect-error 退役 pin 在 packages/spec 里是幽灵检查:tsconfig 把 **/*.test.ts 排除出唯一的 tsc --noEmit #5286 的 phantom-check 形状(本包 tsconfig 排除测试文件,类型层穷尽无 tsc 求值)。mail 侧同形反模式值得留意(未立单 —— 它的三行在被 tsc 求值的包里,危害减半)。
   • 落点判定合格:实测两向不成环后按架构理由选 transport owner 持测试(注册表不反向 import 各 provider 实现),与 mail 先例同向;判定理由写进 PR。
   • TEST_DEBT 台账重测量(3→4,errors 不动)带实测证据,sms.manifest.ts CONTRACT 注释指名测试路径与两方向 —— 一致性半径内的必要 rider。
   • 必答项两条:feat(sms): 短信全局/每租户日发送配额（成本总量闸） #2814 完全无影响(断言面 scope 到 key === 'provider',并在 TSDoc 里点名 feat(sms): 短信全局/每租户日发送配额（成本总量闸） #2814 写死该约束 —— 结构性隔离而非口头承诺);spec: SettingsManifest 的 SpecifierSchema 新增 valueDomain 闭合枚举 —— 声明存在时标准域为执行边界,options 退化为 UI 便利列表(#5712 裁决的 spec 半边) #5933 无影响(其正文自陈注册表背书封闭集照旧)。
   • skip-changeset 已由 dev 以读回并集方式打上(bot 标签零覆盖);Check Changeset 首跑竞态红已按 fix(ci): Check Changeset 实时读 skip-changeset 标签,首个 run 不再永久红 (#5580) #5625 语义 rerun 翻绿。

   经办:services 座位,会话 session_015a5qkLzpGXhLL2F5gvJ7dD(第 3 轮)。

   ---

   Generated by Claude Code
10. hotlong

    closed this as completedin #5982on Aug 6, 2026
11. os-zhuang

    added a commit that references this issue on Aug 7, 2026

    ci(dx): 按裁决(option A)在合并树上全量重校准,三个热包记档 +10 余量

    ...

    5bccbf1