# NousResearch/hermes-agent · Locale-sensitive number formatting breaks tests

## Open now

- Nothing open on GitHub.

These items address vitest/desktop test failures caused by locale-dependent number and currency formatting when the system locale is not en-US. The work involves either pinning 'en-US' explicitly in formatting calls or adjusting test assertions to tolerate host locales.

## TL;DR

- 3 member(s): 0 open issue(s), 1 closed issue(s), 0 open PR(s), 1 merged PR(s), 1 PR(s) closed without merging.
- A fix was merged to main in PR #120267. Verify it addresses your case.
- **Recommended:** read PR #120267 (and any competing PRs below) before filing. Confirm the merged change covers your symptom.

## Maintainer resolution verdict

- Internal-tracker verdict over this cluster's issue members: `resolved` · deciding PR(s): `pr-120267`. This is distinct from the fix status above. Weigh it yourself.

## How this cluster was grouped

- Link evidence (`STRUCTURAL_SPANNING`): every member is connected by structural evidence: shared changed files, or a closing/superseded reference.
- Pairwise link density `0.67`: the share of member pairs that carry a link, counting a structural link in full and a relevance-only link at 0.36. `1.00` means every pair is linked.

## Closed / duplicate issues

- #98849: vitest suite fails under non-en-US system locale (bare \`.toLocaleString()\` / \`Intl.NumberFormat(undefined, ...)\`)

## Prior & competing PRs

- **#120267: test(desktop): respect host locales in UI assertions**. merged to main · fix tier `verified`
  - maintainer direction: category `merged`
  - files: `apps/desktop/src/app/session/hooks/use-prompt-actions/utils.test.ts`, `apps/desktop/src/app/settings/billing/index.test.tsx`, `apps/desktop/src/components/assistant-ui/thread/timestamp.test.ts`, `apps/desktop/src/components/assistant-ui/tool/fallback-model.test.ts`
- **#98850: fix(desktop): pin 'en-US' in number/currency formatting so tests pass under non-en-US locales**. closed without merge
  - maintainer direction: category `superseded_by_pr`; implication `superseded`

## Fix status

- Merged to main in PR #120267 (cited above). No release data is tracked. Verify downstream.
- Fix last verified (merged) `2026-09-23`.