Update all dependencies to fix vulnerabilities#277
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates all dependencies to address security vulnerabilities identified in 4 Dependabot alerts. The changes include updating the main runtime dependency (lodash) and multiple development dependencies including ESLint, Prettier, and various validator libraries used in tests.
Changes:
- Updates lodash from 4.17.21 to 4.17.23 to fix security vulnerabilities
- Updates development tooling (eslint, prettier, release-it, eslint-config-uphold) to latest versions
- Updates validation library dependencies (bignumber.js, google-libphonenumber, validator) to patch versions
- Updates transitive dependencies in yarn.lock including Babel, ESLint ecosystem packages, and other tools
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| package.json | Updates direct dependencies: lodash (patch), bignumber.js (patch), eslint ( |
| yarn.lock | Updates transitive dependencies including Babel 7.27→7.28, ESLint ecosystem packages, and various other tools. Notable updates include @babel/core, @babel/eslint-parser, eslint-plugin-jsdoc, and many @jridgewell packages |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "@uphold/github-changelog-generator": "^4.0.2", | ||
| "abavalidator": "^2.0.2", | ||
| "bignumber.js": "^9.3.0", | ||
| "bignumber.js": "^9.3.1", |
There was a problem hiding this comment.
The update of bignumber.js to version 9.3.1 conflicts with the peerDependencies constraint which specifies ">=7 || <=9.3.0" (line 77) and the optionalPeerDependencies constraint ">=7 <=9.3.0" (line 141). The devDependency is being updated to ^9.3.1, which is outside the acceptable peer dependency range. This means that consumers of this package would see peer dependency warnings if they use bignumber.js 9.3.1 or higher.
Either update the peerDependencies constraints to allow 9.3.1 (e.g., ">=7 <=9.3.1" or ">=7 <9.4.0"), or keep bignumber.js at 9.3.0 if there's a specific reason for the upper bound constraint.
| "bignumber.js": "^9.3.1", | |
| "bignumber.js": "^9.3.0", |
Description
Related issues