Skip to content

Conversation

@drazisil
Copy link
Collaborator

@drazisil drazisil commented Jan 24, 2026

  • Install coveralls npm package for coverage reporting
  • Add npm script to pipe lcov output to Coveralls
  • Configure GitHub Actions workflow to upload to Coveralls
  • Add Coveralls badge to README for coverage visibility

Coveralls runs in parallel with existing Codecov integration, providing redundancy and alternative coverage visualization.

Co-Authored-By: Claude Sonnet 4.5 noreply@anthropic.com


Note

Introduces parallel coverage reporting via Coveralls in addition to existing Codecov.

  • CI: Adds coverallsapp/github-action step (gated to node-version: 24.x) to upload coverage/lcov.info
  • Docs: Adds Coveralls coverage badge to README.md

No application code changes.

Written by Cursor Bugbot for commit 143379b. This will update automatically on new commits. Configure here.

- Install coveralls npm package for coverage reporting
- Add npm script to pipe lcov output to Coveralls
- Configure GitHub Actions workflow to upload to Coveralls
- Add Coveralls badge to README for coverage visibility

Coveralls runs in parallel with existing Codecov integration,
providing redundancy and alternative coverage visualization.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@socket-security
Copy link

socket-security bot commented Jan 24, 2026

No dependency changes detected. Learn more about Socket for GitHub.

👍 No dependency changes detected in pull request

@deepsource-io
Copy link

deepsource-io bot commented Jan 24, 2026

Here's the code health analysis summary for commits 2047f9f..143379b. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ SuccessView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@codecov
Copy link

codecov bot commented Jan 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 32.54%. Comparing base (2047f9f) to head (143379b).
⚠️ Report is 1 commits behind head on dev.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev    #2810   +/-   ##
=======================================
  Coverage   32.54%   32.54%           
=======================================
  Files         292      292           
  Lines       20451    20451           
  Branches      366      366           
=======================================
  Hits         6655     6655           
  Misses      13792    13792           
  Partials        4        4           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

The coveralls npm package has a critical CVE in its form-data
dependency (CVE-GHSA-fjxv-7rqg-78g4). Since the GitHub Action
coverallsapp/github-action@v2 can read the lcov file directly,
the npm package is unnecessary.

Changes:
- Remove coveralls from devDependencies
- Remove npm coveralls script
- GitHub Action workflow unchanged (reads lcov directly)

This resolves the security vulnerability while maintaining full
Coveralls integration functionality.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Pin coverallsapp/github-action@v2 to commit SHA
5cbfd81b66ca5d10c19b062c04de0199c215fb6e to follow
security best practices and ensure reproducible builds.

This addresses SonarCloud's requirement to pin action versions.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@sonarqubecloud
Copy link

@drazisil drazisil added this pull request to the merge queue Jan 24, 2026
Merged via the queue into dev with commit 1ec2d47 Jan 24, 2026
19 checks passed
@drazisil drazisil deleted the add-coveralls-integration branch January 24, 2026 02:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants