Skip to content

chore(deps): bump undici to patch Dependabot alerts - #109

Open
ClaudiaFang wants to merge 1 commit into
mainfrom
agent/fix-dependabot-alerts
Open

chore(deps): bump undici to patch Dependabot alerts#109
ClaudiaFang wants to merge 1 commit into
mainfrom
agent/fix-dependabot-alerts

Conversation

@ClaudiaFang

Copy link
Copy Markdown
Member

Summary

Not fixed (upstream limitation)

Alerts #34, #35, #42 (ip-address), plus tar, brace-expansion, and one undici occurrence, all resolve to copies bundled inside the npm CLI package itself (bundleDependencies in npm's own package.json). npm's overrides field cannot reach into a package's bundled dependencies — verified this is unfixable by testing npm CLI versions 11.18.0, 11.19.0, and 12.0.2, all of which bundle the same vulnerable versions. npm is a transitive dev dependency of @semantic-release/npm (used only during the release-publish CI step, not in the shipped plugin). These will need to stay open until npm CLI ships a release with updated bundled deps.

Test plan

  • npx eslint . → 0 errors
  • npm run build → passes (incl. Obsidian 1.11.0 compat typecheck)
  • npx vitest run → 496 passed (34 files)

🤖 Generated with Claude Code

Refreshes package-lock.json to pick up undici 6.28.0 (via the
existing @actions/http-client override) and 7.29.0 (top-level, used
by @octokit/*, jsdom). Both are already within the semver ranges
already permitted by package.json - no override changes needed.

Fixes Dependabot alerts #36-40, #43-45.

Remaining open alerts (#34, #35, #42 ip-address; brace-expansion,
tar, undici<6.28 under npm/node-gyp) are bundled dependencies of the
npm CLI package itself (bundleDependencies), which npm overrides
cannot reach. Verified across npm 11.18.0, 11.19.0, and 12.0.2 -
all three bundle the same vulnerable versions. These require an
upstream npm CLI release and cannot be fixed from this repo.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@sonarqubecloud

sonarqubecloud Bot commented Aug 6, 2026

Copy link
Copy Markdown

@ClaudiaFang
ClaudiaFang force-pushed the main branch 2 times, most recently from 4f78bfa to 5ab9bf8 Compare August 7, 2026 02:55
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