Skip to content

security: close all 18 open Dependabot alerts - #307

Merged
htilly merged 1 commit into
masterfrom
security/dependency-patch-2026-08
Aug 8, 2026
Merged

security: close all 18 open Dependabot alerts#307
htilly merged 1 commit into
masterfrom
security/dependency-patch-2026-08

Conversation

@htilly

@htilly htilly commented Aug 8, 2026

Copy link
Copy Markdown
Owner

Summary

Closes every currently-open Dependabot security alert (18 total) by bumping the actual vulnerable transitive packages via overrides, plus picks up the routine bumps from the still-open dependency PRs.

Security fixes

Also included

Routine bumps from the remaining open Dependabot PRs, superseded by this branch:

Once this merges, Dependabot should auto-close #292, #296, #299, #300, #301, #304, #306 as superseded — worth a manual check if it doesn't.

Out of scope

ip SSRF via sonos (GHSA-2p57-rm9w-gvfp) has no upstream fix (first_patched_version: null) and was already reviewed/dismissed on Dependabot in 2024. npm audit will still show this one after merge.

Testing

  • npm audit: down to just the one unfixable ip/sonos finding above
  • npm test: 666/666 passing (one unrelated auth-handler.test.mjs timing flake seen once during a background run — confirmed pre-existing on unmodified master too, not caused by this change)

🤖 Generated with Claude Code

Bump transitive deps via overrides + direct dependency bumps to close
every currently-open Dependabot security alert:

- axios ^1.17.0 -> ^1.18.1 (override): closes 10 alerts (#109, #111-119)
- undici ^7.28.0 -> ^7.29.0 (override): closes 5 alerts (#124-128)
- js-yaml: add ^4.3.1 override (mocha pins ^4.1.0 so it never
  self-updates): closes 2 alerts (#108, #120). Note this goes one patch
  past what PR #304 proposed (4.3.0), since GHSA-5p4m-2wfm-xmqj landed
  after that PR was opened and isn't fixed until 4.3.1.
- brace-expansion 2.1.1->2.1.4, 5.0.6->5.0.9 (npm update, already
  within existing minimatch ranges, no override needed): closes #110
  plus two more advisories npm audit flagged that Dependabot hadn't
  opened a PR for yet (GHSA-mh99-v99m-4gvg, GHSA-rgw5-rvv9-x895).

Also picked up the routine bumps from the remaining open Dependabot
PRs (#292, #296, #299, #300, #301): @sefinek/google-tts-api,
@slack/web-api, openai, posthog-node, soundcraft-ui-connection.

Remaining `npm audit` finding (ip SSRF via sonos, GHSA-2p57-rm9w-gvfp)
has no upstream fix (first_patched_version: null) and was already
reviewed/dismissed on Dependabot in 2024. Out of scope here.

Full suite: 666 passing. (One auth-handler timing test flaked once
during a background run and passed cleanly on isolation + two
subsequent full runs — pre-existing flake, unrelated to this change.)
Copilot AI lite review requested due to automatic review settings August 8, 2026 00:28
@cursor

cursor Bot commented Aug 8, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@htilly
htilly merged commit 60a17c0 into master Aug 8, 2026
6 checks passed

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.

Pull request overview

This PR aims to close all currently open Dependabot security alerts by updating vulnerable transitive dependencies via npm overrides, while also rolling up routine dependency bumps already proposed in other Dependabot PRs.

Changes:

  • Bumped several direct dependencies (e.g., @slack/web-api, openai, posthog-node, soundcraft-ui-connection, @sefinek/google-tts-api).
  • Added/updated overrides for vulnerable transitive packages (axios, undici, js-yaml) and refreshed the lockfile to pick up patched transitive versions.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
package.json Updates direct dependency versions and adds/adjusts overrides to force patched transitive dependency versions.
package-lock.json Updates the resolved dependency tree to reflect the new direct deps and override-enforced transitive versions.

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

htilly pushed a commit that referenced this pull request Aug 8, 2026
Security dependency patch (PR #307): closes all 18 open Dependabot
alerts (axios, undici, js-yaml, brace-expansion). See docs/CHANGELOG.md.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@htilly
htilly deleted the security/dependency-patch-2026-08 branch August 8, 2026 02:18
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