From 2073a533935426cd489ce1ce3df569004859cdb2 Mon Sep 17 00:00:00 2001 From: JinHyuk Sung <163989462+sjh9714@users.noreply.github.com> Date: Thu, 21 May 2026 09:03:14 +0900 Subject: [PATCH] docs: localize README context diagram alt text MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Summary - Localized the context diagram alt text in the Japanese README. - Localized the context diagram alt text in the Korean README. - Kept the English README unchanged because its current alt text is already appropriate. ## Problem #2162 requested localized equivalents for README context-diagram alt text. The original missing `bash` fences and basic image `alt` attributes are already present on `main`, but the Japanese and Korean READMEs still used the English `OpenHuman context diagram` alt text. ## Solution - Replaced the remaining English context-diagram `alt` strings in `README.ja-JP.md` and `README.ko.md` with localized text. - Kept the change docs-only and limited to the two localized README files. ## Submission Checklist > If a section does not apply to this change, mark the item as `N/A` with a one-line reason. Do not delete items. - [x] N/A: Tests added or updated — docs-only image alt text change, no executable behavior. - [x] N/A: Diff coverage ≥ 80% — docs-only README change, no coverage-producing code paths. - [x] N/A: Coverage matrix updated — no feature row added, removed, or renamed. - [x] N/A: All affected feature IDs listed — no affected feature IDs for README alt text. - [x] No new external network dependencies introduced. - [x] N/A: Manual smoke checklist updated — not a release-cut surface. - [x] N/A: Linked issue closed via closing keyword — this is a small follow-up for the localized alt text portion only. ## Impact Docs/accessibility metadata only. No runtime, platform, performance, security, migration, or compatibility impact. ## Related - Closes: N/A - Follow-up PR(s)/TODOs: Refs #2162 --- ## AI Authored PR Metadata (required for Codex/Linear PRs) > Keep this section for AI-authored PRs. For human-only PRs, mark each field `N/A`. ### Linear Issue - Key: N/A - URL: N/A ### Commit & Branch - Branch: docs-localize-readme-alt-text - Commit SHA: bd58f7af ### Validation Run - [x] N/A: `pnpm --filter openhuman-app format:check` blocked at Rust formatting step; see Validation Blocked. - [x] N/A: `pnpm typecheck` — docs-only README change. - [x] Focused tests: `PATH="/Users/sungjh/.cache/codex-runtimes/codex-primary-runtime/dependencies/node/bin:$PATH" pnpm --filter openhuman-app exec prettier --check .` - [x] Focused tests: `git diff --check` - [x] N/A: Rust fmt/check — no Rust files changed. - [x] N/A: Tauri fmt/check — no Tauri files changed. ### Validation Blocked - `command:` `pnpm format:check` - `error:` app Prettier reported `All matched files use Prettier code style!`, then `pnpm rust:format:check` failed with `sh: cargo: command not found` because this local environment does not have Rust/Cargo on PATH. - `impact:` Rust/Tauri formatting was not run locally; this PR changes only README markdown image alt text. ### Behavior Changes - Intended behavior change: Japanese and Korean README context-diagram alt text now matches each README locale. - User-visible effect: screen readers and markdown consumers get localized image descriptions in those README files. ### Parity Contract - Legacy behavior preserved: image source, layout, and English README text are unchanged. - Guard/fallback/dispatch parity checks: N/A for docs-only change. ### Duplicate / Superseded PR Handling - Duplicate PR(s): none found for #2162 / README alt cleanup. - Canonical PR: this PR. - Resolution (closed/superseded/updated): N/A. Co-authored-by: jinhyuk9714 Co-authored-by: Steven Enamakel <31011319+senamakel@users.noreply.github.com> Co-authored-by: Steven Enamakel