fix: clean up render exports from external module (#394)

crates/rolldown/src/bundler/renderer/render_esm.rs

@@ -26,7 +26,7 @@ impl<'r> AstRenderer<'r> {
             );
           }
         }
-        Module::External(_) => unreachable!(), // TODO
+        Module::External(_) => {}
       }
       self.ctx.remove_node(named_decl.span);
       RenderControl::Skip

crates/rolldown/src/bundler/renderer/render_wrapped_esm.rs

@@ -77,7 +77,7 @@ impl<'r> AstRenderer<'r> {
             );
           }
         }
-        Module::External(_) => {} // TODO
+        Module::External(_) => {}
       }
       self.ctx.remove_node(named_decl.span);
     } else {

crates/rolldown/src/bundler/stages/bundle_stage.rs

@@ -157,9 +157,7 @@ impl<'a> BundleStage<'a> {
               }
             }
           }
-          Module::External(_) => {
-            // TODO: process external module
-          }
+          Module::External(_) => {}
         }
       }
 

crates/rolldown/tests/fixtures/external/export_external/artifacts.snap

@@ -0,0 +1,30 @@
+---
+source: crates/rolldown/tests/common/case.rs
+expression: content
+input_file: crates/rolldown/tests/fixtures/export_external
+---
+# Assets
+
+## main.mjs
+
+```js
+import * as ext1 from "external";
+import * as ext from "external";
+import { a, a as a$1, a as a1, b, b as b$1, b as b1 } from "external";
+
+// foo.js
+
+
+
+const a1$1 = 1;
+console.log(a1$1);
+// main.js
+
+
+
+console.log(ext, a1, b1, ext1, a, b)
+
+
+
+export { a$1 as a, b$1 as b };
+```

crates/rolldown/tests/fixtures/.export_external/main.js → crates/rolldown/tests/fixtures/external/export_external/main.js

@@ -1,4 +1,6 @@
 import * as ext from 'external'
 import { a, b } from 'external'
 import { a1, b1, ext1 } from './foo'
-console.log(ext, a1, b1, ext1, a, b)
+console.log(ext, a1, b1, ext1, a, b)
+
+export { a, b } from 'external'

crates/rolldown/tests/fixtures/external/export_external/test.config.json

@@ -0,0 +1,9 @@
+{
+    "input": {
+        "external": [
+            "external"
+        ],
+        "treeshake": false
+    },
+    "expectExecuted": false
+}

crates/rolldown/tests/fixtures/external/import_external/artifacts.snap

@@ -0,0 +1,24 @@
+---
+source: crates/rolldown/tests/common/case.rs
+expression: content
+input_file: crates/rolldown/tests/fixtures/external/import_external
+---
+# Assets
+
+## main.mjs
+
+```js
+import * as ext$1 from "external";
+import * as ext from "external";
+import { a, a as a$1, b, b as b$1 } from "external";
+
+// foo.js
+
+
+console.log(ext$1, a$1, b$1)
+// main.js
+
+
+
+console.log(ext, a, b)
+```

crates/rolldown/tests/fixtures/external/import_external/test.config.json

@@ -0,0 +1,9 @@
+{
+    "input": {
+        "external": [
+            "external"
+        ],
+        "treeshake": false
+    },
+    "expectExecuted": false
+}