fix(ci): Fix stale PCH module cache error after mlx-server→SwiftLM rename

solderzzc · solderzzc · commit ce626c1549e8 · 2026-03-30T12:46:28.000-07:00
The CI cache was built when the repo was named 'mlx-server'. After renaming
to 'SwiftLM', clang embedded the old path in the PCH, causing:

  error: PCH was compiled with module cache path '.../mlx-server/.build/...'
  but the path is currently '.../SwiftLM/.build/...'

Fixes:
- build.yml, e2e-test.yml: scope cache key to 'spm-SwiftLM-' so the old
  'spm-' prefixed cache is never restored as a partial match
- Add 'Clear stale module cache' step (rm ModuleCache/) before swift build
  to eliminate any stale PCH artifacts that sneak in via partial cache hits
- tests/test-server.sh: update comment and default binary path to SwiftLM
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -21,13 +21,20 @@ jobs:
         uses: actions/cache@v4
         with:
           path: .build
-          key: ${{ runner.os }}-spm-${{ hashFiles('Package.resolved') }}
+          # Key includes product name so any rename (e.g. mlx-server→SwiftLM)
+          # automatically busts the cache and prevents stale PCH errors.
+          key: ${{ runner.os }}-spm-SwiftLM-${{ hashFiles('Package.resolved') }}
           restore-keys: |
-            ${{ runner.os }}-spm-
+            ${{ runner.os }}-spm-SwiftLM-
 
       - name: Resolve dependencies
         run: swift package resolve
 
+      - name: Clear stale module cache
+        # Prevents: "PCH was compiled with module cache path '…mlx-server…'
+        # but the path is currently '…SwiftLM…'" after repo rename.
+        run: find .build -type d -name ModuleCache -exec rm -rf {} + 2>/dev/null || true
+
       - name: Build (Release)
         run: swift build -c release
 
diff --git a/.github/workflows/e2e-test.yml b/.github/workflows/e2e-test.yml
@@ -24,9 +24,14 @@ jobs:
         uses: actions/cache@v4
         with:
           path: .build
-          key: ${{ runner.os }}-spm-${{ hashFiles('Package.resolved') }}
+          key: ${{ runner.os }}-spm-SwiftLM-${{ hashFiles('Package.resolved') }}
           restore-keys: |
-            ${{ runner.os }}-spm-
+            ${{ runner.os }}-spm-SwiftLM-
+
+      - name: Clear stale module cache
+        # Prevents: "PCH was compiled with module cache path '…mlx-server…'
+        # but the path is currently '…SwiftLM…'" after repo rename.
+        run: find .build -type d -name ModuleCache -exec rm -rf {} + 2>/dev/null || true
 
       - name: Build (Release)
         run: swift build -c release
diff --git a/tests/test-server.sh b/tests/test-server.sh
@@ -1,5 +1,5 @@
 #!/bin/bash
-# test-server.sh — Integration tests for mlx-server
+# test-server.sh — Integration tests for SwiftLM
 #
 # Usage:
 #   ./tests/test-server.sh [binary_path] [port]
@@ -9,7 +9,7 @@
 
 set -euo pipefail
 
-BINARY="${1:-.build/release/mlx-server}"
+BINARY="${1:-.build/release/SwiftLM}"
 PORT="${2:-15413}"
 HOST="127.0.0.1"
 MODEL="mlx-community/Qwen2.5-0.5B-Instruct-4bit"  # Smallest model for CI
