[ui5-tooling-modules] License header removed when using sheetjs with ui5-tooling-modules

[marc4F]

Hello :)

When using the SheetJS library with the following dependency:

"dependencies": {
  "xlsx": "https://cdn.sheetjs.com/xlsx-0.20.3/xlsx-0.20.3.tgz"
}

the ui5-tooling-modules plugin (version ^3.20.6) appears to remove the license comment at the top of the xlsx.js file in the thirdparty folder. Specifically, the following license header is removed:

/*! xlsx.js (C) 2013-present SheetJS -- http://sheetjs.com */

To Reproduce

1. Add the above dependency to your UI5 project.
2. Build the project.
3. Check the generated thirdparty folder and note that the license comment is missing from the top of xlsx.js.

Expected Behavior

The license header should remain at the top of the generated xlsx.js file to maintain legal and attribution requirements.

Additional Context

• ui5-tooling-modules version: 3.20.6
• The license comment is present in the original library but is missing after the UI5 build process.

[petermuessig]

Thanks @marc4F - need to check what rollup can do here...

[petermuessig]

Fixed with version 3.22.0

[marc4F]

Hi @petermuessig

thanks for the fast response. But it seems to still remove the license info

[petermuessig]

@marc4F - I need to check - BTW: did you scroll through the file - the copyright was somewhere in the middle of the file for me.