fix(openapi/upload): restore --working-directory

Author: kanadgupta

src/commands/openapi/convert.ts

@@ -28,7 +28,7 @@ export default class OpenAPIConvertCommand extends BaseCommand<typeof OpenAPICon
   static flags = {
     out: Flags.string({ description: 'Output file path to write converted file to' }),
     title: titleFlag,
-    workingDirectory: workingDirectoryFlag,
+    'working-directory': workingDirectoryFlag,
   };
 
   static examples = [
@@ -45,13 +45,7 @@ export default class OpenAPIConvertCommand extends BaseCommand<typeof OpenAPICon
   ];
 
   async run() {
-    const { out, workingDirectory } = this.flags;
-
-    if (workingDirectory) {
-      const previousWorkingDirectory = process.cwd();
-      process.chdir(workingDirectory);
-      this.debug(`switching working directory from ${previousWorkingDirectory} to ${process.cwd()}`);
-    }
+    const { out } = this.flags;
 
     const { preparedSpec, specPath, specType } = await prepareOas.call(this);
     const parsedPreparedSpec: OASDocument = JSON.parse(preparedSpec);

src/commands/openapi/inspect.ts

@@ -216,7 +216,7 @@ export default class OpenAPIInspectCommand extends BaseCommand<typeof OpenAPIIns
       multiple: true,
       options: getSupportedFeatures(),
     }),
-    workingDirectory: workingDirectoryFlag,
+    'working-directory': workingDirectoryFlag,
   };
 
   static examples = [
@@ -239,19 +239,13 @@ export default class OpenAPIInspectCommand extends BaseCommand<typeof OpenAPIIns
   ];
 
   async run() {
-    const { workingDirectory, feature: features } = this.flags;
+    const { feature: features } = this.flags;
 
     const tableBorder = Object.entries(getBorderCharacters('norc'))
       .map(([border, char]) => ({ [border]: chalk.gray(char) }))
       // biome-ignore lint/performance/noAccumulatingSpread: @todo can this be improved?
       .reduce((prev, next) => Object.assign(prev, next));
 
-    if (workingDirectory) {
-      const previousWorkingDirectory = process.cwd();
-      process.chdir(workingDirectory);
-      this.debug(`switching working directory from ${previousWorkingDirectory} to ${process.cwd()}`);
-    }
-
     const { preparedSpec, definitionVersion } = await prepareOas.call(this);
     const parsedPreparedSpec: OASDocument = JSON.parse(preparedSpec);
 

src/commands/openapi/reduce.ts

@@ -38,7 +38,7 @@ export default class OpenAPIReduceCommand extends BaseCommand<typeof OpenAPIRedu
     path: Flags.string({ description: 'Paths to reduce by', multiple: true }),
     tag: Flags.string({ description: 'Tags to reduce by', multiple: true }),
     title: titleFlag,
-    workingDirectory: workingDirectoryFlag,
+    'working-directory': workingDirectoryFlag,
   };
 
   static examples = [
@@ -61,13 +61,6 @@ export default class OpenAPIReduceCommand extends BaseCommand<typeof OpenAPIRedu
 
   async run() {
     const opts = this.flags;
-    const { workingDirectory } = opts;
-
-    if (workingDirectory) {
-      const previousWorkingDirectory = process.cwd();
-      process.chdir(workingDirectory);
-      this.debug(`switching working directory from ${previousWorkingDirectory} to ${process.cwd()}`);
-    }
 
     const { preparedSpec, specPath, specType } = await prepareOas.call(this);
     const parsedPreparedSpec: OASDocument = JSON.parse(preparedSpec);

src/commands/openapi/resolve.ts

@@ -53,7 +53,7 @@ export default class OpenAPIResolveCommand extends BaseCommand<typeof OpenAPIRes
   static flags = {
     out: Flags.string({ description: 'Output file path to write resolved file to' }),
     title: titleFlag,
-    workingDirectory: workingDirectoryFlag,
+    'working-directory': workingDirectoryFlag,
   };
 
   /**
@@ -345,13 +345,7 @@ export default class OpenAPIResolveCommand extends BaseCommand<typeof OpenAPIRes
   }
 
   async run() {
-    const { out, workingDirectory } = this.flags;
-
-    if (workingDirectory) {
-      const previousWorkingDirectory = process.cwd();
-      process.chdir(workingDirectory);
-      this.debug(`Switching working directory from ${previousWorkingDirectory} to ${process.cwd()}`);
-    }
+    const { out } = this.flags;
 
     const { preparedSpec, specPath, specType } = await prepareOas.call(this);
     if (specType !== 'OpenAPI') {

src/commands/openapi/validate.ts

@@ -33,16 +33,10 @@ export default class OpenAPIValidateCommand extends BaseCommand<typeof OpenAPIVa
 
   static flags = {
     github: githubFlag,
-    workingDirectory: workingDirectoryFlag,
+    'working-directory': workingDirectoryFlag,
   };
 
   async run() {
-    if (this.flags.workingDirectory) {
-      const previousWorkingDirectory = process.cwd();
-      process.chdir(this.flags.workingDirectory);
-      this.debug(`switching working directory from ${previousWorkingDirectory} to ${process.cwd()}`);
-    }
-
     const { specPath, specType } = await prepareOas.call(this);
 
     return this.runCreateGHAHook({

src/lib/flags.ts

@@ -46,6 +46,8 @@ export const titleFlag = Flags.string({
  * Used in the `openapi` family of commands where `workingDirectory` is an option.
  */
 export const workingDirectoryFlag = Flags.string({
+  aliases: ['workingDirectory'],
+  deprecateAliases: true,
   description: 'Working directory (for usage with relative external references)',
 });
 

src/lib/prepareOas.ts

@@ -53,6 +53,12 @@ export default async function prepareOas(this: OpenAPICommands) {
   let specPath = this.args.spec;
   const command = this.id satisfies `openapi ${OpenAPIAction}`;
 
+  if (this.flags.workingDirectory) {
+    const previousWorkingDirectory = process.cwd();
+    process.chdir(this.flags.workingDirectory);
+    this.debug(`switching working directory from ${previousWorkingDirectory} to ${process.cwd()}`);
+  }
+
   if (!specPath) {
     /**
      * Scans working directory for a potential OpenAPI or Swagger file.