ci: promote unit tests to a hard gate

108 deterministic pure-logic tests now block the build job (and thus deploy) on
failure, alongside the Build step. Moved out of the informational quality-checks
section and dropped continue-on-error.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
2026-06-30 13:17:20 -04:00
parent 4f03775e04
commit 1e37b20c6a
+4 -5
View File
@@ -47,14 +47,13 @@ jobs:
NODE_OPTIONS: '--max_old_space_size=4096'
VITE_APP_VERSION: ${{ github.sha }}
# ── Quality checks (informational — pre-existing issues exist) ───────
# Unit tests run on Node's built-in runner via tsx (no vitest — Vite 8 is
# ahead of vitest's supported range). Informational for now to match the
# section's convention; promote to a hard gate by dropping continue-on-error.
# Unit tests are a hard gate too — deterministic pure-logic tests on Node's
# built-in runner via tsx (no vitest — Vite 8 is ahead of vitest's range).
# A failure blocks the deploy.
- name: Unit tests
run: npm test
continue-on-error: true
# ── Quality checks (informational — pre-existing issues exist) ───────
- name: TypeScript
run: npm run typecheck
continue-on-error: true