fix(ci): drop removed baseUrl option from the plugins tsconfig - #205
Merged
Merged
Conversation
The marketplace publish job's tsc run failed with TS5102 (Option 'baseUrl' has been removed) under the pinned TypeScript 6. The option was a leftover with an empty paths map, and the root tsconfig already dropped it when its own paths went tsconfig-relative; removing it here restores the publish.
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
1 participant
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.
Fixes the
Publish marketplace toolkitfailure on main.Root cause: the publish job compiles
packages/plugins/tsconfig.jsonwith the repo-pinned TypeScript 6, which removed thebaseUrlcompiler option (TS5102). The option was a leftover — an emptypathsmap withbaseUrl: ".", and TS 6 resolvespathsrelative to the tsconfig file anyway. The root tsconfig already droppedbaseUrlin the 2.0.16–2.0.20 releases; the package-level one was missed.Change: remove
baseUrl(and the now-pointless emptypaths) frompackages/plugins/tsconfig.json. No compiled-output change — verifiedbunx tsc -p packages/plugins/tsconfig.jsonexits 0 and emitsdist/index.d.ts(run locally with the pinned toolchain, reproducing the exact command the publish job failed on).Note: the open renovate PR #204 (oxfmt bump) is unrelated to this failure and is dispositioned separately.