Skip to content

[codex] fix ws dependabot alert#3963

Open
szdziedzic wants to merge 2 commits into
mainfrom
szdziedzic-codex/fix-dependabot-462-ws
Open

[codex] fix ws dependabot alert#3963
szdziedzic wants to merge 2 commits into
mainfrom
szdziedzic-codex/fix-dependabot-462-ws

Conversation

@szdziedzic

@szdziedzic szdziedzic commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Why

Dependabot reported security alert 462 for ws (GHSA-96hv-2xvq-fx4p, CVE-2026-48779). The worker package pinned vulnerable ws@8.18.0, and the lockfile also contained vulnerable transitive ws@8.17.1 and ws@5.2.4 resolutions. The patched releases are 8.21.0 for 8.x and 5.2.5 for 5.x.

How

Updated @expo/worker to depend on ws@8.21.0 and re-resolved ws with Yarn so the lockfile now uses ws@8.21.0 for 8.x descriptors and ws@5.2.5 for the existing 5.x descriptor. Added a main changelog chore entry for the security bump.

Test Plan

  • corepack yarn fmt
  • corepack yarn install --immutable
  • corepack yarn why ws
  • corepack yarn lint-changelog
  • git diff --check

corepack yarn install --immutable still reports the same pre-existing peer dependency warnings for graphql, oxlint-tsgolint, eslint, and @types/node.

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

✅ Thank you for adding the changelog entry!

@szdziedzic szdziedzic requested a review from sjchmiela July 6, 2026 17:04
@szdziedzic szdziedzic marked this pull request as ready for review July 6, 2026 17:04
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

Subscribed to pull request

File Patterns Mentions
**/* @douglowder

Generated by CodeMention

Warning: The preamble and epilogue options in commentConfiguration are deprecated. Use template instead.

@codecov

codecov Bot commented Jul 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.32%. Comparing base (4855745) to head (12ea548).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3963   +/-   ##
=======================================
  Coverage   59.32%   59.32%           
=======================================
  Files         936      936           
  Lines       41169    41169           
  Branches     8670     8670           
=======================================
  Hits        24421    24421           
  Misses      16654    16654           
  Partials       94       94           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@szdziedzic szdziedzic requested a review from douglowder July 7, 2026 09:10
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