Skip to content

Include issue authors in CI scan dedup searches - #132738

Merged
vitek-karas merged 1 commit into
dotnet:mainfrom
vitek-karas:vitek-karas-investigate-ci-scan-duplicate
Aug 28, 2026
Merged

vitek-karas merged 1 commit into
dotnet:mainfrom
vitek-karas:vitek-karas-investigate-ci-scan-duplicate

Conversation

@vitek-karas

Copy link
Copy Markdown
Member

Summary

Request the user field in every shared KBE issue search so the integrity gateway can recognize trusted bot authors before filtering results.

This prevents Actions-authored KBEs such as #132668 from being hidden during duplicate detection, which led to duplicate #132729.

Validation

  • gh aw compile ci-failure-scan --no-emit --validate

Note

This pull request description was generated by GitHub Copilot.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings August 25, 2026 08:48
@vitek-karas
vitek-karas requested review from a team and jeffhandley as code owners August 25, 2026 08:48
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
15 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 shared KBE duplicate-detection instructions used by CI scan and local tooling so issue searches always request author information needed by downstream integrity filtering.

Changes:

  • Require including user in the requested fields for every search_issues call in the “Search for an existing KBE” flow.
  • Document why this is necessary (integrity gateway uses user.login to recognize trusted bot authors prior to filtering).

@davidnguyen-tech davidnguyen-tech left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thanks!

@vitek-karas
vitek-karas merged commit 019d8d9 into dotnet:main Aug 28, 2026
24 checks passed
@vitek-karas
vitek-karas deleted the vitek-karas-investigate-ci-scan-duplicate branch August 28, 2026 13:49
@dotnet-milestone-bot dotnet-milestone-bot Bot added this to the 12.0-preview1 milestone Aug 29, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 28, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants