fix(frontend): make the unit suite and stylelint able to run at all

[rubenvdlinde]

Supersedes #424 and #425, which deadlocked: each failed exactly the check the other fixes, so neither could ever go green alone.

#424 (hoist @vue/server-renderer)  ->  fails  Vue Quality (stylelint)
#425 (stylelint config)            ->  fails  Frontend Tests (unit)

They are two independent defects that happen to hide each other, so they have to land together.

1. The unit suite could not run at all

All 63 test files failed with zero tests executed:

Error: Cannot find package '@vue/server-renderer' imported from
       node_modules/@vue/test-utils/dist/vue-test-utils.cjs.js

Nothing was missing from the lockfile — it was installed nested at node_modules/vue/node_modules/@vue/server-renderer. @vue/test-utils declares it as a peer, and a peer resolves upward from the importing package's directory, so it looked for node_modules/@vue/server-renderer and found nothing. Declaring it directly pins it at the top level, which is where a peer has to be.

2. stylelint could not run at all

Exit 78 — a configuration failure, not a lint finding:

Could not find "stylelint-config-recommended".

stylelint.config.js extended stylelint-config-recommended-vue, which was never declared in package.json; the config it extends was not installed at all. launchpad already declares @nextcloud/stylelint-config, which is what openregister, opencatalogi, dossiq and shillinq extend — so this points it at the package the app declares and the fleet uses.

That made stylelint run, exposing 44 real violations it had been hiding. 42 were auto-fixable. The last two were a genuine CSS bug:

background-color: #ffffff !important;
background-image: none    !important;
background:       #ffffff !important;   /* silently discards both */

Keeping only the shorthand preserves the computed result exactly — keeping the longhands would not have, since the shorthand also resets the other background sub-properties.

3. …and then the two fixers fought

stylelint's indentation rule wanted 0 tabs on six wrapped selector lines; Prettier wanted them indented. Running either fixer broke the other's check. Both indentation and string-quotes are deprecated in stylelint 15 precisely because formatters do this better, so they are off and Prettier owns whitespace — leaving stylelint to judge what only it can judge.

Verification (all three together, on this branch)

stylelint   exit 0
prettier    exit 0
vitest      63 files passed, 682 tests passed

[github-actions]

Quality Report — ConductionNL/launchpad @ 92c3af0

Check	PHP	Vue	Security	License	Tests
lint	✅
phpcs	✅
phpmd	✅
psalm	✅
phpstan	✅
phpmetrics	✅
eslint		✅
stylelint		✅
build		✅
check-manifest		✅
format		✅
check-schema-l10n		✅
composer			✅	✅ 104/104
npm			✅	✅ 544/544
app:check-code	⏭️
info.xml	✅
REUSE	❌
PHPUnit					✅
Newman					✅
Playwright					🚨 NO VERDICT — enabled but never ran
Hydra gates					✅

---

Quality workflow — 2026-08-30 10:44 UTC

Download the full PDF report from the workflow artifacts.