sam-ui-elements pipeline modernization + Angular 19→21 upgrade (SAM Angular upgrade — 4th tracer bullet) — IAEMOD-61187
Los mantenedores suelen responder en 1 día
@fpigeonjr ya está trabajando en esto.
Desde el 14/8/2026.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
Pipeline modernization and Angular version migration for @gsa-sam/sam-ui-elements — the fourth tracer bullet of the SAM design-library Angular upgrade epic (IAEMOD-57412), following sam-styles (#732 on GSA/sam-styles), ngx-uswds-icons (#26 on GSA/ngx-uswds-icons), and ngx-uswds (#184 on GSA/ngx-uswds). Coordination/tracking lives on the internal research repo (mcaas-iae/iae-angular-upgrade).
sam-ui-elements sits at Angular 19 (published @gsa-sam/[email protected]) and consumes only sam-styles (effectively done), so it is unblocked. It is the only design library that is not a compiled Angular library: it publishes raw TypeScript source (no main/module/exports; ~560 .ts), and the consumer compiles it in-place. Its formly is bundled and load-bearing in prod. See binding decision ADR-0009 on the coordination repo.
What makes this repo different
- Raw-source publish, not ng-packagr. No library
ng build; the consumer's own Angular compiler builds our source. Deep import paths (/src/formly/**,/src/ui-kit/**) are a frozen contract — no file moves this epic. - Branch of record is
ng-19, notmaster.masterlags at Angular 18.0.2;ng-19is a strict content superset (the lone "master-only" commit is an empty merge bubble, 0 unique files). Reconcile by mergingng-19→master(no force). - No GitHub Actions — only a commented-out CircleCI stub on Node 10.
- Test harness is rotten. Tests live in a separate
test-app/that hard-links../srcin, but the harness is Angular 9 / TS 2.3 / ngx-formly 5 against a library at Angular 19 — the 107 specs have not been compiling against a matching Angular. Green-at-19 is the gate before any migration. - formly kept + migrated, not deprecated.
AbstractSamFormlyis imported in 15 files across 3 consumers, incl. prod — unlikeuswds-formly(separate package, 0 consumers, cleanly removed). So ngx-formly 6→7 is in scope, gating Angular 20. - rxjs already on 7 (peer
>=7.5.0) — no rxjs migration; ~6 rxjs-6-only imports survive in specs only, fixed during the green baseline.
Goal
Two bundled tracks, sequenced pipeline-first so the Angular migration lands on green CI:
- Pipeline/tooling — inherit the pattern proven on sam-styles, icons, and ngx-uswds: registry-pinned
.npmrc, normalized Node, ESLint (off tslint) with the a11y gate, Prettier, test + coverage, Dependabot, a PR template, dead-code pruning, Playwright smoke (replacing Protractor), and a tag-gated livenpm publish. - Angular migration — stepped per-major chain 19 → 20 → 21 (LTS), one major at a time, with the ngx-formly 6→7 migration as its own sub-issue gating the 20 step.
Parallelism model
Serialize anything that mutates package.json / angular.json / tsconfig; parallelize disjoint file sets. Prettier and dead-code prune are "stop the world" (run solo). The Angular chain is a serial train — one major at a time, nothing else editing manifests/source in flight.
Deferred (own follow-up issues, NOT this epic)
- Convert to a compiled ng-packagr library. Prereq: break the
src/ui-kit/filters → src/formly(FormlySAMUIModule) dependency cycle. - Replace the
test-appcopy-src-in harness with a standard in-repo Karma/Jest setup; retiretest-app.
Notes / scope boundaries
- Migration is unblocked — sam-ui-elements consumes only sam-styles. Only the live publish sub-issue carries the DevOps dependency (OIDC Trusted Publisher registration, ADR-0007).
- Pin, don't break, the Angular-17 legacy consumer (
iae-sam-front-end3, IBM/legacy SAM). Major-locked peers make the version pin the natural outcome. - Consumer validation is mandatory before publish:
npm pack --dry-run+ audit every frozen deep path against the three consumers. - Jira epic (this repo): IAEMOD-61187
- Program epic (SAM design-library Angular upgrade): IAEMOD-57412
- Lenguaje dominante
- TypeScript
- Estrellas
- 8
- Forks
- 33
- Merge medio
- 2 d 15 h
- PR fusionados (30 d)
- 11
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de GSA/sam-ui-elements
-
github_actions security
Dificultad 2/5 Medio día Aptitud para principiantes 88/100
GSA/sam-ui-elements#761 ·
Los mantenedores suelen responder en 1 día
-
tech-debt
Dificultad 4/5 3-5 días Aptitud para principiantes 56/100
GSA/sam-ui-elements#708 ·
Los mantenedores suelen responder en 1 día
-
disablePageTabIndex() sets aria-hidden on the focused calendar button, and enablePageTabIndex() never removes itQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abiertobugfix
Dificultad 4/5 3-5 días Aptitud para principiantes 68/100
GSA/sam-ui-elements#682 ·
Los mantenedores suelen responder en 1 día
-
bugfix
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
GSA/sam-ui-elements#681 ·
Los mantenedores suelen responder en 1 día
-
Burn down ESLint and Angular lint debtQuizá libre de nuevo @fpigeonjr la tomó hace 51 días y no hay ningún pull request abierto. Abiertoangular-upgrade maintenance
GSA/sam-ui-elements#580 · 1 asignado ·
Los mantenedores suelen responder en 1 día
Todos los issues de GSA/sam-ui-elements
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 83/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
ranxianglei/billion-context#3583 ·
Los mantenedores suelen responder en 1 día
-
bug:new
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
callstackincubator/simlock#673 ·
Los mantenedores suelen responder en 1 día
-
documentation v3
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
anomalyco/opencode#56380 · 1 comentario ·
Los mantenedores suelen responder en 1 día