Skip to content

fix(bds-tools): 修复 #80/#81 合并损坏的 scanDestOccupancy(恢复 ootb/发版构建) - #87

Merged
Tanya7z merged 1 commit into
mainfrom
cursor/bc-00ef740c-dce3-466a-ae77-d60c9fdf6f38-7762
Jul 26, 2026
Merged

fix(bds-tools): 修复 #80/#81 合并损坏的 scanDestOccupancy(恢复 ootb/发版构建)#87
Tanya7z merged 1 commit into
mainfrom
cursor/bc-00ef740c-dce3-466a-ae77-d60c9fdf6f38-7762

Conversation

@cursor

@cursor cursor Bot commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

背景

main 合入 #80readPackDirOccupancy DRY)与 #81(BOM 剥离)后,scanDestOccupancy 合并冲突解决错误:保留了 let factsfacts?.uuid 输出,却把函数体留在对未声明 uuid/version/name 的赋值上。

CI 失败

审查(SOLID)

级别 原则 说明
BLOCKER DRY 占用扫描必须单一事实源 readPackDirOccupancy,禁止再内联 info→header 双读
BLOCKER 合并损坏导致工作区无法构建(ootb + 发版)
MINOR readPackDirOccupancy 文档仍写 Utf8BomError 上抛,与 #81 BOM 剥离不符(本 PR 已更正)

修复

验证

  • npm run build --workspaces --if-present
  • node tools/build-publishable.mjs → ok — 8 workspaces
  • node --test bds-tools/world-packs.test.mjs → 15 pass
Open in Web View Automation 

#80/#81 合并后 facts 声明与旧 uuid/version 赋值体拼在一起,tsc 构建失败并阻断 ootb / changeset-release。恢复 DRY 调用并补占用扫描回归测。

Co-authored-by: Shiroha <Tanya7z@users.noreply.github.com>
@Tanya7z
Tanya7z marked this pull request as ready for review July 26, 2026 11:10
@cursor

cursor Bot commented Jul 26, 2026

Copy link
Copy Markdown
Contributor Author

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@Tanya7z
Tanya7z merged commit fa79536 into main Jul 26, 2026
1 check passed
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