[lens0021]

Timeless has little future, so the docs site stops carrying it. What is left is two skins: Vector from default.yml (the main skin, at the export root) and Citizen from docs/.wikven.yml (under citizen/). The main skin is unchanged — WikvenSettings.php takes it from the head of the merged list, which is still Vector.

What changed

File	Why
docs/.wikven.yml	The enablement itself.
.github/workflows/smoke.yml	The language-bar total is counted once per skin rendered, so the multiplier follows the skin count: * 3 → * 2, comment reworded.
tests/e2e/skin-switcher.spec.js	The three surviving tests drive Citizen instead of Timeless. The fourth, "switching between two non-main skins", is deleted — see below.
includes/Hooks/Adder.php	Dropped the getSkinName() === 'timeless' inline style that hid #user-tools/#page-tools on a cli export.
docs/Skins.wikitext	Removed the Timeless bullet from the bundled-skins list (unit T:4), and the skins: example now says MonoBook.
docs/Configuration.wikitext	The .wikven.yaml example's skins: entry now says MonoBook.
docs/Skins/ko.wikitext	Same bullet removed; T:4 and T:6 restamped.
docs/Configuration/ko.wikitext	T:3 restamped.

Translation stamps

Both YAML examples sit inside a <syntaxhighlight> block that falls between two <!--T:n--> markers, so StalenessComputer::splitUnits counts it as part of the preceding unit's body — editing the example changes that unit's hash even though no prose moved. That is why Skins T:6 and Configuration T:3 restamped alongside the real prose change in Skins T:4.

Restamping is StalenessComputer::restamp, exactly what maintenance/stampTranslations.php calls, with the same $pageTitle (Skins / Configuration) that TranslationSource::translatableTitle derives. Both pages were fully ok before this change, so nothing pre-existing was marked fresh, and both are fully ok after.

Two calls worth a second look

The deleted e2e test. "Switching between two non-main skins on a Korean page" covered a skin-directory to skin-directory hop at equal depth. With one non-main skin left, that path is unreachable from the docs site, so the test goes rather than being faked. The main↔non-main hops in both directions, and both page depths, are still covered.

The Adder.php fixup. It is generic wikven code, not docs-site code: a downstream site that enables Timeless itself loses the fix. It was only ever exercised by this repo's own bake, though, so keeping it would mean carrying an untested Timeless-shaped branch — which is the thing being dropped. Easy to restore if you would rather keep it.

Also left alone deliberately: the docs/.wikven.yml comment ending "The other skins keep working SifterSearch search" now describes exactly one other skin, but it is Citizen-adjacent prose and reads fine either way.

Verified

• uv run --frozen --group lint yamllint --strict ., biome ci (clean; the one warning is the pre-existing !important in docs/MediaWiki:Citizen.css), typos, rumdl check ., composer test (minus-x), php -l on Adder.php — all pass.
• Translation staleness checked before and after with StalenessComputer directly, per above.
• The smoke workflow's expected loop replayed against docs/: 108 language-bar entries per skin, so 216 total where it was 324.

Not verified

The bake did not run here. docker build fails resolving the mediawiki:1.46-fpm-alpine base — the registry CDN returns 403 in this environment — so no dist tree was produced and the browser tests were never run against one. The smoke workflow's own bake is the first real check of both the 216 figure and the rewritten switcher tests. mago format --check / mago lint and composer phpcs also could not run (mago is not installed; the local phpcs install is missing its PHPCSExtra sniffs).

---

Generated by Claude Code

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[lens0021]

Two changes after review.

The includes/Hooks/Adder.php deletion is reverted. Not listing Timeless on the docs site says nothing about sites that enable it themselves, and includes/ ships to them — removing the inline rule would have left their exports showing the dead personal-tools dropdown and page-actions sidebar it exists to hide. Unsupported means untested, not deliberately broken, which is the same reading #346 keeps its Timeless CSS rule on.

It also made the stack incoherent: this PR deleted those seven lines and #352 restored them, three PRs later.

Retitled feat: → docs:. With includes/ out of the diff, this touches only docs/, .github/ and tests/ — nothing that reaches the distribution.

Knock-on for #352

#352 currently restores the same seven lines. That hunk is now redundant and will conflict; it should be dropped from that branch. The rest of #352 — declaring Vector, Minerva and Citizen supported — is unaffected.

---

Generated by Claude Code