Warn when workflow dispatches share conclusion concurrency - #55836
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Adds compiler diagnostics for shared conclusion-job concurrency in manually dispatched workflows.
Changes:
- Emits an actionable missing-discriminator warning.
- Adds trigger and discriminator test cases.
Show a summary per file
| File | Description |
|---|---|
pkg/workflow/compiler_validators.go |
Adds the concurrency warning. |
pkg/workflow/workflow_dispatch_concurrency_warning_test.go |
Tests warning conditions. |
Review details
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
- Files reviewed: 2/2 changed files
- Comments generated: 2
- Review effort level: Balanced
| } | ||
|
|
||
| func (c *Compiler) emitGeneralToolWarnings(workflowData *WorkflowData, markdownPath string) { | ||
| if strings.Contains(workflowData.On, "workflow_dispatch") && workflowData.ConcurrencyJobDiscriminator == "" { |
| } | ||
|
|
||
| func (c *Compiler) emitGeneralToolWarnings(workflowData *WorkflowData, markdownPath string) { | ||
| if strings.Contains(workflowData.On, "workflow_dispatch") && workflowData.ConcurrencyJobDiscriminator == "" { |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
Please do one focused follow-up pass:
Run: https://github.com/github/gh-aw/actions/runs/32896118186
|
|
@copilot PR Sous Chef triage: this PR still needs a forward-progress pass before maintainer investigation is likely to be productive. Please do one focused follow-up pass:
Run: https://github.com/github/gh-aw/actions/runs/32897609333
|
Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
Completed the focused follow-up in 51b1f54: the warning now requires configured safe outputs, with a no-safe-outputs regression case. The branch is refreshed from main, and PR-finisher validation passed ( |
The unresolved feedback is already implemented on the current HEAD: a478d98 parses top-level |
|
@copilot resolve the merge conflicts on this branch. |
|
Please do one focused follow-up pass:
Run: https://github.com/github/gh-aw/actions/runs/32907942669
|
Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
Ran
Resolved and pushed in merge commit f86eac9. The conflict retained the dispatch-input warning helper while accepting the upstream bounded-query removal; post-merge lint, unit/full tests, and recompilation pass. |
|
Category: bug | Risk: medium | Score: 55/100 (impact 24/50, urgency 15/30, quality 16/20) | Action: batch_review | Batch: workflow-runtime
|
|
Please do one focused follow-up pass:
Run: https://github.com/github/gh-aw/actions/runs/32949103424
|
The branch is already refreshed in merge commit b8478f5, whose second parent is the current base d846ab2. PR-finisher validation passes ( |
|
Triage update: Category: bug | Risk: medium | Score: 54/100 (impact 24 + urgency 14 + quality 16) | Action: auto_merge | Batch: workflow-runtime
|
|
Triage update: Category: bug | Risk: medium | Score: 54/100 (impact 24 + urgency 14 + quality 16) | Action: auto_merge | Batch: workflow-runtime
|
workflow_dispatchfan-out silently serializes generated conclusion jobs whenconcurrency.job-discriminatoris omitted, reducing throughput while runs remain green.Compiler warning
Warn when a workflow includes
workflow_dispatchwithout a job discriminator.Explain that dispatches share one conclusion concurrency group.
Recommend a per-run discriminator:
Coverage
Run: https://github.com/github/gh-aw/actions/runs/32897609333