feature: improve support of ExportNamespaceSpecifiers

lib/printer.ts

@@ -3053,12 +3053,12 @@ function printExportDeclaration(path: any, options: any, print: any) {
         }
       }
     } else {
-      parts.push(
-        shouldPrintSpaces ? "{ " : "{",
-        fromString(", ").join(path.map(print, "specifiers")),
-        shouldPrintSpaces ? " }" : "}",
-      );
-    }
+    parts.push(
+      shouldPrintSpaces ? "{ " : "{",
+      fromString(", ").join(path.map(print, "specifiers")),
+      shouldPrintSpaces ? " }" : "}",
+    );
+  }
 
     if (decl.source) {
       parts.push(

package.json

@@ -53,6 +53,8 @@
     "@babel/core": "7.20.5",
     "@babel/parser": "7.20.5",
     "@babel/preset-env": "7.20.2",
+    "@types/babel__template": "^7.4.1",
+    "@types/babel__traverse": "^7.18.1",
     "@types/esprima": "4.0.3",
     "@types/glob": "8.0.0",
     "@types/mocha": "10.0.1",

test/babel.ts

@@ -1,8 +1,13 @@
 import assert from "assert";
 import { EOL as eol } from "os";
 import * as recast from "../main";
+
+import traverse, {NodePath, Node} from '@babel/traverse';
+import template from '@babel/template';
+
 const n = recast.types.namedTypes;
 const b = recast.types.builders;
+
 const nodeMajorVersion = parseInt(process.versions.node, 10);
 
 describe("Babel", function () {
@@ -455,6 +460,41 @@ describe("Babel", function () {
     );
   });
 
+  it("should not add curly braces in ExportNamedDeclaration used with ExportNamespaceSpecifier", function () {
+    const code = 'export * as fs2 from "fs/promises"';
+    const ast = recast.parse(code, parseOptions);
+
+    traverse(ast, {
+      ExportNamedDeclaration(path: NodePath) {
+        path.replaceWith(template.ast('export * as fs from "xx"') as Node);
+        path.stop();
+      }
+    })
+
+    assert.strictEqual(
+      recast.print(ast).code,
+      'export * as fs from "xx";'
+    );
+  });
+
+  it("should not add curly braces in ExportNamedDeclaration used with ExportNamespaceSpecifier: couple specifiers", function () {
+    const code = 'export * as fs2, {x, y} from "fs/promises"';
+    const ast = recast.parse(code, parseOptions);
+
+    traverse(ast, {
+      ExportNamedDeclaration(path: NodePath) {
+        path.replaceWith(template.ast('export * as fs, {x, y} from "xx"') as Node);
+        path.stop();
+      }
+    })
+
+    assert.strictEqual(
+      recast.print(ast).code,
+      'export * as fs, { x, y } from "xx";'
+    );
+  });
+
+
   it("should keep braces in !(a && b)", function () {
     const code = "(options || !options.bidirectional) ? false : true;";
     const ast = recast.parse(code, parseOptions);