You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
.github/workflows/ci-failure-fix.md (Step 3 — Existing-artifact dedup, ~line 211) — replace the "enumerate every [ci-fix] PR ... parsing head.ref" instruction with an explicit note that search_pull_requests returns no head field, and direct the fixer to read head branches from list_pull_requests (state: open) for open candidates and pull_request_read get for merged/closed candidates. Add that an empty branch map means "not yet enumerated", not "no open PR".
Expected behavior change
The next fixer run builds the <kbe> -> [PRs] dedup map from head.ref values actually present in the response — list_pull_requests for open PRs and pull_request_read get for merged/closed ones — instead of from search_pull_requests items that never carry a head field. This closes the residual dedup gap that let help-wanted PR #134186 be filed against KBE #134061 while confident #134134 was already open: the fixer detects the open ci-fix/134061- branch and skips with open fix PR #134134 already exists rather than opening a duplicate. Landing directly (not as a re-filed feedback issue) because this identical prompt change has now orphaned three times on failed PR pushes (#132616, #134276, #134305).
The bundle file is available in the agent artifact in the workflow run linked above.
To create a pull request with the changes:
# Download the artifact from the workflow run
gh run download 35602411113 -n agent -D /tmp/agent-35602411113
# Fetch the bundle into a temporary ref, then update the local branch
git fetch /tmp/agent-35602411113/aw-ci-scan-feedback-fixer-headref-listget.bundle refs/heads/ci-scan-feedback-fixer-headref-listget:refs/bundles/create-pr-ci-scan-feedback-fixer-headref-listget-9147d48586987407-59344a87
git update-ref refs/heads/ci-scan-feedback-fixer-headref-listget-9147d48586987407 refs/bundles/create-pr-ci-scan-feedback-fixer-headref-listget-9147d48586987407-59344a87
git checkout ci-scan-feedback-fixer-headref-listget-9147d48586987407
# Ensure the working tree matches the updated branch
git reset --hard
# Remove the temporary bundle ref
git update-ref -d refs/bundles/create-pr-ci-scan-feedback-fixer-headref-listget-9147d48586987407-59344a87
# Push the branch to origin
git push origin ci-scan-feedback-fixer-headref-listget-9147d48586987407
# Create the pull request
gh pr create --title '[ci-scan-feedback] Read ci-fix head.ref via list/get, not search, to close dedup gap' --base main --head ci-scan-feedback-fixer-headref-listget-9147d48586987407 --repo dotnet/runtime
Triggering signals
[ci-fix]PR [ci-fix] Needs review: mark Apple runner interop as safe (refs #134061) #134186 (kind: help, Linked KBE [ci-scan] Build break: SYSLIB1064 in AppleTestRunner.cs blocks Apple NativeAOT test legs #134061) was filed while confident[ci-fix]PR [ci-fix] Mark Apple test runner LibraryImports safe (refs #134061) #134134 (kind: fix, same KBE, merged 09-18) was already open — both onci-fix/134061-head branches. The fixer's Step 3 branch-prefix dedup key never matched. [ci-fix] Needs review: mark Apple runner interop as safe (refs #134061) #134186search_pull_requestsresult items expose noheadfield (confirmed against [ci-fix] Needs review: mark Apple runner interop as safe (refs #134061) #134186, [ci-fix] Mark Apple test runner LibraryImports safe (refs #134061) #134134, [ci-fix] Needs review: dual-stack accept-reset socket test fails on Windows/Linux (refs #133778) #133925, [ci-fix] Fix Android NativeAOT template for updated memory safety rules #134323 — allhead.refMISSING), whereaslist_pull_requestsandpull_request_read getboth returnhead.ref. Step 3 instructed the fixer to parsehead.reffrom search results, so theci-fix/<kbe>-dedup map was always empty and never matched.Proposed edits
.github/workflows/ci-failure-fix.md(Step 3 — Existing-artifact dedup, ~line 211) — replace the "enumerate every[ci-fix]PR ... parsinghead.ref" instruction with an explicit note thatsearch_pull_requestsreturns noheadfield, and direct the fixer to read head branches fromlist_pull_requests(state: open) for open candidates andpull_request_read getfor merged/closed candidates. Add that an empty branch map means "not yet enumerated", not "no open PR".Expected behavior change
The next fixer run builds the
<kbe> -> [PRs]dedup map fromhead.refvalues actually present in the response —list_pull_requestsfor open PRs andpull_request_read getfor merged/closed ones — instead of fromsearch_pull_requestsitems that never carry aheadfield. This closes the residual dedup gap that let help-wanted PR #134186 be filed against KBE #134061 while confident #134134 was already open: the fixer detects the openci-fix/134061-branch and skips withopen fix PR #134134 already existsrather than opening a duplicate. Landing directly (not as a re-filed feedback issue) because this identical prompt change has now orphaned three times on failed PR pushes (#132616, #134276, #134305).Resolves #134305
Note
AI/Copilot-generated (
ci-scan-feedback). Prompt-only edit to.github/workflows/ci-failure-fix.md.Note
This was originally intended as a pull request, but the git push operation failed.
Original error: The process '/usr/bin/git' failed with exit code 1
Workflow Run: View run details and download bundle artifact
The bundle file is available in the
agentartifact in the workflow run linked above.To create a pull request with the changes: