fix(deps): resolve js-yaml CVE-2026-84375 - #256
Conversation
js-yaml is a transitive devDependency flagged HIGH (CVE-2026-84375). Added scoped npm override(s) to force the patched line without breaking any consumer. npm audit now reports 0 vulnerabilities.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
Included review availability: Your plan provides up to 10 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughThe change adds scoped npm overrides for patched Changesjs-yaml security remediation
Priority: ➖ Normal Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Other Merge Risk: ⚪ Minimal · up to The scoped overrides and committed lockfile align with the CI installation path, with no merge-blocking issue identified. 🚥 Pre-merge checks | ✅ 6✅ Passed checks (6 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
Summary
Resolves the open HIGH
js-yamlDependabot advisory CVE-2026-84375 so Vanta's "High vulnerabilities" test clears.js-yamlis only a transitive devDependency here, flagged on both lines:>= 4.0.0, < 4.3.2) — pulled viacosmiconfig>= 3.0.0, < 3.15.2) — pulled via@istanbuljs/load-nyc-configFix
Added scoped npm
overridesso each consumer stays on its own patched line (a blanket top-leveljs-yamloverride would force the istanbul v3 consumer onto 4.x and break it):Lockfile now resolves
js-yamlto 4.3.2 (root) and 3.15.2 (under@istanbuljs/load-nyc-config).Verification
npm audit→ 0 vulnerabilitiesnpm run build✅ ·npm run lint✅ ·npm run test:unit✅ (1918 passed, 42 skipped)Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.Summary by CodeRabbit
Bug Fixes
Documentation