Skip to content

ci: unit+build job and integration job (e2e vs production server)#11

Merged
snk-js merged 1 commit into
feat/game-reconnectfrom
feat/ci
Jul 7, 2026
Merged

ci: unit+build job and integration job (e2e vs production server)#11
snk-js merged 1 commit into
feat/game-reconnectfrom
feat/ci

Conversation

@snk-js

@snk-js snk-js commented Jul 6, 2026

Copy link
Copy Markdown
Owner

Stacked on #10 (needs the in-repo e2e/ suite). Implements docs/07 stages 1–2.

Jobs

  • unit: pnpm install (frozen lockfile) → prisma generatevitest (54 tests) → pnpm build
  • integration: postgres:16 service container → migrate deploy → build → node server.js (with ORIGIN) → healthcheck → pnpm e2e — the auth/rooms/match suites over real HTTP + WebSocket against the production server

Also: the auth e2e Secure-cookie assertion is now mode-aware (E2E_SECURE=1 in CI, since the production build sets Secure), and the legacy gh-pages workflow (pre-auth static demo) is retired.

Verified

Rehearsed the integration job's exact steps locally: production build → node server.js → healthcheck → full e2e suite, all green. The workflow runs for real on this PR.

🤖 Generated with Claude Code

…rver)

Implements docs/07 stages 1-2 as one workflow:
- unit job: pnpm install (frozen lockfile), prisma generate, vitest, build
- integration job: postgres:16 service container -> migrate deploy ->
  pnpm build -> node server.js (ORIGIN set) -> healthcheck -> pnpm e2e
  (auth + rooms + match over real HTTP/WS)
- auth e2e: the Secure cookie assertion is now mode-aware (E2E_SECURE=1
  for production servers, which set the attribute; scripts replay the
  cookie header so both modes work)
- retire the legacy gh-pages workflow (pre-auth static demo)

Rehearsed locally with the exact integration steps: build, node
server.js, healthcheck, full e2e suite — all green.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@snk-js snk-js merged commit d0eb997 into feat/game-reconnect Jul 7, 2026
3 checks passed
snk-js added a commit that referenced this pull request Jul 7, 2026
PR #11 was merged into its stacked base (feat/game-reconnect) after that
branch had already been merged to master via #10, leaving the approved CI
changes stranded. This merge brings them home unchanged.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant