Skip to content

build(deps): migrate to Nx 22 and Angular 21 - Phase 1#1

Open
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1768166626-nx-22-migration-phase1
Open

build(deps): migrate to Nx 22 and Angular 21 - Phase 1#1
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1768166626-nx-22-migration-phase1

Conversation

@devin-ai-integration
Copy link
Copy Markdown

build(deps): migrate to Nx 22 and Angular 21 - Phase 1

Summary

This PR migrates the workspace from Nx 21.5.2 to Nx 22.3.3 and Angular 20.3.0 to Angular 21.0.8. This is Phase 1 of the migration, focusing on foundation libraries.

Key dependency updates:

  • Nx: 21.5.2 → 22.3.3
  • Angular: 20.3.0 → 21.0.8
  • Angular CDK: 20.2.3 → 21.0.6
  • Storybook: 9.0.15 → 10.1.9
  • Jest: 29.7.0 → 30.2.0
  • Cypress: 14.5.1 → 15.8.2
  • Vitest: 3.2.4 → 4.0.9
  • zone.js: 0.15.1 → 0.16.0

All 60+ library package.json files have been updated with new peer dependency versions.

Review & Testing Checklist for Human

  • Verify Angular migrations completed: The Nx migration command hung on router-current-navigation migration and was interrupted. Verify that Angular 21 migrations (control-flow, router signals, ApplicationConfig) are properly applied or not needed.
  • Check Jest 30 ESM compatibility: The packages library tests failed with "Must use import to load ES Module: ora" errors. This may require Jest configuration updates for ESM module handling.
  • Review snapshot change: libs/components/packages/src/schematics/utility/typescript/__snapshots__/ng-ast.spec.ts.snap was modified - verify this is expected.
  • Run full CI test suite: Local Karma tests couldn't run due to ChromeHeadless environment issues. CI should validate all unit tests pass.
  • Verify Storybook 10 compatibility: Major version upgrade from Storybook 9 to 10 - test that storybook builds and runs correctly.

Recommended test plan:

  1. Wait for CI to complete and review any failures
  2. Run npx nx serve playground to verify the playground app works
  3. Run npx nx storybook storybook to verify Storybook 10 works
  4. Spot-check a few component builds: npx nx build core, npx nx build theme

Notes

  • Builds passed locally for foundation libraries (packages, core, theme, i18n)
  • Lint checks passed for all 164 projects
  • The CommonModule import was removed from list-summary.component.ts as part of Angular's control-flow migration (no longer needed with standalone components)

Link to Devin run: https://app.devin.ai/sessions/5fdf1acedde845e28e4adef7b52bf77d
Requested by: @bcmake

- Update Nx from 21.5.2 to 22.3.3
- Update Angular from 20.3.0 to 21.0.8
- Update Angular CDK from 20.2.3 to 21.0.6
- Update Storybook from 9.0.15 to 10.1.9
- Update Jest from 29.7.0 to 30.2.0
- Update Cypress from 14.5.1 to 15.8.2
- Update zone.js from 0.15.1 to 0.16.0
- Update all library package.json files with new dependency versions
- Update e2e-schematics peer dependencies for Nx 22
- Update cypress-commands peer dependencies for Cypress 15

Co-Authored-By: benc@cognition.ai <Benc@windsurf.com>
@devin-ai-integration
Copy link
Copy Markdown
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

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.

0 participants