Conversation
Contributor
mnkiefer
commented
Feb 12, 2026
- Header is already in issue title, so not needed in issue body.
Contributor
There was a problem hiding this comment.
Pull request overview
Removes a redundant top-level Markdown header from the bot-detection workflow’s generated GitHub issue body (the issue title already provides the header), aiming to align the report body formatting with repo guidance.
Changes:
- Dropped the
# ...header line from the generated issue body in the bot-detection workflow.
Comments suppressed due to low confidence (1)
.github/workflows/bot-detection.yml:156
- The generated issue body still uses h2 headers (e.g.
## @${login}), which conflicts with the repo’s report formatting guidelines (use###or lower; never#/##). Consider downgrading account headings to###and adjusting the nested section headers accordingly to preserve hierarchy.
body = `Recently-created accounts often indicate bots, spam accounts, or coordinated attacks.\n\n`;
const sorted = Array.from(highRiskAccounts.entries()).sort((a, b) => a[1].daysOld - b[1].daysOld);
for (const [login, data] of sorted) {
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| const today = new Date().toISOString().split('T')[0]; | ||
| let body = `# 🚨 HIGH RISK: Brand New Accounts Detected — ${today}\n\n`; | ||
| body += `Recently-created accounts often indicate bots, spam accounts, or coordinated attacks.\n\n`; | ||
| body = `Recently-created accounts often indicate bots, spam accounts, or coordinated attacks.\n\n`; |
There was a problem hiding this comment.
body is assigned without being declared (body = ...). This creates an implicit global in non-strict mode and will throw in strict mode. Declare it here (e.g., let body = ...) to keep the script scope-safe and reliable.
Suggested change
| body = `Recently-created accounts often indicate bots, spam accounts, or coordinated attacks.\n\n`; | |
| let 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.