We need to understand what changes, if any, are required in PageTriage given the IP Masking changes which are coming.
The scope of this investigation should be to understand what technical changes are needed and create tasks for each unit of work.
Findings
PageTriage sets preferences for temporary accounts
- backbone.js article model (used to present the toolbar and the new pages feed)
- direct checks for isAnon(): https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/extensions/PageTriage/+/refs/heads/master/modules/ext.pageTriage.util/models/ext.pageTriage.article.js
- Impact: This would throw errors
- onArticleViewFooter hook (used to present the toolbar)
- direct check for isRegistered(): https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/extensions/PageTriage/+/refs/heads/master/includes/Hooks.php
- Impact: Temporary accounts would make it past an early return check in the hook implementation. This would not break anything, but we lose the benefit of the optimization.
Mitigation
New page creation + Draft creation
- New page creation
- Currently, only registered users can create new pages on enwiki (the only production site using the extension that I am aware of), so I don't believe ip masking will have an meaningful immediate impact on the new pages feed queue.
- Draft creation
- Anonymous users can currently create drafts that populate the afc queue on enwiki (verified by scrolling through the afc queue on the new pages feed). Articles that currently show IP addresses as the creator would instead show temporary user accounts. Drafts from single IP address may spawn multiple temporary accounts over time.
Mitigation
Impacts to draft not look like an immediate problem, so no mitigation is required at this time. After IP masking rolls out, the community will asses the impact to their processes and may request implementation of a "reaveal ip" workflow in the PageTriage extension.