chore(a22): retire deprecated Angular packages — build-angular executor, @angular/animations, platform-browser-dynamic #725

New Issue

2026-07-20T00:37:33Z

spikerj commented

2026-07-20 00:37:33 +00:00

pnpm install flags three direct dependencies as deprecated on Angular 22. Worked as three PRs in series — all merged:

PR 1 — `@angular-devkit/build-angular` (dev) → spikersoft-angular#447 ✅

extract-i18n target swapped to @angular/build:extract-i18n. The package itself stays installed: @storybook/angular 10.5 hard-peers on it and imports it at runtime in its webpack preset. Drop the dep only when Storybook does.

PR 2 — `@angular/animations` → spikersoft-angular#448 ✅

book-shelf translateTab trigger was dead code (never bound in the template) — deleted, no CSS replacement needed
commented-out DSL in classes.component.ts deleted
~50 specs: NoopAnimationsModule/provideAnimations etc. removed; profile spec needed the one real replacement: { provide: MATERIAL_ANIMATIONS, useValue: { animationsDisabled: true } }
dep removed from package.json
verified: 1993 tests green, production build green

PR 3 — `@angular/platform-browser-dynamic` (dev) → spikersoft-angular#450 ✅

Dep removed; peerDependencyRules.ignoreMissing added for Storybook's phantom required peer (verified it never imports the package). Fresh install shows no missing-peer warning.

Follow-up

The "Storybook still builds" acceptance check could not be validated: the static Storybook build is broken on master pre-existing (addon-actions import, node built-ins, worker tsconfig) — split out to #728.

`pnpm install` flags three direct dependencies as deprecated on Angular 22. Worked as three PRs in series — **all merged**: ## PR 1 — `@angular-devkit/build-angular` (dev) → spikersoft-angular#447 ✅ `extract-i18n` target swapped to `@angular/build:extract-i18n`. The package itself stays installed: `@storybook/angular` 10.5 hard-peers on it and imports it at runtime in its webpack preset. Drop the dep only when Storybook does. ## PR 2 — `@angular/animations` → spikersoft-angular#448 ✅ - book-shelf `translateTab` trigger was dead code (never bound in the template) — deleted, no CSS replacement needed - commented-out DSL in classes.component.ts deleted - ~50 specs: `NoopAnimationsModule`/`provideAnimations` etc. removed; profile spec needed the one real replacement: `{ provide: MATERIAL_ANIMATIONS, useValue: { animationsDisabled: true } }` - dep removed from package.json - verified: 1993 tests green, production build green ## PR 3 — `@angular/platform-browser-dynamic` (dev) → spikersoft-angular#450 ✅ Dep removed; `peerDependencyRules.ignoreMissing` added for Storybook's phantom required peer (verified it never imports the package). Fresh install shows no missing-peer warning. ## Follow-up The "Storybook still builds" acceptance check could not be validated: the static Storybook build is broken on master **pre-existing** (addon-actions import, node built-ins, worker tsconfig) — split out to #728.

spikerj closed this issue

2026-07-20 02:24:45 +00:00

spikerj referenced this issue

2026-07-22 02:46:00 +00:00

[Angular][Tech-debt] Migrate off deprecated @angular/animations DSL + animations modules (Angular 22 — SonarQube S1874 ×85) #686

Sign in to join this conversation.