Fix tsup configuration for vanilla Webpack 4 without babel-loader
#5322
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.
Changelog Entry
Fixed
babel-loader
, by @compulim, in PR #5322Description
There are some limitations on vanilla Webpack 4 without
babel-loader
:noExternal: 'globalize'
only for ESMglobalize
is CJS-only, we bundle it inside Web Chat so it don't do "ESM -> CJS"??
operator.js
, otherwise, Webpack 4 may point to.mjs
which will require BabelAlso, in
tsup
, it disabled code splitting for CJS asesbuild
does not support it. We are using experimental flag to re-enable it.Design
Specific Changes
globalize
only in ESM but not CJSglobalize
is CJS only and it would cross the ESM -> CJS boundarybotframework-webchat-[api|component]/[decorator|internal]
I have added tests and executed them locallyCHANGELOG.md
I have updated documentationReview Checklist
Accessibility reviewed (tab order, content readability, alt text, color contrast)Browser and platform compatibilities reviewedCSS styles reviewed (minimal rules, noz-index
)Documents reviewed (docs, samples, live demo)Internationalization reviewed (strings, unit formatting)package.json
andpackage-lock.json
reviewedSecurity reviewed (no data URIs, check for nonce leak)Tests reviewed (coverage, legitimacy)