Upgrade to Angular 22 (combines Renovate #11 + #13)#28
Conversation
ng update @angular/core@22 @angular/cli@22 — all @angular/* -> 22.0.4, TypeScript 6.0.3. Migrations applied (provideClientHydration withNoIncrementalHydration; tsconfig.app.json). Combines Renovate #11 + #13 (Angular framework and CLI were split into two PRs whose lockfiles couldn't resolve) now that TypeScript 6 (#16) is merged — @angular/build@22 requires typescript >=6.0 <6.1. Verified: build + 9/9 tests green; home (WebGL shader, portrait, AI section) and /projects (18 covers) render with zero console errors; SSR returns 200 and server-renders content (NG_ALLOWED_HOSTS already set on Railway for the new v22 SSRF host check). renovate.json now groups @angular/*, devkit, and typescript into one "angular" PR so the interdependent upgrade never splits again.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (3)
📒 Files selected for processing (2)
Cache: Disabled due to data retention organization setting Knowledge base: Disabled due to data retention organization setting Summary by CodeRabbit
WalkthroughTwo unrelated changes: Renovate configuration now groups Angular-related packages (framework, CLI/devkit, schematics, ng-packagr, zone.js, TypeScript) under one ChangesRenovate Angular Grouping
Client Hydration Configuration
Estimated code review effort: 1 (Trivial) | ~3 minutes Suggested labels: Comment |
There was a problem hiding this comment.
Pull request overview
This PR completes the repo’s upgrade to Angular 22 (framework + CLI/build tooling) on top of the already-merged TypeScript 6 upgrade, applies the Angular 22 hydration migration in app bootstrap, and adds Renovate configuration intended to keep Angular-related packages updated in a single lockstep PR going forward.
Changes:
- Upgrades
@angular/*, devkit/build tooling, and TypeScript to Angular22.0.4+ TypeScript6.0.3. - Updates app bootstrap hydration configuration to include
withNoIncrementalHydration()as per Angular 22 migration. - Adds compiler diagnostic suppressions and updates Renovate rules intended to group Angular-related updates.
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tsconfig.app.json | Adds Angular compiler extended diagnostics suppressions for specific nullability checks. |
| src/app/app.config.ts | Updates hydration setup to include withNoIncrementalHydration() and reformats imports. |
| package.json | Bumps Angular dependencies/devDependencies to 22.0.4 and TypeScript to 6.0.3. |
| package-lock.json | Updates resolved dependency tree for Angular 22 + TS 6, including updated engine constraints and toolchain packages. |
| .github/renovate.json | Adds a Renovate grouping rule intended to keep Angular-related updates in a single PR. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "matchPackageNames": [ | ||
| "@angular/**", | ||
| "@angular-devkit/**", | ||
| "@schematics/angular", | ||
| "ng-packagr", | ||
| "zone.js", | ||
| "typescript" | ||
| ], |
Finishes the Angular 22 upgrade that Renovate split across #11 (framework) and #13 (CLI) — neither could resolve alone because it's an interdependent chain: TypeScript 6 → @angular/build 22 (needs
typescript >=6.0 <6.1) → @angular/core 22. With TS 6 (#16) merged,ng updatecompletes cleanly.What ran:
ng update @angular/core@22 @angular/cli@22→ all@angular/*→ 22.0.4, TypeScript 6.0.3. Migrations applied toapp.config.ts(provideClientHydration(..., withNoIncrementalHydration())) andtsconfig.app.json.Verified (no build CI here, so done manually):
npm run build+npm test(9/9) green/projects(all 18 covers) render, 0 console errors/and/projectsreturn 200 and server-render real content. Angular 22 adds an SSRF host-allow-list; Railway already hasNG_ALLOWED_HOSTS=alexander-production-2841.up.railway.app+trustProxyHeaders:true, so SSR works on deploy. (Caught locally: a disallowed host hard-400s in v22 vs CSR-fallback in v21.)@defer, nativefetchnotHttpClient). ERESOLVE warnings are install-time only (lenisreact/vue peers, SDK optionalzod).Also groups
@angular/*+ devkit +typescriptinto one Renovate "angular" PR so this split can't recur. Merging this closes #11 and #13.🤖 Generated with Claude Code