chore: resolve open dependabot security alerts#8
Conversation
- js-yaml <3.15.0 -> 3.15.0 via npm overrides on @istanbuljs/load-nyc-config (medium, alert #66) - js-yaml >=4.0.0,<=4.1.1 -> 4.2.0+ via npm overrides on @expo/xcpretty (medium, alert #65) Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
There was a problem hiding this comment.
Pull request overview
This PR addresses Dependabot-reported security vulnerabilities in transitive js-yaml dependencies by adding targeted npm overrides, ensuring patched js-yaml versions are selected without broadly upgrading parent toolchain packages.
Changes:
- Added scoped npm
overridesto force patchedjs-yamlversions for@istanbuljs/load-nyc-configand@expo/xcpretty. - Updated
package-lock.jsonto reflect resolved patched versions (js-yaml@3.15.0andjs-yaml@4.3.0).
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| package.json | Adds scoped overrides to pin vulnerable transitive js-yaml paths to patched ranges. |
| package-lock.json | Updates the lockfile to reflect the patched js-yaml resolutions (3.15.0 and 4.3.0). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
overridesto the specific transitive dependency paths pulling in vulnerablejs-yamlversions.Dependabot Alerts Resolved
js-yaml(< 3.15.0, via@istanbuljs/load-nyc-config->babel-plugin-istanbul->babel-jest->react-native)^3.15.0js-yaml(>= 4.0.0, <= 4.1.1, via@expo/xcpretty->@expo/cli->expo)^4.2.0Both are transitive dev-tooling dependencies (Jest/Istanbul coverage tooling and Expo's xcpretty log formatter), so scoped
overrideswere used rather than bumping the parent packages, avoiding pulling in unrelated major version changes.npm installconfirms both resolve to patched versions (js-yaml@3.15.0andjs-yaml@4.3.0), andnpm auditreports 0 vulnerabilities.This repo has no
test/lint/buildscripts and no CI workflows;npx tsc --noEmitpasses with no errors.