Skip to content

ci: shard Vitest suite#1584

Open
stipsan wants to merge 6 commits into
mainfrom
cursor/vitest-sharding-116b
Open

ci: shard Vitest suite#1584
stipsan wants to merge 6 commits into
mainfrom
cursor/vitest-sharding-116b

Conversation

@stipsan

@stipsan stipsan commented Jul 14, 2026

Copy link
Copy Markdown
Member

Summary

  • run the complete Vitest suite in four parallel GitHub Actions shards
  • collect V8 coverage from each shard and merge the blob reports
  • post merged coverage results as a pull request comment
  • require the shard matrix to succeed before the aggregate test check passes
  • ignore locally generated Vitest blob reports

Testing

  • pnpm install --frozen-lockfile
  • pnpm lint
  • pnpm knip
  • pnpm build (Node 22.22.2)
  • all four coverage-enabled Vitest shards
  • pnpm vitest run --merge-reports --coverage (167 files and 893 tests)
  • actionlint .github/workflows/ci.yml
  • GitHub Actions run 29404888837: all shards, aggregate test check, and coverage report passed
  • verified the coverage report comment posted successfully

vitest_coverage_comment_validation.md

To show artifacts inline, enable in settings.

Open in Web Open in Cursor 

@vercel

vercel Bot commented Jul 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
plugins-studio Ready Ready Preview, Comment Jul 15, 2026 8:53pm

Request Review

@changeset-bot

changeset-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 8482351

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@socket-security

socket-security Bot commented Jul 14, 2026

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Jul 14, 2026

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 27.08% 4238 / 15649
🔵 Statements 26.71% 4432 / 16589
🔵 Functions 22.34% 1048 / 4690
🔵 Branches 17.51% 1914 / 10929
File CoverageNo changed files found.
Generated in workflow #7427 for commit 8482351 by the Vitest Coverage Report Action

@stipsan
stipsan marked this pull request as ready for review July 14, 2026 23:47
@stipsan
stipsan requested a review from a team as a code owner July 14, 2026 23:47
@stipsan
stipsan requested review from Copilot and pedrobonamin and removed request for a team July 14, 2026 23:47

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 043397e. Configure here.

Comment thread .github/workflows/ci.yml

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the repository’s CI pipeline to run the Vitest suite in parallel shards, collect per-shard blob reports with V8 coverage, and merge/post consolidated coverage results on pull requests while preserving a single aggregate test status check.

Changes:

  • Shard the Vitest CI run into 4 parallel jobs and add an aggregate test job.
  • Enable V8 coverage and add @vitest/coverage-v8, plus ignore generated .vitest-reports/.
  • Add a PR-only coverage merge step and post coverage results as a PR comment.

Reviewed changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
.github/workflows/ci.yml Replace single test job with 4 sharded runs, upload blob artifacts, add aggregate test check, and merge/report coverage on PRs.
vitest.config.ts Configure Vitest coverage provider/reporters for consolidated coverage output.
package.json Add @vitest/coverage-v8 to devDependencies (catalog).
pnpm-workspace.yaml Add @vitest/coverage-v8 to the workspace catalog.
pnpm-lock.yaml Lockfile updates for @vitest/coverage-v8 and its transitive deps.
.gitignore Ignore .vitest-reports generated by Vitest blob reporting.
Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/ci.yml Outdated
@stipsan
stipsan enabled auto-merge (squash) July 15, 2026 05:22
Copilot AI review requested due to automatic review settings July 15, 2026 09:32
@stipsan
stipsan removed the request for review from Copilot July 15, 2026 09:32
Copilot AI review requested due to automatic review settings July 15, 2026 20:52
@stipsan
stipsan removed the request for review from Copilot July 15, 2026 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants