Conversation
Signed-off-by: Mridul Tailor <mtailor729@gmail.com>
PR SummaryLow Risk Overview Updates CI ( Reviewed by Cursor Bugbot for commit 4234e42. Bugbot is set up for automated code reviews on this repo. Configure here. |
| - name: Run Tests | ||
| run: node --run test |
There was a problem hiding this comment.
We should upload to codecov, no?
There was a problem hiding this comment.
Sure, I can add that. Just want to check, does the repo already have Codecov enabled on the org side?
Co-authored-by: Matt Cowley <me@mattcowley.co.uk> Signed-off-by: Mridul Tailor <71081929+MridulTailor@users.noreply.github.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit d48f408. Configure here.

Closes #789
Summary
Sets up code coverage reporting using
@vitest/coverage-istanbul.@vitest/coverage-v8was attempted first but is incompatible with the Cloudflare Workers Vitest pool, it depends onnode:inspectorwhich is unavailable in that runtime. Istanbul is the supported alternative per the Cloudflare docs.Changes
package.json: addstest:coveragescriptvitest.config.ts: enables Istanbul coverage with text/lcov reporters and modest line/function thresholds.github/workflows/test.yml: CI now runstest:coverage.gitignore: addscoverage/to prevent generated artifacts from being committedpackage-lock.json: updated for@vitest/coverage-istanbul@3.2.4