Skip to content

ci: Add GitHub Actions workflow gating PRs on typecheck + tests - #23

Merged
pdettori merged 2 commits into
mainfrom
ci/add-github-actions-20
Jun 29, 2026
Merged

pdettori merged 2 commits into
mainfrom
ci/add-github-actions-20

Conversation

@pdettori

Copy link
Copy Markdown
Member

Summary

  • Adds .github/workflows/ci.yml that runs on pull_request and push to main
  • Gates PRs on tsc --noEmit (typecheck) across all workspace packages with tsconfigs
  • Gates PRs on pnpm -r test (unit tests across all workspace packages)
  • Builds pi-fork submodule type declarations (required for linked @earendil-works/* deps)
  • Fixes a stale @ts-expect-error in packages/k8s-sandbox/test/m3-live-smoke.test.ts

CI Steps

  1. Checkout with submodules: recursive
  2. Setup Node 22 + pnpm 9
  3. Cache pnpm store + pi-fork node_modules
  4. cd pi-fork && npm ci && npm run build (produces .d.ts files for linked deps)
  5. pnpm install --frozen-lockfile
  6. Typecheck: tsc --noEmit in harness, k8s-sandbox, knative-server, experiments
  7. Test: pnpm -r test

Notes

  • pi-fork's dist/ is gitignored — CI must build it for type declarations
  • The work-queue and session-backend packages have no tsconfig yet; they're type-checked transitively via harness imports
  • Live/env-gated tests (E5, M3 smoke) remain skipped in CI by default

Closes #20

Assisted-By: Claude Code

Paolo Dettori added 2 commits June 29, 2026 16:09
- Checkout with submodules (pi-fork is a linked dep)
- Build pi-fork type declarations (tsgo via npm run build)
- pnpm install --frozen-lockfile for workspace
- tsc --noEmit across harness, k8s-sandbox, knative-server, experiments
- pnpm -r test for all workspace packages
- Cache pnpm store and pi-fork node_modules
- Fix stale @ts-expect-error in k8s-sandbox test

Closes #20

Signed-off-by: Paolo Dettori <paolo.dettori@example.com>
Signed-off-by: Paolo Dettori <paolo.dettori@example.com>
@pdettori
pdettori merged commit 7ca7748 into main Jun 29, 2026
2 checks passed
@pdettori
pdettori deleted the ci/add-github-actions-20 branch June 29, 2026 21:46
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.

ci: Add GitHub Actions workflow gating PRs on tsc --noEmit + tests

1 participant