Skip to content

fix(i18n): complete locale coverage#404

Merged
Harry19081 merged 1 commit into
perf/visibility-aware-polling-auditfrom
feat/i18n-locale-completeness
Jul 17, 2026
Merged

fix(i18n): complete locale coverage#404
Harry19081 merged 1 commit into
perf/visibility-aware-polling-auditfrom
feat/i18n-locale-completeness

Conversation

@beruro

@beruro beruro commented Jul 17, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • localize AppUpdater check, download, install, timeout, and retry messaging
  • complete existing locale gaps across all supported languages and preserve interpolation placeholders
  • add a BOM-aware i18n completeness check for missing keys, placeholder mismatches, duplicate keys, and invalid locale files
  • expose the locale audit as pnpm check:i18n

Verification

  • pnpm check:i18n
    • Total missing: 0
    • Placeholder mismatches: 0
    • Duplicate keys: 0
    • Invalid files: 0
  • pnpm exec vitest run src/scaffold/AppUpdater/index.test.ts
  • pnpm exec eslint src/scaffold/AppUpdater/index.tsx src/scaffold/AppUpdater/index.test.ts
  • git diff --check

Stack

This PR is stacked on #400 because AppUpdater uses the visibility-aware polling hook introduced there. Only the new i18n/AppUpdater commit is included in this PR's diff.

Pre-commit hook ran. Total eslint: 1, total circular: 0
@Harry19081
Harry19081 merged commit 83c8830 into perf/visibility-aware-polling-audit Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants