fix(wf@3d15949): #33 合并 TS6133 + banned_items/canDisable LSP

[cursor]

Summary

main@3d15949（合入 #33）ootb 再次失败（run 29940895451）：#32/#33 合并残留未使用的 MODULE_HELP_LABEL，触发 TS6133，sfmc workspace build 红。

同批审查（DRY · OCP · DIP · LSP · Demeter）发现并修复两处可验证 MAJOR LSP，以及相关 DRY/防御加固。

触发背景：此前 #31 合入后 ootb 失败（module/mod HELP）已由 #32/#33 修复；本 PR 修 #33 合并回归 + 残留 SOLID。

审查

级别	原则	问题	处理
BLOCKER	DRY	#32 MODULE_HELP_LABEL 与 #33 paintModuleCmdAlias 合并后双源，前者未删 → TS6133 阻塞 ootb	删除死代码，HELP 只走 paintModuleCmdAlias
MAJOR	LSP / DRY	configs/all.banned_items 发 {item_id}，ConfigManager/AllConfigs/GET /banned_items 契约是 string[] → 缓存恒空	getAllConfigs 复用 helpers，返回 string[]
MAJOR	LSP / DRY	catalog 省略 canDisable 时 list 报 can_disable:true，resolveModuleByKey 裸返回 → disable 被 !undefined 拦	moduleCanDisable() 同源归一化
MINOR	LSP	service.get 失败一律 code:"internal"（db 客户端已保留 code）	对齐保留服务端 code
MINOR	LSP	ConfigManager 仅认 string	兼容历史 {item_id}
MINOR	—	ootb 未锁 banned_items 形状	断言元素为 string

未展开（仍为设计债）：

• DIP：运行时 enable 热更新 token/enabledSet
• LSP：db.tx 两阶段读回协议

验证

npm run build --workspaces --if-present
SFMC_ROOT=$PWD npm run check-ootb   # 11/11 PASS
# 隔离 SFMC_ROOT：configs/all.banned_items 为 string[]；
# catalog 无 canDisable 时 POST disable → success

Test plan

• CI ootb 在本分支绿
• GET /api/sfmc/configs/all 的 banned_items 为 string[]
• catalog 省略 canDisable 的模块可 POST .../disable

  

[cursor]

并行 PR 说明

本 PR 与 #34/#36 同修 main@3d15949 的 TS6133 + SOLID LSP。本 PR 中的 moduleCanDisable / getAllConfigs helpers / ConfigManager 兼容 / service.get code / ootb 断言已吸收进 #36。

建议关闭本 PR，改合 #36：超集，另含 enable 热同步（本 PR Out of scope 的 MAJOR DIP）。

View PR

  

_{Sent by Cursor Automation: SFMC-Review}