Skip to content

fix: remediate high & critical Dependabot vulnerabilities - #165

Merged
pkaeding merged 2 commits into
mainfrom
devin/1781209884-fix-dependabot-vulnerabilities
Jun 12, 2026
Merged

fix: remediate high & critical Dependabot vulnerabilities#165
pkaeding merged 2 commits into
mainfrom
devin/1781209884-fix-dependabot-vulnerabilities

Conversation

@pkaeding

Copy link
Copy Markdown
Contributor

Summary

Remediates all 19 high and 32 of 33 moderate severity vulnerabilities identified by yarn audit.

Direct dependency changes:

  • ajv: ^6.12.6^6.14.0 (fixes ReDoS with $data option)
  • uuid: ^9.0.1^11.1.1 (fixes missing buffer bounds check in v3/v5/v6)
  • @vscode/test-cli: ^0.0.4^0.0.12 (fixes high-severity minimatch ReDoS and glob command injection)
  • Removed unused request dependency (deprecated, unfixable SSRF vulnerability)

Yarn resolutions added for transitive dependencies:

  • tar >=7.5.16, brace-expansion >=1.1.13, tough-cookie >=4.1.3, micromatch >=4.0.8, qs >=6.14.1, ip-address >=10.1.1

Before: 56 vulnerabilities (19 high, 33 moderate, 4 low)
After: 3 vulnerabilities (1 moderate, 2 low) — all in deep transitive dev-only dependency chains (nyc, vscode-test)

Link to Devin session: https://app.devin.ai/sessions/0ebfb6d0e5c241a089740010ad5e6167
Requested by: @pkaeding

- Update ajv from ^6.12.6 to ^6.14.0 (fixes moderate ReDoS)
- Update uuid from ^9.0.1 to ^11.1.1 (fixes moderate buffer bounds check)
- Update @vscode/test-cli from ^0.0.4 to ^0.0.12 (fixes high minimatch/glob CVEs)
- Remove unused request dependency (deprecated, unfixable SSRF)
- Add yarn resolutions for tar, brace-expansion, tough-cookie, micromatch, qs, ip-address

Reduces vulnerabilities from 56 (19 high, 33 moderate) to 3 (1 moderate, 2 low).

Co-Authored-By: Patrick Kaeding <patrick@kaeding.name>
@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@pkaeding
pkaeding requested a review from a team June 11, 2026 20:43
@pkaeding
pkaeding marked this pull request as ready for review June 11, 2026 20:44
The >=1.1.13 resolution resolved to v5.x which is incompatible with
minimatch's ESM import of brace-expansion.

Co-Authored-By: Patrick Kaeding <patrick@kaeding.name>
@pkaeding
pkaeding requested review from a team June 11, 2026 20:44
@pkaeding
pkaeding merged commit f3fef80 into main Jun 12, 2026
4 checks passed
@pkaeding
pkaeding deleted the devin/1781209884-fix-dependabot-vulnerabilities branch June 12, 2026 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants