chore: migrate to Corepack-managed Yarn 4#38
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (12)
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Comment |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
.github/workflows/CI.yml
Outdated
| cache: 'yarn' | ||
| - run: HUSKY=0 yarn install --frozen-lockfile --network-concurrency 1 | ||
| - run: corepack enable | ||
| - run: HUSKY=0 yarn install --immutable --network-concurrency 1 |
There was a problem hiding this comment.
CI dependency caching removed without replacement
Medium Severity
The cache: 'yarn' option was removed from actions/setup-node in both the checks and e2e jobs without adding any alternative Yarn dependency caching. Every CI run now does a fresh yarn install with no cache, which can add several minutes per run. The corepack enable step runs after actions/setup-node, but moving it before setup-node and restoring cache: 'yarn' would fix the compatibility issue that likely motivated the removal.


Summary
Testing
Note
Medium Risk
Moderate risk because it changes dependency installation semantics in CI and local dev (Corepack +
yarn install --immutable), which can cause workflow breakage if the pinned Yarn/lockfile expectations differ across environments.Overview
Pins the repo to Corepack-managed Yarn 4 by adding
packageManager: "yarn@4.13.0",.yarnrc.yml, and.nvmrc, and updates contributor guidance to runnvm+corepack enable.Updates CI to enable Corepack and switch installs from Yarn Classic (
--frozen-lockfile) to Yarn 4’syarn install --immutable, and expands workflow triggers to include.yarnrc.yml. Adds.yarn/*ignore rules (with Yarn-managed subfolders allowlisted) and lightly aligns hook/docs wording around “Yarn install”.Written by Cursor Bugbot for commit a3c2184. This will update automatically on new commits. Configure here.