Skip to content

update[angular]: Upgrade Angular SDK and test apps to Angular 19#4633

Open
floating-dynamo wants to merge 6 commits into
mainfrom
ENG-11558
Open

update[angular]: Upgrade Angular SDK and test apps to Angular 19#4633
floating-dynamo wants to merge 6 commits into
mainfrom
ENG-11558

Conversation

@floating-dynamo
Copy link
Copy Markdown
Contributor

@floating-dynamo floating-dynamo commented Jun 1, 2026

Description

  • We need to move the Angular SDK from v17 to v19 so that the generated output can drop the deprecated allowWriteSignals flag (deprecated in 19+) (Mitosis PR)
  • So going forward the SDK supports only Angular 19+

Screenshot
If relevant, add a screenshot or two of the changes you made.


Note

High Risk
Breaking peer dependency and generated effect() behavior for all Angular SDK consumers; large removal/rename of test apps and CI matrix changes.

Overview
Breaking change: @builder.io/sdk-angular now requires Angular ≥ 19.0.0 and drops the deprecated allowSignalWrites option from generated effect() calls (documented in changeset).

The Angular 17 e2e apps and snippets are removed; existing angular-19 / angular-19-ssr apps are renamed and retargeted as the sole Angular test matrix (CI, SERVER_NAME, Playwright skips, and spec targets). Dependencies move to 19.2.x (CLI, SSR, TypeScript ~5.6, zone.js ~0.15).

SSR entry points are updated for Angular 19: bootstrapApplication receives BootstrapContext, CommonEngine uses allowedHosts, and snippet/e2e servers align with @angular/ssr/node and ** route patterns where applicable.

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

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jun 1, 2026

🦋 Changeset detected

Latest commit: fea0361

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@builder.io/sdk-angular Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@nx-cloud
Copy link
Copy Markdown

nx-cloud Bot commented Jun 1, 2026

View your CI Pipeline Execution ↗ for commit fea0361

Command Status Duration Result
nx test @snippet/react ✅ Succeeded 59s View ↗

☁️ Nx Cloud last updated this comment at 2026-06-02 11:00:10 UTC

@nx-cloud
Copy link
Copy Markdown

nx-cloud Bot commented Jun 1, 2026

View your CI Pipeline Execution ↗ for commit 961d39f

Command Status Duration Result
nx test @builder.io/sdk ✅ Succeeded 5s View ↗
nx test @builder.io/react -- packages/react/src ✅ Succeeded 12s View ↗

☁️ Nx Cloud last updated this comment at 2026-06-01 04:15:58 UTC

@nx-cloud
Copy link
Copy Markdown

nx-cloud Bot commented Jun 1, 2026

🤖 Nx Cloud AI Fix Eligible

An automatically generated fix could have helped fix failing tasks for this run, but Self-healing CI is disabled for this workspace. Visit workspace settings to enable it and get automatic fixes in future runs.

To disable these notifications, a workspace admin can disable them in workspace settings.


View your CI Pipeline Execution ↗ for commit 961d39f

Command Status Duration Result
nx test @snippet/angular-19-ssr ❌ Failed 2m 16s View ↗
nx test @e2e/qwik-city ✅ Succeeded 9m 12s View ↗
nx test @e2e/nextjs-sdk-next-app ✅ Succeeded 8m 29s View ↗
nx test @e2e/angular-19 ✅ Succeeded 8m 11s View ↗
nx test @e2e/nuxt ✅ Succeeded 7m 7s View ↗
nx test @e2e/hydrogen ✅ Succeeded 6m 23s View ↗
nx test @e2e/angular-19-ssr ✅ Succeeded 6m 27s View ↗
nx test @e2e/gen1-next15-app ✅ Succeeded 5m 53s View ↗
Additional runs (37) ✅ Succeeded ... View ↗

☁️ Nx Cloud last updated this comment at 2026-06-01 04:25:07 UTC

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.

1 participant