Skip to content

Fix codegen-agentic-fix: remove dependencies label requirement for PR pushes#174

Merged
edburns merged 4 commits into
mainfrom
copilot/aw-fix-codegen-agentic-workflow
May 11, 2026
Merged

Fix codegen-agentic-fix: remove dependencies label requirement for PR pushes#174
edburns merged 4 commits into
mainfrom
copilot/aw-fix-codegen-agentic-workflow

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 8, 2026

The codegen-agentic-fix workflow was failing with push_to_pull_request_branch: Pull request is missing required labels: dependencies because the push-to-pull-request-branch safe output required the target PR to carry the dependencies label — but PRs without that label (e.g. manually-triggered runs targeting non-Dependabot PRs) caused the push to be blocked entirely.

Before the change?

  • The push-to-pull-request-branch safe output in codegen-agentic-fix.md required target PRs to carry the dependencies label (labels: [dependencies]), causing the workflow to fail with a label mismatch error when run against PRs that don't have that label.

After the change?

  • codegen-agentic-fix.md: Removed labels: [dependencies] from push-to-pull-request-branch safe output — the workflow can now push to any PR (target: "*") regardless of labels.
  • codegen-agentic-fix.lock.yml: Regenerated via gh aw compile to reflect the updated frontmatter hash and remove the label constraint from all safe-output handler configs.
  • Branch has been updated to incorporate the latest main commits (PR Close stale sync PRs in reference-impl-sync agentic workflow #186 reference-impl-sync cleanup and related changes); the fix remains intact post-merge.

Pull request checklist

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)
  • mvn spotless:apply has been run to format the code
  • mvn clean verify passes locally

Does this introduce a breaking change?

  • Yes
  • No

Copilot AI linked an issue May 8, 2026 that may be closed by this pull request
…to-pull-request-branch

Co-authored-by: edburns <75821+edburns@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix codegen agentic workflow failure due to missing labels Fix codegen-agentic-fix: remove dependencies label requirement for PR pushes May 8, 2026
Copilot finished work on behalf of edburns May 8, 2026 22:27
Copilot AI requested a review from edburns May 8, 2026 22:27
@edburns
Copy link
Copy Markdown
Collaborator

edburns commented May 8, 2026

Dear @copilot ,

But there has been a subsequent run of the agentic fix that seems to have not exhibited this failure:

#177

Please investigate. Are these changes necessary? Or is the existence of issue 177 really not related to the changes in this PR 174?

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 8, 2026

Dear @copilot ,

But there has been a subsequent run of the agentic fix that seems to have not exhibited this failure:
...

Issue #177 is a reference-impl-sync workflow issue — a different workflow entirely, not a subsequent run of codegen-agentic-fix. The two are unrelated.

The original failure was the codegen-agentic-fix workflow blocking its push because the target Dependabot PR lacked the dependencies label. That constraint is what this PR removes. Without this fix, any future codegen-agentic-fix run targeting a PR without that label will fail in the same way.

Copilot finished work on behalf of edburns May 8, 2026 23:18
@edburns edburns marked this pull request as ready for review May 11, 2026 17:15
Copilot AI review requested due to automatic review settings May 11, 2026 17:15
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

This PR updates the codegen-agentic-fix agentic workflow configuration so that the Safe Output push-to-pull-request-branch is no longer gated on the target PR having the dependencies label, preventing manual workflow runs from being blocked on non-Dependabot PRs.

Changes:

  • Removed the labels: [dependencies] requirement from the push-to-pull-request-branch safe-output configuration in the workflow source.
  • Regenerated the compiled workflow lock file to reflect the updated frontmatter hash and Safe Outputs configuration (including removal of the label constraint).
Show a summary per file
File Description
.github/workflows/codegen-agentic-fix.md Removes the Safe Outputs label requirement so pushes aren’t blocked on PR labeling.
.github/workflows/codegen-agentic-fix.lock.yml Regenerated compiled workflow to match the updated Safe Outputs frontmatter/config.

Copilot's findings

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

Comment thread .github/workflows/codegen-agentic-fix.lock.yml Outdated
@edburns
Copy link
Copy Markdown
Collaborator

edburns commented May 11, 2026

@copilot some relevant code was merged to main since you authored this.

Please incorporate that code into your work here and re-evaluate its necessity, especially your review comment #174 (comment) .

You'll need to re-run an agentic session to do this work.

…n-agentic-workflow

Co-authored-by: edburns <75821+edburns@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 11, 2026

@copilot some relevant code was merged to main since you authored this.

Please incorporate that code into your work here and re-evaluate its necessity, especially your review comment https://gith...

Merged main into this branch (commit b7b62de). The changes merged cleanly — the commits on main since this PR was branched didn't touch codegen-agentic-fix.md or its lock file, so the fix is still intact.

Regarding the reviewer comment about the AGENTS.md mismatch between config.json and GH_AW_SAFE_OUTPUTS_HANDLER_CONFIG: this pre-exists on main as well (both have exactly one occurrence of AGENTS.md, only in the handler config). It's a compiler artifact from gh aw compile v0.68.3 and can't be hand-edited without breaking the hash validation. No action needed on our side.

The labels: [dependencies] fix remains necessary — main still carries it, and any future codegen-agentic-fix run against a PR without that label will fail the same way.

Instead of removing the labels:[dependencies] safety constraint from the
codegen-agentic-fix safe-output, fix the root cause: the PR created by
update-copilot-dependency.yml was missing the label. Now the PR is
created with --label dependencies, matching the safe-output constraint.

Recompile codegen-agentic-fix.lock.yml.
@edburns edburns merged commit 51545f3 into main May 11, 2026
7 checks passed
@edburns edburns deleted the copilot/aw-fix-codegen-agentic-workflow branch May 11, 2026 21:05
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.

[aw] Codegen Agentic Fix failed

3 participants