Merge pull request #1391 from benjamn/fix-1390

fix: print export namespace specifiers without braces
benjamn · Mar 9, 2024 · 51d5c89 · 51d5c89
2 parents 34ca956 + 88a2bae
commit 51d5c89
Show file tree

Hide file tree

Showing 4 changed files with 61 additions and 5 deletions.
diff --git a/lib/printer.ts b/lib/printer.ts
@@ -3005,13 +3005,19 @@ function printExportDeclaration(path: any, options: any, print: any) {
       parts.push("*");
     } else if (decl.specifiers.length === 0) {
       parts.push("{}");
-    } else if (decl.specifiers[0].type === "ExportDefaultSpecifier") {
+    } else if (
+      decl.specifiers[0].type === "ExportDefaultSpecifier" ||
+      decl.specifiers[0].type === "ExportNamespaceSpecifier"
+    ) {
       const unbracedSpecifiers: any[] = [];
       const bracedSpecifiers: any[] = [];
 
       path.each(function (specifierPath: any) {
         const spec = specifierPath.getValue();
-        if (spec.type === "ExportDefaultSpecifier") {
+        if (
+          spec.type === "ExportDefaultSpecifier" ||
+          spec.type === "ExportNamespaceSpecifier"
+        ) {
           unbracedSpecifiers.push(print(specifierPath));
         } else {
           bracedSpecifiers.push(print(specifierPath));

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,7 +1,7 @@
 {
   "author": "Ben Newman <bn@cs.stanford.edu>",
   "name": "recast",
-  "version": "0.23.5",
+  "version": "0.23.6",
   "description": "JavaScript syntax tree transformer, nondestructive pretty-printer, and automatic source map generator",
   "keywords": [
     "ast",

diff --git a/test/printer.ts b/test/printer.ts
@@ -508,6 +508,56 @@ describe("printer", function () {
     assert.strictEqual(printer.printGenerically(ast).code, code);
   });
 
+  it("export namespace", function () {
+    const printer = new Printer();
+
+    assert.strictEqual(
+      printer.print({
+        type: "ExportNamedDeclaration",
+        exportKind: "value",
+        specifiers: [
+          {
+            type: "ExportNamespaceSpecifier",
+            exported: {
+              type: "Identifier",
+              name: "Foobar",
+            },
+          },
+        ],
+        source: {
+          type: "StringLiteral",
+          value: "./foo",
+        },
+      }).code,
+      `export * as Foobar from "./foo";`,
+    );
+  });
+
+  it("export type namespace", function () {
+    const printer = new Printer();
+
+    assert.strictEqual(
+      printer.print({
+        type: "ExportNamedDeclaration",
+        exportKind: "type",
+        specifiers: [
+          {
+            type: "ExportNamespaceSpecifier",
+            exported: {
+              type: "Identifier",
+              name: "Foobar",
+            },
+          },
+        ],
+        source: {
+          type: "StringLiteral",
+          value: "./foo",
+        },
+      }).code,
+      `export type * as Foobar from "./foo";`,
+    );
+  });
+
   it("export default of IIFE", function () {
     const printer = new Printer();
     let ast = b.exportDefaultDeclaration(