chore: mention maintainer in bot detection report#15186
Merged
Conversation
Contributor
mnkiefer
commented
Feb 12, 2026
- Ensures the maintainer is directly mentioned in the generated report for quicker incident response.
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the bot-detection workflow’s generated alert issue body to include a direct maintainer mention, aiming to speed up incident triage when high-risk new accounts are detected.
Changes:
- Prepend the bot detection alert report with a maintainer mention (
@pelikhan). - Add a short inline comment documenting the intent of the mention.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+339
to
+340
| // Mention a maintainer for timely triage. | ||
| let body = `@pelikhan\n\nRecently-created accounts often indicate bots, spam accounts, or coordinated attacks.\n\n`; |
There was a problem hiding this comment.
The maintainer handle is hard-coded into the report body. This makes the workflow brittle if maintainers change (or in forks where that user doesn’t exist) and requires a code change to update. Consider making the mention configurable via a repo variable/env (and only prepending it when set) so it can be updated without editing the workflow and can be disabled when desired.
Suggested change
| // Mention a maintainer for timely triage. | |
| let body = `@pelikhan\n\nRecently-created accounts often indicate bots, spam accounts, or coordinated attacks.\n\n`; | |
| // Mention a maintainer for timely triage (configurable via env). | |
| const maintainerHandle = process.env.BOT_DETECTION_MAINTAINER; | |
| let body = ''; | |
| if (maintainerHandle) { | |
| body += `@${maintainerHandle}\n\n`; | |
| } | |
| body += `Recently-created accounts often indicate bots, spam accounts, or coordinated attacks.\n\n`; |
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.