Merge ci/required-checks -> main #68

Closed

Mark wants to merge 3 commits from ci/required-checks into main

Mark commented

2026-09-12 18:02:12 +00:00

Owner

Auto-opened batch merger.

Mark added 3 commits

2026-09-12 18:02:13 +00:00

ci: fix broken workflow YAML, add unit tests to required checks

CI / quality (pull_request) Failing after 1m2s

Details

CI / e2e (pull_request) Has been skipped

Details

9d1e16f205

- Repair mis-indented Type check step that made ci.yml invalid YAML
  (every Actions run was failing at parse time)
- Add Unit tests step (vitest) so lint, typecheck, unit + integration
  tests all gate the quality job
- npm ci is the frozen-lockfile install for this repo (package-lock.json);
  pnpm lockfile does not exist

test: add vitest devDependency so unit tests and typecheck can run in CI

CI / quality (pull_request) Successful in 1m21s

Details

CI / e2e (pull_request) Failing after 49s

Details

93e51b11d0

The test suite imports from 'vitest' but the package was never declared,
so 'npm run typecheck' failed (TS2307) and 'npm run test:unit' could not
run at all. Adds vitest ^4.1.10.

ci(e2e): reach postgres service via localhost on host-networked runner

CI / quality (pull_request) Successful in 1m21s

Details

CI / e2e (pull_request) Failing after 51s

Details

51ac43fedd

The runner runs job containers with container.network: host, so the
service hostname 'postgres' does not resolve and prisma db push fails
with P1001. The postgres service container binds the host network, so
localhost:5432 reaches it. Also fixes the redacted password to match
the service definition (test).