Skip to content

Verify manual-trigger exclusions for event-context workflows - #53897

Draft
pelikhan with Copilot wants to merge 1 commit into
mainfrom
copilot/deep-report-add-manual-trigger
Draft

Verify manual-trigger exclusions for event-context workflows#53897
pelikhan with Copilot wants to merge 1 commit into
mainfrom
copilot/deep-report-add-manual-trigger

Conversation

Copilot AI commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

The workflow fleet has 279 manual-dispatchable lockfiles. The remaining seven depend on event-specific payloads and cannot safely support a context-free workflow_dispatch.

  • Assessment

    • Retained event-only triggers for deployment, PR, issue, command, and workflow-run handlers.
    • No workflow source or lockfile changes are proposed.
  • Rationale

    • Adding a bare manual trigger would leave required event fields unset, e.g.:
if: github.event.pull_request.merged == true

Copilot AI changed the title [WIP] Add workflow_dispatch trigger to seven workflows Verify manual-trigger exclusions for event-context workflows Aug 19, 2026
Copilot AI requested a review from pelikhan August 19, 2026 03:36
@github-actions

Copy link
Copy Markdown
Contributor

Thanks for the investigation! 🔍 This PR provides a clear assessment of why 7 workflows cannot safely support workflow_dispatch. Here are some thoughts:

The technical reasoning (preventing unset event fields) is solid and well-documented. Moving this to discussion or merging the rationale into an issue tracker would help the team make a final decision.

Generated by ✅ Contribution Check · auto · 57.7 AIC · ⌖ 4.11 AIC · ⊞ 9.2K ·

@github-actions

Copy link
Copy Markdown
Contributor

Triage: category=docs · risk=low · score=26/100 (impact 10/50, urgency 8/30, quality 8/20)
Recommended action: defer

Assessment-only PR with 0 file changes; already labeled needs-work. Consider closing or converting to an issue since no code/lockfile diff is proposed.

Generated by 🔧 PR Triage Agent · auto · 58.8 AIC · ⌖ 2.56 AIC · ⊞ 8.3K ·

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[deep-report] Add workflow_dispatch trigger to the 7 workflows currently lacking it

2 participants