Fix contributor reputation check comment deduping#2150
Merged
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request updates the contributor reputation check workflow to make its “risk” PR/issue comment idempotent across runs, preventing repeated/duplicated AGT marker comments and reducing thread noise.
Changes:
- Collect all existing comments that contain the AGT marker and choose the first as the canonical comment.
- When risk is MEDIUM/HIGH, update the canonical comment and delete any duplicate marker comments.
- When risk drops below MEDIUM, delete all marker comments to avoid leaving stale warnings behind.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/contributor-check.yml | Adjusts the workflow’s comment sync logic to update one canonical AGT comment and remove duplicates (or delete all when risk is below MEDIUM). |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 1
- Review effort level: Low
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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.
Pull Request Checklist
npm startand verified thatREADME.mdis up to date.mainbranch for this pull request.Description
The contributor reputation workflow was leaving repeated PR comments across runs, which made review threads noisy. This change makes the risk comment behavior idempotent so each PR keeps a single canonical AGT comment.
The workflow now collects all comments containing the AGT marker, updates the first match, and removes any duplicates. When risk is below MEDIUM, it deletes all matching marker comments so stale warnings do not remain.
Type of Contribution
Additional Notes
N/A
By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.