Repository navigation
fix(build): align TypeScript dependency and lockfile - #218
Merged
Merged
Conversation
Contributor
Author
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
Use the supported JavaScript compiler API from TypeScript 6 instead of the incompatible TypeScript 7 package root. Remove redundant overrides, refresh the lockfile and constrain Renovate for build dependencies only.
jerome-benoit
changed the title
|
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.
PR Checklist
Does this PR introduce a breaking change?
Changes
build/package.jsononly. Consumer peer dependencies remain unchanged.Root cause
All three bundle jobs failed at
ts.sys.readFile. The lockfile still recorded TypeScript 5.9.3 while the manifest requested 7.0.2. Installation reconciled this mismatch and selected TypeScript 7, whose package root no longer exposes the legacy compiler API used by the build. The original manifest also contained duplicate override keys. An isolated installation of the base commit with Bun 1.4.4 resolves TypeScript 5.9.3, confirming why the base build passed despite its inconsistent declarations.No diagnostics, declaration generation or DOM/Web Worker consumer checks are removed.
Verification
d75b21bd6fba629eb6bcc0d5a6d72969fa86d2c9.