fix(bds-tools): scanDestOccupancy 合并冲突导致 changeset-release/--dts 失败 - #86
Merged
Tanya7z merged 4 commits intoJul 27, 2026
Merged
Conversation
#80/#81 合入时 scanDestOccupancy 保留了 facts 外壳却残留未声明的 uuid/version/name 赋值，changeset-release/ootb 的 tsc7 --dts 挂掉； 运行时 ReferenceError 被 catch 吞掉会使占用 uuid 恒空。 改为 facts = readPackDirOccupancy(dir)（DRY），补 scan 单测； 清理已写入 pre.json 的残留 changeset 文件，避免与 pending 混淆。 Co-authored-by: Shiroha <Tanya7z@users.noreply.github.com>
pre 模式下 apply-release-plan 不会删除 .md（仅 exit/非 pre 才删）， 须留盘至 pre exit；listPendingChangesetFiles 靠 pre.json 排除。 误删会破坏 pre exit 时对历史 changeset 的再计算。 Co-authored-by: Shiroha <Tanya7z@users.noreply.github.com>
Tanya7z
marked this pull request as ready for review
July 27, 2026 06:52
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. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
合入 #80 后
changeset-release/ootb在bds-tools--dts失败（TS2304: Cannot find name 'uuid'|'version'|'name'）。根因（BLOCKER · DRY）
#80 抽取
readPackDirOccupancy后，与 #81（BOM/catch 拓宽）合并时冲突解决拼坏：scanDestOccupancy保留了facts外壳与facts?.…写出，却残留旧的未声明uuid=/version=/name=赋值，且从未facts = readPackDirOccupancy(dir)。运行时这些赋值抛
ReferenceError，被catch吞掉 → 占用 uuid 恒空 → 同 uuid 覆盖决策失效。修复
facts = readPackDirOccupancy(dir)（单一占用读取权威来源）scanDestOccupancy单测（含 kind）pre.json#changesets的残留.md（align-release-token-dry/bds-log-filter-solid），只留 pendingbuild-deps-world-packs-solid验证
node tools/build-publishable.mjs（含 bds-tools--dts）node --test bds-tools/world-packs.test.mjs（15 pass）node --test tools/changeset-release.test.mjs（12 pass）Trigger
workflow_run failure @
e75f2d6（Merge #80）→ runs30199481775/30199481763