Skip to content

Conversation

alan-agius4
Copy link
Collaborator

This refactoring moves the substitution for @angular/ssr and beasties from a global configuration to a local one within the @angular/ssr package definition.

This is achieved by introducing an extra_substitutions parameter to the ng_package function, allowing package-specific substitutions to be defined. This change improves modularity and removes hardcoded paths from the global substitution rules, making the build system cleaner and easier to maintain.

Additionally, this fixes an issue where the generated TypeScript definition files (.d.ts) for @angular/ssr had incorrect relative paths for the beasties dependency, causing type resolution failures in consuming projects. The new local substitution corrects these paths.

@alan-agius4 alan-agius4 added target: major This PR is targeted for the next major release action: review The PR is still awaiting reviews from at least one requested reviewer labels Oct 10, 2025
@angular-robot angular-robot bot added the area: build & ci Related the build and CI infrastructure of the project label Oct 10, 2025
This refactoring moves the substitution for `@angular/ssr` and `beasties` from a global configuration to a local one within the `@angular/ssr` package definition.

This is achieved by introducing an `extra_substitutions` parameter to the `ng_package` function, allowing package-specific substitutions to be defined. This change improves modularity and removes hardcoded paths from the global substitution rules, making the build system cleaner and easier to maintain.

Additionally, this fixes an issue where the generated TypeScript definition files (.d.ts) for `@angular/ssr` had incorrect relative paths for the `beasties` dependency, causing type resolution failures in consuming projects. The new local substitution corrects these paths.
@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 10, 2025
@alan-agius4 alan-agius4 merged commit 7134dfe into angular:main Oct 10, 2025
31 of 32 checks passed
@alan-agius4 alan-agius4 deleted the fix/types/bundle branch October 10, 2025 08:04
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: build & ci Related the build and CI infrastructure of the project target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants