Optimize CI: skip docs, merge build+e2e, improve turbo cache

Author: hotlong

.github/workflows/ci.yml

@@ -3,8 +3,20 @@ name: CI
 on:
   push:
     branches: [main, develop]
+    paths-ignore:
+      - '**/*.md'
+      - 'content/**'
+      - 'docs/**'
+      - 'apps/site/**'
+      - '.changeset/**'
   pull_request:
     branches: [main, develop]
+    paths-ignore:
+      - '**/*.md'
+      - 'content/**'
+      - 'docs/**'
+      - 'apps/site/**'
+      - '.changeset/**'
 
 concurrency:
   group: ci-${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
@@ -44,28 +56,43 @@ jobs:
           node-version: '20.x'
           cache: 'pnpm'
 
+      # Per-branch turbo cache. PRs restore from base branch via restore-keys
+      # for maximum cache hit rate; only changed packages re-run.
       - name: Turbo Cache
         uses: actions/cache@v5
         with:
           path: node_modules/.cache/turbo
-          key: turbo-${{ runner.os }}-${{ github.sha }}
+          key: turbo-${{ runner.os }}-test-${{ github.sha }}
           restore-keys: |
+            turbo-${{ runner.os }}-test-
             turbo-${{ runner.os }}-
 
       - name: Install dependencies
         run: pnpm install --frozen-lockfile
 
-      - name: Run tests with coverage
+      # PRs: skip coverage instrumentation (v8 adds 40-100% overhead) and
+      # leverage turbo per-package cache so unchanged packages are skipped.
+      - name: Run tests (PR — fast, no coverage)
+        if: github.event_name == 'pull_request'
+        run: pnpm turbo run test --concurrency=4
+
+      # Push to main/develop: still run coverage so Codecov stays current.
+      - name: Run tests with coverage (push)
+        if: github.event_name == 'push'
         run: pnpm test:coverage --coverage.reporter=json --coverage.reporter=text
 
       - name: Upload coverage to Codecov
+        if: github.event_name == 'push'
         uses: codecov/codecov-action@v6
         with:
           fail_ci_if_error: false
           token: ${{ secrets.CODECOV_TOKEN }}
 
-  build:
-    name: Build Core
+  # Build + E2E merged into one job: avoids artifact upload/download (~30s)
+  # and a duplicate `pnpm install` (~60s). Turbo cache makes the build
+  # essentially free on subsequent runs.
+  e2e:
+    name: Build & E2E
     runs-on: ubuntu-latest
 
     steps:
@@ -90,8 +117,9 @@ jobs:
         uses: actions/cache@v5
         with:
           path: node_modules/.cache/turbo
-          key: turbo-${{ runner.os }}-${{ github.sha }}
+          key: turbo-${{ runner.os }}-build-${{ github.sha }}
           restore-keys: |
+            turbo-${{ runner.os }}-build-
             turbo-${{ runner.os }}-
 
       - name: Install dependencies
@@ -102,64 +130,14 @@ jobs:
           pnpm turbo run build --filter='./packages/*'
           pnpm --filter @object-ui/console build
 
-      - name: Check for build artifacts
+      - name: Verify build artifacts
         run: |
           if [ ! -d "packages/core/dist" ]; then
             echo "Protocol package build failed"
             exit 1
           fi
           echo "All packages built successfully"
 
-      - name: Upload build artifacts
-        uses: actions/upload-artifact@v7
-        with:
-          name: build-output
-          path: |
-            packages/*/dist
-            apps/console/dist
-          retention-days: 1
-
-  e2e:
-    name: E2E Tests
-    runs-on: ubuntu-latest
-    needs: build
-    permissions:
-      contents: read
-
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v6
-        with:
-          submodules: true
-
-      - name: Enable Corepack
-        run: corepack enable
-
-      - name: Verify pnpm version
-        run: pnpm --version
-
-      - name: Setup Node.js
-        uses: actions/setup-node@v6
-        with:
-          node-version: '20.x'
-          cache: 'pnpm'
-
-      - name: Turbo Cache
-        uses: actions/cache@v5
-        with:
-          path: node_modules/.cache/turbo
-          key: turbo-${{ runner.os }}-${{ github.sha }}
-          restore-keys: |
-            turbo-${{ runner.os }}-
-
-      - name: Install dependencies
-        run: pnpm install --frozen-lockfile
-
-      - name: Download build artifacts
-        uses: actions/download-artifact@v8.0.1
-        with:
-          name: build-output
-
       - name: Get Playwright version
         id: playwright-version
         run: echo "version=$(pnpm list @playwright/test --depth=0 --json | jq -r '.[0].devDependencies["@playwright/test"].version')" >> $GITHUB_OUTPUT
@@ -184,7 +162,7 @@ jobs:
 
       - name: Upload Playwright report
         uses: actions/upload-artifact@v7
-        if: ${{ !cancelled() }}
+        if: ${{ !cancelled() && failure() }}
         with:
           name: playwright-report
           path: playwright-report/
@@ -240,8 +218,9 @@ jobs:
         uses: actions/cache@v5
         with:
           path: node_modules/.cache/turbo
-          key: turbo-${{ runner.os }}-${{ github.sha }}
+          key: turbo-${{ runner.os }}-docs-${{ github.sha }}
           restore-keys: |
+            turbo-${{ runner.os }}-docs-
             turbo-${{ runner.os }}-
 
       - name: Install dependencies

turbo.json

@@ -10,7 +10,22 @@
     "test": {
       "dependsOn": ["^build"],
       "outputs": ["coverage/**"],
-      "cache": true
+      "cache": true,
+      "inputs": [
+        "src/**",
+        "test/**",
+        "tests/**",
+        "vitest.config.*",
+        "vitest.setup.*",
+        "tsconfig*.json",
+        "package.json",
+        "$TURBO_DEFAULT$",
+        "!**/*.md",
+        "!**/CHANGELOG.md",
+        "!storybook-static/**",
+        "!.storybook/**",
+        "!**/*.stories.*"
+      ]
     },
     "test:watch": {
       "cache": false,