Page MenuHomePhabricator

Deprecate WVUI in favor of Codex
Closed, ResolvedPublic3 Estimated Story Points

Description

WVUI is currently in maintenance mode, see T290051. This means no new features are added, but changes are still made and new releases done when necessary to fix bugs. This still happens sometimes, because there is still code using WVUI.

We would like to deprecate WVUI, migrate all code that uses it to Codex, and then remove WVUI completely.

After that, we would migrate non-production code from WVUI to Codex, and remove Codex from MW core, which is captured in T310244: Remove WVUI from MediaWiki core.

All of these are also blocked on T302772: Application Security Review Request : Codex, as we can't use Codex in any production code until its security review is completed.

Related, but not a blocker, is T286835: Port RelatedArticles to Codex. There is a patch to port RelatedArticles to WVUI, but it was never merged. There is now an alternative patch to port RelatedArticles to Codex instead of WVUI. That patch can be merged once the other issues on T286835 are addressed (and once the security review is done).

Also look at https://codesearch.wmcloud.org/things/?q=wvui&i=nope&files=&excludeFiles=&repos=

Related Objects

Event Timeline

Can we get this task into a current or upcoming DST sprint?

Change 824250 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/core@master] Set intention for deprecating wvui library

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

Change 824250 merged by jenkins-bot:

[mediawiki/core@master] Deprecate use of wvui library

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

DAbad subscribed.

Marking this to review for integration into roadmap

Volker_E updated the task description. (Show Details)

A non-Wikimedia skin named Cosmos (repo) has been also making use of WVUI.

Change 831148 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/extensions/WikimediaEvents@master] [WIP] Replace WVUI by Codex

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

Change 831148 merged by jenkins-bot:

[mediawiki/extensions/WikimediaEvents@master] SearchSatisfaction: Remove extraArgs logging for WVUI

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

Change 837746 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/core@master] Deprecate use of 'wvui-search' package

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

Change 837747 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/extensions/DarkMode@master] Replace WVUI related code with Codex

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

With clarification on two latest patches, this task can be resolved successfully.

GlobalWatchlist can be migrated whenever needed in future, as WVUI code is not active. Filed two tasks for skins (Cosmos, Snap!WikiSkin) relying on 'wvui-search' package. Both skins haven't seen any activity though in last year.

ldelench_wmf set the point value for this task to 3.

Pushing to code review; seeking @Catrope's input on decision to move GlobalWatchlist migration out-of-scope for this task.

Change 837747 merged by jenkins-bot:

[mediawiki/extensions/DarkMode@master] Replace WVUI related code with Codex

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

Change 837746 merged by jenkins-bot:

[mediawiki/core@master] Deprecate use of 'wvui-search' package

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

@Catrope My plan would be to resolve this task now with the two additional patches merged, untangle the remove task T310244 from here and leave only that on our radar.

@Catrope My plan would be to resolve this task now with the two additional patches merged, untangle the remove task T310244 from here and leave only that on our radar.

Untangling done, T310244 is now its own task, and all the tasks about migrating non-production code have been moved to under T310244. This task is now done.

Change 838262 had a related patch set uploaded (by Anne Tomasevich; author: Anne Tomasevich):

[mediawiki/extensions/MediaSearch@master] Add comments to Icon component about WVUI and Codex

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

Change 838813 had a related patch set uploaded (by Jforrester; author: VolkerE):

[mediawiki/core@REL1_39] Deprecate use of 'wvui-search' package

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

Change 838813 merged by jenkins-bot:

[mediawiki/core@REL1_39] Deprecate use of 'wvui-search' package

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

Change 838262 merged by jenkins-bot:

[mediawiki/extensions/MediaSearch@master] Add comments to Icon component about WVUI and Codex

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

Catrope claimed this task.

For completion, volunteer snapwikiskin has also just replaced WVUI with Codex.