Repository navigation
[ci-scan-feedback] Read ci-fix head.ref via list/get, not search, to close dedup gap #134305
Description
Triggering signals
- [ci-fix] Needs review: mark Apple runner interop as safe (refs #134061) #134186 / [ci-fix] Mark Apple test runner LibraryImports safe (refs #134061) #134134 — help-wanted
[ci-fix]PR [ci-fix] Needs review: mark Apple runner interop as safe (refs #134061) #134186 (2026-09-18,kind: help, Linked KBE [ci-scan] Build break: SYSLIB1064 in AppleTestRunner.cs blocks Apple NativeAOT test legs #134061) is an un-deduped twin of confident[ci-fix]PR [ci-fix] Mark Apple test runner LibraryImports safe (refs #134061) #134134 (kind: fix, same KBE [ci-scan] Build break: SYSLIB1064 in AppleTestRunner.cs blocks Apple NativeAOT test legs #134061), both single-file 2-lineLibraryImportsafe-annotation changes onci-fix/134061-head branches. [ci-fix] Mark Apple test runner LibraryImports safe (refs #134061) #134134 merged 09-18; [ci-fix] Needs review: mark Apple runner interop as safe (refs #134061) #134186 remains open. The fixer's Step 3 branch-prefix dedup key never matched. [ci-fix] Needs review: mark Apple runner interop as safe (refs #134061) #134186 - [ci-scan-feedback] Read ci-fix head.ref via list/get, not search, to close dedup gap #134276 / [ci-scan-feedback] Read ci-fix head.ref via list/get, not search, to close dedup gap #132616 — this exact fix orphaned twice: last tick's
[ci-scan-feedback]PR failed its git push and orphaned as issue [ci-scan-feedback] Read ci-fix head.ref via list/get, not search, to close dedup gap #134276 (2026-09-19); a month earlier the same diagnosis orphaned as [ci-scan-feedback] Read ci-fix head.ref via list/get, not search, to close dedup gap #132616 (2026-08-21). The fix never landed, so the failure mode reproduced. [ci-scan-feedback] Read ci-fix head.ref via list/get, not search, to close dedup gap #134276 - Empirical confirmation —
search_pull_requestsresult items expose keys[number, title, state, pull_request, body, labels, ...]with nohead/head.ref, whereaslist_pull_requestsandpull_request_read getboth returnhead.ref(e.g. [ci-fix] Needs review: mark Apple runner interop as safe (refs #134061) #134186 ->ci-fix/134061-apple-runner-safety-...). Step 3 instructed the agent to parsehead.reffrom search results, so theci-fix/-dedup map was always empty.
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 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 #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. This edit lands directly (not as a re-filed feedback issue) because the identical prompt change has now orphaned twice on failed PR pushes (#132616, #134276).
Note
AI/Copilot-generated (ci-scan-feedback). Prompt-only edit to .github/workflows/ci-failure-fix.md.
Resolves #134276
Generated by CI Outer-Loop Failure Scanner — Feedback · opus48 · 961.2 AIC · ⌖ 19.2 AIC · ⊞ 18.2K · ◷
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 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 35511965948 -n agent -D /tmp/agent-35511965948
# Fetch the bundle into a temporary ref, then update the local branch
git fetch /tmp/agent-35511965948/aw-ci-scan-feedback-fixer-dedup-headref-listget.bundle refs/heads/ci-scan-feedback-fixer-dedup-headref-listget:refs/bundles/create-pr-ci-scan-feedback-fixer-dedup-headref-listget-7090a0294c34b8a3-12246dde
git update-ref refs/heads/ci-scan-feedback-fixer-dedup-headref-listget-7090a0294c34b8a3 refs/bundles/create-pr-ci-scan-feedback-fixer-dedup-headref-listget-7090a0294c34b8a3-12246dde
git checkout ci-scan-feedback-fixer-dedup-headref-listget-7090a0294c34b8a3
# 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-dedup-headref-listget-7090a0294c34b8a3-12246dde
# Push the branch to origin
git push origin ci-scan-feedback-fixer-dedup-headref-listget-7090a0294c34b8a3
# 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-dedup-headref-listget-7090a0294c34b8a3 --repo dotnet/runtimeMetadata
Metadata
Assignees
Type
Projects
- StatusShow more project fieldsDone
Activity
dotnet-policy-service commentedon Sep 21, 2026
Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.