[jp-sync] Replay failed Angular + xplat JP syncs for EN PRs #522, #438, #535, #491 - #556
[jp-sync] Replay failed Angular + xplat JP syncs for EN PRs #522, #438, #535, #491#556ChronosSF wants to merge 6 commits into
Conversation
Manual replay of the four failed "Sync Japanese Documentation (Angular)" runs (33877005832, 34238718782, 34250163709, 34329511530). Each died in the Copilot CLI step with transient_bad_request before doing any work; the workflows have since been recompiled with gh-aw v0.88.7 in #555. - chat: translate the `messages` row and the new bidirectional messages collection section (#438) - theming-mcp: add the WCAG contrast note linking to the new Accessibility topic (#522) - palettes: hsla() contrast variables in the CSS excerpt (#522) - toc: add Themes > Accessibility; Button now points at inputs/button.mdx (#522, #491) - retarget IgxButton links in dialog, time-picker, tooltip, roundness, material/bootstrap theming and the row-editing/search grid templates to the xplat-generated inputs/button page (#491, #535) - remove the hand-authored button.mdx and layouts/avatar.mdx, superseded by the xplat-generated pages exactly as EN did (#491, #522) - xplat jp toc: lift the Angular exclude on Button, Icon, Icon Button and Avatar and add Themes > Accessibility, mirroring the EN xplat toc so the Angular JP site gains the generated pages the Angular toc now references - add the jp xplat themes/accessibility.mdx translation (target of the new toc entry and of the theming-mcp note) - mirror the new button anatomy / do-don't images into jp/images, which the Angular build resolves per language via @xplat-images Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Manual replay of the four failed "Sync Japanese Documentation (xplat)" runs (33877005838, 34238718784, 34250163990, 34329511576), which died in the Copilot CLI step exactly like their Angular counterparts. Change set is the EN diff under docs/xplat/src/content/en/ since the last successful xplat sync (d346871), filtered to TOC-covered files per the workflow. - inputs/button: re-mirror the rewritten EN topic (anatomy, getting started, usage/variants/states/layout/size/download/do-don't, properties, styling incl. Tailwind, accessibility, troubleshooting, dependencies, related, FAQ), reusing existing JP prose where EN kept it. The Getting Started link is split per platform like the JP badge and button-group pages, because the JP xplat toc still excludes general-getting-started.mdx for Angular (EN does not) - inputs/button-group: Angular-specific PlatformBlocks throughout, {PackageCommon} install block, per-platform properties tables, API references and dependencies, new Related/FAQ content - inputs/badge: {PackageCommon} install block, Angular import sentence, CSS variables table, troubleshooting/FAQ updates - interactivity/accessibility-compliance: replace the Section 508 / WCAG matrices with the new standards, criteria, per-component documentation index (JP heading anchors), configuration, VPAT and FAQ sections - interactivity/chat: Blazor MessageCreated guidance, per-platform property names, bidirectional messages collection section - themes/overview: configureTheme variant argument, new Customizing with Sass section; themes/accessibility now anchors to it - themes/palettes: runtime relative-color contrast, seed variables, --ig-wcag-* excerpt, contrast-level note - ai/theming-mcp: WCAG contrast note Not mirrored: the EN toc href fix for grids/grid/theming-grid.mdx (#535) only corrected EN after a July file move (#355) that was never applied to JP; JP still keeps the page at grids/theming-grid.mdx consistently. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
|
@jsakamotoIGJP , this run does not exactly replay the GH AW workflow but rather uses the same rules but passed on Fable 5.1 Max . I know you guys have issues with the quality of the default GH AW run and I am wondering if Fable 5.1 produced better results translating this. The model quality variable in the output result will also be important when choosing the models for the next translation process. Let me know what you think about the work Fable 5.1 did with these. |
|
@ChronosSF I've already started reviewing this PR, but I'd appreciate until tomorrow to share my findings. |
…tion in troubleshooting intros and known-limitations lead-in)
…tion in the Before You Start lead-in)
…rousel and tooltip links)
|
@ChronosSF Thank you for your patience. In summaryI think the quality is significantly better than before, but I'm not sure Fable is required to achieve that, since our AI review was running on cheaper models. DetailsI do not have any measured numbers, so this is only my personal impression. Compared to the past JP syncs, the translation quality is much better. The improvement is not only in the translation itself. I didn't find any untranslated English paragraphs, and I didn't find structural problems such as missing or misplaced PlatformBlocks. This is a very good improvement. That said, a few fixes were still needed. To be transparent, our "human review" is now heavily assisted by AI agents. For this review, I used Claude Code with Opus 5 (High effort) and a custom prompt. I did not use Fable for the review side. Claude Code's fixes fell into three main areas. First, one translation issue. The word "boundaries" was translated correctly at the word level, but the result sounded unnatural in context. A more natural interpretation was needed. This was the only translation correction I made, and it was a difficult case. Second, consistency issues. Some terms and expressions were different from existing Japanese documentation. My custom prompt checks existing .mdx files and treats them as translation examples. This helps keep wording consistent across pages, even without a formal glossary. Third, anchor links. When headings are translated, anchor names can change, and links can break. The review process detected such issues, and they were fixed. Overall, the amount of translation correction was very small compared to previous reviews. In fact, there was only one real translation fix. Because of that, I am not sure how much of the result comes from the model itself and how much comes from the review prompt and workflow. It may be possible to get similar results with Opus 5, Sonnet 5, or even less expensive models if the prompt is well designed. One more thought. The remaining translation issue was a difficult wording choice. This suggests that writing English documentation in a simpler, more consistent style may also improve translation quality across all languages. |
Original authors: Marin Popov (desig9stein@users.noreply.github.com) — #522 · Stamen Stoychev (chronos.stz@gmail.com) — #438 · Dobromir Tsvetkov (dobromirts@users.noreply.github.com) — #535 · Hristo Hristov (Hristo313@users.noreply.github.com) — #491
Summary
Manual replay of the eight failed JP sync runs for the four
vnextpushes #522, #438, #535 and #491 — both the Angular workflow (33877005832, 34238718782, 34250163709, 34329511530) and the xplat workflow (33877005838, 34238718784, 34250163990, 34329511576). Every run died in the Execute GitHub Copilot CLI step withtransient_bad_requeston all retries before doing any work, so nothing partial exists; the workflows have since been recompiled with gh-aw v0.88.7 in #555.Each change set is the union of EN changes since the respective last successful sync (
34497daa95for Angular,d34687170dfor xplat), filtered to TOC-covered files exactly as the workflow specs prescribe. Two commits: Angular first, xplat second.Angular JP (
docs/angular/src/content/jp/)components/chat.mdxjp/components/chat.mdxmessagesrow + new "双方向のメッセージ コレクション" section (#438)components/ai/theming-mcp.mdxjp/components/ai/theming-mcp.mdxDocsAsidebefore the tools table (#522)components/themes/palettes.mdxjp/components/themes/palettes.mdxhsl()→hsla(… / 1)contrast variables in the CSS excerpt (#522)components/toc.jsonjp/components/toc.jsonnew: true); Button →inputs/button.mdx(#522, #491)components/dialog.mdx,time-picker.mdx,tooltip.mdxjp/IgxButtonlink →./inputs/button.mdx(#491)components/themes/misc/angular-material-theming.mdx,bootstrap-theming.mdxjp/../../inputs/button.mdx(#491)components/themes/roundness.mdxjp/components/themes/roundness.mdx../inputs/button.mdx(#491, #535)grids_templates/row-editing.mdx,search.mdxjp/inputs/button/inputs/button#スタイル設定(#491)components/list.mdxjp/components/list.mdx./inputs/button-group.mdx(#535)components/button.mdx(deleted in EN)jp/components/button.mdxinputs/button.mdx(#491)components/layouts/avatar.mdx(deleted in EN)jp/components/layouts/avatar.mdximages/button/*.png,images/anatomy-content-light/button-lt-a.pngjp/images/@xplat-imagesper language, so the generated JP button page needs themxplat JP (
docs/xplat/src/content/jp/)components/inputs/button.mdxcomponents/inputs/button-group.mdxPlatformBlocks,{PackageCommon}install block, per-platform Properties/API References/Dependencies, new Related Components and FAQ (#491)components/inputs/badge.mdx{PackageCommon}install block, Angular import sentence, CSS variables table, troubleshooting intro, FAQ (#491)components/interactivity/accessibility-compliance.mdx#キーボード-ナビゲーション,#アクセシビリティ,#アクセシビリティと-aria-サポート), and link to the page only where the JP target lacks the section yet (Angular carousel/tooltip, xplat carousel) (#522, #491)components/interactivity/chat.mdxMessageCreatedguidance, per-platform property names in the table, new bidirectional messages section with per-platform snippets (#438)components/themes/overview.mdxconfigureThemevariant argument, new Sass によるカスタマイズ section (#522)components/themes/palettes.mdxseed/--ig-wcag-*variables, seed and contrast-level paragraphs (#522)components/themes/accessibility.mdxcomponents/ai/theming-mcp.mdxtoc.json"exclude": ["Angular"]on ボタン, アイコン, アイコン ボタン (EN #491) and アバター (EN had already lifted it; the stale JP exclude is why the hand-authored JP avatar page was still being maintained),"updated": truewhere EN has it, new テーマ › アクセシビリティ entry (#522)Not mirrored on purpose: the EN toc href fix
grids/theming-grid.mdx→grids/grid/theming-grid.mdx(#535) only corrected EN after a July file move (#355) that was never applied to JP. JP still keeps the page atgrids/theming-grid.mdxwith ~22 consistent links, so the relocation is a separate follow-up rather than part of this replay.Verification
npm run check:llms-metadata— passed (1096 frontmatter blocks, 964 EN+JP topics)markdownlintandcspellon all 21 changed.mdxfiles — 0 issuescheck-relative-linksafter the full CI-equivalent generation (Angular EN/JP sync + grid templates, React/WC/Blazor EN+JP): Angular 0 broken, xplat 0 brokentoc.jsonexclude comparison: only one pre-existing, unrelated drift remains (general-cli-overview.mdx)Checklist:
preview/beta./page.mdxor../relative/path.mdx(.mdxextension required)IgxSelectComponent,<igx-combo>code blocksfor the names of classes / tags / propertiescode blockscheck-api/mdxnpm commandsllms.descriptionmetadata and runnpm run check:llms-metadata🤖 Generated with Claude Code