Skip to content

Override terser and babel-traverse deps#34245

Open
GoodDayForSurf wants to merge 3 commits into
DevExpress:25_2from
GoodDayForSurf:25_2_override_systemjs_builder_deps
Open

Override terser and babel-traverse deps#34245
GoodDayForSurf wants to merge 3 commits into
DevExpress:25_2from
GoodDayForSurf:25_2_override_systemjs_builder_deps

Conversation

@GoodDayForSurf

Copy link
Copy Markdown
Contributor
  • use patches for compatibility systemjs-builder with updated terser and babel-traverse

+ use patches for compatibility systemjs-builder with updated terser and babel-traverse
@GoodDayForSurf GoodDayForSurf marked this pull request as ready for review July 9, 2026 18:35

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates dependency resolution and applies targeted pnpm patches to keep systemjs-builder@0.16.15 working with newer terser and to align legacy babel-core@6.26.3 behavior with the @babel/traverse implementation.

Changes:

  • Add pnpm overrides to alias babel-traverse to @babel/traverse and to force systemjs-builder to use terser@5.46.2.
  • Add pnpm patchedDependencies entries for babel-core@6.26.3 and systemjs-builder@0.16.15.
  • Introduce patch files updating systemjs-builder minification/sourcemap handling for modern terser and adding defensive guards in babel-core’s shadow-functions plugin.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.

File Description
pnpm-workspace.yaml Adds overrides for babel-traverse and systemjs-builder>terser, and registers new patched dependencies.
pnpm-lock.yaml Reflects the new overrides and patch hashes; updates resolved dependency graph accordingly.
patches/systemjs-builder@0.16.15.patch Refactors lib/output.js minification to use the terser v5 API and make the minification path Promise-based.
patches/babel-core@6.26.3.patch Adds guards for missing traversal path APIs when running with @babel/traverse-backed NodePath objects.
Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants