Skip to content

fix: restore CI and turbo build pipeline - #26

Merged
sarthakagrawal927 merged 3 commits into
mainfrom
wip/opt-snapshot
Jun 20, 2026
Merged

sarthakagrawal927 merged 3 commits into
mainfrom
wip/opt-snapshot

Conversation

@sarthakagrawal927

@sarthakagrawal927 sarthakagrawal927 commented Jun 20, 2026 •

Copy link
Copy Markdown
Member

Summary

Split from the former WIP snapshot (#26). This PR restores a green CI/build pipeline on main and removes the orphaned email block that blocked API typecheck.

  • Add missing turbo.json (build/lint/typecheck were migrated to turbo without it)
  • Fix CI pnpm setup: drop conflicting version: 10 so packageManager in package.json wins
  • Remove dead build:email CI steps (no root script; package deleted)
  • Restore root vitest run for tests (320 tests; per-package turbo test was broken)
  • Simplify cockpit build — turbo ^build handles shared-types/ui/sdk deps
  • Delete @saas-maker/email block; inline API telemetry in workers/api/src/lib/telemetry.ts
  • Drop feedback/waitlist owner email notifications (were fire-and-forget Resend sends)

Remaining slices (separate PRs)

  • RAG service integration — feat/rag-service-integration (from backup wip/opt-snapshot-full)
  • Astro landing tooling — packages/tooling/astro-landing/

Full WIP snapshot preserved at branch wip/opt-snapshot-full and ref backup/wip-opt-snapshot-6318592.

Test plan

  • pnpm test — 320 passed
  • pnpm check:openapi
  • pnpm build:types && pnpm build:db && pnpm build:widget
  • pnpm -F @saas-maker/api typecheck
  • pnpm build:cockpit
  • CI green on PR

Main was broken after the partial turbo migration: missing turbo.json,
pnpm version conflict in CI, dead build:email steps, and a test script
that ran broken per-package turbo tests instead of the root vitest suite.

Also removes the orphaned @saas-maker/email block and inlines API
telemetry so the worker no longer depends on email/ops workspace packages
for typecheck and deploy.

Co-authored-by: Cursor <cursoragent@cursor.com>
@sarthakagrawal927 sarthakagrawal927 changed the title wip: optimization snapshot (INCOMPLETE — mixed with WIP) fix: restore CI and turbo build pipeline Jun 20, 2026
sarthakagrawal927 and others added 2 commits June 20, 2026 15:20
`next build` alone misses @saas-maker/ui and @saas-maker/sdk when CI
runs sequential build:* steps instead of a full turbo graph.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.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