Skip to content

Consolidate security bot to one comment per push#105

Merged
wesm merged 1 commit intomainfrom
disable-security-bot
Feb 7, 2026
Merged

Consolidate security bot to one comment per push#105
wesm merged 1 commit intomainfrom
disable-security-bot

Conversation

@wesm
Copy link
Owner

@wesm wesm commented Feb 7, 2026

Summary

  • Replaces per-finding inline comments + summary comment with a single consolidated PR comment containing all high/medium findings
  • Deletes old bot comments on each push so stale reviews don't accumulate
  • Low severity issues are counted but omitted from the comment body

Test plan

  • Open a test PR from a non-trusted contributor and verify only 1 comment is posted
  • Push again to the same PR and verify the old comment is replaced, not duplicated
  • Verify low-severity-only PRs get the "No High/Medium Issues Found" single comment

🤖 Generated with Claude Code

Instead of posting individual inline comments for each finding plus a
separate summary comment, the bot now posts a single consolidated
comment containing all high/medium findings. Old bot comments are
deleted on each push so noise never accumulates.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@wesm wesm merged commit b49da71 into main Feb 7, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant