fix(module-tools): set isModule is true in swc transform (#5053)

.changeset/olive-keys-destroy.md

@@ -0,0 +1,6 @@
+---
+'@modern-js/module-tools': patch
+---
+
+fix(module-tools): Modify the configuration of isModule when converting umd products.
+fix(module-tools): 修改转换 umd 产物过程中 isModule 的配置

packages/solutions/module-tools/src/builder/feature/swc.ts

@@ -166,7 +166,11 @@ export const swcRenderChunk = {
                     type: 'umd',
                   }
                 : undefined,
-            isModule: 'unknown',
+            // If `chunk.contents` is recognized as [Script](https://swc.rs/docs/configuration/compilation#ismodule),
+            // then it will not be converted to umd format content.
+            // so we need to set `isModule` to `true`.
+            // eg: when `autoExternal` is false, then chunk.contents will be recognized as [Script]
+            isModule: format === 'umd' ? true : 'unknown',
           });
           const result = await swcCompiler.transform(
             name,

tests/integration/module/fixtures/build/autoExternal/autoExternal.test.ts

@@ -50,4 +50,11 @@ describe('autoExternal usage', () => {
     expect(content.includes(`require("postcss")`)).toBeTruthy();
     expect(content.includes(`require("path-browserify")`)).toBeFalsy();
   });
+
+  it('autoExternal is false, and format is umd', async () => {
+    const distFilePath = path.join(fixtureDir, './dist/6/umd.js');
+    const content = await fs.readFile(distFilePath, 'utf-8');
+    expect(await fs.pathExists(distFilePath)).toBeTruthy();
+    expect(content.includes(`function(global, factory)`)).toBeTruthy();
+  });
 });

tests/integration/module/fixtures/build/autoExternal/config.ts

@@ -36,5 +36,12 @@ export default defineConfig({
       },
       outDir: './dist/5',
     },
+    {
+      buildType: 'bundle',
+      input: ['./src/umd.ts'],
+      format: 'umd',
+      autoExternal: false,
+      outDir: './dist/6',
+    },
   ],
 });

tests/integration/module/fixtures/build/autoExternal/src/umd.ts

@@ -0,0 +1,3 @@
+import react from 'react';
+
+console.info(react);