Skip to content

fix(deps): bump dompurify 3.4.1 → 3.4.10 (resolves 8 security alerts)#386

Merged
jackmusick merged 1 commit into
mainfrom
fix/bump-dompurify-3.4.10
Jun 17, 2026
Merged

fix(deps): bump dompurify 3.4.1 → 3.4.10 (resolves 8 security alerts)#386
jackmusick merged 1 commit into
mainfrom
fix/bump-dompurify-3.4.10

Conversation

@jackmusick

Copy link
Copy Markdown
Collaborator

Dependabot's client npm group (#383) bumped 41 packages but left dompurify at 3.4.1 (it's not in the ignore list — just wasn't grouped), leaving 5 open dompurify Dependabot alerts + transitive babel/js-yaml ones.

Change

  • dompurify ^3.4.1^3.4.10 in both dependencies and overrides (the override ensures transitive consumers like monaco-editor also get the patched version). Lock resolves to 3.4.11 — above all alert fix versions (3.4.6–3.4.9).

Alerts resolved

  • 2× medium (IN_PLACE clobber / shadow-root bypass), 1× medium (hook mutation), lows (SAFE_FOR_TEMPLATES bypass, Trusted-Types clearConfig survival)
  • The one remaining dompurify alert (IN_PLACE nodeName, no patch available) does not apply — SafeHTMLRenderer sanitizes in WHOLE_DOCUMENT mode, not IN_PLACE.

On the 3.4.10 × happy-dom history

The incompatibility that bit us earlier is already mitigated — SafeHTMLRenderer's test runs under jsdom (per-file env), so the newer dompurify works.

Verification

  • Full client suite: 198 files / 1441 tests, 0 failures
  • SafeHTMLRenderer: 4/4
  • tsc clean (1 pre-existing unrelated lint warning)
  • npm ci exits 0 under both node:20 (test runner) and node:26 (image build)

🤖 Generated with Claude Code

Dependabot's client npm group (#383) bumped 41 packages but left dompurify at
3.4.1, leaving 5 open dompurify alerts (2 med IN_PLACE/clobber, lows for
SAFE_FOR_TEMPLATES bypass + Trusted-Types clearConfig) plus transitive babel/
js-yaml. dompurify isn't in the dependabot ignore list — it just wasn't grouped.
Bump the direct dep AND the overrides pin (so transitive consumers like
monaco-editor also get the patched version); lock resolves to 3.4.11, above all
the alert fix versions (3.4.6-3.4.9). The one remaining dompurify alert
(IN_PLACE nodeName, no patch available) does not apply — SafeHTMLRenderer
sanitizes in WHOLE_DOCUMENT mode, not IN_PLACE.

The 3.4.10 happy-dom incompatibility that bit us earlier is already mitigated:
SafeHTMLRenderer's test runs under jsdom. Verified: full client suite 198 files
/ 1441 tests pass, SafeHTMLRenderer 4/4, tsc clean, npm ci exits 0 under both
node:20 (test runner) and node:26 (image build).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@jackmusick jackmusick enabled auto-merge June 17, 2026 14:11
@jackmusick jackmusick added this pull request to the merge queue Jun 17, 2026
Merged via the queue into main with commit 380bfc7 Jun 17, 2026
16 checks passed
@jackmusick jackmusick deleted the fix/bump-dompurify-3.4.10 branch June 17, 2026 14:43
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