Merge pull request #138 from SebastienGllmt/fix-export-list

bugfix: export list of types

Author: orta

src/__tests__/__snapshots__/type-exports.spec.ts.snap

@@ -1,5 +1,21 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
+exports[`should handle export list syntax 1`] = `
+"declare type ComplexType =
+  | {
+      type: number,
+      ...
+    }
+  | {
+      type: string,
+      ...
+    };
+export type { ComplexType };
+declare var foo: 5;
+declare export { foo };
+"
+`;
+
 exports[`should handle exported types 1`] = `
 "export type FactoryOrValue<T> = T | (() => T);
 export type Maybe<T> =

src/__tests__/type-exports.spec.ts

@@ -10,3 +10,19 @@ export type Maybe<T> = {type: 'just', value: T} | {type: 'nothing'}
   expect(beautify(result)).toMatchSnapshot();
   expect(result).toBeValidFlowTypeDeclarations();
 });
+
+it("should handle export list syntax", () => {
+  const ts = `
+declare type ComplexType = {
+    type: number
+} | {
+    type: string
+};
+export type { ComplexType };
+const foo = 5;
+export { foo };
+`;
+  const result = compiler.compileDefinitionString(ts, { quiet: true });
+  expect(beautify(result)).toMatchSnapshot();
+  expect(result).toBeValidFlowTypeDeclarations();
+});

src/nodes/export-declaration.ts

@@ -19,12 +19,21 @@ export default class ExportDeclaration extends Node<ExportDeclarationType> {
     if (this.raw.exportClause) {
       // @ts-expect-error todo(flow->ts)
       const elements = this.raw.exportClause.elements;
+      const isTypeImport = this.raw.isTypeOnly;
+
       let specifier = "";
       if (this.raw.moduleSpecifier)
         specifier = `from '${this.raw.moduleSpecifier.text}';`;
-      return `declare export {
-        ${elements.map(node => printers.node.printType(node))}
-      }${specifier}\n`;
+
+      const generateOutput = prefix => {
+        return `${prefix} {
+          ${elements.map(node => printers.node.printType(node))}
+        }${specifier}\n`;
+      };
+
+      return isTypeImport
+        ? generateOutput(`export type`)
+        : generateOutput(`declare export`);
     } else {
       return `declare export * from '${this.raw.moduleSpecifier.text}';\n`;
     }