fix(ci): run isolated tests separately to avoid mock.module() leakage

Bun's mock.module() leaks between test files in the same run. Keep
test:unit and test:isolated as separate invocations, add coverage
flags to test:isolated, and merge lcov reports before upload.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: betegon

.github/workflows/ci.yml

@@ -169,10 +169,17 @@ jobs:
         run: bun install --frozen-lockfile
       - name: Unit Tests
         run: bun run test:unit
+      - name: Save Unit Coverage
+        run: cp coverage/lcov.info coverage/unit-lcov.info
+      - name: Isolated Tests
+        run: bun run test:isolated
+      - name: Merge Coverage
+        run: cat coverage/unit-lcov.info coverage/lcov.info > coverage/merged-lcov.info
       - name: Upload Coverage
         uses: getsentry/codecov-action@main
         with:
           token: ${{ secrets.GITHUB_TOKEN }}
+          files: coverage/merged-lcov.info
 
   build-binary:
     name: Build Binary (${{ matrix.target }})

package.json

@@ -63,9 +63,9 @@
     "typecheck": "tsc --noEmit",
     "lint": "bunx ultracite check",
     "lint:fix": "bunx ultracite fix",
-    "test": "bun run test:unit",
-    "test:unit": "bun test test/lib test/commands test/types test/isolated --coverage --coverage-reporter=lcov",
-    "test:isolated": "bun test test/isolated",
+    "test": "bun run test:unit && bun run test:isolated",
+    "test:unit": "bun test test/lib test/commands test/types --coverage --coverage-reporter=lcov",
+    "test:isolated": "bun test test/isolated --coverage --coverage-reporter=lcov",
     "test:e2e": "bun test test/e2e",
     "test:init-eval": "bun test test/init-eval --timeout 600000 --concurrency 6",
     "generate:skill": "bun run script/generate-skill.ts",