Fix news workflow agent timeouts: prohibit manual article generation#1288
Merged
Fix news workflow agent timeouts: prohibit manual article generation#1288
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 |
…ime budgets
- Add "Article Generation Safety" section to interpellations, propositions,
motions, and committee-reports workflow prompts prohibiting python3/heredoc
manual HTML generation that causes agent timeouts
- Fix misleading time budgets ("all 14 languages" → "core languages EN, SV")
- Add fallback instruction: use safeoutputs___noop if script fails
- Add 3 new tests enforcing script-based generation safety rules
Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
github-actions
bot
added
documentation
Contributor
🔍 Lighthouse Performance Audit
📥 Download full Lighthouse report Budget Compliance: Performance budgets enforced via |
Copilot
AI
changed the title
[WIP] [aw] Debug workflow failure in news interpellation debates
Fix news workflow agent timeouts: prohibit manual article generation
Mar 19, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the gh-aw news workflow prompts to prevent agent timeouts by disallowing manual article construction and enforcing the existing TypeScript-based generation pipeline, with accompanying architecture tests to keep the constraint in place.
Changes:
- Add an “Article Generation Safety” section to four dedicated news workflows, prohibiting python/manual HTML/heredoc generation and directing agents to
npx tsx scripts/generate-news-enhanced.ts. - Update workflow time budgets to focus on core language generation (EN/SV) and then dispatch
news-translate. - Add a Vitest suite enforcing the new “script-based generation” safety constraints across the four workflows.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/workflow-architecture.test.ts | Adds tests to enforce python/manual-HTML prohibitions and require generate-news-enhanced.ts usage for script-based workflows. |
| .github/workflows/news-interpellations.md | Updates time budget and adds “Article Generation Safety” guidance directing generation via generate-news-enhanced.ts. |
| .github/workflows/news-propositions.md | Updates time budget and adds “Article Generation Safety” guidance directing generation via generate-news-enhanced.ts. |
| .github/workflows/news-motions.md | Updates time budget and adds “Article Generation Safety” guidance directing generation via generate-news-enhanced.ts. |
| .github/workflows/news-committee-reports.md | Updates time budget and adds “Article Generation Safety” guidance directing generation via generate-news-enhanced.ts. |
Comment on lines
+147
to
+149
| The repository provides a complete article generation pipeline. You MUST use it: | ||
| ```bash | ||
| source scripts/mcp-setup.sh && npx tsx scripts/generate-news-enhanced.ts --types=interpellations --languages="en,sv" --skip-existing |
|
|
||
| The repository provides a complete article generation pipeline. You MUST use it: | ||
| ```bash | ||
| source scripts/mcp-setup.sh && npx tsx scripts/generate-news-enhanced.ts --types=propositions --languages="en,sv" --skip-existing |
.github/workflows/news-motions.md
Outdated
|
|
||
| The repository provides a complete article generation pipeline. You MUST use it: | ||
| ```bash | ||
| source scripts/mcp-setup.sh && npx tsx scripts/generate-news-enhanced.ts --types=motions --languages="en,sv" --skip-existing |
Comment on lines
+151
to
+153
| The repository provides a complete article generation pipeline. You MUST use it: | ||
| ```bash | ||
| source scripts/mcp-setup.sh && npx tsx scripts/generate-news-enhanced.ts --types=committee-reports --languages="en,sv" --skip-existing |
Member
|
@copilot apply changes based on the comments in this thread |
Replace hardcoded --languages="en,sv" with --languages="$LANG_ARG" in the safety section of all 4 news workflow prompts, referencing the Generation Steps section where LANG_ARG is derived from the dispatch languages input. Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
Contributor
Author
Updated in 379343e — replaced hardcoded |
Contributor
🔍 Lighthouse Performance Audit
📥 Download full Lighthouse report Budget Compliance: Performance budgets enforced via |
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.
build-en-article.py) instead of usingnpx tsx scripts/generate-news-enhanced.tssafeoutputs___noopinstead of manually building HTMLtests/workflow-architecture.test.tsenforcing script-based generation safety$LANG_ARGin safety section examples instead of hardcoded"en,sv"Original prompt
💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.