refactor: Bump js-yaml from 3.14.2 to 3.15.2 - #3451
Conversation
|
🚀 Thanks for opening this pull request! We appreciate your effort in improving the project. Please let us know once your pull request is ready for review. Tip
Note Please respond to review comments from AI agents just like you would to comments from a human reviewer. Let the reviewer resolve their own comments, unless they have reviewed and accepted your commit, or agreed with your explanation for why the feedback was incorrect. Caution Pull requests must be written using an AI agent with human supervision. Pull requests written entirely by a human will likely be rejected, because of lower code quality, higher review effort and the higher risk of introducing bugs. Please note that AI review comments on this pull request alone do not satisfy this requirement. Our CI and AI review are safeguards, not development tools. If many issues are flagged, rethink your development approach. Invest more effort in planning and design rather than using review cycles to fix low-quality code. |
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Essentials Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review. 📝 WalkthroughWalkthroughThe lockfile updates the nested Changesjs-yaml dependency update
Priority: ➖ Normal Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to The dependency update is mergeable after normal checks; no concrete issue requiring a pre-merge fix is established. 🚥 Pre-merge checks | ✅ 7✅ Passed checks (7 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
🎉 This change has been released in version 9.3.0-alpha.6 |
Pull Request
Issue
Closes #3450
Approach
Bumps the transitive dev dependency
js-yamlfrom 3.14.2 to 3.15.2. The change is limited to the lock file. It updates only the nested 3.x copy used by@istanbuljs/load-nyc-config(viajest→babel-plugin-istanbul). The rootjs-yaml4.1.1 copy is left unchanged and will be handled in a separate update. This resolves the 3.x security advisories GHSA-2883-xcg3-v3hh, GHSA-5p4m-2wfm-xmqj, GHSA-52cp-r559-cp3m and GHSA-h67p-54hq-rp68. The dependency is development-only.Changes
maxTotalMergeKeys(10000) loader option, which limits the total number of keys processed by YAML merge (<<) in onesafeLoad()/safeLoadAll()call!!omapduplicate key detectionmaxTotalMergeKeys, to limit CPU usage (fix: count empty merge sources against maxTotalMergeKeys nodeca/js-yaml#797)Breaking Changes
None
Code Changes Required
None. This is a drop-in replacement.
@istanbuljs/load-nyc-configonly callsload()to parse a.nycrc.yml/.nycrc.yamlfile, and this repository has none.Tasks
N/A. This is a lock-file-only dependency update.
Summary by CodeRabbit