chore: split types:build and build scripts for speedier deploys. Make typechecking and prettier not mandatory for deploys.

Author: geclos

.github/workflows/docker-build.yml .github/workflows/build-and-push.yml.github/workflows/docker-build.yml renamed to .github/workflows/build-and-push.yml

@@ -64,7 +64,7 @@ jobs:
               - 'packages/**'
 
   build-and-push:
-    needs: [check-changes]
+    needs: [lint, check-changes]
     runs-on: ubuntu-latest
     permissions:
       contents: read

.github/workflows/lint.yml

@@ -1,4 +1,4 @@
-name: Linter & Types
+name: Linter
 
 on:
   pull_request:
@@ -51,15 +51,9 @@ jobs:
       - name: Use Turbo to build affected packages
         run: pnpm turbo build --cache-dir=.turbo --filter="./packages/**"
 
-      - name: Prettier
-        run: pnpm prettier:check
-
       - name: Node.js Lint
         run: pnpm lint
 
-      - name: TypeScript
-        run: pnpm tc --cache-dir=.turbo
-
   python-lint:
     runs-on: ubuntu-latest
 

.github/workflows/typecheck.yml

@@ -0,0 +1,58 @@
+name: Typecheck
+
+on:
+  pull_request:
+    branches:
+      - main
+  workflow_call:
+    inputs:
+      workflow_call:
+        type: boolean
+        required: false
+        default: false
+
+jobs:
+  node-lint:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v3
+        with:
+          fetch-depth: 2
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v3
+        with:
+          node-version: 22.x
+
+      - name: Install pnpm
+        uses: pnpm/action-setup@v3
+        with:
+          version: 9
+          run_install: false
+
+      - name: Get pnpm store directory
+        shell: bash
+        run: |
+          echo "STORE_PATH=$(pnpm store path --silent)" >> $GITHUB_ENV
+
+      - name: Setup pnpm cache
+        uses: actions/cache@v3
+        with:
+          path: ${{ env.STORE_PATH }}
+          key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }}
+          restore-keys: |
+            ${{ runner.os }}-pnpm-store-
+
+      - name: Install Node.js dependencies
+        run: pnpm install
+
+      - name: Use Turbo to build affected packages
+        run: pnpm turbo build --cache-dir=.turbo --filter="./packages/**"
+
+      - name: Build package types
+        run: pnpm turbo types:build --cache-dir=.turbo --filter="./packages/**"
+
+      - name: Typecheck
+        run: pnpm tc

.tmuxinator.yml

@@ -5,12 +5,12 @@ windows:
       layout: main-horizontal
       panes:
         - pnpm dev --filter='./apps/*'
-        - cd packages/core && pnpm build:watch
-        - cd packages/constants && pnpm build:watch
+        - cd packages/core && pnpm types:build:watch
+        - cd packages/constants && pnpm types:build:watch
   - packages:
       layout: main-horizontal
       panes:
-        - pnpm build --filter='./packages/sdks/typescript' --filter='./packages/telemetry/typescript' --filter='./packages/constants' --filter='./packages/web-ui'
+        - pnpm build --filter='./packages/**'
         - cd apps/web && pnpm workers:build
   - docker: docker compose up db redis mailpit --menu=false
   - studio: cd packages/core && pnpm db:studio

packages/constants/package.json

@@ -7,8 +7,8 @@
   "description": "Common constants without any dependencies",
   "type": "module",
   "scripts": {
-    "build": "tsc -p tsconfig.build.json",
-    "build:watch": "tsc --watch -p tsconfig.build.json",
+    "types:build": "tsc -p tsconfig.build.json",
+    "types:build:watch": "tsc --watch -p tsconfig.build.json",
     "lint": "eslint src",
     "tc": "tsc --noEmit"
   },

packages/core/package.json

@@ -114,8 +114,8 @@
     }
   },
   "scripts": {
-    "build": "tsc -p tsconfig.build.json",
-    "build:watch": "tsc --watch -p tsconfig.build.json",
+    "types:build": "tsc -p tsconfig.build.json",
+    "types:build:watch": "tsc --watch -p tsconfig.build.json",
     "db:generate": "drizzle-kit generate",
     "db:generate:custom": "pnpm run db:generate --custom",
     "db:migrate": "drizzle-kit migrate --config=./drizzle.config.ts",

packages/web-ui/package.json

@@ -4,7 +4,7 @@
   "description": "Latitude web ui elements",
   "private": true,
   "scripts": {
-    "build": "tsc -p tsconfig.build.json",
+    "types:build": "tsc -p tsconfig.build.json",
     "lint": "eslint ./src",
     "tc": "tsc --noEmit",
     "test": "vitest run",