ci: add code coverage with 80% line threshold (#63)

* ci: add code coverage with 80% line threshold

Add test:coverage npm script using Node.js built-in test coverage
with --test-coverage-lines=80 threshold. Excludes thin VS Code API
wrapper files (extension.ts, statusBar.ts, configureMcp.ts,
setupWorkspace.ts, showStatus.ts) that are tested by integration
tests rather than unit tests.

Add coverage check step to CI workflow (Linux-only in unit-test job).
Include coverage in the npm run check gate.

Current coverage: 82.65% line, 86.27% branch, 86.69% function.

Closes #61

Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>

* fix: exclude patchloomCli.test.js from coverage

The patchloomCli tests depend on the patchloom binary being on PATH.
In CI on Ubuntu, the binary is not available so tests skip, causing
the test file's coverage to drop from 88% to 17% and failing the
80% threshold. Exclude this environment-dependent test file from
coverage measurement.

Also add --test-coverage-include to limit coverage to source files
only (not test files) for cleaner measurement.

Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>

---------

Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>

Author: SebTardif

.github/workflows/ci.yml

@@ -38,6 +38,10 @@ jobs:
       - name: Run tests
         run: npm test
 
+      - name: Check code coverage
+        if: matrix.os == 'ubuntu-latest'
+        run: npm run test:coverage
+
   build:
     runs-on: ubuntu-latest
     timeout-minutes: 10

AGENTS.md

@@ -16,8 +16,9 @@ Patchloom for VS Code is the official VS Code extension for [Patchloom](https://
 | `npm run test:ui` | Run ExTester UI tests (downloads VS Code if needed) |
 | `npm run test:all` | Compile + unit tests + extension integration tests |
 | `npm run test` | Compile + compile-tests + unit tests |
+| `npm run test:coverage` | Unit tests with line coverage (80% threshold) |
 | `npm run package` | Package the `.vsix` using `@vscode/vsce` |
-| `npm run check` | Full CI gate: test + package |
+| `npm run check` | Full CI gate: test + coverage + package |
 
 Always run `npm run check` before committing.
 

package.json

@@ -140,13 +140,14 @@
     "compile-uitests": "tsc -p ./tsconfig.uitest.json",
     "watch": "tsc -watch -p ./",
     "test:unit": "node --test ./out-test/test/unit/*.test.js",
+    "test:coverage": "node --test --experimental-test-coverage --test-coverage-lines=80 --test-coverage-exclude='out-test/src/extension*' --test-coverage-exclude='out-test/src/commands/showStatus*' --test-coverage-exclude='out-test/src/commands/configureMcp*' --test-coverage-exclude='out-test/src/commands/setupWorkspace*' --test-coverage-exclude='out-test/src/status/statusBar*' --test-coverage-exclude='out-test/test/unit/patchloomCli*' ./out-test/test/unit/*.test.js",
     "test:extension": "node ./out-test/test/suite/runExtensionTests.js",
     "test:ui": "npm run compile && npm run compile-uitests && extest setup-and-run './out-uitest/test/ui/*.test.js' --code_version max --extensions_dir .vscode-test/extensions",
     "test": "npm run compile && npm run compile-tests && npm run test:unit",
     "test:all": "npm run compile && npm run compile-tests && npm run test:unit && npm run test:extension",
     "vscode:prepublish": "npm run compile",
     "package": "vsce package",
-    "check": "npm run test && npm run package"
+    "check": "npm run test && npm run test:coverage && npm run package"
   },
   "overrides": {
     "mocha": {