Skip to content

fix(i18n): fix Korean translation errors, untranslated entries, and terminology consistency - #10667

Merged
alexhancock merged 2 commits into
aaif-goose:mainfrom
soolmuk:fix/korean-translation
Jul 24, 2026
Merged

fix(i18n): fix Korean translation errors, untranslated entries, and terminology consistency#10667
alexhancock merged 2 commits into
aaif-goose:mainfrom
soolmuk:fix/korean-translation

Conversation

@soolmuk

@soolmuk soolmuk commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixed 120 Korean translation entries in ui/desktop/src/i18n/messages/ko.json covering:

  • 16 untranslated entries → translated to Korean (e.g. "Delete app" → "앱 삭제", "This approval request is no longer active." → "이 승인 요청은 더 이상 활성 상태가 아닙니다.")
  • 10 meaning-mismatched translations aligned to English source (e.g. chatSettings.modeDescription had completely different meaning, externalBackendSection.* lost ACP references)
  • 16 "Goose" → "goose" lowercase per branding guidelines (except sessionViewComponents.role.assistant which keeps "Goose" as a role label)
  • 3 cron picker prepositions differentiated (at→시각, in→월, on→지정; previously all translated to "에")
  • 5 loading messages changed to polite "-요" style (e.g. "goose가 작업 중…" → "goose가 작업 중이에요…")
  • 19 terminology consistency fixes (e.g. 크론→cron, 감지→탐지, 호환 가능→호환, 서브레시피 유지)
  • 17 spacing fixes (변경사항→변경 사항, 일시중지→일시 중지, 괄호 앞 띄어쓰기)
  • 4 placeholder values kept in English (model-a, command-name, daily-summary-job — these are input format examples, not translatable text)
  • 18 expression/grammar fixes (e.g. "사용자화합니다"→"맞춤 설정합니다", "좋아요"→"확인" for OK button, "초점"→"포커스" for focus terminology)

Testing

  • pnpm i18n:validate-locale ko — passed (1536 messages, structure + ICU placeholder validation)
  • pnpm typecheck — passed

…erminology consistency

## Summary

Fixed 120 Korean translation entries in ui/desktop/src/i18n/messages/ko.json covering:

- 16 untranslated entries → translated to Korean
- 10 meaning-mismatched translations aligned to English source
- 16 Goose → goose lowercase per branding guidelines (except role label)
- 3 cron picker prepositions differentiated (at→시각, in→월, on→지정)
- 5 loading messages changed to polite -요 style
- 19 terminology consistency fixes (크론→cron, 감지→탐지, 호환 가능→호환, 서브레시피 유지)
- 17 spacing fixes (변경사항→변경 사항, 일시중지→일시 중지, 괄호 앞 띄어쓰기)
- 4 placeholder values kept in English (input format examples)
- 18 expression/grammar fixes (사용자화→맞춤 설정, 좋아요→확인, 초점→포커스)

### Testing
- pnpm i18n:validate-locale ko — passed (1536 messages)
- pnpm typecheck — passed
@soolmuk
soolmuk force-pushed the fix/korean-translation branch from 8f967a0 to efddef4 Compare July 24, 2026 13:23

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8f967a035b

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread ui/desktop/src/i18n/messages/ko.json Outdated
},
"cronPicker.inMonth": {
"defaultMessage": ""
"defaultMessage": ""

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Keep the yearly month label as a preposition

For Korean users configuring a yearly schedule, this string is rendered immediately before the month dropdown in CronPicker.tsx; the options are already translated as 1월, 2월, etc., so the UI now reads like 월 1월 instead of conveying “in January.” This makes the schedule sentence harder to interpret and can lead users to misread which part of the cron expression they are editing; use a phrase/particle that works before the selector rather than the noun .

Useful? React with 👍 / 👎.

@soolmuk soolmuk Jul 24, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch. We decided to keep the cron picker labels and options in English since cron expressions are inherently English-based. This avoids the "월 1월" duplication issue and maintains consistency across all period modes.

@alexhancock
alexhancock merged commit 192b5db into aaif-goose:main Jul 24, 2026
24 checks passed
alexhancock added a commit that referenced this pull request Jul 27, 2026
* origin/main: (24 commits)
  upgrade to rmcp 2.0 (#10584)
  chore: opus5 support (including adaptive thinking) (#10686)
  fix(desktop): use shell working directory on initial launch (#10655)
  fix(providers): enable inventory refresh for OpenRouter model picker (#10641)
  Add configurable GOOSE_DOCS_ROOT for air-gapped docs access (#10294)
  fix(i18n): fix Korean translation errors, untranslated entries, and terminology consistency (#10667)
  fix(acp): preserve read lines and reload tool output (#10662)
  fix(acp): surface tool call output in CLI and Desktop (#10654)
  fix overlong function names in provider requests (#10659)
  fix (desktop): preserve ACP tool call update fields (#10653)
  feat: support latest Gemini models (#10630)
  Merge commit from fork
  feat (acp): Gate tool-call label enrichment on ACP client capability (#10644)
  fix(session): use configured session manager for tool summaries (#10628)
  refactor (acp): simplify ACP tool-call handling and fix chain summaries (#10599)
  fix: omit disabled OpenRouter reasoning (#10517)
  enhance the uniffi API layer (#10427)
  remove sampling from smoke test (#10639)
  rebuild canonical model db (#10633)
  fix(ui): clear stale pending ACP connection after terminal recovery failure (#10552)
  ...

# Conflicts:
#	ui/desktop/src/i18n/messages/ko.json
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