Skip to content

Commit 2c7b2e9

Browse files
DanWahlinCopilot
andcommitted
fix translation polisher safe outputs
Constrain the Translation Polisher to emit each safe output type at most once and require exact managed PR-body markers so safe output validation can complete. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 9b4f0b4 commit 2c7b2e9

1 file changed

Lines changed: 13 additions & 2 deletions

File tree

.github/workflows/translation-polisher.md

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -129,6 +129,17 @@ Do not edit:
129129
15. Add the `translation-polished` label only when every changed target-language file is A- or higher.
130130
16. Add a short pull request comment summarizing what was polished, which languages were touched, and the final grade range.
131131

132+
## Safe output limits
133+
134+
Emit each safe output type at most once:
135+
136+
- At most one `push_to_pull_request_branch`.
137+
- At most one `update_pull_request`.
138+
- At most one `add_labels`.
139+
- At most one `add_comment`.
140+
141+
Do not emit duplicate label or comment requests. If a label is already present, do not emit an `add_labels` request for it. If you need to report both a quality summary and polishing summary, combine them into the single allowed pull request comment.
142+
132143
## Quality checklist
133144

134145
For every translated Markdown file you edit:
@@ -207,7 +218,7 @@ Apply the profile only when that language is present in the pull request.
207218

208219
## Pull request body update
209220

210-
After the final review, update the pull request body with a managed translation-quality section. Replace only the managed block between these markers:
221+
After the final review, update the pull request body with a managed translation-quality section. Replace only the managed block between these exact lowercase markers:
211222

212223
```markdown
213224
<!-- translation-quality-review:start -->
@@ -220,7 +231,7 @@ The body must include exactly one managed block and exactly one section inside t
220231
## Translation Quality Review
221232
```
222233

223-
If an older unmarked `## Translation Quality Review` section already exists, replace it with the marked block. Do not append duplicates. Do not place generated workflow footers, integrity notes, or unrelated comments inside the managed block.
234+
If an older unmarked `## Translation Quality Review` section already exists, replace it with the marked block. Do not append duplicates. Do not change the marker casing. Do not place generated workflow footers, integrity notes, or unrelated comments inside the managed block.
224235

225236
Use this format:
226237

0 commit comments

Comments
 (0)