fix: only delete generated files instead of whole output directory

hey-api · Apr 12, 2024 · b29a9e6 · b29a9e6
1 parent a4b0c22
commit b29a9e6
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 18 deletions.
diff --git a/.changeset/fuzzy-vans-leave.md b/.changeset/fuzzy-vans-leave.md
@@ -0,0 +1,5 @@
+---
+"@hey-api/openapi-ts": patch
+---
+
+fix: only delete generated files instead of whole output directory
diff --git a/packages/openapi-ts/src/compiler/index.ts b/packages/openapi-ts/src/compiler/index.ts
@@ -1,4 +1,4 @@
-import { writeFileSync } from 'node:fs';
+import { PathLike, writeFileSync } from 'node:fs';
 
 import ts from 'typescript';
 
@@ -15,9 +15,9 @@ export class TypeScriptFile {
     private _headers: Array<string> = [];
     private _imports: Array<ts.Node> = [];
     private _items: Array<ts.Node | string> = [];
-    private _path: string;
+    private _path: PathLike;
 
-    public constructor(path: string, header: boolean = true) {
+    public constructor(path: PathLike, header: boolean = true) {
         this._path = path;
         if (header) {
             const text = 'This file is auto-generated by @hey-api/openapi-ts';

diff --git a/packages/openapi-ts/src/utils/write/client.ts b/packages/openapi-ts/src/utils/write/client.ts
@@ -1,4 +1,4 @@
-import { mkdirSync, rmSync } from 'node:fs';
+import { existsSync, mkdirSync } from 'node:fs';
 import path from 'node:path';
 
 import type { OpenApi } from '../../openApi';
@@ -20,10 +20,6 @@ import { writeServices } from './services';
  */
 export const writeClient = async (openApi: OpenApi, client: Client, templates: Templates): Promise<void> => {
     const config = getConfig();
-    await rmSync(config.output, {
-        force: true,
-        recursive: true,
-    });
 
     if (typeof config.exportServices === 'string') {
         const regexp = new RegExp(config.exportServices);
@@ -35,6 +31,10 @@ export const writeClient = async (openApi: OpenApi, client: Client, templates: T
         client.models = client.models.filter(model => regexp.test(model.name));
     }
 
+    if (!existsSync(path.resolve(config.output))) {
+        mkdirSync(path.resolve(config.output), { recursive: true });
+    }
+
     const sections = [
         {
             dir: 'core',
@@ -60,15 +60,6 @@ export const writeClient = async (openApi: OpenApi, client: Client, templates: T
 
     for (const section of sections) {
         const sectionPath = path.resolve(config.output, section.dir);
-        if (section.dir) {
-            await rmSync(sectionPath, {
-                force: true,
-                recursive: true,
-            });
-        }
-        await mkdirSync(sectionPath, {
-            recursive: true,
-        });
         await section.fn(openApi, sectionPath, client, templates);
     }
 

diff --git a/packages/openapi-ts/src/utils/write/core.ts b/packages/openapi-ts/src/utils/write/core.ts
@@ -1,4 +1,4 @@
-import { copyFileSync, existsSync, writeFileSync } from 'node:fs';
+import { copyFileSync, existsSync, mkdirSync, rmSync, writeFileSync } from 'node:fs';
 import path from 'node:path';
 
 import type { OpenApi } from '../../openApi';
@@ -28,6 +28,14 @@ export const writeCore = async (
         version: client.version,
     };
 
+    rmSync(path.resolve(outputPath), {
+        force: true,
+        recursive: true,
+    });
+    mkdirSync(path.resolve(outputPath), {
+        recursive: true,
+    });
+
     if (config.exportCore) {
         await writeFileSync(
             path.resolve(outputPath, 'OpenAPI.ts'),