Skip to content

Fix dependabot security vulnerabilities in npm dependencies#233

Merged
lcollins merged 2 commits intomasterfrom
copilot/fix-dependabot-alerts
Mar 5, 2026
Merged

Fix dependabot security vulnerabilities in npm dependencies#233
lcollins merged 2 commits intomasterfrom
copilot/fix-dependabot-alerts

Conversation

Copy link

Copilot AI commented Mar 5, 2026

9 vulnerabilities (1 high, 8 moderate) in transitive and direct dependencies, primarily ReDoS and undici decompression chain issues.

Dependency updates

  • @actions/github: ^6.0.0^8.0.0 — v8 pulls in @actions/http-client ^3.x which uses undici ^6.23.0, fixing the unbounded decompression chain CVE. v9 was skipped — it's ESM-only and incompatible with this project's CommonJS output.
  • @octokit/rest: latest (resolved 20.1.1) → ^22.0.1 — fixes ReDoS in @octokit/plugin-paginate-rest (GHSA-h5c3-5r3r-rr8q)
  • Added overrides: { "undici": ">=6.23.0" } — forces safe undici version for the @actions/core → @actions/http-client → undici chain, which remains on v5.x in the 1.x release line

TypeScript config

Added "lib": ["ES2022"] to tsconfig.json@octokit/request-error v7 (pulled in by @actions/github v8) references ErrorOptions, introduced in ES2022. Output target remains ES6.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…, override undici

Co-authored-by: lcollins <528874+lcollins@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix Dependabot alerts in the project Fix dependabot security vulnerabilities in npm dependencies Mar 5, 2026
@lcollins lcollins marked this pull request as ready for review March 5, 2026 14:39
Copilot AI review requested due to automatic review settings March 5, 2026 14:39
@lcollins lcollins merged commit fc630db into master Mar 5, 2026
6 of 7 checks passed
@lcollins lcollins deleted the copilot/fix-dependabot-alerts branch March 5, 2026 14:40
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses 9 security vulnerabilities (1 high, 8 moderate) in npm dependencies, primarily ReDoS and undici decompression chain issues. The changes are limited to dependency version bumps, an npm override, and a minor TypeScript config adjustment needed by the new dependency types.

Changes:

  • Bumped @actions/github from ^6.0.0 to ^8.0.0 and @octokit/rest from latest to ^22.0.1 to fix known CVEs in transitive dependencies.
  • Added overrides: { "undici": ">=6.23.0" } to force a safe undici version for the @actions/core → @actions/http-client → undici chain.
  • Added "lib": ["ES2022"] to tsconfig.json to support ErrorOptions type referenced by @octokit/request-error v7.

Reviewed changes

Copilot reviewed 1 out of 6 changed files in this pull request and generated 1 comment.

File Description
package.json Bumped @actions/github and @octokit/rest versions; added undici override
package-lock.json Updated lockfile reflecting all transitive dependency changes
tsconfig.json Added "lib": ["ES2022"] for ES2022 type definitions
dist/licenses.txt Updated bundled licenses: removed @fastify/busboy, deprecation, once, wrappy; added fast-content-type-parse, json-with-bigint

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

"unescape": "latest"
},
"overrides": {
"undici": ">=6.23.0"
Copy link

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

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

The unbounded override "undici": ">=6.23.0" resolves to v7.22.0 in the current lockfile, which is fine. However, the open-ended >= range means a future npm install (without a lockfile) could pull in undici v8+, potentially introducing breaking changes. Consider pinning to a major range like "^6.23.0" or ">=6.23.0 <8" to limit the blast radius to tested major versions. That said, since the lockfile pins v7.22.0 and @actions/github v8 itself depends on ^6.23.0, a narrower override would still be effective.

Suggested change
"undici": ">=6.23.0"
"undici": ">=6.23.0 <8"

Copilot uses AI. Check for mistakes.
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.

3 participants