Skip to content

[codex] fix handlebars dependabot alert#3958

Open
szdziedzic wants to merge 2 commits into
mainfrom
szdziedzic-codex/fix-dependabot-334-handlebars
Open

[codex] fix handlebars dependabot alert#3958
szdziedzic wants to merge 2 commits into
mainfrom
szdziedzic-codex/fix-dependabot-334-handlebars

Conversation

@szdziedzic

@szdziedzic szdziedzic commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Why

Dependabot reported security alert 334 for the transitive dev dependency handlebars (GHSA-2w6w-674q-4c4q, CVE-2026-33937). The existing lockfile resolved the ^4.7.7 and ^4.7.8 ranges to vulnerable 4.7.8; 4.7.9 is the patched release.

How

Re-resolved handlebars with Yarn so the lockfile now pins handlebars@4.7.9 for the existing transitive ranges. Added a main changelog chore entry for the security bump.

Test Plan

  • corepack yarn fmt
  • corepack yarn install --immutable
  • corepack yarn why handlebars
  • 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 16:24
@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 (a848411) to head (7fa7571).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3958      +/-   ##
==========================================
+ Coverage   59.28%   59.32%   +0.05%     
==========================================
  Files         935      936       +1     
  Lines       41093    41169      +76     
  Branches     8660     8670      +10     
==========================================
+ Hits        24356    24421      +65     
- Misses      16643    16654      +11     
  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 marked this pull request as ready for review July 6, 2026 16:31
@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.

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