refactor(frontend): unify dateFormat + numberFormat into formatters #50

dmolinari · 2026-04-19T22:26:45Z

dmolinari commented

2026-04-19 22:26:45 +00:00

Summary

Crea src/web/src/lib/formatters.ts como punto unificado de formateo (fechas + números)
Migra los 14 import sites a @/lib/formatters (Opción A — migración inmediata)
Reemplaza dateFormat.test.ts por formatters.test.ts con 10 smoke tests + suite completa
Elimina src/web/src/lib/dateFormat.ts y numberFormat.ts

Opción elegida: A (migración inmediata)

Conteo de imports previo al refactor:

@/lib/dateFormat → 13 archivos
@/lib/numberFormat → 1 archivo
Total: 14 archivos — muy por debajo del umbral de 20, migración directa sin riesgo

No se renombró ningún export. Las firmas de todas las funciones son idénticas.

Test plan

npx vitest run → 464 tests passed, 0 failed (70 test files) — +11 nuevos smoke tests
npx tsc --noEmit → 0 errores de TypeScript (TS5101 es warning preexistente de baseUrl, no relacionado)

Closes #46

## Summary - Crea `src/web/src/lib/formatters.ts` como punto unificado de formateo (fechas + números) - Migra los **14 import sites** a `@/lib/formatters` (Opción A — migración inmediata) - Reemplaza `dateFormat.test.ts` por `formatters.test.ts` con 10 smoke tests + suite completa - Elimina `src/web/src/lib/dateFormat.ts` y `numberFormat.ts` ## Opción elegida: A (migración inmediata) Conteo de imports previo al refactor: - `@/lib/dateFormat` → **13 archivos** - `@/lib/numberFormat` → **1 archivo** - **Total: 14 archivos** — muy por debajo del umbral de 20, migración directa sin riesgo No se renombró ningún export. Las firmas de todas las funciones son idénticas. ## Test plan - `npx vitest run` → **464 tests passed, 0 failed** (70 test files) — +11 nuevos smoke tests - `npx tsc --noEmit` → 0 errores de TypeScript (TS5101 es warning preexistente de `baseUrl`, no relacionado) Closes #46

dmolinari added 1 commit 2026-04-19 22:26:45 +00:00

refactor(frontend): unify dateFormat + numberFormat into formatters (closes #46 ) 7d06ac721b

- Create src/web/src/lib/formatters.ts with all exports from both modules
- Migrate all 14 import sites to @/lib/formatters (Opción A — immediate migration)
- Replace dateFormat.test.ts with formatters.test.ts including 10 smoke tests + full suite
- Delete src/web/src/lib/dateFormat.ts and numberFormat.ts
- 464 tests green, tsc clean (TS5101 warning is pre-existing)

dmolinari merged commit da063ad677 into main

2026-04-19 22:26:48 +00:00

dmolinari deleted branch refactor/unify-formatters

2026-04-19 22:26:48 +00:00

dmolinari referenced this issue from a commit

2026-04-19 22:26:48 +00:00

Merge pull request 'refactor(frontend): unify dateFormat + numberFormat into formatters' (#50) from refactor/unify-formatters into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: dmolinari/SIG-CM2.0#50