chore(deps): resolve fixable npm audit advisories - #157
Merged
Merged
Conversation
jjeff
force-pushed
the
chore/drop-deprecated-transitives
branch
from
August 23, 2026 14:04
710f547 to
6ab75ca
Compare
Drops the audit count from 13 to 7. Every remaining advisory is unfixable from this repository; see below. - `npm audit fix` — in-range lock bumps for undici, js-yaml, nanoid, postcss and the top-level brace-expansion. - `brace-expansion@2` override 2.0.3 -> 2.1.4 and a new `brace-expansion@5` override at 5.0.9. GHSA-rgw5-rvv9-x895 patches each major line separately, and the existing override was pinning the v2 line below its own patched release. - `eslint-plugin-tsdoc` 0.4.0 -> 0.5.2, which moves `@microsoft/tsdoc-config` past the ajv ReDoS advisory. Bump the direct `@microsoft/tsdoc` dependency to ^0.16.0 to match what the plugin now pulls in. The 7 that remain all live inside `node_modules/npm/` — the npm CLI that `@semantic-release/npm` depends on — plus the three parent entries that only report them. npm vendors its own dependency tree, so `overrides` cannot reach into it, and npm 12.0.2 still ships undici 6.27.0, so there is no newer npm to move to. `@semantic-release/npm` is already on its latest release; the fix npm audit suggests is a downgrade to 12.0.2. Verified: npm run check, 469 unit tests, and typedoc all pass. Co-Authored-By: Claude Opus 5 (1M context) <[email protected]> Claude-Session: https://claude.ai/code/session_01TkCEqrapNbBcjRxviEJBm5
jjeff
force-pushed
the
chore/audit-fix-transitives
branch
from
August 23, 2026 14:06
a3d3ca4 to
153aaab
Compare
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.OpenSSF ScorecardScorecard details
Scanned Files
|
This was referenced Aug 23, 2026
|
🎉 This PR is included in version 3.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stacked on PR #156 — deprecated glob and inflight transitives. The base will retarget to
mainautomatically once that one merges.Result
npm auditgoes from 13 vulnerabilities to 7. Every one of the remaining 7 is unfixable from this repository — see the last section.Changes
npm audit fix— in-range lock bumps forundici,js-yaml,nanoid,postcss, and the top-levelbrace-expansion. Nopackage.jsonchange needed for these.brace-expansion@2override^2.0.3→^2.1.4, plus a newbrace-expansion@5override at^5.0.9. GHSA-rgw5-rvv9-x895 patches each major line separately (>=2.0.0 <2.1.4and>=4.0.0 <5.0.9). The existing@2override was actively pinning that line below its own patched release.eslint-plugin-tsdoc^0.4.0→^0.5.2, which moves@microsoft/tsdoc-configpast theajvReDoS advisory. This is a semver-major bump of the plugin. The direct@microsoft/tsdocdependency goes to^0.16.0to match what the plugin now resolves.What remains, and why it cannot be fixed here
Four of these live inside
node_modules/npm/— the npm CLI that@semantic-release/npmdepends on. The other three (npm,@semantic-release/npm,semantic-release) are parent entries that only report those same four.Three reasons there is no fix available:
overridescannot reach insidenode_modules/npm/node_modules/.npm: ^12.0.2override; npm 12.0.2 still ships[email protected], so the audit count did not change. Reverted.@semantic-release/npmis already on its latest release (13.1.5). The fixnpm auditsuggests is a downgrade to 12.0.2.None of this reaches shipped library code — it is release tooling that runs in CI on tagged releases only.
Verification
npm run check— 0 errorsnpm run test:unit— 31 files, 469 tests passednpm run docs:build— typedoc generates cleanly, confirming the@microsoft/tsdocmajor bump is safetsdoc/syntaxis set tooffineslint.config.js, and the plugin still loads without error under 0.5.2🤖 Generated with Claude Code
https://claude.ai/code/session_01TkCEqrapNbBcjRxviEJBm5
🔖 Claude agent: resortable-ea
session id:
f17a668a-16a6-415b-b0b5-7fe91bfefd74