Wikipedia:Bots/Requests for approval/Ahechtbot

The following discussion is an archived debate. Please do not modify it. To request review of this BRFA, please start a new section at WT:BRFA. The result of the discussion was

Approved.

Ahechtbot

Operator: Ahecht (talk · contribs · SUL · edit count · logs · page moves · block log · rights log · ANI search)

Time filed: 14:37, Friday, July 27, 2018 (UTC)

Automatic, Supervised, or Manual: Automatic (with frequent spot checks)

Programming language(s): AutoWikiBrowser

Source code available: AWB, replacement strings posted at User:Ahechtbot#Task 1

Function overview: Fixes the specific signatures with unclosed formatting tags listed at User:Ahechtbot#Task 1. These are now causing formatting issues on entire pages due to the Change from HTML Tidy to RemexHTML. Also fixes unclosed <s>...</s> tags where found.

Links to relevant discussions (where appropriate): Wikipedia:Village pump (technical)#Remex: My page is recently broken and I can't figure out why. Also, at Wikipedia:Bots/Requests for approval/Fluxbot 7, while there didn't seem to be consensus for blanket fixes for linter errors, there did not seem to be an issue with more targeted fixes.

Edit period(s): Initial one-time run on batch of pages determined by searching for the KoshVorlon, Ginsengbomb, Transhumanist, Tellyaddict, and Ralbot signatures. ~~Three~~ Four of these editors are not longer active, and the last has since changed their signature, so I don't anticipate new errors of this type being introduced. I may do additional runs if I find pages that were missed.

Estimated number of pages affected: ~~4000~~ 5000

Namespace(s): User, Wikipedia, and all ...talk namespaces

Exclusion compliant (Yes/No): Yes (via AWB)

Function details: Find and replace specific strings of text that are causing formatting problems on pages. These strings are listed at User:Ahechtbot#Task 1. The first 5 8 strings will be replaced whenever they are found, as they are actively causing formatting problems. The last 4 strings will be marked as "minor" replacements in AWB, and the "Skip if only minor replacements" option will be enabled. No "Automatic changes" (genfixes, etc.) will be enabled. All edits will be marked as "minor".

AWB permission is requested for User:Ahechtbot, as is the Bot flag (so that I can avoid triggering notifications when editing user pages). I have been running these replacements in small batches with supervision under my main account, and have not seen any bad edits made, but it got to the point where it was probably inappropriate to do too many more without a BRFA. Additional task requests will be created if new replacement strings are to be added.

Discussion

Important note for you Ahecht, probably best for all, but for any user_talk base pages, ensure that you flag edits as both BOT and MINOR to avoid triggering the "new messages" alert. — xaosflux ^Talk 15:11, 27 July 2018 (UTC)[reply]
Yes, that was the plan. I've clarified above. --Ahecht (TALK
PAGE) 19:00, 27 July 2018 (UTC)[reply]

Approved for trial (80 edits). Please provide a link to the relevant contributions and/or diffs when the trial is complete. (NO usertalk basepages). OK to trial, 10 of each of the 8 types, please provide results of the trial below. — xaosflux ^Talk 20:01, 27 July 2018 (UTC)[reply]

@Xaosflux: ~~The bot does not yet have AWB permissions. Should I do the test edits under my main account?~~ Nevermind, I was looking under bots, but it was listed under "Approved users". --Ahecht (TALK
PAGE) 00:10, 28 July 2018 (UTC)[reply]

Yes, you will need to manually press save during trials. — xaosflux ^Talk 00:35, 28 July 2018 (UTC)[reply]

Trial complete. 10 edits of each type are below. I ran it with only one replacement enabled at a time, so some pages still have signatures that would've been fixed if all the replacements were enabled.

KoshVorlon
Ginsengbomb (The first 2 had mislabeled edit summaries, but were actually for Ginsengbomb)
Ralbot
The Transhumanist (some signatures were missed -- I'll use a wider regex during the next BRFA)
Tellyaddict 1
Tellyaddict 2 (The first 5 had mislabeled edit summaries, but were actually for Tellyaddict 2)
Tellyaddict 3
Strikeout

--Ahecht (TALK
PAGE) 01:59, 28 July 2018 (UTC)[reply]

@Ahecht: the edits look OK, I don't love your edit summaries, they are a bit too busy, take a look at some I've used here: Special:Contributions/Fluxbot at least maybe use "do not use edit section summaries"? — xaosflux ^Talk 03:02, 28 July 2018 (UTC)[reply]

@Xaosflux: Sure, the summaries' extra information was really just for me tracking the trials. I was planning on using "Task 1: Fix LintErrors (missing end tag)". I can turn off section summaries. --Ahecht (TALK
PAGE) 14:02, 28 July 2018 (UTC)[reply]

@Ahecht: OK, I'd like to leave this on hold for about a week should there be any feedback. — xaosflux ^Talk 14:23, 28 July 2018 (UTC)[reply]

An (exceedingly) minor comment, but I believe there's an error on the bot's Task 1 page: the RalBot regex is not meant to include <span class="nowrap"> at the start, I don't think. ~ Amory (u • t • c) 10:27, 29 July 2018 (UTC)[reply]

Yes, that was a documentation error. The regex in AWB doesn't include that. --Ahecht (TALK
PAGE) 16:57, 29 July 2018 (UTC)[reply]

@Xaosflux: It's been about a week. Any chance this could be approved so that I could run the first batch today (4 August 2018)? I won't have any time for wikipedia from 8/5 through 8/11 and I'd like to run a supervised batch before that. --Ahecht (TALK
PAGE) 02:34, 4 August 2018 (UTC)[reply]

Approved. — xaosflux ^Talk 02:39, 4 August 2018 (UTC)[reply]

The above discussion is preserved as an archive of the debate. Please do not modify it. To request review of this BRFA, please start a new section at WT:BRFA.