Skip to content

feat(i18n): add Korean (ko) locale#20630

Closed
RyanRana wants to merge 1 commit into
NousResearch:mainfrom
RyanRana:feat/i18n-korean
Closed

feat(i18n): add Korean (ko) locale#20630
RyanRana wants to merge 1 commit into
NousResearch:mainfrom
RyanRana:feat/i18n-korean

Conversation

@RyanRana

@RyanRana RyanRana commented May 6, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds Korean (ko) as the 9th supported UI locale, following the exact pattern of feat(i18n): add Turkish (tr) locale (#20474) and feat(i18n): add Ukrainian locale (#20467).

Translates the small set of static user-facing messages — CLI approval prompts and a handful of gateway slash-command replies. Agent-generated output, log lines, error tracebacks, and tool outputs continue to stay in English by design.

Changes

  • locales/ko.yaml — 18-key catalog (Korean translations of every key in locales/en.yaml)
  • agent/i18n.py"ko" added to SUPPORTED_LANGUAGES; aliases korean, 한국어, ko-kr, kr added to _LANGUAGE_ALIASES
  • hermes_cli/config.py — supported-language comment updated
  • locales/en.yaml — header note updated to list ko
  • website/docs/user-guide/configuration.md — both docs lines updated
  • tests/agent/test_i18n.py — alias assertions + a t("approval.denied", lang="ko") round-trip assertion

Community signal

Closes the UI-locale half of two long-standing open issues requesting Korean support: #18126 and #18124. The issues focus on docs translation (larger scope); a UI locale catalog is the surgical first step.

Test plan

  • pytest tests/agent/test_i18n.py — all 29 tests pass locally
  • Catalog parity test (test_catalog_keys_match_english) automatically covers ko via parametrize over SUPPORTED_LANGUAGES
  • Placeholder parity test (test_catalog_placeholders_match_english) confirms {description}, {count}, {error} tokens preserved
  • Manual: HERMES_LANGUAGE=ko hermes shows the Korean approval prompt

🤖 Generated with Claude Code

Adds Korean as the 9th supported UI locale for static user-facing
messages (CLI approval prompt, gateway slash-command replies). Mirrors
the Turkish (#20474) and Ukrainian (#20467) PRs.

- locales/ko.yaml: 18-key catalog (Korean translations of en.yaml)
- agent/i18n.py: ko in SUPPORTED_LANGUAGES, aliases korean / 한국어 / ko-kr / kr
- hermes_cli/config.py + locales/en.yaml: supported-language comments
- website/docs/user-guide/configuration.md: docs updated
- tests/agent/test_i18n.py: alias + round-trip assertions

Closes the UI-locale half of #18126 / #18124 (Korean translation requests).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@alt-glitch alt-glitch added type/feature New feature or request P3 Low — cosmetic, nice to have comp/cli CLI entry point, hermes_cli/, setup wizard comp/gateway Gateway runner, session dispatch, delivery labels May 6, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Duplicate of #20297 — same feature (Korean static message locale). Competing PRs for the same i18n addition.

@RyanRana RyanRana closed this by deleting the head repository May 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard comp/gateway Gateway runner, session dispatch, delivery P3 Low — cosmetic, nice to have type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants