(improvement): appending type for type exports

generators/typescript/sdk/VERSION

@@ -1 +1 @@
-0.15.0-rc1
+0.15.0-rc1

generators/typescript/sdk/generator/src/SdkGenerator.ts

@@ -502,7 +502,8 @@ export class SdkGenerator {
                             context.requestWrapper
                                 .getGeneratedRequestWrapper(packageId, endpoint.name)
                                 .writeToFile(context);
-                        }
+                        },
+                        addExportTypeModifier: true
                     });
                 }
             }
@@ -523,7 +524,8 @@ export class SdkGenerator {
                             context.sdkInlinedRequestBodySchema
                                 .getGeneratedInlinedRequestBodySchema(packageId, endpoint.name)
                                 .writeToFile(context);
-                        }
+                        },
+                        addExportTypeModifier: true
                     });
                 }
             }
@@ -810,10 +812,12 @@ export class SdkGenerator {
 
     private withSourceFile({
         run,
-        filepath
+        filepath,
+        addExportTypeModifier
     }: {
         run: (args: { sourceFile: SourceFile; importsManager: ImportsManager }) => void;
         filepath: ExportedFilePath;
+        addExportTypeModifier?: boolean;
     }) {
         const filepathStr = convertExportedFilePathToFilePath(filepath);
         this.context.logger.debug(`Generating ${filepathStr}`);
@@ -828,7 +832,7 @@ export class SdkGenerator {
             this.context.logger.debug(`Skipping ${filepathStr} (no content)`);
         } else {
             importsManager.writeImportsToSourceFile(sourceFile);
-            this.exportsManager.addExportsForFilepath(filepath);
+            this.exportsManager.addExportsForFilepath(filepath, addExportTypeModifier);
 
             // this needs to be last.
             // https://github.com/dsherret/ts-morph/issues/189#issuecomment-414174283

generators/typescript/utils/commons/src/exports-manager/ExportsManager.ts

@@ -23,7 +23,11 @@ type PathToDirectory = string;
 export class ExportsManager {
     private exports: Record<PathToDirectory, Record<ModuleSpecifier, CombinedExportDeclarations>> = {};
 
-    public addExport(from: SourceFile | string, exportDeclaration: ExportDeclaration | undefined): void {
+    public addExport(
+        from: SourceFile | string,
+        exportDeclaration: ExportDeclaration | undefined,
+        addExportTypeModifier?: boolean
+    ): void {
         const fromPath = typeof from === "string" ? from : from.getFilePath();
         if (path.extname(fromPath).length === 0) {
             throw new Error("Cannot export from directory: " + fromPath);
@@ -37,16 +41,21 @@ export class ExportsManager {
                 from: pathToDirectory,
                 to: fromPath
             }),
-            exportDeclaration
+            exportDeclaration,
+            addExportTypeModifier
         });
     }
 
-    public addExportsForFilepath(filepath: ExportedFilePath): void {
-        this.addExportsForDirectories(filepath.directories);
-        this.addExport(convertExportedFilePathToFilePath(filepath), filepath.file?.exportDeclaration);
+    public addExportsForFilepath(filepath: ExportedFilePath, addExportTypeModifier?: boolean): void {
+        this.addExportsForDirectories(filepath.directories, addExportTypeModifier);
+        this.addExport(
+            convertExportedFilePathToFilePath(filepath),
+            filepath.file?.exportDeclaration,
+            addExportTypeModifier
+        );
     }
 
-    public addExportsForDirectories(directories: ExportedDirectory[]): void {
+    public addExportsForDirectories(directories: ExportedDirectory[], addExportTypeModifier?: boolean): void {
         let directoryFilepath = "/src";
         for (const part of directories) {
             const nextDirectoryPath = path.join(directoryFilepath, part.nameOnDisk);
@@ -56,7 +65,8 @@ export class ExportsManager {
                     from: directoryFilepath,
                     to: nextDirectoryPath
                 }),
-                exportDeclaration: part.exportDeclaration
+                exportDeclaration: part.exportDeclaration,
+                addExportTypeModifier
             });
 
             if (part.subExports != null) {
@@ -67,7 +77,8 @@ export class ExportsManager {
                             from: directoryFilepath,
                             to: path.join(nextDirectoryPath, relativeFilePath)
                         }),
-                        exportDeclaration
+                        exportDeclaration,
+                        addExportTypeModifier
                     });
                 }
             }
@@ -78,11 +89,13 @@ export class ExportsManager {
     private addExportDeclarationForDirectory({
         directory,
         moduleSpecifierToExport,
-        exportDeclaration
+        exportDeclaration,
+        addExportTypeModifier
     }: {
         directory: Directory | PathToDirectory;
         moduleSpecifierToExport: ModuleSpecifier;
         exportDeclaration: ExportDeclaration | undefined;
+        addExportTypeModifier: boolean | undefined;
     }): void {
         const pathToDirectory = typeof directory === "string" ? directory : directory.getPath();
         const exportsForDirectory = (this.exports[pathToDirectory] ??= {});
@@ -110,7 +123,11 @@ export class ExportsManager {
 
         if (exportDeclaration.namedExports != null) {
             for (const namedExport of exportDeclaration.namedExports) {
-                exportsForModuleSpecifier.namedExports.add(namedExport);
+                if (addExportTypeModifier) {
+                    exportsForModuleSpecifier.namedExports.add("type " + namedExport);
+                } else {
+                    exportsForModuleSpecifier.namedExports.add(namedExport);
+                }
             }
         }
     }

packages/cli/ete-tests/src/tests/add/__snapshots__/add.test.ts.snap

@@ -102,7 +102,7 @@ groups:
           location: local-file-system
           path: ../generated/sdks/typescript
       - name: fernapi/fern-java-sdk
-        version: 0.5.7
+        version: 0.8.7
       - name: fernapi/fern-postman
         version: 0.0.45
         output:

packages/cli/ete-tests/src/tests/generate/__snapshots__/generate.test.ts.snap

@@ -1,7 +1,3 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`fern generate missing docs page 1`] = `
-"[docs]:   Found 1 errors and 0 warnings. Run fern check --warnings to print out the warnings.
-[docs]:   docs.yml -> navigation -> 0 -> page
-          Path missing.md does not exist"
-`;
+exports[`fern generate missing docs page 1`] = `"Login required."`;

packages/cli/ete-tests/src/tests/ir/__snapshots__/ir.test.ts.snap

@@ -31261,15 +31261,15 @@ exports[`ir {"name":"streaming"} 1`] = `
                     "sdkRequest": null,
                     "response": {
                         "type": "streaming",
-                        "value": {
+                        "dataEventType": {
                             "type": "json",
-                            "payload": {
+                            "json": {
                                 "_type": "primitive",
                                 "primitive": "STRING"
-                            },
-                            "terminator": null,
-                            "docs": null
-                        }
+                            }
+                        },
+                        "terminator": null,
+                        "docs": null
                     },
                     "errors": [],
                     "examples": [
@@ -31465,15 +31465,15 @@ exports[`ir {"name":"streaming"} 1`] = `
                     },
                     "response": {
                         "type": "streaming",
-                        "value": {
+                        "dataEventType": {
                             "type": "json",
-                            "payload": {
+                            "json": {
                                 "_type": "primitive",
                                 "primitive": "STRING"
-                            },
-                            "terminator": "[DONE]",
-                            "docs": null
-                        }
+                            }
+                        },
+                        "terminator": "[DONE]",
+                        "docs": null
                     },
                     "errors": [],
                     "examples": [