fix(build): replace tsconfck with native typescript config parsing to support ts 6#3967
Conversation
🦋 Changeset detectedLatest commit: f0a875e The changes in this PR will be included in the next version bump. This PR includes changesets to release 25 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
WalkthroughThe ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Hi @Rahulchaudharyji2, thanks for your interest in contributing! This project requires that pull request authors are vouched, and you are not in the list of vouched users. This PR will be closed automatically. See https://github.com/triggerdotdev/trigger.dev/blob/main/CONTRIBUTING.md for more details. |
Closes #3953
✅ Checklist
Testing
@trigger.dev/buildpackage successfully:Result: Passed successfully.
Changelog
Removed tsconfck dependency from @trigger.dev/build to fix the peer dependency installation conflict when using TypeScript 6 and pnpm.
Implemented native tsconfig.json parsing in typescript.ts using the already runtime-imported typescript package APIs (findConfigFile, readConfigFile, and parseJsonConfigFileContent).
Added Map-based caching for resolved compiler options to avoid duplicate disk reads and parsing.
💯