From 219ccd3f2ccd5dc695e63ba608d9d0c2f834f1c8 Mon Sep 17 00:00:00 2001 From: Till JS Date: Wed, 22 Apr 2026 17:33:50 +0200 Subject: [PATCH] chore(web): harden pnpm check with --fail-on-warnings MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit svelte-check emits a11y + dead-CSS + Svelte-5 $state warnings that were previously non-blocking — pre-push only caught hard type errors. The a11y-30 cleanup commit (3e09ff66d) brought the warning count to 0, so flipping `--fail-on-warnings` on now makes the checker hold the line: any new warning fails the pre-push hook that runs `pnpm check`. Covers: a11y_click_events_have_key_events, a11y_consider_explicit_label, css_unused_selector, state_referenced_locally, and the other svelte-check diagnostic categories. No behaviour change with current codebase (0 warnings); prevents drift going forward. Co-Authored-By: Claude Opus 4.7 (1M context) --- apps/mana/apps/web/package.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/mana/apps/web/package.json b/apps/mana/apps/web/package.json index 6a56633a1..4991aba10 100644 --- a/apps/mana/apps/web/package.json +++ b/apps/mana/apps/web/package.json @@ -7,7 +7,7 @@ "build": "vite build", "preview": "vite preview", "prepare": "svelte-kit sync || echo ''", - "check": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json", + "check": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json --fail-on-warnings", "check:watch": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json --watch", "format": "prettier --write .", "lint": "prettier --check . && NODE_OPTIONS=--max-old-space-size=8192 eslint .",