Skip to content

fix(wf@894436a): tools@0.1.1 发包失败 + HttpDB/db.tx SOLID - #30

Closed
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/bc-7e2665ad-acd4-4db7-b555-46b0a65f5951-8011
Closed

fix(wf@894436a): tools@0.1.1 发包失败 + HttpDB/db.tx SOLID#30
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/bc-7e2665ad-acd4-4db7-b555-46b0a65f5951-8011

Conversation

@cursor

@cursor cursor Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

npm-publish run 29928821721 失败:tag @sfmc-bds/tools@v0.1.1 指向的 commit 里 tools/package.json 仍是 0.1.0

修复

  • BLOCKER / DRY@sfmc-bds/tools bump 0.1.00.1.1(对齐已打 tag)
  • MAJOR / DIPHttpDB.resolveAuthToken — 空串不再用 ?? 阻断平台默认 token
  • MAJOR / LSPtx.call 改为 Promise<void>(去掉 undefined as U);post 保留 tx 失败信封的 step/code;去掉 post 成功后的死 !ok 分支
  • MAJOR / DIP:db/service 在未 set*ModuleContext 时 fail-fast(对齐 config)
  • MINOR / DRYassertPublishPackageInWorkspaces 额外校验清单路径存在 + name 一致;resolve-tag 版本不匹配时打印 workflow_dispatch 修复指引

合入后如何真正发出 0.1.1

现有 tag @sfmc-bds/tools@v0.1.1 仍指向旧 commit(含 0.1.0)。请任选:

  1. 推荐:本 PR 合入后,对 @sfmc-bds/tools@v0.1.1workflow_dispatch(checkout 默认分支,已含 0.1.1)
  2. 或删除并重打该 tag 到合入后的 main tip

审查残留(未改,下轮)

级别 原则
BLOCKER DIP setDb/Config/ServiceModuleContext 全仓无调用;module-tokens.json 无 SAPI 注入路径;AFK config.get 必炸
MAJOR Demeter/DIP config 客户端是进程单例(一 _configKey),多模块会互相覆盖
MINOR LSP tx.update 仍返回 patch as U;平台 legacy success 方言

验证

  • PUBLISH_TAG=@sfmc-bds/tools@v0.1.1 node tools/resolve-npm-publish-tag.mjs → Version OK
  • npm run typecheck -w @sfmc-bds/sdk 通过
  • SFMC_ROOT=$PWD node tools/check-ootb.mjs11/11
Open in Web View Automation 

npm-publish 在 tag @sfmc-bds/tools@v0.1.1 失败:package.json 仍为 0.1.0。
同步 bump tools 到 0.1.1;空 authToken 回落平台默认(DIP);tx.call 改为
Promise<void> 并保留 tx 失败 step/code(LSP);publish 清单校验路径/name(DRY)。

Co-authored-by: Shiroha <Tanya7z@users.noreply.github.com>
cursor Bot pushed a commit that referenced this pull request Jul 22, 2026
npm-publish tag @sfmc-bds/tools@v0.1.1 曾因 package.json 仍为 0.1.0 失败。
同步 bump;HttpDB 空串回落平台默认;publish 清单校验路径/name。

Co-authored-by: Shiroha <Tanya7z@users.noreply.github.com>

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

审查结论(本轮 push@894436a 后续)

本 PR 对 wf 失败(tools@0.1.1 版本漂移)与部分 SOLID(空 token / tx.call void / tx step-code)的修复方向正确。

建议:请优先合入(或关闭后改合)#31#31 已吸收本 PR 的 wf/HttpDB/publish 门禁改动,并继续修了本 PR 正文「审查残留」里的 BLOCKER:

级别 原则 #30 残留 #31
BLOCKER DIP set*ModuleContext 无注入 configs/all.module_tokens + bootModule 注入
BLOCKER DRY configs/all.modules 不再恒 []
MAJOR OCP ModuleId=string + id/configKey 双索引
MAJOR Demeter config 单例互踩 configKey 分桶 Map
MINOR LSP tx.update patch as U 录制期 Promise<void>

合入 #31 后对 @sfmc-bds/tools@v0.1.1workflow_dispatch(checkout 默认分支)即可真正发包。

View PR

Open in Web View Automation 

Sent by Cursor Automation: SFMC-Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants