Skip to content

[Snyk] Security upgrade @angular/compiler from 8.2.13 to 19.2.17 #78

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
thiennn wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[Snyk] Security upgrade @angular/compiler from 8.2.13 to 19.2.17 #78

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion myshop-angular/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"dependencies": {
"@angular/animations": "~8.2.13",
"@angular/common": "~8.2.13",
"@angular/compiler": "~8.2.13",
"@angular/compiler": "~19.2.17",

Copilot AI Dec 5, 2025

Copy link

Choose a reason for hiding this comment

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

This upgrade introduces a critical version mismatch. The @angular/compiler is being upgraded from v8.2.13 to v19.2.17 (a jump of 11 major versions), but all other Angular packages remain at v8.2.13:

  • @angular/animations: ~8.2.13
  • @angular/common: ~8.2.13
  • @angular/core: ~8.2.13
  • @angular/forms: ~8.2.13
  • @angular/platform-browser: ~8.2.13
  • @angular/platform-browser-dynamic: ~8.2.13
  • @angular/router: ~8.2.13
  • @angular/compiler-cli: ~8.2.13 (in devDependencies)

Angular packages must be kept in sync at the same major version. This mismatch will cause compilation errors and runtime failures because the compiler version is incompatible with the other Angular packages.

Recommendation: Upgrade all Angular packages together to v19.2.17, or find a security patch for Angular v8 if a full migration is not feasible. A major version upgrade from v8 to v19 requires careful migration following Angular's update guides for each major version.

Suggested change
"@angular/compiler": "~19.2.17",
"@angular/compiler": "~8.2.13",

Copilot uses AI. Check for mistakes.

Copilot AI Dec 5, 2025

Copy link

Choose a reason for hiding this comment

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

Angular 19 requires TypeScript 5.6 or later, but this project is using TypeScript ~3.5.3. This version incompatibility will cause build failures even if the Angular version mismatch is resolved.

If upgrading all Angular packages to v19, the TypeScript dependency must also be updated to at least v5.6 to match Angular 19's requirements.

Copilot uses AI. Check for mistakes.
"@angular/core": "~8.2.13",
"@angular/forms": "~8.2.13",
"@angular/platform-browser": "~8.2.13",
Expand Down
Loading