fix(deps): resolve js-yaml CVE-2026-84375 - #89
Conversation
Force js-yaml >= 4.3.2 via a scoped npm override on cosmiconfig (the transitive parent(s) of js-yaml), clearing the HIGH advisory GHSA-2883-xcg3-v3hh (CVE-2026-84375, vulnerable range >= 4.0.0 < 4.3.2). js-yaml is a dev-only transitive dependency. npm audit: 0 vulnerabilities. Build, lint, typecheck, and tests all pass.
|
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 (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughThe package configuration now forces ChangesDependency override
Priority: ⬆️ High Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The dependency security fix is lockfile-consistent, with no unresolved merge risk identified. 🚥 Pre-merge checks | ✅ 5 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (5 passed)
Full details: Changelog EntryExplanation The PR adds an npm
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
Summary
Resolves the open HIGH Dependabot advisory GHSA-2883-xcg3-v3hh / CVE-2026-84375 for
js-yaml(vulnerable range>= 4.0.0 < 4.3.2), clearing Vanta's High vulnerabilities test.js-yamlis a dev-only transitive dependency pulled bycosmiconfig(viasemantic-release). Applied a scoped npm override to force>= 4.3.2(same pattern used in wyre-technology/afkbot):Verification
npm audit: 0 vulnerabilities (js-yaml advisory gone)npm run build,npm run lint,npm run typecheck, andnpm testall passSupersedes Dependabot PR #88 (bare lockfile bump) — the scoped override is self-documenting and prevents regression on re-resolution.
Closes advisory #45.
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.Summary by CodeRabbit