Skip to content

feat: support TypeScript 6 transpile builds#1316

Merged
styfle merged 1 commit intovercel:mainfrom
roryabraham:fix-typescript-6-rootdir-build
Apr 28, 2026
Merged

feat: support TypeScript 6 transpile builds#1316
styfle merged 1 commit intovercel:mainfrom
roryabraham:fix-typescript-6-rootdir-build

Conversation

@roryabraham
Copy link
Copy Markdown
Contributor

@roryabraham roryabraham commented Apr 28, 2026

Summary

Updates ts-loader to 9.5.7 so ncc's bundled TypeScript loader preserves configured rootDir during transpileModule builds.

This fixes TypeScript 6 TS5011 failures like:

TS5011: The common source directory of 'tsconfig.json' is './actions/javascript/...'. The 'rootDir' setting must be explicitly set to this or another path to adjust your output's file layout.

ts-loader fixed this upstream in TypeStrong/ts-loader#1679.

Verification

  • Ran yarn build

@socket-security
Copy link
Copy Markdown

socket-security Bot commented Apr 28, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatednpm/​ts-loader@​9.5.1 ⏵ 9.5.79810010089100

View full report

Update ts-loader to include the rootDir handling fix needed by TypeScript 6.

Made-with: Cursor
@roryabraham roryabraham force-pushed the fix-typescript-6-rootdir-build branch from 4986753 to f8f55f9 Compare April 28, 2026 08:33
@styfle styfle changed the title fix: support TypeScript 6 transpile builds feat: support TypeScript 6 transpile builds Apr 28, 2026
Copy link
Copy Markdown
Member

@styfle styfle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I marked as feat since this is enabling new features of typescript@6.

Ideally this would have a test also to confirm the new features work.

@styfle styfle enabled auto-merge (squash) April 28, 2026 15:48
@styfle styfle merged commit e7c0c70 into vercel:main Apr 28, 2026
12 checks passed
@roryabraham
Copy link
Copy Markdown
Contributor Author

@styfle thanks for reviewing! For what it's worth, I tested this via patch-package for the bundling of our JS GitHub Actions over at Expensify/App#89061

By the way, it looks like the npm publish for 0.39 failed with a 403 error. Seems unrelated to these changes. I see ncc hasn't been published for ~7 months - perhaps it's using a classic npm token that's been revoked.

More info: https://github.blog/changelog/2025-10-10-strengthening-npm-security-important-changes-to-authentication-and-token-management/

@styfle
Copy link
Copy Markdown
Member

styfle commented Apr 28, 2026

Thanks, I'll see if IT can generate a new token so we can publish again 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants