Repository navigation
fix(ci): give the release npm job time to build pnpm - #1844
Conversation
The v20.0.27 npm job was cancelled twice at the ten-minute limit while `nix profile install` was still compiling the pinned pnpm 12.8.1 on the arm runner, so nothing was published.
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
ccusage-guide | 726b56e | Commit Preview URL Branch Preview URL |
Oct 09 2026, 11:56 PM |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe npm release job timeout increased from 10 to 45 minutes. A comment explains that building pinned pnpm from source can take more than ten minutes on an ARM runner after a cache miss. ChangesRelease timeout
Priority: ➖ Normal Estimated code review effort: 1 (Trivial) | ~3 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The npm release job now has 45 minutes for setup, compilation, and publication instead of 10, with its release ordering unchanged. No concrete merge-blocking risk is evident; it is ready to merge subject to ordinary checks. Pre-merge checks |
 |
| 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: increasing the release npm job timeout so pnpm can finish building. |
| 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.
Reviewed changes
- Release
npmjob timeout increase: raisestimeout-minutesfrom 10 to 45 with a comment noting the pinnedpnpmcompiles from source on a cache miss onubuntu-24.04-arm.
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: Performance comparison skipped. Base package URL was not ready before 300.000s. Fixture performance comparison requires a base package when --base-dir is not provided. Base package: |
ccusage performance comparisonPR SHA: Performance comparison skipped. Base package URL was not ready before 300.000s. Fixture performance comparison requires a base package when --base-dir is not provided. Base package: |
The v20.0.27 release tagged successfully, but the
npmjob was cancelled twice at its 10-minute timeout whilenix profile install --inputs-from . .#pnpm .#publintwas still compiling the pinned pnpm 12.8.1 (built from source vianix/pnpm.nix) onubuntu-24.04-arm. Nothing was published to npm and no GitHub release was created.This raises the job timeout to 45 minutes so a cache miss can finish;
setup-nix-cachesaves the result for later releases. Re-running the v20.0.27 jobs would reuse the old workflow file, so the next tagpr release will carry these changes to npm.Summary by cubic
Raises the release npm job timeout from 10 to 45 minutes so the pinned pnpm can finish compiling from source on a cache miss.
The v20.0.27 npm job was cancelled twice at the 10-minute limit while
nix profile installwas still building pnpm 12.8.1 on the arm runner, so nothing reached npm.setup-nix-cachesaves the build for later releases; re-running v20.0.27 jobs would reuse the old workflow, so the next tagpr release carries this fix.Written for commit 726b56e. Summary will update on new commits.
Summary by CodeRabbit