Repository navigation
Bump vercel cli version - #3990
Conversation
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ 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: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Review rate limit: 0/1 reviews remaining, refill in 36 minutes and 44 seconds.Comment |
There was a problem hiding this comment.
Code Review
This pull request updates the vercel package to version 53.1.0, leading to a large-scale update of transitive dependencies in the lockfile. The review feedback correctly identifies a critical Node.js compatibility issue where the new undici dependency requires version 20.18.1+, breaking support for Node.js 18. Additionally, the update introduces a legacy version of js-yaml, creating a duplicate dependency that should be unified with the existing version 4.x to minimize the maintenance surface.
| undici@7.25.0: | ||
| resolution: {integrity: sha512-xXnp4kTyor2Zq+J1FfPI6Eq3ew5h6Vl0F/8d9XU5zZQf1tX9s2Su1/3PiMmUANFULpmksxkClamIZcaUqryHsQ==} | ||
| engines: {node: '>=20.18.1'} |
There was a problem hiding this comment.
The introduction of undici@7.25.0 (via @vercel/sandbox) imposes a Node.js requirement of >= 20.18.1. This contradicts the engines field of the main vercel@53.1.0 package (line 7036), which specifies >= 18. This mismatch breaks compatibility for users on Node.js 18 or older versions of Node.js 20. Consider pinning undici to a compatible 6.x version to maintain broader Node.js support.
| '@yarnpkg/parsers': 3.0.3 | ||
| execa: 3.2.0 | ||
| fs-extra: 11.1.0 | ||
| js-yaml: 3.14.2 |
There was a problem hiding this comment.
This update introduces js-yaml@3.14.2 as a dependency for @vercel/backends. Since js-yaml@4.1.1 is already present in the project (e.g., line 9682), this creates a duplicate dependency on a legacy version. Version 3.x is deprecated and has a different security profile. Unifying the js-yaml versions to 4.x is recommended to reduce maintenance surface and bundle size.
Check if the Pull Request fulfils these requirements
Greptile Summary
This PR bumps the Vercel CLI from
50.25.4to53.1.0by updating the catalog entry inpnpm-workspace.yamland regenerating the lock file. The lock file changes are consistent with the version bump — all@vercel/*sub-packages are updated, new packages such as@vercel/sandbox,@vercel/oidc, and@vercel/prepare-flags-definitionsare introduced as transitive dependencies, and obsolete packages likepip-requirements-jsandohm-jsare removed.Confidence Score: 5/5
This PR is safe to merge — it is a clean dependency version bump with no application code changes.
Only dependency files are modified. The lock file is fully consistent with the catalog change, all integrity hashes are present, and no application logic is touched.
No files require special attention.
Reviews (1): Last reviewed commit: "Bump vercel cli version" | Re-trigger Greptile