fix(docs): correct translation fidelity issues in Vietnamese docs#2192
fix(docs): correct translation fidelity issues in Vietnamese docs#2192
Conversation
Sync Vietnamese translations with current English source: - Update agent table to consolidated Developer agent architecture - Fix bmad-dev -> bmad-agent-dev skill ID references - Replace Quinn/QA agent framing with built-in QA workflow - Fix SM agent -> Developer agent in getting-started - Fix broken platform-codes.yaml URL - Add missing Validation Commands section to style guide - Fix malformed table row in style guide - Remove unsourced content additions in project-context - Fix roadmap section heading and index link format - Fix accountability softening in party-mode dialogue
🤖 Augment PR SummarySummary: This PR expands and corrects the Vietnamese (vi-vn) docs to better match the current BMad Method/BMM v6 behavior. Changes:
Technical Notes: Most changes are documentation-only; correctness hinges on links, skill IDs, and menu triggers matching the actual installed skills. 🤖 Was this summary useful? React with 👍 or 👎 |
| order: 5 | ||
| --- | ||
|
|
||
| BMad cung cấp hai hướng kiểm thử: workflow QA tích hợp sẵn để tạo test nhanh và module Test Architect có thể cài thêm cho chiến lược kiểm thử c��p doanh nghiệp. |
There was a problem hiding this comment.
docs/vi-vn/reference/testing.md:8 contains mojibake (e.g., c��p, tích h��p, S���n), which will render incorrectly in the published docs and likely indicates an encoding/copy issue. It’d be good to ensure these Vietnamese strings are valid UTF-8 and display correctly everywhere they appear.
Severity: medium
Other Locations
docs/vi-vn/reference/testing.md:21docs/vi-vn/reference/testing.md:57
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
|
|
||
| | Agent | Skill ID | Trigger | Workflow chính | | ||
| | --------------------------- | -------------------- | ---------------------------------- | --------------------------------------------------------------------------------------------------- | | ||
| | Analyst (Mary) | `bmad-analyst` | `BP`, `RS`, `CB`, `WB`, `DP` | Brainstorm Project, Research, Create Brief, PRFAQ Challenge, Document Project | |
There was a problem hiding this comment.
docs/vi-vn/reference/agents.md:20 lists several agent skill IDs / triggers (e.g., bmad-analyst, bmad-pm, RS, bmad-tech-writer, US) that don’t match the actual installed agent skill names/menu codes (which are bmad-agent-* in this repo). As written, readers following this page will try invoking skills/triggers that don’t exist.
Severity: medium
Other Locations
docs/vi-vn/reference/agents.md:14docs/vi-vn/reference/agents.md:21docs/vi-vn/reference/agents.md:22docs/vi-vn/reference/agents.md:24docs/vi-vn/reference/agents.md:25
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
| | Ví dụ skill | Agent | Vai trò | | ||
| | --- | --- | --- | | ||
| | `bmad-agent-dev` | Amelia (Developer) | Triển khai story với mức tuân thủ đặc tả nghiêm ngặt | | ||
| | `bmad-pm` | John (Product Manager) | Tạo và kiểm tra PRD | |
There was a problem hiding this comment.
docs/vi-vn/reference/commands.md:83 uses bmad-pm / bmad-architect as agent skill examples, but the actual agent skills in this repo are bmad-agent-pm / bmad-agent-architect (and similarly for other roles). This mismatch can make the Vietnamese docs feel “broken” because the example invocations won’t resolve in an IDE.
Severity: medium
Other Locations
docs/vi-vn/reference/commands.md:84
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
843e7a2 to
11a362a
Compare
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughThis PR adds comprehensive Vietnamese localization for the BMad Method documentation suite, including a new README, documentation style guide, and 25+ reference pages covering installation, workflows, tutorials, explanations, and reference material across all major documentation categories. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
⚔️ Resolve merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Fixes translation fidelity issues found during editorial review of #2110.
bmad-agent-dev) — affects agents.md, commands.md, testing.md, getting-started.mdTest plan
npm run qualitypasses (format, lint, markdown lint, docs build, install tests, ref validation, skill validation)Context
Full translation fidelity review report was produced comparing all 30 translated files against English sources. Zero injections found; all fixes address drift, omissions, and link issues.