revert some to all

Author: fi3ework

packages/core/src/config.ts

@@ -339,7 +339,7 @@ export function composeMinifyConfig(config: LibConfig): RsbuildConfig {
               toplevel: format !== 'mf',
             },
             format: {
-              comments: 'some',
+              comments: 'all',
               preserve_annotations: true,
             },
           },

tests/integration/minify/index.test.ts

@@ -7,13 +7,14 @@ describe('minify config', () => {
     const fixturePath = join(__dirname, 'default');
     const { entries } = await buildAndGetResults({ fixturePath });
     expect(entries.esm).toMatchInlineSnapshot(`
-    "/*! For license information please see index.js.LICENSE.txt */
-    import * as __WEBPACK_EXTERNAL_MODULE_react_jsx_runtime__ from "react/jsx-runtime";
-    /*! Legal Comment */ const foo = ()=>{};
-    const Button = ()=>/*#__PURE__*/ (0, __WEBPACK_EXTERNAL_MODULE_react_jsx_runtime__.jsx)('button', {});
-    export { Button, foo };
-    "
-  `);
+      "/*! For license information please see index.js.LICENSE.txt */
+      import * as __WEBPACK_EXTERNAL_MODULE_react_jsx_runtime__ from "react/jsx-runtime";
+      /*! Legal Comment */ const foo = ()=>{};
+      // normal comment
+      const Button = ()=>/*#__PURE__*/ (0, __WEBPACK_EXTERNAL_MODULE_react_jsx_runtime__.jsx)('button', {});
+      export { Button, foo };
+      "
+    `);
   });
 
   test('minify is disabled, nothing will be stripped', async () => {
@@ -55,9 +56,10 @@ describe('minify config (mf)', () => {
     const { mfExposeEntry } = await buildAndGetResults({ fixturePath });
     // biome-ignore format: snapshot
     expect(mfExposeEntry).toMatchInlineSnapshot(`
-    "/*! For license information please see __federation_expose_default_export.js.LICENSE.txt */
-    "use strict";(globalThis["default_minify"]=globalThis["default_minify"]||[]).push([["249"],{163:function(__unused_webpack_module,__webpack_exports__,__webpack_require__){__webpack_require__.r(__webpack_exports__);__webpack_require__.d(__webpack_exports__,{Button:function(){return Button},foo:function(){return foo}});var react_jsx_runtime__WEBPACK_IMPORTED_MODULE_0__=__webpack_require__(37);/*! Legal Comment */const foo=()=>{};const Button=()=>/*#__PURE__*/(0,react_jsx_runtime__WEBPACK_IMPORTED_MODULE_0__.jsx)("button",{})}}]);"
-  `);
+      "/*! For license information please see __federation_expose_default_export.js.LICENSE.txt */
+      "use strict";(globalThis["default_minify"]=globalThis["default_minify"]||[]).push([["249"],{163:function(__unused_webpack_module,__webpack_exports__,__webpack_require__){__webpack_require__.r(__webpack_exports__);__webpack_require__.d(__webpack_exports__,{Button:function(){return Button},foo:function(){return foo}});/* ESM import */var react_jsx_runtime__WEBPACK_IMPORTED_MODULE_0__=__webpack_require__(37);/*! Legal Comment */const foo=()=>{};// normal comment
+      const Button=()=>/*#__PURE__*/(0,react_jsx_runtime__WEBPACK_IMPORTED_MODULE_0__.jsx)("button",{})}}]);"
+    `);
   });
 
   test('minify is disabled by the user, nothing not be stripped', async () => {