Give CI the node packages its web hooks have always needed
Some checks failed
dependency freshness / dependencies on newest stable (push) Has been cancelled
md-naming / markdown naming convention (push) Has been cancelled
pre-commit / pre-commit (push) Has been cancelled
Tests / test (push) Has been cancelled
Tests / lock-popup-safety (push) Has been cancelled
release-apk / release (APK) (push) Has been cancelled

The pre-commit workflow has been red since 8f91158 added the status UI: it
installs Python and Flutter deps but never node ones, so `tsc --build` could
not find the 'vite/client' and 'node' type libraries and failed TS2688 on
every run. Nothing was wrong with the code; the job simply could not build it.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BJUtseRczWVBJW8LvL1aQH
This commit is contained in:
Krzysztof Rudnicki 2026-08-30 15:32:51 +02:00
parent 1d5965b9a7
commit d35097f461

View File

@ -34,6 +34,19 @@ jobs:
flutter-version: 3.47.2
channel: stable
cache: true
# The web-lint / web-typecheck / web-coverage hooks run npm inside web/.
# Without its installed packages `tsc --build` cannot find the
# 'vite/client' and 'node' type libraries and fails with TS2688 -- which
# is what this workflow had been doing on every run since the status UI
# landed in 8f91158.
- uses: actions/setup-node@v4
with:
node-version: "24.20.0"
cache: npm
cache-dependency-path: web/package-lock.json
- name: Install status-UI packages
run: npm ci
working-directory: web
- name: Install dependencies
run: pip install -r requirements.txt
# analyze needs resolved packages (.dart_tool); the hook runs analyze