Skip to content

docs(agents): require translations when source strings change - #557

Merged
DeepDiver1975 merged 1 commit into
masterfrom
docs/agents-translations-rule
Jul 26, 2026
Merged

docs(agents): require translations when source strings change#557
DeepDiver1975 merged 1 commit into
masterfrom
docs/agents-translations-rule

Conversation

@DeepDiver1975

Copy link
Copy Markdown
Member

Adds a Translations rule to agents.md so AI agents and contributors keep the l10n/ catalogs in sync whenever translatable source strings change.

Why

Translations are moving from Transifex to in-repo maintenance (see companion PR retiring the Transifex sync). This codifies the new expectation: a source-string change must carry its translation updates in the same PR.

Change

New ### Translations subsection under ## OSPO Policy Constraints:

  • Source-string changes ($l->t()/p() in templates/, t()/n() in js/) must update every l10n/<lang>.js + .json.
  • Keep .js/.json pairs in sync; preserve each file's pluralForm; plural arrays need one element per nplurals.
  • Translations are maintained in-repo (no Transifex); include them in the same PR.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: Thomas Müller <1005065+DeepDiver1975@users.noreply.github.com>

@phil-davis phil-davis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How will an ordinary person be able to know what is the correct translation for each language when making a PR to change some text, or add a new feature that has front-end UI text added?

@DeepDiver1975

Copy link
Copy Markdown
Member Author

We will provide a proper AI skill. Research ongoing.
But this instructions should be enough for the first step.

@DeepDiver1975
DeepDiver1975 merged commit 5b1ae35 into master Jul 26, 2026
21 of 22 checks passed
@DeepDiver1975
DeepDiver1975 deleted the docs/agents-translations-rule branch July 26, 2026 12:00
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.

2 participants