Skip to content

fix(i18n): pt-BR review pass over the retranslated leaves (172 corrections) + review-locale script - #13885

Merged
diegosouzapw merged 4 commits into
release/v3.8.51from
fix/i18n-pt-br-review
Sep 16, 2026
Merged

diegosouzapw merged 4 commits into
release/v3.8.51from
fix/i18n-pt-br-review

Conversation

@diegosouzapw

Copy link
Copy Markdown
Owner

Summary

Task 8 of the i18n completion plan. PR #13782 retranslated 1,865 pt-BR leaves by machine; this PR sends exactly those leaves through a reviewer pass (native-speaker prompt, dashboard context, placeholders/ICU/product names preserved) and applies only the corrections it returns.

  • scripts/i18n/review-locale.mjs (--locale=<code> --since=<git ref> [--dry-run] [--batch-size=30]): exports changedLeaves(before, after) and parseReviewResponse(text, ids); loads the repo .env like sync-ui-keys; writes the catalog in place and a markdown report (_artifacts/i18n-review/<code>.md, untracked). Test: tests/unit/i18n-review-locale.test.ts (3 cases).
  • src/i18n/messages/pt-BR.json: 172 corrections out of 1,865 reviewed leaves.

Validation

Check Result
i18n-review-locale, i18n-pt-br, i18n-placeholder-parity, i18n-cli-namespaces, server-owned-tool-loop-flag, i18n-home-recent-requests-topology-legend 41/41
npm run i18n:check-ratio OK — 65 locales within baseline
prettier on the catalog and scripts clean

Sample of the corrections:

key en before after
common.retries Retries Tentativas Novas tentativas
common.expirationBannerExpiringSoon Expiration Banner Expiring Soon Banner de expiração próxima Banner de expiração iminente
common.safeSearchModerate Safe Search Moderate Pesquisa segura moderada Pesquisa segura: moderada
common.disableConfirm Disable Confirm Confirmar desativação Confirmação de desativação

⚠️ base-red inherited: #12732

@diegosouzapw
diegosouzapw merged commit 060f331 into release/v3.8.51 Sep 16, 2026
15 of 21 checks passed
@diegosouzapw
diegosouzapw deleted the fix/i18n-pt-br-review branch September 16, 2026 15:21
diegosouzapw added a commit that referenced this pull request Sep 18, 2026
… quality (#14078)

* fix(i18n): review-locale reviews every leaf of a catalog that did not exist at --since

* fix(i18n): review the 64 non-pt-BR dashboard catalogs for translation quality

Runs scripts/i18n/review-locale.mjs over every locale except pt-BR (done in
#13885): 75,263 corrections applied against the English source, 1,677 of
them reverted because the "correction" replaced a real translation with the
plain English term (the real-translation ratio gate counts those as
untranslated). zh-CN/zh-TW provider term normalised after the run.

review-locale.mjs hardening found by the run: per-batch retries with backoff
(a skipped batch is listed, not fatal), catalog checkpoint every 25 batches,
and setDeep resolving leaf keys that contain a dot.
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
…tions) + review-locale script (diegosouzapw#13885)

Reviewer pass (native-speaker prompt) over the 1,865 pt-BR leaves retranslated in diegosouzapw#13782: 172 corrections applied; new scripts/i18n/review-locale.mjs with tests. ⚠️ base-red inherited: diegosouzapw#12732
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
… quality (diegosouzapw#14078)

* fix(i18n): review-locale reviews every leaf of a catalog that did not exist at --since

* fix(i18n): review the 64 non-pt-BR dashboard catalogs for translation quality

Runs scripts/i18n/review-locale.mjs over every locale except pt-BR (done in
diegosouzapw#13885): 75,263 corrections applied against the English source, 1,677 of
them reverted because the "correction" replaced a real translation with the
plain English term (the real-translation ratio gate counts those as
untranslated). zh-CN/zh-TW provider term normalised after the run.

review-locale.mjs hardening found by the run: per-batch retries with backoff
(a skipped batch is listed, not fatal), catalog checkpoint every 25 batches,
and setDeep resolving leaf keys that contain a dot.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant