Page MenuHomePhabricator

Automoderator should ignore self-reverts and reverts of Automoderator
Closed, ResolvedPublic

Description

AutoModerator should check some reverts, but not others.
When AutoModerator is reverted by another user, we should assume this was because AutoModerator made an error. We don't want AutoModerator to get into edit wars with human editors.
An editor reverting their own edit might look like a large removal of information, but it's actually an innocuous edit, because the editor is just undoing their own work. As such, we would like to skip these edits entirely.

  • If Editor A makes an edit and Editor B reverts that edit, the revert should be checked
  • If AutoModerator reverts an edit, and Editor A reverts AutoModerator's revert, the second revert from Editor A should not be checked
  • If Editor A makes an edit and Editor A reverts that edit, the revert should not be checked

Details

Event Timeline

Samwalton9-WMF renamed this task from Automoderator should handle revert edits as a special case to Automoderator should ignore self-reverts and reverts of Automoderator.Mar 14 2024, 4:42 PM
Samwalton9-WMF raised the priority of this task from Medium to High.Apr 16 2024, 4:19 PM
Scardenasmolinar changed the task status from Open to In Progress.Apr 25 2024, 4:08 PM
Scardenasmolinar claimed this task.

Change #1024535 had a related patch set uploaded (by Scardenasmolinar; author: Scardenasmolinar):

[mediawiki/extensions/AutoModerator@master] Change the revert pre-check

https://gerrit.wikimedia.org/r/1024535

Change #1024535 merged by jenkins-bot:

[mediawiki/extensions/AutoModerator@master] Change the revert pre-check

https://gerrit.wikimedia.org/r/1024535