Repository navigation
chore(deps): clear Dependabot security alerts - #121
Conversation
Force patched versions of the dev-scope transitive dependencies js-yaml and brace-expansion via pnpm overrides and regenerate the lockfile. - js-yaml 4.1.1 -> 4.3.2 (pulled in by eslint 8 / @eslint/eslintrc) - brace-expansion 1.1.14 -> 1.1.21 (pulled in by minimatch 3) The brace-expansion override is scoped to the 1.x line so minimatch 10's brace-expansion 5.x resolution is untouched. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013S4pYSjwUsiZtdtMMpW7bw
|
ⓘ Qodo reviews are paused because the subscription is no longer active. Ask your workspace admin to reactivate the subscription to resume reviews. Manage billing |
Code Review by Qodo🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)
Great, no issues found!Qodo reviewed your code and found no material issues that require reviewTip of the day💡 Did you know, you can add REVIEW.md to your repo root and Qodo follows it on every PR |
PR Summary by QodoPin secure transitive development dependencies with pnpm overrides
AI Description
Diagram
High-Level Assessment
Files changed (2)
|
Clears all open Dependabot security alerts on this repo. Both are development-scope transitive dependencies coming in through
pnpm-lock.yaml, not from any direct dependency, so the fix is apnpm.overridesentry plus a regenerated lockfile.5 alerts closed: 4 high, 1 medium.
Notes
pnpm update --recursive --latestdoes not move either package, because neither is a direct dependency — hence the overrides.js-yamlis overridden to^4.3.2rather than>=4.3.2so it stays on the 4.x line (js-yaml 5.x would break eslint 8).brace-expansionoverride is scoped to the 1.x line (brace-expansion@1) so that minimatch 10's brace-expansion 5.0.12 resolution is left alone.pnpm run lintpasses with 0 errors.🤖 Generated with Claude Code
https://claude.ai/code/session_013S4pYSjwUsiZtdtMMpW7bw