FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

security: fix Dependabot alerts in dompurify, astro, and @babel/core by sauravpanda · Pull Request #158 · browser-use/browsercode · GitHub

security: fix Dependabot alerts in dompurify, astro, and @babel/core - #158

Merged
sauravpanda merged 4 commits into
browser-use:mainfrom
sauravpanda:saurav/eng-5625-fix-dependabot-vulnerabilities-in-browser-usebrowsercode-24
Aug 26, 2026
Merged

security: fix Dependabot alerts in dompurify, astro, and @babel/core#158
sauravpanda merged 4 commits into
browser-use:mainfrom
sauravpanda:saurav/eng-5625-fix-dependabot-vulnerabilities-in-browser-usebrowsercode-24

Conversation

sauravpanda commented Aug 26, 2026
edited by cubic-dev-ai Bot
Loading

Copy link
Copy Markdown
Collaborator

Fixes ENG-5625.

Clears 23 of the 24 open Dependabot alerts (0 high, 17 medium, 7 low) by bumping three packages:

Package From To Alerts closed
dompurify 3.3.1 (session-ui) / 3.4.11 (ui, catalog) 3.4.13 19 of 20
astro 6.4.8 7.1.0 3
@babel/core 7.28.4 7.29.6 1

The remaining alert, dompurify CVE-2026-65901 (low), has no fixed release — Vanta marks it not fixable, so it needs an exception there rather than a code change.

astro 7 migration notes

packages/web moves to astro 7.1.0 (CVE-2026-73422 requires >= 7.1.0), which drags the integrations along:

  • @astrojs/starlight 0.40.0 → 0.41.7 (the astro-7-compatible line; 0.41.8+ is still inside the bunfig minimumReleaseAge window)
  • @astrojs/solid-js 6.0.1 → 7.0.2
  • @astrojs/cloudflare 13.7.0 → 14.1.7 — deliberately not 14.2.x: 14.2.0 imports beginContentEntryCollection from astro internals, which only exists from astro 7.2.0, despite the 14.2.0 peer range claiming ^7.0.0. The 14.2.x releases that fixed the peer range (^7.2.0) can't pair with astro 7.1.0 at all.
  • The root overrides pin for astro/starlight and the toolbeam-docs-theme peer patch are refreshed the same way as the astro 6 migration (security: bump vulnerable npm dependencies + migrate docs to astro 6 #109), so the unmaintained theme keeps deduping onto the site's versions instead of pulling a nested older astro.
  • prerenderEnvironment: "node" carries over unchanged — the option still exists with the same workerd default in adapter v14.

The nested @babel/core@7.28.0 pinned by @opentui/solid stays as-is; Dependabot keys these alerts off the package manifests and has never flagged that copy.

Verification

  • Full astro build of packages/web: all locales prerender, config schema generated, Pagefind indexed 649 pages, sitemap created
  • bun run typecheck + bun test src in packages/ui (9 pass) and packages/session-ui (75 pass — includes the DOMPurify.sanitize/addHook consumer in markdown-cache.tsx)
  • Root bun run typecheck: 17/17 tasks pass

nix/hashes.json is left to the post-merge bot that regenerates node_modules hashes, matching previous dependency PRs.


Summary by cubic

Bumps dompurify, astro, and @babel/core to clear 23 of the 24 open Dependabot alerts, and moves solid-js to 1.9.15 so the docs site's Solid island satisfies @astrojs/solid-js@7.0.2's peer range. The remaining low-severity dompurify alert has no fixed release and needs a Vanta exception, not a code change.

Migration Notes

  • packages/web moves to astro 7.1.0; this brings @astrojs/starlight, @astrojs/solid-js, and @astrojs/cloudflare major bumps (14.1.7 is pinned deliberately, because 14.2.x imports an astro internal that only exists from 7.2.0).
  • @astrojs/markdown-remark moves 7.2.0 → 7.2.1 to match astro 7.1.0's exact optional peer.
  • solid-js moves 1.9.10 → 1.9.15: the old patch is dropped because Stale read from <Show> under a transition solidjs/solid#2046 ships upstream in 1.9.14+.
  • A root solid-js override keeps solid a singleton; @opentui/solid, @opentui/keymap, and @solidjs/start pin 1.9.10 and would otherwise grow nested copies.
  • The root overrides pin and the toolbeam-docs-theme peer patch are refreshed so the theme still dedupes onto the site's astro/starlight versions.
  • prerenderEnvironment: "node" carries over unchanged.
  • The nested @babel/core under @opentui/solid stays as-is; Dependabot flags only manifest-declared copies.

Written for commit 67bd12b. Summary will update on new commits.

…(ENG-5625)

Clears 23 of the 24 open Dependabot alerts (17 medium, 7 low):

- dompurify 3.3.1 (session-ui) / 3.4.11 (ui, catalog) -> 3.4.13: closes 19
  of 20 dompurify alerts. CVE-2026-65901 (low) has no fixed release and
  needs a Vanta exception instead.
- astro 6.4.8 -> 7.1.0 (CVE-2026-59727, CVE-2026-59729, CVE-2026-73422;
  the last one requires >= 7.1.0). Along for the major bump:
  @astrojs/starlight 0.40.0 -> 0.41.7, @astrojs/solid-js 6.0.1 -> 7.0.2,
  and @astrojs/cloudflare 13.7.0 -> 14.1.7 (14.2.0 imports an astro
  internal that only exists from 7.2.0 despite its ^7.0.0 peer range).
  The astro/starlight overrides pin and the toolbeam-docs-theme peer
  patch are refreshed the same way as the astro 6 migration (PR browser-use#109) so
  the unmaintained theme keeps deduping onto the site's versions.
- @babel/core 7.28.4 -> 7.29.6 (CVE-2026-49356).

Verified: full astro build of packages/web (all locales + Pagefind),
typecheck + tests for ui and session-ui, root turbo typecheck.

chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e812c25c72

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread packages/web/package.json

cubic-dev-ai Bot left a comment
edited
Loading

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

All reported issues were addressed across 8 files

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

Comment thread packages/web/package.json
Comment thread packages/web/package.json
…ange

Review feedback on browser-use#158: @astrojs/solid-js@7.0.2 peer-requires
solid-js ^1.9.13 while the catalog pinned 1.9.10, leaving the docs
site's Solid island on an unsupported peer combination.

- Catalog solid-js 1.9.10 -> 1.9.15.
- Drop patches/solid-js@1.9.10.patch: it cherry-picked solidjs/solid#2046,
  which ships upstream in 1.9.14+.
- Add a root solid-js override: @opentui/solid, @opentui/keymap, and
  @solidjs/start pin exactly 1.9.10, and without the override they each
  grew a nested solid-js copy — solid must stay a singleton per app.
  They previously ran against the deduped, patched 1.9.10, which is
  behaviorally 1.9.15's runComputation fix anyway.

Verified: single solid-js@1.9.15 in bun.lock, no peer warnings on
install, full astro build of packages/web, typecheck for tui/ui/
session-ui + root turbo (17/17), ui and session-ui tests.

chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 657519ad3c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread packages/web/package.json
astro@7.1.0 optionally peers @astrojs/markdown-remark at exactly 7.2.1;
packages/web pinned 7.2.0. Starlight's ^7.2.0 peer is satisfied either
way. Verified with a full astro build.
sauravpanda merged commit b48dbef into browser-use:main Aug 26, 2026
3 checks passed
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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL