In T248494#6272976, @gerritbot wrote:

Change 588837 merged by jenkins-bot:
[mediawiki/core@master] Add watchlist expiry to edit form

https://gerrit.wikimedia.org/r/c/mediawiki/core/ /588837

This change introduces a new top-level bundle entry point for what appears to be a small amount of code to simply load on an existing experience (the edit screen). This should be added to an existing bundle instead.

See also:

• https://www.mediawiki.org/wiki/Wikimedia_Performance_Team/Page_load_performance#Size_of_scripts
• https://www.mediawiki.org/wiki/ResourceLoader/Developing_with_ResourceLoader#Introduction

Change 649463 had a related patch set uploaded (by Krinkle; owner: Krinkle):
[mediawiki/core@master] mediawiki.action.edit: Move module files to their own subdirectory

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

In T250215#6316488, @gerritbot wrote:

Change 597145 merged by jenkins-bot:
[mediawiki/core@master] Display remaining watch period in watchstar

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

This commit introduces a call to mw.msg() where a message parameter is set to null which is invalid both as regular paramter, but also invalid as plural value (which expects integer). This makes the code difficult to understand, and seems to only work by accident (tech debt) where it is never used in the message passed from alternate conditions.

I suggest either using a real integer as default and thus trust your own code not to use it (instead of the unsupported reliance for mw.msg to never step on it), or to structure the code to acknowledge this split and call the function differently based on the feature being enabled or not.

	daysLeftExpiry = null,
…
			.attr( 'title', mw.msg( 'tooltip-ca-' + tooltipAction, daysLeftExpiry ) )

… … …

	function updateWatchLink( $link, action, state, expiry ) {

		if ( expiry === undefined ) {
			expiry = null;
		}
		…
	}

	updateWatchLink( $link, action, 'loading', null );

As a general principle, we rarely if ever use the null in our JavaScript codebases, so that can be a good indicator for something that stands out as odd.

Gerrit:

rMW9074c0f0715b: Preload mediawiki.watchstar.widgets before api request
https://gerrit.wikimedia.org/r/c/mediawiki/core/+/623063

This introduces a second HTTP request for preloading instead of adding to the existing one.

Note that mw.loader.load() accepts an array. I suggest using that, e.g. if … load [ ,, ] else load [ , ] .

Gerrit:

rMW6224ffd03ccf: Replace toast with OOUI popup on watchlist message
https://gerrit.wikimedia.org/r/c/mediawiki/core/+/591159/18/resources/src/mediawiki.page.watch.ajax.js

Minor nit pick: There seems to be a lazy-load pattern here around watchlistPopup !=| new WatchlistExpiryWidget but without any use of it. Possibly a left-over from a previous approach that was abandoned?

Change 649463 merged by jenkins-bot:
[mediawiki/core@master] mediawiki.action.edit: Move module files to their own subdirectory

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

Change 654300 had a related patch set uploaded (by MusikAnimal; owner: MusikAnimal):
[mediawiki/core@master] mediawiki.page.watch.ajax: consolidate loading of modules

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

Change 654300 merged by jenkins-bot:
[mediawiki/core@master] mediawiki.page.watch.ajax: consolidate loading of modules

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