Skip to content

Commit 347ca3c

Browse files
Copilotpethers
andauthored
Align batch count wording with create-pull-request.max: 5 and clarify recoverable files
Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/f13db3b0-0295-4404-9263-f987b5558ca7 Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
1 parent 81e59a5 commit 347ca3c

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/workflows/news-translate.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -410,7 +410,7 @@ news/translate/{YYYY-MM-DD}/{article-type}
410410
411411
## ⏱️ Time Budget (55 minutes of work, hard stop at 55)
412412

413-
The budget is now organised around **3 rolling PR batches**, not one monolithic PR. This is the direct fix for PR #1835 where 3 completed translations (`fr`, `es`, `nl`) were lost because they were committed *after* the single `safeoutputs___create_pull_request` call had frozen the patch.
413+
The budget is now organised around **typically 2–3 rolling PR batches (up to 5 allowed by `create-pull-request.max: 5`)**, not one monolithic PR. This is the direct fix for PR #1835 where 3 completed translations (`fr`, `es`, `nl`) were lost because they were committed *after* the single `safeoutputs___create_pull_request` call had frozen the patch. If time allows and the workload spans multiple articles/types, open additional batch PRs up to the 5-per-run safe-outputs cap.
414414

415415
| Phase | Minutes | Action |
416416
|-------|---------|--------|
@@ -849,7 +849,7 @@ npx tsx scripts/validate-news-translations.ts
849849
| Time running out (current batch ≥18 min) | Stop adding languages → validate → commit → `safeoutputs___create_pull_request` with what you have, then start the next batch on `main` |
850850
| HTMLHint errors | Fix with `edit` tool or run `npx tsx scripts/article-quality-enhancer.ts --fix` |
851851
| safeoutputs "session not found" | Session expired — all uncreated PR intents are LOST. **Prevention: call `safeoutputs___create_pull_request` for the first batch by minute 22.** Later batches must also be called promptly (never more than 15 minutes between successive calls). |
852-
| Committed files on a branch that already has a PR | Those files are lost — the patch was frozen at the first `safeoutputs___create_pull_request` call. Switch to `main`, re-create the translations fresh, and call `safeoutputs___create_pull_request` again for a new PR. |
852+
| Committed files on a branch that already has a PR | Not included in the existing PR — the patch was frozen at the first `safeoutputs___create_pull_request` call. Switch to `main` and create a new batch PR containing only the new translations; if the files still exist in the workspace (or on the previous branch), re-apply/cherry-pick or recommit them onto `main` so you don't need to re-translate from scratch. Then call `safeoutputs___create_pull_request` again for the new batch. |
853853
854854
## 🎯 Execution Summary
855855

0 commit comments

Comments
 (0)