ci: use voidzero-dev/setup-vp instead of setup-node + pnpm/action-setup#109
ci: use voidzero-dev/setup-vp instead of setup-node + pnpm/action-setup#109kodiakhq[bot] merged 1 commit intomainfrom
Conversation
Replace the two-step setup pattern (actions/setup-node + pnpm/action-setup + pnpm install) with the unified voidzero-dev/setup-vp action across all CI workflows. This simplifies setup to a single step with node-version, cache, and automatic dependency installation via vp. - Replace all pnpm vp commands with vp directly - Use cache: true for jobs needing full install - Use run-install: false for jobs with partial install - Pin to v1.6.0 (8ecb3917)
✅ Deploy Preview for wolfstar-rocks-et34281 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughGitHub Actions workflows across four files replace Node.js and pnpm setup actions with a consolidated Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Suggested labels
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
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 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #109 +/- ##
=======================================
Coverage 67.03% 67.03%
=======================================
Files 76 76
Lines 1878 1878
Branches 567 567
=======================================
Hits 1259 1259
Misses 489 489
Partials 130 130 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR standardizes CI bootstrapping across workflows by replacing the actions/setup-node + pnpm/action-setup + pnpm install pattern with the unified voidzero-dev/setup-vp action, aligning CI with the repo’s vp-based tooling.
Changes:
- Migrated CI workflow jobs to
voidzero-dev/setup-vp(pinned tov1.6.0/8ecb3917) for Node setup + caching/install behavior. - Updated workflow commands from
pnpm .../pnpm vp ...tovp ...(including install, exec, run, and test invocations). - Applied the same setup pattern to release workflows and the autofix workflow (with
run-install: falsewhere appropriate).
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
.github/workflows/continuous-integration.yml |
Migrates all CI jobs to setup-vp, switches job commands to vp equivalents, and uses cache: true where full installs are required. |
.github/workflows/autofix.yml |
Replaces multi-step Node+pnpm setup with setup-vp and runs autofix via vp run lint:fix. |
.github/workflows/release-tag.yml |
Uses setup-vp (no auto-install) and switches dependency installation to vp install for release-note generation. |
.github/workflows/release-pr.yml |
Uses setup-vp (no auto-install) for Node availability during release PR automation steps. |
|
👋 Thanks for the contribution, @RedStar071! This PR meets the repository's contribution guidelines — the title follows Conventional Commits format, the description clearly explains the what, why, changes, and impact, and no screenshots or migration files are needed for a CI-only change. The
|
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
CI-only simplification — clean and well-structured. ✅
What looks good:
- All 4 workflow files updated consistently with the same pattern.
- Action pinned to full commit SHA (
8ecb391...) with a# v1.6.0version comment, matching the repo's existing security practice. cache: true(full auto-install) vsrun-install: false(manual partial install) is applied correctly per job — jobs that need everything getcache: true, jobs that do partial/no installs getrun-install: false.- The
vp installstep inrelease-tag.ymlcorrectly retains itsif: steps.check.outputs.skip == 'false'guard. pnpm prisma:generate→vp run prisma:generateis correct sinceprisma:generateis a defined package.json script.
CI at review time: Lint ✅ · Unit tests ✅ · Component tests ✅ · Knip ✅ · GitGuardian ✅ · Browser/A11y/Benchmarks ⏳ (in progress, expected duration).
Generated by WolfStar Code Quality Reviewer for issue #109
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
What
Replace the two-step setup pattern (`actions/setup-node` + `pnpm/action-setup` + `pnpm install`) with the unified `voidzero-dev/setup-vp` action across all CI workflows.
Why
Simplifies CI setup to a single step with `node-version`, `cache`, and automatic dependency installation via `vp`. Based on npmx-dev/npmx.dev#2145.
Changes
Key patterns
Impact
CI-only change. No runtime code changes.
Summary by CodeRabbit