chore: migrate from nyc to c8 and enhance coverage reporting

.c8rc.json

@@ -0,0 +1,27 @@
+{
+  "src": ["src"],
+  "extension": [".ts"],
+  "exclude": [
+    "**/*.d.ts",
+    "**/*.test.ts",
+    "test/**/*",
+    "dist/**/*",
+    "coverage/**",
+    "bin/**/*",
+    "scripts/**/*",
+    "tmp/**/*",
+    "**/*.mjs"
+  ],
+  "include": [
+    "src/**/*.ts"
+  ],
+  "reporter": [
+    "text-summary",
+    "html",
+    "lcov"
+  ],
+  "statements": 60,
+  "branches": 60,
+  "functions": 60,
+  "lines": 60
+}

.github/workflows/ci.yml

@@ -1,9 +1,30 @@
 name: Tests
 
 on:
-  [push, workflow_dispatch]
+  push:
+    # Only run on pushes to main branches and long-lived feature branches
+    # PRs will trigger via pull_request event to avoid duplicate runs
+    branches:
+      - main
+      - v11.0.0
+      # Add any long-lived feature branches here
+  pull_request:
+  workflow_dispatch:
 
 jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v6
+      - name: Use Node.js
+        uses: actions/setup-node@v6
+        with:
+          node-version: 22.x
+          cache: npm
+      - run: npm ci
+      - name: linting
+        run: npm run lint
+
   test:
     runs-on: ${{ matrix.os }}
     strategy:
@@ -19,9 +40,88 @@ jobs:
           cache: npm
       - run: npm ci
       - name: unit tests
-        run: npm test
-      - name: linting
-        run: npm run lint
+        shell: bash
+        run: |
+          if [ "${{ matrix.os }}" = "ubuntu-latest" ] && [ "${{ matrix.node-version }}" = "22.x" ]; then
+            npm run pretest && npm run test:unit:justTest:ci:coverage
+          else
+            npm test
+          fi
+      - name: Upload coverage reports
+        if: always() && github.event_name == 'pull_request' && matrix.os == 'ubuntu-latest' && matrix.node-version == '22.x'
+        uses: actions/upload-artifact@v4
+        with:
+          name: coverage-report
+          path: coverage/
+          retention-days: 30
+      - name: Coverage summary
+        if: always() && matrix.os == 'ubuntu-latest' && matrix.node-version == '22.x'
+        run: |
+          if [ -f coverage/lcov.info ]; then
+            echo "## 📊 Coverage Summary" >> $GITHUB_STEP_SUMMARY
+            echo "" >> $GITHUB_STEP_SUMMARY
+            echo "| Metric | Coverage |" >> $GITHUB_STEP_SUMMARY
+            echo "|--------|----------|" >> $GITHUB_STEP_SUMMARY
+
+            # Parse coverage from lcov.info
+            LINES_FOUND=$(grep -o "LF:[0-9]*" coverage/lcov.info | cut -d: -f2 | awk '{s+=$1} END {print s}')
+            LINES_HIT=$(grep -o "LH:[0-9]*" coverage/lcov.info | cut -d: -f2 | awk '{s+=$1} END {print s}')
+            FUNCS_FOUND=$(grep -o "FNF:[0-9]*" coverage/lcov.info | cut -d: -f2 | awk '{s+=$1} END {print s}')
+            FUNCS_HIT=$(grep -o "FNH:[0-9]*" coverage/lcov.info | cut -d: -f2 | awk '{s+=$1} END {print s}')
+            BRANCHES_FOUND=$(grep -o "BRF:[0-9]*" coverage/lcov.info | cut -d: -f2 | awk '{s+=$1} END {print s}')
+            BRANCHES_HIT=$(grep -o "BRH:[0-9]*" coverage/lcov.info | cut -d: -f2 | awk '{s+=$1} END {print s}')
+
+            LINES_PCT=$(awk "BEGIN {printf \"%.2f\", ($LINES_HIT/$LINES_FOUND)*100}")
+            FUNCS_PCT=$(awk "BEGIN {printf \"%.2f\", ($FUNCS_HIT/$FUNCS_FOUND)*100}")
+            BRANCHES_PCT=$(awk "BEGIN {printf \"%.2f\", ($BRANCHES_HIT/$BRANCHES_FOUND)*100}")
+
+            echo "| Lines | ${LINES_PCT}% (${LINES_HIT}/${LINES_FOUND}) |" >> $GITHUB_STEP_SUMMARY
+            echo "| Functions | ${FUNCS_PCT}% (${FUNCS_HIT}/${FUNCS_FOUND}) |" >> $GITHUB_STEP_SUMMARY
+            echo "| Branches | ${BRANCHES_PCT}% (${BRANCHES_HIT}/${BRANCHES_FOUND}) |" >> $GITHUB_STEP_SUMMARY
+          else
+            echo "⚠️ Coverage report not found" >> $GITHUB_STEP_SUMMARY
+          fi
+
+  base-coverage:
+    runs-on: ubuntu-latest
+    if: github.event_name == 'pull_request'
+    steps:
+      - uses: actions/checkout@v6
+        with:
+          ref: ${{ github.event.pull_request.base.ref }}
+      - name: Use Node.js
+        uses: actions/setup-node@v6
+        with:
+          node-version: 22.x
+          cache: npm
+      - run: npm ci
+      - name: Run tests on base branch with full coverage
+        run: npm run pretest && npm run test:unit:justTest:ci:coverage
+      - name: Upload base coverage
+        uses: actions/upload-artifact@v6
+        with:
+          name: coverage-base
+          path: coverage/
+          retention-days: 1
+
+  coverage-check:
+    runs-on: ubuntu-latest
+    needs: [test, base-coverage]
+    if: github.event_name == 'pull_request' && needs.test.result == 'success' && needs.base-coverage.result == 'success'
+    steps:
+      - uses: actions/checkout@v6
+      - name: Download PR coverage
+        uses: actions/download-artifact@v7
+        with:
+          name: coverage-report
+          path: coverage-pr/
+      - name: Download base coverage
+        uses: actions/download-artifact@v7
+        with:
+          name: coverage-base
+          path: coverage-base/
+      - name: Compare coverage and fail if decreased
+        run: ./scripts/ci/compare-coverage.sh coverage-pr coverage-base --fail-on-decrease
 
   integration:
     runs-on: ${{ matrix.os }}
@@ -94,7 +194,7 @@ jobs:
   # dummy job needed to pass changeling compliance because it only watches one build
   done:
     runs-on: macos-latest
-    needs: [test, integration, acceptance]
+    needs: [lint, test, integration, acceptance]
     steps:
       - run: echo done
         working-directory: /

cspell-dictionary.txt

@@ -83,6 +83,8 @@ ECCN
 echoerr
 Edmonds
 elif
+EOFCOMMENT
+EOFTABLE
 EMEA
 envfile
 envl
@@ -315,6 +317,7 @@ resetstat
 reviewapps
 rikki
 rollbar
+romeovs
 rootdir
 rootfulroot
 routable