Skip to content

ci: add test reporting coverage summary#13

Merged
jasonsaayman merged 11 commits into
mainfrom
feat/report-test-results-in-pr
May 17, 2026
Merged

ci: add test reporting coverage summary#13
jasonsaayman merged 11 commits into
mainfrom
feat/report-test-results-in-pr

Conversation

@jasonsaayman
Copy link
Copy Markdown
Member

@jasonsaayman jasonsaayman commented May 17, 2026

Summary

  • Add the Vite/Vitest/Oxlint tooling changes present on this branch.
  • Add a CI-only Vitest reporting script that emits JUnit results and coverage summary data.
  • Publish test results with dorny/test-reporter@v3 in the pull request workflow.
  • Add coverage results to the GitHub Actions job summary and archive the completed OpenSpec change.

Verification

  • pnpm run lint
  • pnpm run typecheck
  • pnpm test
  • pnpm run test:ci
  • pnpm run build

Notes

  • This is repository CI reporting only and does not change the bundle-size action runtime API.
  • The test reporter is skipped for fork PRs/read-only-token contexts by requiring the PR head repo to match this repository.

Summary by cubic

Adds CI test reporting for Vitest and a coverage summary in PR checks, and migrates repo tooling to Vite/Vitest/Oxlint without changing the action’s runtime or API.

  • New Features

    • PR workflow runs lint, typecheck, and Vitest with coverage.
    • Publishes JUnit results via dorny/test-reporter@v3 when the PR head repo matches this repo; skips on forks/read-only tokens.
    • Adds a coverage summary to the job summary; artifacts at reports/vitest-junit.xml and coverage/coverage-summary.json with checks: write permission.
  • Refactors

    • Build uses vite to bundle src/index.ts to dist/index.js (CommonJS); tests run on vitest; lint uses oxlint.
    • Adds @/* alias and bundler-style TS config; updates scripts: lint, typecheck, test, test:ci, build.
    • Removes @vercel/ncc, intermediate lib/, and dist/licenses.txt; runtime behavior and action API unchanged.

Written for commit 7b8af4b. Summary will update on new commits. Review in cubic

@jasonsaayman jasonsaayman self-assigned this May 17, 2026
@jasonsaayman jasonsaayman added commit::ci The PR is related to CI priority::medium A medium priority labels May 17, 2026
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

2 issues found across 51 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="THREAT_MODEL.md">

<violation number="1" location="THREAT_MODEL.md:75">
P2: This bullet overstates workflow hardening: at least one third-party action (`dorny/test-reporter@v3`) is not pinned by commit SHA.</violation>
</file>

<file name="src/index.ts">

<violation number="1" location="src/index.ts:16">
P2: The `endsWith("dist/index.js")` fallback can trigger `run()` for unrelated entry scripts, causing unintended side effects when this module is imported in processes whose argv script path matches that suffix.</violation>
</file>

Note: This PR contains a large number of files. Free-tier reviews are limited to 40 files per PR, so some files may not have been reviewed. cubic prioritizes the most important files to review. Paid plans can review more files.
You're on the cubic free plan with 10 free PR reviews remaining this month. Upgrade for unlimited reviews.
Re-trigger cubic

Comment thread THREAT_MODEL.md Outdated
Comment thread src/index.ts Outdated
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 17, 2026

Bundle Size Report

File Baseline gzip Current gzip Difference Status
index.js 302 B 43 B -259 B (-85.76%) 🟢
Total 302 B 43 B -259 B (-85.76%) 🟢

jasonsaayman and others added 4 commits May 17, 2026 15:54
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
@jasonsaayman jasonsaayman merged commit 0ab1edd into main May 17, 2026
2 checks passed
@jasonsaayman jasonsaayman deleted the feat/report-test-results-in-pr branch May 17, 2026 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

commit::ci The PR is related to CI priority::medium A medium priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant