fix: push@72e24e7 SOLID 审查 — ootb npm install + registry/unzip 契约对齐

[cursor]

Summary

main push e6525bb（fix tools）+ 72e24e7（lodash 挪入 deps）后 ootb 仍红（run 29897268620）：Install root deps → Cannot read properties of undefined (reading 'extraneous')。

本 PR 在最新 main 上直接给出可合并补丁（#15/#16 基于旧 tools/.js 树，已被 e6525bb ESM 重写 supersede，请优先合本 PR）。

根因与修复

级别	原则	问题	修复
BLOCKER	DRY	package-lock.json 仍 link 已删 modules/packages/{daily-task,economy,…}	清悬空 link，登记 afk workspace
BLOCKER	LSP	sfmc/src/registry.ts 把 index 当扁平 map；真实契约是 { modules: {…} }（fetch-module 已正确）	parseRegistryIndex + tools/registry-index.mjs 对齐
BLOCKER	LSP	unzip 未归一化 Windows \\	\\→/ 后再 join
MAJOR	DRY	folderFromEntry 在 catalog 与 packages 重复；check-modules 用 split('/')[2]	统一 folderFromEntryPath
MAJOR	DIP	fetch-module 动态 import('jszip') 但未声明依赖	补 jszip
MINOR	—	dirSize 未 await walk → 体积恒 0	补 await

验证

• npm install --no-audit --no-fund ✅
• npm run build --workspaces --if-present ✅
• node tools/check-ootb.mjs → 10/10 ✅
• node tools/smoke-modules.mjs（含 feature-afk 启停翻转）✅

不在本 PR

• 审查+修复:v2 SOLID(DRY/OCP/DIP/LSP/迪米特) #13 host token / service HTTP 门
• fix: push@c559ab4 SOLID 审查补丁 — economy SQL/? 绑定 + service 契约对齐 #14 economy SQL/?（业务包已外置；需 rebase 或关闭）
• 关闭过时草稿 fix(ci): 修复 ootb 工作流连续失败的 4 处根因 #5/fix(ci): 修复 ootb 工作流长期失败的四处根因 #6/fix(ci): 修复 ootb workflow 持续失败 — check-catalog ESM 崩溃、db-server 入口路径、模块启停缓存 #7/fix(ci): 模块外置后 npm install 崩溃 + 空 catalog OOTB/registry 对齐 #15/fix(ci): 修复模块抽取后 ootb npm install 崩溃 + 空 catalog 工具链 #16（建议人工 close as superseded）

  

[cursor]

审查摘要（push e6525bb + 72e24e7）

结论：ootb 仍红；本 PR 已给可合并补丁。

BLOCKER

1. DRY — package-lock.json 仍 link 已删 packages → npm install extraneous（run 29897268620）
2. LSP — sfmc/src/registry.ts 未解析 { modules: … }，与 fetch-module / 真实 index.json 契约不一致
3. LSP — unzip 未归一化 \\，Linux 安装布局错乱

MAJOR

4. DRY / Demeter — folder 解析重复且 check-modules 用脆弱 split('/')[2]
5. DIP — 动态 import('jszip') 但未声明依赖

MINOR

6. dirSize 未 await → 体积统计恒 0

本地验证：check-ootb 10/10、smoke-modules 通过。

#15/#16 基于旧 tools 树，已被本 PR supersede，建议关闭。

View PR

  

_{Sent by Cursor Automation: SFMC-Review}

[Tanya7z]

Superseded by main tools rewrite (e6525bb) and PR #17 CI/registry fixes. Closing as duplicate/outdated.