Skip to content

ci: add anti-slop action#44847

Merged
tarekziade merged 6 commits intomainfrom
ai-slop-action
Mar 26, 2026
Merged

ci: add anti-slop action#44847
tarekziade merged 6 commits intomainfrom
ai-slop-action

Conversation

@tarekziade
Copy link
Copy Markdown
Collaborator

@tarekziade tarekziade commented Mar 19, 2026

What does this PR do?

Activated anti-slop action.

Enabled checks:

  • min-account-age: 30 to catch brand-new throwaway accounts, which are common in automated spam waves.
  • max-daily-forks: 7 to catch accounts that fork many repositories in a 24-hour window.

This workflow is label-only and does not close or lock PRs. It also keeps max-failures: 2, so a PR is only flagged when both signals fail.

@tarekziade tarekziade self-assigned this Mar 19, 2026
@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@tarekziade tarekziade requested a review from ydshieh March 19, 2026 07:31
Copy link
Copy Markdown
Collaborator

@ydshieh ydshieh left a comment

Choose a reason for hiding this comment

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

Nice

Comment thread .github/workflows/anti-slop.yml Outdated
@ydshieh
Copy link
Copy Markdown
Collaborator

ydshieh commented Mar 24, 2026

Let's see what @glegendre01 or @paulinebm says before merge.

@tarekziade
Copy link
Copy Markdown
Collaborator Author

Let's see what @glegendre01 or @paulinebm says before merge.

Thanks for the review and pings @ydshieh

I'd also like to hear from @Rocketknight1 , since there were since then more discussion on applying more immediate remedies.

@Rocketknight1
Copy link
Copy Markdown
Member

Hey, I'm not certain we want all of this yet! The minimum account age might be good, but the rest are quite difficult to hard filter. In particular, I think the max-comments check will catch e.g. all model addition PRs, and max-description-length is also a bit unreliable. Also, sometimes people have good reason for skipping the PR template, and I'm seeing plenty of agent PRs that keep it, so I don't know if it's a good signal. Can we start with a much smaller list and then iterate over time?

Comment thread .github/workflows/anti-slop.yml Outdated
@Rocketknight1
Copy link
Copy Markdown
Member

Rocketknight1 commented Mar 25, 2026

Also one other thing: The check for the maximum number of repos forked by an account in 24 hours is probably a very good way to catch the worst OpenClaw offenders, and it should catch very few legit accounts - maybe we could try adding that one?

Copy link
Copy Markdown
Contributor

@paulinebm paulinebm left a comment

Choose a reason for hiding this comment

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

LGTM

@tarekziade
Copy link
Copy Markdown
Collaborator Author

Hey, I'm not certain we want all of this yet! The minimum account age might be good, but the rest are quite difficult to hard filter. In particular, I think the max-comments check will catch e.g. all model addition PRs, and max-description-length is also a bit unreliable. Also, sometimes people have good reason for skipping the PR template, and I'm seeing plenty of agent PRs that keep it, so I don't know if it's a good signal. Can we start with a much smaller list and then iterate over time?

Sounds good to me, I'll shrink it

@tarekziade tarekziade added this pull request to the merge queue Mar 26, 2026
Merged via the queue into main with commit 61efa32 Mar 26, 2026
16 of 18 checks passed
@tarekziade tarekziade deleted the ai-slop-action branch March 26, 2026 13:39
zucchini-nlp pushed a commit to zucchini-nlp/transformers that referenced this pull request Mar 27, 2026
* add anti-slop action

* remove maintainers names

* pin the action to 0.2.1

* tweaks @Rocketknigth1 reviews
NielsRogge pushed a commit to NielsRogge/transformers that referenced this pull request Mar 30, 2026
* add anti-slop action

* remove maintainers names

* pin the action to 0.2.1

* tweaks @Rocketknigth1 reviews
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.

5 participants