fix(lint): let prettier have the last word after eslint --fix

[rubenvdlinde]

Closes #739.

Running the repo's own lint fixer produced a file the repo's own format check
rejected — so fixing lint the documented way landed a red Frontend Check (format).

It is not a rule disagreement

eslint-config-prettier is already applied last in eslint.config.mjs, and it
cannot prevent this. That package only turns rules off, and the culprit is
not a formatting rule.

import-extensions/ban-inline-type-imports rewrites an import, and its
autofixer emits its own text:

import type {Page} from '@playwright/test';    // what eslint --fix writes
import type { Page } from '@playwright/test'   // what prettier demands

Any autofixer that constructs source can do this. Disabling formatting rules
does not stop it, because the rule doing the damage is not one of them.

The fix

lint-fix now re-runs the formatter afterwards:

"lint-fix": "npm run lint -- --fix && npm run format:fix"

That makes the two tools ordered rather than competing: eslint decides what
the code says, prettier decides how it looks, in that order, always. It is
also what the config's own doctrine already says — "exactly one of them is
allowed an opinion and prettier is it".

Verified both directions on a clean tree:

after eslint --fix alone:   prettier REJECTS   (the bug)
after chaining format:fix:  prettier OK        (fixed)

Why this went unnoticed — worth knowing

The two tools do not cover the same files:

script	scope
lint	eslint src — src only
format	prettier "**/*.{js,ts,vue,css,scss}" — everything, including tests/

tests/e2e/** is format-checked but never linted, so an eslint --fix run
there is outside the workflow CI exercises — which is exactly where I hit this.

Whether to widen eslint's scope to tests/ is a genuine decision with a findings
backlog behind it, so it stays in #739 rather than riding along in a one-line
script fix.

[github-actions]

Quality Report — ConductionNL/stackiq @ 392cee2

Check	PHP	Vue	Security	License	Tests
lint	✅
phpcs	✅
phpmd	✅
psalm	✅
phpstan	✅
phpmetrics	✅
eslint		✅
stylelint		✅
build		✅
check-manifest		✅
check-vue-demi		✅
test-l10n		✅
format		✅
check-schema-l10n		✅
composer			✅	✅ 130/130
npm			✅	✅ 720/720
app:check-code	⏭️
info.xml	✅
REUSE	❌
PHPUnit					✅
Newman					⏭️
Playwright					✅
Hydra gates					✅

---

Quality workflow — 2026-08-24 22:52 UTC

Download the full PDF report from the workflow artifacts.