Skip to content

Fix push-to-pull-request-branch target for workflow_dispatch#138

Merged
edburns merged 1 commit intomainfrom
edburns/resolve-trigger-dependabot-problem
Apr 30, 2026
Merged

Fix push-to-pull-request-branch target for workflow_dispatch#138
edburns merged 1 commit intomainfrom
edburns/resolve-trigger-dependabot-problem

Conversation

@edburns
Copy link
Copy Markdown
Collaborator

@edburns edburns commented Apr 30, 2026

The codegen-agentic-fix workflow is always triggered via workflow_dispatch (from codegen-check.yml), which has no PR context. The push-to-pull-request-branch safe-output defaulted to target "triggering", causing every run to fail with "requires pull request context".

Set target: "*" with labels: [dependencies] so the agent can push to the PR branch identified by the pr_number input, constrained to PRs carrying the "dependencies" label (Dependabot PRs).

The codegen-agentic-fix workflow is always triggered via
workflow_dispatch (from codegen-check.yml), which has no PR context.
The push-to-pull-request-branch safe-output defaulted to target
"triggering", causing every run to fail with "requires pull request
context".

Set target: "*" with labels: [dependencies] so the agent can push to
the PR branch identified by the pr_number input, constrained to PRs
carrying the "dependencies" label (Dependabot PRs).
Copilot AI review requested due to automatic review settings April 30, 2026 02:32
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the agentic codegen-fix workflow’s safe-outputs configuration so it can successfully push fixes back to a PR branch even when the workflow is invoked via workflow_dispatch (i.e., without PR event context), while restricting the capability to Dependabot-style PRs via label gating.

Changes:

  • Configure push-to-pull-request-branch safe-output with target: "*" to avoid requiring triggering PR context.
  • Add labels: [dependencies] constraint so pushes only apply to PRs labeled dependencies.
  • Regenerate/update the compiled .lock.yml workflow to reflect the frontmatter change.
Show a summary per file
File Description
.github/workflows/codegen-agentic-fix.md Updates safe-outputs config for push-to-pull-request-branch to work under workflow_dispatch and restrict by PR label.
.github/workflows/codegen-agentic-fix.lock.yml Regenerated compiled workflow reflecting the updated safe-outputs configuration.

Copilot's findings

  • Files reviewed: 2/2 changed files
  • Comments generated: 0

@edburns edburns merged commit 5419ac8 into main Apr 30, 2026
11 checks passed
@edburns edburns deleted the edburns/resolve-trigger-dependabot-problem branch April 30, 2026 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants