fix: news-translate workflow timeout — add batch limiting and time guards#1293
Merged
fix: news-translate workflow timeout — add batch limiting and time guards#1293
Conversation
Contributor
🏷️ Automatic Labeling SummaryThis PR has been automatically labeled based on the files changed and PR metadata. Applied Labels: size-xs Label Categories
For more information, see |
Contributor
🔍 Lighthouse Performance Audit
📥 Download full Lighthouse report Budget Compliance: Performance budgets enforced via |
… news-translate workflow - Increase timeout-minutes from 45 to 60 to prevent premature timeout - Add batch limiting: max 2 article types per run when scanning all - Add time guard: skip parity check if 40+ min elapsed - Simplify enhancement step (3c) to only fix critical issues - Update error handling table with batch limit guidance - Recompile lock.yml with gh-aw v0.62.0 Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
github-actions
bot
added
documentation
…med time guard constant Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
Contributor
🔍 Lighthouse Performance Audit
📥 Download full Lighthouse report Budget Compliance: Performance budgets enforced via |
Contributor
🔍 Lighthouse Performance Audit
📥 Download full Lighthouse report Budget Compliance: Performance budgets enforced via |
Copilot
AI
changed the title
[WIP] Debug workflow failure in news translate articles
fix: news-translate workflow timeout — add batch limiting and time guards
Mar 19, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the agentic news-translate workflow prompt and compiled lock workflow to reduce scheduled-run timeouts when many untranslated article types exist, aiming to ensure the workflow reaches validation + PR creation reliably.
Changes:
- Increased workflow timeout from 45 to 60 minutes (prompt + compiled lock).
- Added batch limiting guidance (process max 2 untranslated types per run) and introduced time-guard guidance to prioritize creating a partial PR over timing out.
- Simplified the translation “enhancement” step into a time-conscious “Quick Parity Check”.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| .github/workflows/news-translate.md | Adjusts timeout/time budget guidance and adds batch limiting + time guards in the agent prompt. |
| .github/workflows/news-translate.lock.yml | Recompiled lock workflow to apply the 60-minute timeout in the executed job and env. |
| .github/aw/actions-lock.json | Adds the github/gh-aw-actions/setup@v0.62.0 lock entry consistent with the updated compiler version. |
| ELAPSED=$(( $(date +%s) - START_TIME )) | ||
| if [ "$ELAPSED" -gt "$TIME_GUARD_SECONDS" ]; then | ||
| echo "⏰ 40+ minutes elapsed — skipping remaining article types to avoid timeout" | ||
| echo " Remaining types will be handled by the next scheduled run." |
| echo "⚠️ BATCH LIMIT: Processing only first 2 types (sorted): $BATCH_TYPES" | ||
| echo " Deferred to next run: $REMAINING" | ||
| UNTRANSLATED_TYPES="$BATCH_TYPES" | ||
| fi |
|
|
||
| If no untranslated articles are found, call `safeoutputs___noop` with message: "All articles are fully translated. No translation work needed." | ||
|
|
||
| **IMPORTANT**: When batch limiting is applied, the agent MUST only process the `BATCH_TYPES` (first 2 article types). Do NOT attempt to translate deferred types — they will be handled by the next scheduled run. |
Comment on lines
410
to
+416
| # Re-derive variables (each code block runs in its own shell session) | ||
| source /tmp/gh-aw/agent/timing.env 2>/dev/null || START_TIME=$(date +%s) | ||
| TIME_GUARD_SECONDS=2400 # 40 minutes | ||
| ELAPSED=$(( $(date +%s) - START_TIME )) | ||
| echo "⏰ Elapsed time: $((ELAPSED / 60)) minutes" | ||
| if [ "$ELAPSED" -gt "$TIME_GUARD_SECONDS" ]; then | ||
| echo "⚠️ 40+ minutes elapsed — skipping parity check to avoid timeout" |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The
news-translatescheduled workflow timed out at 45 minutes when 3+ EN article types needed translation into 12 languages (36 files). The agent spent its entire budget on translation work without ever reaching PR creation.Changes
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.