Fix policy bot bugs and expand label coverage#6202
Merged
Conversation
Trenly
reviewed
May 3, 2026
Trenly
reviewed
May 3, 2026
Trenly
reviewed
May 4, 2026
Trenly
reviewed
May 4, 2026
Trenly
reviewed
May 4, 2026
Trenly
left a comment
Contributor
There was a problem hiding this comment.
As long as you're touching files, I'd recommend removing the automergeTriggers.yml and the associated label since it's not used at all
- Convert regex patterns to (?in) inline flags with label-matching casing - Add Needs-Author-Feedback back to triageLabels (safe for issues) - Remove unused automergeTriggers.yml policy - Fix Needs-Triage removal to exclude auto-applied labels - Fix stale issue flow and issueUpdated handlers Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
3825078 to
96890da
Compare
Trenly
approved these changes
May 4, 2026
Trenly
approved these changes
May 4, 2026
florelis
approved these changes
May 4, 2026
8 tasks
denelon
pushed a commit
that referenced
this pull request
May 28, 2026
## 📖 Description <!-- Describe what this PR changes, why, and any limitations. --> Although these three labels get applied when the issue is opened, it is technically a separate event that happens *after* the the issue is opened. This means that the policy service is catching this event and removing issues from triage before they are actually triaged. This is a faulty implementation from #6202 ## 🔗 References <!-- Link related issues, PRs, or docs. Use "Resolves #1234" to auto-close. --> See recently opened issue #6245 ## 🔍 Validation None <!-- How did you test? List manual steps or note automated test coverage. --> ## ✅ Checklist <!-- Place an "x" between the brackets to check an item. e.g: [x] --> - [x] Signed the [Contributor License Agreement](https://cla.opensource.microsoft.com) - [ ] Linked to an issue - [ ] Updated [Release Notes](../doc/ReleaseNotes.md) (if applicable) - [ ] Updated documentation (if applicable) - [ ] Updated [Copilot instructions](.github/copilot-instructions.md) (if build, architecture, or conventions changed) ## 📋 Issue Type <!-- Select the type that best describes this PR --> - [x] Bug fix - [ ] Feature - [ ] Task ###### Microsoft Reviewers: [Open in CodeFlow](https://microsoft.github.io/open-pr/?codeflow=https://github.com/microsoft/winget-cli/pull/6254)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📖 Description
Modernizes policy bot regex handling by switching to inline flags, fixes triageLabels so auto-applied labels do not clear triage too early, removes unused automergeTriggers, restores Needs-Author-Feedback coverage, and cleans up related stale-flow behavior.
Created with GitHub Copilot assistance.
🔗 References
Resolves #6208
🔍 Validation
✅ Checklist
📋 Issue Type