Skip to content

fix: upgrade @babel/plugin-transform-modules-systemjs to address secu…#8732

Merged
rohitsaw115 merged 1 commit into
masterfrom
CGD-1102-upgrade-babel
May 11, 2026
Merged

fix: upgrade @babel/plugin-transform-modules-systemjs to address secu…#8732
rohitsaw115 merged 1 commit into
masterfrom
CGD-1102-upgrade-babel

Conversation

@yashvanthbl137-crypto
Copy link
Copy Markdown
Contributor

@yashvanthbl137-crypto yashvanthbl137-crypto commented May 11, 2026

Summary

Upgrades @babel/plugin-transform-modules-systemjs to address a high-severity security vulnerability blocking the beta release.

Security Fix

@babel/plugin-transform-modules-systemjs (7.12.0-7.29.3 → 7.29.4)

  • CVE-2026-44728 (High): Arbitrary Code Execution vulnerability
    • Affects versions 7.12.0-7.29.3 and 8.0.0-alpha.0-8.0.0-alpha.12
    • Patched in version 7.29.4

Changes

  • Added yarn resolution for @babel/plugin-transform-modules-systemjs@7.29.4 in package.json
  • Updated yarn.lock to ensure all transitive dependencies use the patched version

Ticket: CGD-1102

@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 11, 2026

CGD-1102

@yashvanthbl137-crypto yashvanthbl137-crypto marked this pull request as ready for review May 11, 2026 10:46
@yashvanthbl137-crypto yashvanthbl137-crypto requested a review from a team as a code owner May 11, 2026 10:46
Copy link
Copy Markdown
Contributor

@mohd-kashif mohd-kashif left a comment

Choose a reason for hiding this comment

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

this PR should be reviewed by appSec

Copy link
Copy Markdown

@diksha190 diksha190 left a comment

Choose a reason for hiding this comment

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

Approved

This PR correctly remediates CVE-2026-44728 (High) : an arbitrary code execution vulnerability in @babel/plugin-transform-modules-systemjs.

  • Upgrades vulnerable package 7.27.1 → 7.29.4 (patched version)
  • Uses Yarn resolutions to enforce fix across all dependencies
  • All security checks passed (Aikido, internal checks, tests)
  • Minimal attack surface - dependency-only change, no code modifications
  • No regressions detected

@rohitsaw115 rohitsaw115 requested a review from mohd-kashif May 11, 2026 13:18
@rohitsaw115 rohitsaw115 merged commit 7afac15 into master May 11, 2026
22 checks passed
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.

4 participants