Skip to content

feat: add Phase 1A English and zh-CN landing localization - #715

Merged
lilyshen0722 merged 2 commits into
mainfrom
codex/709-zh-cn-phase1
Jul 22, 2026
Merged

feat: add Phase 1A English and zh-CN landing localization#715
lilyshen0722 merged 2 commits into
mainfrom
codex/709-zh-cn-phase1

Conversation

@lilyshen0722

@lilyshen0722 lilyshen0722 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add static react-i18next infrastructure for English and Simplified Chinese with v2.lang persistence, navigator detection, English fallback, and <html lang> synchronization
  • add the EN / 中文 switcher to the landing header and the top of the v2 nav rail
  • migrate all landing-page chrome into one English locale file and one Simplified Chinese locale file; add a scoped ESLint manifest covering the landing and switcher so translated JSX cannot regress to hardcoded copy
  • fix and guard a mobile landing overflow exposed by the CJK browser pass (<pre> min-content widening adapter cards)

Part of #709. This is PR A only; auth and shell/chat chrome intentionally move to PR B after this merges.

Translation review gate

BLOCKED: Sam must complete a native-speaker review of frontend/src/i18n/locales/zh-CN.json before merge.

The Simplified Chinese copy is machine-drafted and is not represented as native-reviewed. All draft translations are centralized in that single file.

Verification

  • npm test -- --watchAll=false --runInBand — 60 suites, 259 tests passed
  • npm run build — passed
  • scoped ESLint over every changed TS/TSX file — passed
  • locale parity check — 176 EN / 176 zh-CN keys, no missing or one-sided keys
  • real Chromium: EN + zh-CN at 1280×900 and 390×844 on landing + nav rail; no horizontal overflow or clipped switch controls
  • npm ci --ignore-scripts --dry-run — passed

Repository-wide lint/typecheck still report pre-existing errors in unchanged files (PersonalityBuilder.tsx, V2GithubPrCard.tsx, and the existing V2MessageBubble _id type mismatch); changed-file lint and the production build are green.

@lilyshen0722 lilyshen0722 added the blocked-on-Sam-review Requires Sam native-speaker review before merge label Jul 22, 2026
@lilyshen0722
lilyshen0722 force-pushed the codex/709-zh-cn-phase1 branch from 7a5bf24 to 151baba Compare July 22, 2026 11:52
lilyshen0722 and others added 2 commits July 22, 2026 10:38
Two-pass review (voice + web-verified terminology) + adjudication per the
new zh-cn-ui-localization skill. Highlights: fork was mistranslated as
创建分支 (=branch); Pod stays English as the product noun; 你 throughout;
agent=智能体, memory=记忆, self-host=自托管, metered compute=云端算力.
Full change table + locked glossary: commonly-skills/zh-cn-ui-localization.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DMeWzgFxsfBcjoLVLewEES
@samxu01
samxu01 force-pushed the codex/709-zh-cn-phase1 branch from a38393b to 0af9c41 Compare July 22, 2026 17:39
@lilyshen0722
lilyshen0722 merged commit 36137f4 into main Jul 22, 2026
10 checks passed
@lilyshen0722
lilyshen0722 deleted the codex/709-zh-cn-phase1 branch July 22, 2026 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked-on-Sam-review Requires Sam native-speaker review before merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant