Skip to content

refactor(@angular/build): use import resolution-mode for rolldown-plugin-dts types - #34334

Merged
alan-agius4 merged 1 commit into
angular:mainfrom
alan-agius4:refactor/rolldown-plugin-dts-types
Oct 13, 2026
Merged

alan-agius4 merged 1 commit into
angular:mainfrom
alan-agius4:refactor/rolldown-plugin-dts-types

Conversation

@alan-agius4

Copy link
Copy Markdown
Collaborator

Use { with: { 'resolution-mode': 'import' } } on the dynamic import type assertion for rolldown-plugin-dts and remove the custom ambient module declaration file (types.d.ts).

…gin-dts types

Use `{ with: { 'resolution-mode': 'import' } }` on the dynamic import type assertion for `rolldown-plugin-dts` and remove the custom ambient module declaration file (`types.d.ts`).

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates the dynamic import of "rolldown-plugin-dts" in "bundler.ts" to explicitly use ESM resolution mode via import attributes. Additionally, the local ambient module declaration file for "rolldown-plugin-dts" has been removed. I have no feedback to provide.

@alan-agius4 alan-agius4 added target: minor This PR is targeted for the next minor release action: review The PR is still awaiting reviews from at least one requested reviewer labels Oct 13, 2026
@alan-agius4
alan-agius4 requested a review from clydin October 13, 2026 15:26
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Oct 13, 2026
@alan-agius4
alan-agius4 merged commit 9c645ec into angular:main Oct 13, 2026
43 checks passed
@alan-agius4
alan-agius4 deleted the refactor/rolldown-plugin-dts-types branch October 13, 2026 15:55
@alan-agius4

Copy link
Copy Markdown
Collaborator Author

This PR was merged into the repository. The changes were merged into the following branches:

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

Labels

action: merge The PR is ready for merge by the caretaker area: @angular/build target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants