Repository navigation
ci: add bun to E2E runners - #1900
Conversation
Co-authored-by: ryoppippi <[email protected]>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Walkthrough
Merge Risk: ⚪ Minimal · up to
|
| Check name | Status | Explanation |
|---|---|---|
| Description Check |  |
Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | |
The title clearly and concisely describes the main change: adding Bun to the E2E runner matrix. |
| Docstring Coverage | |
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
✨ Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
![pullfrog[bot]](https://avatars.githubusercontent.com/in/1768019?s=60&v=4){kind=small}
There was a problem hiding this comment.
✅ No new issues found.
CI-only change expanding E2E coverage. No runtime, security, or release risk. Note: per repo policy this review does not approve — a human approval is still needed.
Reviewed changes
- Added
bunto theccusage-preview-e2erunner matrix alongsidenpmandpnpm. - Added a conditional pinned
oven-sh/setup-buninstall step mirroring the existingpnpmconditional install. - Added the
bunx --package $url ccusageinvocation branch, with a comment explaining why the binary name is explicit.
muse-spark-1.3-contributor (free via Pullfrog for OSS) ｜ 𝕏
ccusage
@ccusage/ccusage-darwin-arm64
@ccusage/ccusage-darwin-x64
@ccusage/ccusage-linux-arm64
@ccusage/ccusage-linux-x64
@ccusage/ccusage-win32-x64
commit: |
ccusage performance comparisonPR SHA: This compares the Rust PR release binary against the configured base package on the same CI runner. Package runtime diagnosticsCompares the PR package wrapper, the installed native optional dependency binary, and the workspace release binary on the same large fixture. This identifies whether slow package results come from JavaScript wrapper overhead, the published native binary build, or the Rust core itself. Fixtures: Claude
Committed fixture performanceCommitted small fixtures for stable PR-to-PR feedback and explicit Claude/Codex command coverage. Fixtures: Claude
Large real-world-shaped fixture performanceGenerated fixtures shaped from aggregate local log statistics: thousands of JSONL files, many small sessions, and a long tail of larger sessions. No real prompts, paths, or outputs are stored in the fixtures. Fixtures: Claude
Artifact size
Lower medians and smaller artifacts are better. CI runner noise still applies; use same-run ratios as directional PR feedback, not release guarantees. |
ccusage performance comparisonPR SHA: This compares the PR package against the configured base package on the same CI runner. Package runtime diagnosticsCompares the PR package wrapper, the installed native optional dependency binary, and the workspace release binary on the same large fixture. This identifies whether slow package results come from JavaScript wrapper overhead, the published native binary build, or the Rust core itself. Fixtures: Claude
Committed fixture performanceCommitted small fixtures for stable PR-to-PR feedback and explicit Claude/Codex command coverage. Fixtures: Claude
Large real-world-shaped fixture performanceGenerated fixtures shaped from aggregate local log statistics: thousands of JSONL files, many small sessions, and a long tail of larger sessions. No real prompts, paths, or outputs are stored in the fixtures. Fixtures: Claude
Artifact size
Lower medians and smaller artifacts are better. CI runner noise still applies; use same-run ratios as directional PR feedback, not release guarantees. |
Summary
Adds bun to the
ccusage-preview-e2erunner matrix in.github/workflows/ci.yaml. The matrix is nownpm,pnpm,bun.oven-sh/setup-bun, pinned to the v2.2.0 commit SHA like the other actions.bunx --package <preview-url> ccusage .... bunx rejects a bare URL as the package (unrecognised dependency format), so the binary name is given explicitly.Validation
d755584.--versionanddaily/monthly/session --offline --jsonall succeed and produce valid JSON, and the@ccusage/ccusage-*native packages install.Summary by cubic
Adds bun to the
ccusage-preview-e2erunner matrix in.github/workflows/ci.yamlso the preview is tested with npm, pnpm, and bun.oven-sh/setup-bunpinned to v2.2.0, matching the other action pins.bunx --package <url> ccusage ...because bunx rejects a bare URL as the package.Written for commit 9364c8b. Summary will update on new commits.
Summary by CodeRabbit