Skip to content

feat(i18n-ko): localize Desktop slice 2 catalogs - #4710

Open
testikun wants to merge 4 commits into
apache:mainfrom
testikun:codex/issue-3976-ko
Open

feat(i18n-ko): localize Desktop slice 2 catalogs#4710
testikun wants to merge 4 commits into
apache:mainfrom
testikun:codex/issue-3976-ko

Conversation

@testikun

@testikun testikun commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add Korean (ko) catalog entries for the remaining Desktop domains in issue feat(i18n-ko): desktop slice 2 — remaining settings/conversation/artifact/browser/mcp/plan-mode #3976: conversation, artifacts, browser, external-session import, MCP, Plan Mode, and settings.
  • Extend UiLocale Korean variants and localize dynamic Desktop labels/messages, including health capabilities, conversation inspector metadata, permissions/retries, and provider context-window copy.
  • Refresh the Korean Runtime Host quit-dialog catalog for the current main-branch contract.
  • Add Korean catalog coverage plus a real Electron settings-navigation regression test.
  • Keep screenshot evidence in this first PR comment instead of committing image binaries to the source diff.

Pre-submit review

  • Standards review completed: no blocking standards, header, commit, lint, format, or architecture findings.
  • Spec review completed: all issue-listed Desktop catalogs have matching en/ko keys and dynamic copy is covered.
  • The remaining English fallback in shared packages/ui catalogs is outside this Desktop slice and belongs to the shared UI follow-up (feat(i18n-ko): shared UI — packages/ui and core error copy #3978).

Validation

  • npm --workspace @maka/desktop run build:with-deps
  • npm --workspace @maka/desktop run typecheck
  • npm --workspace @maka/desktop run check:architecture
  • Biome check for all 58 changed TypeScript/TSX/MJS files
  • Korean catalog tests (3/3)
  • git diff --check
  • Built and launched the local Electron app in a visible window from pre-rebase feature tree df4918571; current head 7c133a4b7 is the rebased equivalent on apache/main 19ac204ab, opened Korean Settings, verified the localized Data page, navigated to Sub-agents, and captured the resulting real window. These are direct App captures, not mockups or generated images.

The repository Playwright Electron launcher currently stalls during its macOS inspector/CDP handshake before the first window is created; an existing #4701 test reproduces the same host-level failure. The direct visible real-window/CDP verification above completed successfully, and CI will exercise the committed Playwright regression test on Linux.

Real App screenshots

Korean Data settings

Korean Data settings

Korean Sub-agents settings

Korean Sub-agents settings

Fixes #3976

AI use

  • Used OpenAI Codex for implementation and review.

@github-actions github-actions Bot added the effort/XXL Over 2500 readable lines label Sep 4, 2026
@testikun
testikun force-pushed the codex/issue-3976-ko branch 2 times, most recently from 9a15a0d to df49185 Compare September 4, 2026 07:44
testikun added 4 commits September 4, 2026 16:45
Refresh the Korean quit dialog catalog for the current main branch, add a localized settings regression test, and keep real UI evidence out of the source diff so it can live in the PR description.\n\nGenerated-by: OpenAI Codex
Carry the Korean provider and usage copy across the latest model-settings and usage-trace contracts, with regression assertions for the newly added fields.\n\nGenerated-by: OpenAI Codex
@testikun
testikun force-pushed the codex/issue-3976-ko branch from 17d71c7 to 7c133a4 Compare September 4, 2026 08:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

effort/XXL Over 2500 readable lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(i18n-ko): desktop slice 2 — remaining settings/conversation/artifact/browser/mcp/plan-mode

1 participant