Objective
Separate canonical values from localized presentation and enforce explicit locale/timezone rules for all financial flows.
Why this matters
Locale-dependent parsing or formatting can display the wrong amount/date or send a value different from what the user reviewed.
This is a substantive production-quality improvement. It must change runtime behavior, security guarantees, correctness, reliability, or meaningful user functionality. It is not a documentation-only, formatting-only, or trivial dependency task.
Scope
Area: internationalization and financial display
Starting points: lib/i18n/formatters.ts, lib/utils/format-currency.ts, lib/i18n/resolve-locale.ts, lib/remittance
The contributor should verify the current implementation before changing it and keep the PR limited to this issue. Do not introduce unrelated refactors or weaken existing CI/security gates.
Acceptance criteria
Required validation
Contributor deliverables
- Open a focused feature branch and do not begin implementation until assigned.
- Reference this issue with
Closes #<issue-number> or Fixes #<issue-number>.
- Check off every acceptance criterion in the PR with links to the relevant code and tests.
- Include a security/correctness note explaining why adversarial inputs cannot bypass the new guarantee.
Maintainer quality bar
The PR must be independently reviewable, preserve existing behavior outside this scope, exercise failure paths, and pass the repository's complete required CI/CD checks. Reward eligibility is not guaranteed by this issue or by merging.
Objective
Separate canonical values from localized presentation and enforce explicit locale/timezone rules for all financial flows.
Why this matters
Locale-dependent parsing or formatting can display the wrong amount/date or send a value different from what the user reviewed.
This is a substantive production-quality improvement. It must change runtime behavior, security guarantees, correctness, reliability, or meaningful user functionality. It is not a documentation-only, formatting-only, or trivial dependency task.
Scope
Area: internationalization and financial display
Starting points: lib/i18n/formatters.ts, lib/utils/format-currency.ts, lib/i18n/resolve-locale.ts, lib/remittance
The contributor should verify the current implementation before changing it and keep the PR limited to this issue. Do not introduce unrelated refactors or weaken existing CI/security gates.
Acceptance criteria
Required validation
Test multiple locales, currencies, negative/zero/large amounts, and DST boundaries.
Test round-trip display/edit/submit behavior.
Add E2E assertions for localized confirmation values.
The PR explains the failure mode, the chosen design, backward-compatibility impact, and rollback or migration considerations.
The PR includes CI evidence and does not contain secrets, generated noise, unrelated cleanup, or disabled checks.
Contributor deliverables
Closes #<issue-number>orFixes #<issue-number>.Maintainer quality bar
The PR must be independently reviewable, preserve existing behavior outside this scope, exercise failure paths, and pass the repository's complete required CI/CD checks. Reward eligibility is not guaranteed by this issue or by merging.