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

This comment has been minimized.

@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 ·

@github-actions

Copy link
Copy Markdown
Contributor

Thanks for looking into workflow_dispatch support! This PR is authored by the core team's Copilot SWE agent, so the direct-PR restriction does not apply. However, the diff is currently empty (0 files changed) - it's an assessment-only PR explaining why 7 event-context workflows cannot safely support manual triggering. Consider either closing this in favor of a tracking issue/discussion, or following up with the actual code changes if any are planned, since as-is there's nothing to merge. Also noted (non-blocking): repository topics could be expanded per the guidelines, but that is a maintainer/admin task outside this PR's scope.

Generated by ✅ Contribution Check · auto · 71.9 AIC · ⌖ 7.45 AIC · ⊞ 9.2K ·

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