Skip to content

feat(messageCreate): Act on suspicious amount of attachments (mrbeast)#382

Merged
XboxBedrock merged 4 commits into
BuildTheEarth:mainfrom
Nudelsuppe42:main
Jun 21, 2026
Merged

feat(messageCreate): Act on suspicious amount of attachments (mrbeast)#382
XboxBedrock merged 4 commits into
BuildTheEarth:mainfrom
Nudelsuppe42:main

Conversation

@Nudelsuppe42

@Nudelsuppe42 Nudelsuppe42 commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

For now this is very basic. It catches all messages with 2 or 4 attachments and if a message with at least one identical attachment was sent in the past 30 seconds the message is deleted.

If a banned word SUS_ATTACHMENT is present, the message will be treated with the moderation actions connected to that word. Else, a log will be pushed.

Original poc by @Sandy

@Nudelsuppe42 Nudelsuppe42 self-assigned this Jun 15, 2026
@Nudelsuppe42 Nudelsuppe42 added feature This issue or pull request requests or integrates a new feature change This issue or pull request requests or makes a refactor labels Jun 15, 2026
@Nudelsuppe42
Nudelsuppe42 requested a review from Copilot June 15, 2026 16:25

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.

Comment thread src/events/messageCreate.event.ts
Comment thread src/events/messageCreate.event.ts Outdated
Comment thread src/events/messageCreate.event.ts Outdated
@Nudelsuppe42
Nudelsuppe42 requested a review from XboxBedrock June 15, 2026 18:37
@XboxBedrock

Copy link
Copy Markdown
Member

Why is there a block of commented code? Unless it is absolutely necessary, we shouldn't add comments to the codebase

@Nudelsuppe42

Copy link
Copy Markdown
Contributor Author

forgot to remove it. Will also remove the 2/4 image check as we've seen posts with just one the psst few days.

@XboxBedrock

Copy link
Copy Markdown
Member

Only one image is problematic because then memes and stuff get flagged

@Nudelsuppe42

Copy link
Copy Markdown
Contributor Author

Only one image is problematic because then memes and stuff get flagged

Wdym? as long as the exact same image isnt sent in <30sec its fine. And if it is, then a mod can still pardon.
If we find that 30sec is too much we can also reduce it to 10 or something

@XboxBedrock
XboxBedrock merged commit 7ac8d78 into BuildTheEarth:main Jun 21, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change This issue or pull request requests or makes a refactor feature This issue or pull request requests or integrates a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants