From d10417a7b1f8970d188db900b6f48460857f8495 Mon Sep 17 00:00:00 2001 From: Claude Date: Sun, 18 Jan 2026 18:05:18 +0000 Subject: [PATCH 1/2] fix(ci): use merged coverage from coverage-tryscript/ Update CI workflow to read coverage from coverage-tryscript/ instead of coverage/ directory. This ensures badges and reports show the combined coverage from both vitest unit tests and tryscript CLI tests (merged via --merge-lcov). Previously, CI read from coverage/ which only contains vitest's unit test coverage. The merged coverage is written to coverage-tryscript/ by tryscript's --merge-lcov feature. --- .github/workflows/ci.yml | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 08c5d20d..7f6fd9fd 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -31,23 +31,25 @@ jobs: - run: pnpm --filter markform test:coverage # Post coverage report (PR comment + job summary) + # Uses merged coverage from tryscript (vitest + tryscript CLI tests combined) - name: Coverage Report uses: davelosert/vitest-coverage-report-action@v2 if: always() with: # Working directory for monorepo support - action looks for vite config and coverage files here working-directory: packages/markform - json-summary-path: coverage/coverage-summary.json + json-summary-path: coverage-tryscript/coverage-summary.json json-final-path: coverage/coverage-final.json # Only show changed files on PRs, show all on main file-coverage-mode: ${{ github.event_name == 'pull_request' && 'changes' || 'all' }} # Generate and commit coverage badges (main branch only) + # Uses merged coverage from tryscript (vitest + tryscript CLI tests combined) - name: Coverage Badges uses: jpb06/coverage-badges-action@v1.4.6 if: github.ref == 'refs/heads/main' && github.event_name == 'push' with: - coverage-summary-path: packages/markform/coverage/coverage-summary.json + coverage-summary-path: packages/markform/coverage-tryscript/coverage-summary.json output-folder: ./badges branches: main From 9ecbfce9454625a26067afaa0a3916ae4295960d Mon Sep 17 00:00:00 2001 From: Claude Date: Sun, 18 Jan 2026 18:11:15 +0000 Subject: [PATCH 2/2] fix(config): add json-summary reporter to tryscript coverage Add json-summary to tryscript coverage reporters so that coverage-tryscript/coverage-summary.json is generated. This file is required by the CI coverage report and badge actions. Addresses Codex review feedback on PR #107. --- packages/markform/tryscript.config.mjs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/markform/tryscript.config.mjs b/packages/markform/tryscript.config.mjs index 581311ca..c50763dc 100644 --- a/packages/markform/tryscript.config.mjs +++ b/packages/markform/tryscript.config.mjs @@ -19,7 +19,7 @@ export default defineConfig({ // See: https://github.com/jlevy/tryscript/blob/main/docs/tryscript-reference.md coverage: { reportsDir: 'coverage-tryscript', - reporters: ['text', 'text-summary', 'html', 'lcov'], + reporters: ['text', 'text-summary', 'html', 'lcov', 'json-summary'], include: ['dist/**'], src: 'src', excludeNodeModules: true,