Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix(systemjs): improve module string name support#17973

Merged
nicolo-ribaudo merged 3 commits into
babel:mainfrom
JLHwung:fix-string-export-specifier-systemjs
May 5, 2026
Merged

fix(systemjs): improve module string name support#17973
nicolo-ribaudo merged 3 commits into
babel:mainfrom
JLHwung:fix-string-export-specifier-systemjs

Conversation

@JLHwung
Copy link
Copy Markdown
Contributor

@JLHwung JLHwung commented May 3, 2026

Q                       A
Fixed Issues? transform-systemjs outputs invalid AST when module string specifier is used with export *
Patch: Bug Fix? Yes
Major: Breaking Change?
Minor: New Feature?
Tests Added + Pass? Yes
Documentation PR Link
Any Dependency Changes?
License MIT

This PR fixes incomplete implementation in #12091, where we missed the string module name support in the exportStarTarget branch of the constructExportCall call.

@JLHwung JLHwung added PR: Bug Fix 🐛 A type of pull request used for our changelog categories area: modules 7.x: needs backport labels May 3, 2026
@babel-bot
Copy link
Copy Markdown
Collaborator

babel-bot commented May 3, 2026

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/61451

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 3, 2026

Open in StackBlitz

commit: c76d4b0

Comment thread packages/babel-plugin-transform-modules-systemjs/src/index.ts
@nicolo-ribaudo nicolo-ribaudo merged commit 5abf7bb into babel:main May 5, 2026
56 checks passed
@nicolo-ribaudo nicolo-ribaudo deleted the fix-string-export-specifier-systemjs branch May 5, 2026 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: modules PR: Bug Fix 🐛 A type of pull request used for our changelog categories

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants