fix: split CI into quality + test jobs to fix format check

Root cause: the test matrix removes pnpm-lock.yaml so different PMs
can install deps. Without the lockfile, vite-plus resolves to a newer
version with different oxfmt rules, causing vp check to fail on
package.json formatting.

Fix: quality job (format, lint, typecheck) runs once with the lockfile
intact for deterministic deps. Test job runs the full OS/runtime/PM
matrix without format checking.

Author: thejackshelton

.github/workflows/ci.yml

@@ -11,6 +11,28 @@ concurrency:
   cancel-in-progress: ${{ github.event_name == 'pull_request' }}
 
 jobs:
+  quality:
+    name: Quality checks
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: pnpm/action-setup@v4
+
+      - uses: voidzero-dev/setup-vp@v1
+        with:
+          node-version: "24"
+          cache: true
+
+      - name: Format check
+        run: vp check
+
+      - name: Lint
+        run: vp lint
+
+      - name: Type check
+        run: npx tsc --noEmit
+
   test:
     name: Test on ${{ matrix.runtime }} / ${{ matrix.os }} / ${{ matrix.pm }}
     runs-on: ${{ matrix.os }}
@@ -43,12 +65,6 @@ jobs:
           node-version: "24"
           cache: true
 
-      - name: Lint
-        run: vp lint
-
-      - name: Type check
-        run: npx tsc --noEmit
-
       - name: Build
         run: vp pack
 

package.json

@@ -53,5 +53,6 @@
   },
   "engines": {
     "node": ">=24.0.0"
-  }
+  },
+  "packageManager": "pnpm@10.25.0"
 }