fix: 收敛 tsc7 双轨入口(审查 d9cf0d2) - #58
Merged
Tanya7z merged 1 commit intoJul 23, 2026
Merged
Conversation
d9cf0d2 引入 TS6/TS7 双轨后,tools/package.json 未声明 tsc7 bin、 emit 路径仍 npx tsc、根 typecheck 写死包列表、脚手架仍用裸 tsc。 统一经 tools/tsc7.mjs(runTsc7 + realpath 主入口)与 bin tsc7。 Co-authored-by: Shiroha <Tanya7z@users.noreply.github.com>
Tanya7z
marked this pull request as ready for review
July 23, 2026 17:05
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
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.
背景
main@61b921b合入d9cf0d2(TypeScript 6/7 双轨 +tools/tsc7.mjs)。ootb 已绿(run30025918061)。本 PR 收敛审查中可验证的 SOLID 债。审查(DRY / OCP / DIP / LSP / Demeter)
package-lock已声明tsc7bin,但tools/package.json未声明 — 权威源不一致;npm install会按 package.json 回写 lockesbuild-transpile/ SDKbuild.mjs仍npx tsc,与 typecheck 入口分叉;依赖.bin/tsc链接胜出typecheck写死 7 个-p路径 — 新增 workspace 需改根脚本new-module脚手架仍tsc --noEmit,与已装模块tsc7契约不一致node ../tools/tsc7.mjsvs../../../tsc7.mjs经.bin符号链接调用时若用 href 比入口会空跑(本 PR 用 realpath)改动
tools/package.json声明tsc7bin;tsc7.mjs导出runTsc7/resolveNativeTsc,主入口用 realpath(兼容 symlink)scripts/esbuild-transpile.mjs、@sfmc-sdk/build.mjs改调runTsc7(唯一 emit 权威)typecheck/build:types统一tsc7 …;根脚本改为npm run typecheck --workspaces --if-presentnew-module.mjs脚手架对齐tsc7验证
npm run build --workspaces --if-presentnpm run typecheck(全 workspace)npm run test -w @sfmc-bds/bds-tools/db-server未改
npx tsc(docs drift,非阻塞)