fix: make logFile optional in config

Author: Matsuuu

packages/openapi-ts/bin/index.cjs

@@ -118,6 +118,8 @@ async function start() {
       userConfig.logs.level = 'silent';
     }
 
+    userConfig.logs.file = userConfig.logFile;
+
     if (typeof params.watch === 'string') {
       userConfig.watch = Number.parseInt(params.watch, 10);
     }

packages/openapi-ts/src/getLogs.ts

@@ -2,6 +2,7 @@ import type { Config, UserConfig } from './types/config';
 
 export const getLogs = (userConfig: UserConfig | undefined): Config['logs'] => {
   let logs: Config['logs'] = {
+    file: true,
     level: 'info',
     path: process.cwd(),
   };

packages/openapi-ts/src/index.ts

@@ -67,11 +67,11 @@ export const createClient = async (
   } catch (error) {
     const config = configs[0] as Config | undefined;
     const dryRun = config ? config.dryRun : resolvedConfig?.dryRun;
-    const writeLogFile = config ? config.logFile : resolvedConfig?.logFile;
+
     // TODO: add setting for log output
     if (!dryRun) {
       const logs = config?.logs ?? getLogs(resolvedConfig);
-      if (logs.level !== 'silent' && writeLogFile) {
+      if (logs.level !== 'silent' && logs.file) {
         const logName = `openapi-ts-error-${Date.now()}.log`;
         const logsDir = path.resolve(process.cwd(), logs.path ?? '');
         ensureDirSync(logsDir);

packages/openapi-ts/src/initConfigs.ts

@@ -269,7 +269,6 @@ export const initConfigs = async (
       dryRun = false,
       experimentalParser = true,
       exportCore = true,
-      logFile = true,
       name,
       request,
       useOptions = true,
@@ -312,7 +311,6 @@ export const initConfigs = async (
       experimentalParser,
       exportCore: false,
       input,
-      logFile,
       logs,
       name,
       output,

packages/openapi-ts/src/types/config.d.ts

@@ -160,6 +160,10 @@ export interface UserConfig {
   logs?:
     | string
     | {
+        /**
+         * Whether or not error logs should be written to a file or not
+         * */
+        file?: boolean;
         /**
          * The logging level to control the verbosity of log output.
          * Determines which messages are logged based on their severity.
@@ -185,17 +189,14 @@ export interface UserConfig {
           | 'silent'
           | 'trace'
           | 'warn';
+
         /**
          * The relative location of the logs folder
          *
          * @default process.cwd()
          */
         path?: string;
       };
-  /**
-   * Whether or not error logs should be written to a file or not
-   * */
-  logFile: boolean;
   /**
    * Regenerate the client when the input file changes? You can alternatively
    * pass a numeric value for the interval in ms.