Skip to content

chore(deps): clear Dependabot security alerts - #307

Merged
JohnMcLear merged 1 commit into
mainfrom
chore/security-deps
Sep 21, 2026
Merged

JohnMcLear merged 1 commit into
mainfrom
chore/security-deps

Conversation

@JohnMcLear

Copy link
Copy Markdown
Member

Clears all 5 open Dependabot alerts on this repo. All are transitive, development-scope only (they arrive via eslint / minimatch), so nothing user-facing changes.

severity package first patched
high js-yaml 4.3.2
high js-yaml 4.3.1
high brace-expansion 1.1.16
high js-yaml 4.3.0
medium js-yaml 4.2.0

pnpm update js-yaml brace-expansion --recursive --latest does not move them (neither is a direct dependency and the parents' ranges stay satisfied), so this adds scoped pnpm.overrides and regenerates pnpm-lock.yaml:

  • js-yaml@^4>=4.3.2 <5 (lock: 4.1.1 → 4.3.2)
  • brace-expansion@^1>=1.1.16 <2 (lock: 1.1.14 → 1.1.21)

The overrides are version-scoped, so the already-patched brace-expansion@5.x / js-yaml@5.x branches of the tree are untouched. No other package versions change; pnpm audit goes from 5 advisories to 0 and pnpm run lint passes with 0 errors.

🤖 Generated with Claude Code

https://claude.ai/code/session_013S4pYSjwUsiZtdtMMpW7bw

Force patched versions of the transitive development-scope deps
js-yaml (>=4.3.2) and brace-expansion (>=1.1.16) via pnpm overrides
and regenerate the lockfile. Both arrive through eslint /
minimatch and are not direct dependencies, so a plain
`pnpm update` cannot move them.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013S4pYSjwUsiZtdtMMpW7bw
@qodo-code-review

Copy link
Copy Markdown

ⓘ Qodo reviews are paused because the subscription is no longer active. Ask your workspace admin to reactivate the subscription to resume reviews. Manage billing

@qodo-free-for-open-source-projects

Copy link
Copy Markdown

PR Summary by Qodo

Patch vulnerable transitive development dependencies

🐞 Bug fix ⚙️ Configuration changes 🕐 Less than 10 minutes

Grey Divider

AI Description

• Enforces patched js-yaml and brace-expansion versions across development dependencies.
• Regenerates dependency resolution to eliminate five security advisories without runtime changes.
Diagram

graph TD
  CFG["package.json"] --> OVR["pnpm overrides"] --> LOCK["Lockfile resolution"]
  LOCK --> ESL["ESLint tree"] --> YAML["js-yaml 4.3.2"]
  LOCK --> MINI["minimatch 3.1.5"] --> BRACE["brace-expansion 1.1.21"]
Loading
High-Level Assessment

The following are alternative approaches to this PR:

1. Upgrade parent development tools
  • ➕ Could remove the need for explicit transitive dependency overrides.
  • ➕ Would adopt upstream dependency selections and other maintenance fixes.
  • ➖ Introduces unrelated ESLint and toolchain changes.
  • ➖ Parent ranges currently resolve without moving the vulnerable transitive versions.
  • ➖ Creates a broader compatibility and regression-testing surface.

Recommendation: Keep the version-scoped pnpm overrides as the immediate remediation. They patch only vulnerable major-version branches, avoid unrelated toolchain upgrades, and leave already-safe major versions untouched; parent upgrades can remove the overrides later when available and independently justified.

Files changed (2) +111 / -119

Other (2) +111 / -119
package.jsonAdd scoped overrides for vulnerable transitive dependencies +6/-0

Add scoped overrides for vulnerable transitive dependencies

• Adds pnpm overrides requiring patched js-yaml 4.x and brace-expansion 1.x releases. The constraints avoid altering already-safe newer major-version branches.

package.json

pnpm-lock.yamlResolve development dependencies to patched versions +105/-119

Resolve development dependencies to patched versions

• Records the overrides and replaces vulnerable js-yaml and brace-expansion resolutions with patched releases. Regeneration also normalizes optional supports-color peer snapshots without changing other package versions.

pnpm-lock.yaml

@qodo-free-for-open-source-projects

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can add REVIEW.md to your repo root and Qodo follows it on every PR

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@JohnMcLear
JohnMcLear merged commit 878ca26 into main Sep 21, 2026
4 checks passed
@JohnMcLear
JohnMcLear deleted the chore/security-deps branch September 21, 2026 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant