Bump @babel/plugin-transform-modules-systemjs from 7.25.7 to 7.29.4#1463
Open
dependabot[bot] wants to merge 1 commit into
Open
Bump @babel/plugin-transform-modules-systemjs from 7.25.7 to 7.29.4#1463dependabot[bot] wants to merge 1 commit into
dependabot[bot] wants to merge 1 commit into
Conversation
Bumps [@babel/plugin-transform-modules-systemjs](https://github.com/babel/babel/tree/HEAD/packages/babel-plugin-transform-modules-systemjs) from 7.25.7 to 7.29.4. - [Release notes](https://github.com/babel/babel/releases) - [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md) - [Commits](https://github.com/babel/babel/commits/v7.29.4/packages/babel-plugin-transform-modules-systemjs) --- updated-dependencies: - dependency-name: "@babel/plugin-transform-modules-systemjs" dependency-version: 7.29.4 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
There was a problem hiding this comment.
Stale comment
No blocking findings.
Security
- This bump resolves
GHSA-fv7c-fp4j-7gwp/CVE-2026-44728, a High-severity code-generation/code-injection issue in@babel/plugin-transform-modules-systemjsaffecting7.12.0through7.29.3;7.29.4is the patched release.- I did not find any additional advisories introduced by the new transitive helper packages pulled in alongside the fix.
- Supply-chain note: Dependabot reports a new npm releaser (
GitHub Actions), but the package release corresponds to Babel's officialv7.29.4release and a verified upstream fix commit, so I did not see a separate ownership or provenance red flag from this update.Safety Of Merging
- This PR is lockfile-only (
yarn.lock).yarn whyshows the package is only present transitively via@babel/preset-env.- The upstream patch is narrowly scoped: it fixes how the SystemJS transform emits string-named imports/exports, especially around
import */export *interop.- I could not find any checked-in repo config that enables
modules: "systemjs"or emitsSystem.register(...); the Babel paths inpackage.json,webpack.config.js, andconfig/jest/babelTransform.jsall look like the standardbabel-preset-react-appflow. That makes behavioral regression risk low for this repository's normal build/test path.Assumptions
- This assessment assumes there is no external, untracked Babel configuration in CI or downstream packaging that forces
@babel/preset-envto usemodules: "systemjs".Recommendation
- Merge. This closes a newly published High advisory with low observed compatibility risk for this repo.
- The only remaining thing to watch is the PR's Cypress check, since I did not run Cypress locally.
Local Verification
yarn install --immutableyarn lintCI=true yarn test --coverage --maxWorkers=4 --workerThreads=true --reporters=default --reporters=jest-junit --reporters=jest-github-actions-reporter-> 92 suites passed, 816 tests passedyarn build-> succeeded; only the existing webpack asset-size warnings were reportedSent by Cursor Automation: Editor-UI - Tests Dependabot PRs
There was a problem hiding this comment.
No blocking findings.
Security
- I did not find any published CVEs or advisory hits for
@babel/plugin-transform-modules-systemjsin either7.25.7or7.29.4from public package scanners. - This update does not appear to resolve a known security issue; it looks like a routine bug-fix / maintenance refresh.
- Supply-chain signal looks normal: npm metadata still points at the official
babel/babelrepository, and the listed maintainers are the expected Babel maintainers. Dependabot notes the release was published by GitHub Actions, but I did not find evidence of a suspicious ownership or source change.
Safety Of Merging
- This PR only changes
yarn.lock; the dependency is indirect via@babel/preset-env. - In this repository, Babel is used in the build/test toolchain (
babel-loader,babel-jest,babel-preset-react-app), and I found no references toSystemJS,System.register, or related configuration in the codebase. - Upstream
7.29.4is a Babel bug fix for SystemJS module string-name handling. Because this project does not appear to emit SystemJS modules, behavioral risk here is very low. - The lockfile also refreshes a small set of Babel helper packages (
@babel/traverse,@babel/types,@babel/parser, and related helpers), so the practical risk is limited to build-time transpilation internals rather than runtime application behavior.
Recommendation
- Recommend merge. I do not see a security reason to hold this PR, and the codebase does not appear to exercise the only behavior changed upstream.
Local Verification
yarn install --immutable✅yarn lint✅CI=true yarn test✅ (92suites /816tests passed)yarn start✅ (webpack compiled successfully)yarn exec cypress runcould not be completed in this environment because the required Cypress binary was not present, and downloading it fromdownload.cypress.iofailed with an SSL/network error from the cloud runner.
Residual Risk
- If you want full end-to-end coverage before merging, rerun Cypress in CI or from an environment that can reach
download.cypress.io. I do not see anything in this particular dependency bump that suggests an e2e-only regression.
Sent by Cursor Automation: Editor-UI - Tests Dependabot PRs
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
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Bumps @babel/plugin-transform-modules-systemjs from 7.25.7 to 7.29.4.
Release notes
Sourced from @babel/plugin-transform-modules-systemjs's releases.
... (truncated)
Commits
a458f66v7.29.432ebd5a[7.x backport]fix(systemjs): improve module string name support (#17974)aa8394ev7.29.00053db6Update polyfill packages (#17727)61647aev7.28.5a177d55[Babel 8] Uset.traverseFastto replace somepath.traverse(#17518)eebd3a0v7.27.1317e332Enforce node protocol import (#17207)fdc0fb5[Babel 8] Bump nodejs requirements to^20.19.0 || >= 22.12.0(#17204)cd24cc0chore: Update TS 5.7 (#17053)Maintainer changes
This version was pushed to npm by GitHub Actions, a new releaser for
@babel/plugin-transform-modules-systemjssince your current version.Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)You can disable automated security fix PRs for this repo from the Security Alerts page.