Skip to content

chore(deps): fix npm audit findings - #2514

Closed
tunc-d wants to merge 3 commits into
actions:mainfrom
tunc-d:tunc-d-npm-audit-fixes
Closed

tunc-d wants to merge 3 commits into
actions:mainfrom
tunc-d:tunc-d-npm-audit-fixes

Conversation

@tunc-d

@tunc-d tunc-d commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Fixes the pre-existing npm audit failures that make the Audit check fail on main.

  • The repo has no npm workspaces and npm run audit-all installs and audits each package separately, so root overrides don't apply. Overrides are declared in each affected package's own package.json.
  • Every lockfile was regenerated with npm install; no hand edits. Each lockfile diff only moves the targeted packages.
  • Dependency changes only: no package version bumps, release notes, source or test changes.

Advisories fixed

Package package.json change Mechanism Advisories fixed
http-client undici ^6.23.0 → ^6.28.1 range bump undici: GHSA-p88m-4jfj-68fv, GHSA-vxpw-j846-p89q, GHSA-g8m3-5g58-fq7m, GHSA-8xcm-r25x-g524, GHSA-m8rv-5g2x-5cg5, GHSA-v3r7-h72x-cjcm, GHSA-35p6-xmwp-9g52, GHSA-r53p-7pc4-xj5r, GHSA-rfgv-xxqx-mfg5
core overrides.undici: ^6.28.1 override (undici comes in through published @actions/http-client, which declares ^6.23.0) undici: same 9 as http-client
tool-cache overrides.undici: ^6.28.1 override undici: same 9 as http-client, plus GHSA-3wwx-pv8p-q78v
glob minimatch ^10.2.5 → ^10.2.6; overrides: undici: ^6.28.1, brace-expansion: ^5.0.12 range bump (minimatch) + override (undici, brace-expansion) undici: same 9 as http-client; brace-expansion: GHSA-mh99-v99m-4gvg, GHSA-rgw5-rvv9-x895, GHSA-q2hr-2g5m-vwhr, GHSA-qhr7-859c-m2p7, GHSA-6j4f-fj2g-mc7p
artifact add overrides: brace-expansion@^2: ^2.1.7, brace-expansion@^5: ^5.0.12, undici: ^6.28.1, markdown-it: ^14.3.1 (existing overrides kept) override undici: GHSA-3wwx-pv8p-q78v, GHSA-r53p-7pc4-xj5r, GHSA-rfgv-xxqx-mfg5; brace-expansion: GHSA-3jxr-9vmj-r5cp, GHSA-mh99-v99m-4gvg, GHSA-rgw5-rvv9-x895, GHSA-q2hr-2g5m-vwhr, GHSA-qhr7-859c-m2p7, GHSA-6j4f-fj2g-mc7p; markdown-it (via typedoc): GHSA-253c-mchw-3w2r
attest devDependency undici ^6.23.0 → ^6.28.1; overrides.ip-address: ^10.7.1 range bump + override undici: GHSA-3wwx-pv8p-q78v, GHSA-r53p-7pc4-xj5r, GHSA-rfgv-xxqx-mfg5; ip-address: GHSA-mwp4-54f8-5fhr, GHSA-4xrf-jv44-h6hh, GHSA-22jq-vg5j-6vgg, GHSA-rpw4-54j3-4h4q, GHSA-2vr4-cq9g-pvrc, GHSA-j6r3-76f7-8jcv, GHSA-h3mg-xc3c-68pw

cache, github, exec and io are already clean on main and are unchanged.

Not addressed

Supersedes Dependabot #2472 (open) and #2471 / #2473 (closed).

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings September 30, 2026 14:15
@tunc-d
tunc-d requested review from a team as code owners September 30, 2026 14:15

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

The dependency declarations and regenerated lockfiles consistently resolve the targeted packages to patched, Node-compatible versions.

Review effort: Balanced
Findings: None

What changed in this PR

Updates vulnerable transitive and direct dependencies so package-level npm audits pass.

Changes:

  • Updates Undici across affected packages.
  • Patches brace-expansion, markdown-it, minimatch, and ip-address.
  • Regenerates corresponding lockfiles.
File Description
packages/​tool-cache/​package.json Overrides Undici.
packages/​tool-cache/​package-lock.json Locks updated Undici.
packages/​http-client/​package.json Raises Undici minimum.
packages/​http-client/​package-lock.json Locks updated Undici.
packages/​glob/​package.json Updates minimatch and overrides Undici.
packages/​glob/​package-lock.json Locks patched dependencies.
packages/​core/​package.json Overrides Undici.
packages/​core/​package-lock.json Locks updated Undici.
packages/​attest/​package.json Updates Undici and overrides ip-address.
packages/​attest/​package-lock.json Locks patched dependencies.
packages/​artifact/​package.json Adds security-related overrides.
packages/​artifact/​package-lock.json Locks patched transitive dependencies.
Files not reviewed (6)
  • packages/artifact/package-lock.json: Generated file
  • packages/attest/package-lock.json: Generated file
  • packages/core/package-lock.json: Generated file
  • packages/glob/package-lock.json: Generated file
  • packages/http-client/package-lock.json: Generated file
  • packages/tool-cache/package-lock.json: Generated file

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

tunc-d and others added 2 commits September 30, 2026 16:27
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@tunc-d

tunc-d commented Sep 30, 2026

Copy link
Copy Markdown
Contributor Author

Folded into #2498

@tunc-d tunc-d closed this Sep 30, 2026
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.

2 participants