Skip to content

chore: replace rollup-plugin-dts with tsc for declaration emission#385

Merged
kinyoklion merged 1 commit intomainfrom
rlamb/sdk-2192/remove-rollup-plugin-dts
Apr 16, 2026
Merged

chore: replace rollup-plugin-dts with tsc for declaration emission#385
kinyoklion merged 1 commit intomainfrom
rlamb/sdk-2192/remove-rollup-plugin-dts

Conversation

@kinyoklion
Copy link
Copy Markdown
Member

@kinyoklion kinyoklion commented Apr 16, 2026

Remove rollup-plugin-dts and emit type declarations directly with tsc via a new tsconfig.build.json. The build now produces lib/types/*.d.ts alongside the existing lib/cjs and lib/esm bundles; the types / exports.types entries in package.json point at lib/types/index.d.ts.

The rollup-plugin-dts was just combining all the types into a single .d.ts file for this project. It wasn't rolling up the .d.ts from its dependencies.


Note

Medium Risk
Primarily build/publish metadata changes, but it affects the emitted/packaged TypeScript declaration layout and could break consumers if the new lib/types output differs or is incomplete.

Overview
Build output now generates .d.ts files via tsc -p tsconfig.build.json into lib/types/ (declarations-only), rather than using Rollup to bundle a single lib/index.d.ts.

package.json and exports.types are updated to point consumers at ./lib/types/index.d.ts, and rollup-plugin-dts is removed from the Rollup config and dev dependencies.

Reviewed by Cursor Bugbot for commit 9227a7d. Bugbot is set up for automated code reviews on this repo. Configure here.

Remove rollup-plugin-dts and emit type declarations directly with tsc
via a new tsconfig.build.json. The build now produces lib/types/*.d.ts
alongside the existing lib/cjs and lib/esm bundles; the `types` /
`exports.types` entries in package.json point at lib/types/index.d.ts.
@kinyoklion kinyoklion requested a review from a team as a code owner April 16, 2026 16:46
@kinyoklion kinyoklion merged commit a6742ae into main Apr 16, 2026
7 checks passed
@kinyoklion kinyoklion deleted the rlamb/sdk-2192/remove-rollup-plugin-dts branch April 16, 2026 17:37
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