Skip to content

fix(@angular-devkit/build-angular): fix app-shell route format and improve package resolution#33017

Merged
alan-agius4 merged 1 commit intoangular:mainfrom
alan-agius4:app-shell
Apr 17, 2026
Merged

fix(@angular-devkit/build-angular): fix app-shell route format and improve package resolution#33017
alan-agius4 merged 1 commit intoangular:mainfrom
alan-agius4:app-shell

Conversation

@alan-agius4
Copy link
Copy Markdown
Collaborator

improve package resolution

  • In the app-shell builder, ensure the route URL has a leading slash.
  • In webpack helpers, use createRequire in isPackageInstalled to improve package resolution robustness.

improve package resolution

- In the `app-shell` builder, ensure the route URL has a leading slash.
- In webpack helpers, use `createRequire` in `isPackageInstalled` to improve package resolution robustness.
@alan-agius4 alan-agius4 requested a review from clydin April 17, 2026 12:00
@alan-agius4 alan-agius4 added the action: review The PR is still awaiting reviews from at least one requested reviewer label Apr 17, 2026
@alan-agius4 alan-agius4 changed the title fix(@angular-devkit/build-angular): fix app-shell route format and fix(@angular-devkit/build-angular): fix app-shell route format and improve package resolution Apr 17, 2026
@alan-agius4 alan-agius4 added the target: patch This PR is targeted for the next patch release label Apr 17, 2026
Copy link
Copy Markdown

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

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 modifies the app-shell builder to ensure routes are correctly prefixed with a slash and refactors the package installation check to use createRequire for module resolution. Review feedback identifies a potential bug where an undefined route would result in the string "/undefined" and recommends using path.resolve for more robust cross-platform path handling when creating a require instance.

@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 Apr 17, 2026
@alan-agius4 alan-agius4 merged commit cf5b49e into angular:main Apr 17, 2026
39 checks passed
@alan-agius4 alan-agius4 deleted the app-shell branch April 17, 2026 13:15
@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-devkit/build-angular target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants