fix: made studio start on different port if one is already in use (#1815)

Co-authored-by: Tushar Anand <tusharannand@gmail.com>
Co-authored-by: asyncapi-bot <info@asyncapi.io>
Co-authored-by: Shurtu-gal <ashishpadhy1729@gmail.com>

Author: neoandmatrix

.changeset/1815.md

@@ -0,0 +1,18 @@
+---
+'@asyncapi/cli': patch
+---
+
+fix: made studio start on different port if one is already in use
+
+- 43a8cb8: fix: made studio start on differnt port if one is already in use
+
+Signed-off-by: Tushar Anand <tusharannand@gmail.com>
+- ce4b061: Merge remote-tracking branch 'remote/master' into fix-studio-multiple-instance
+- 122812a: chore: new approach for port allocation done
+
+Signed-off-by: Tushar Anand <tusharannand@gmail.com>
+- 5946001: fix: removed comment and fix linting errors
+
+Signed-off-by: Tushar Anand <tusharannand@gmail.com>
+
+

src/apps/cli/commands/generate/fromTemplate.ts

@@ -51,6 +51,7 @@ export default class Template extends Command {
 
   parser = new Parser();
 
+  // eslint-disable-next-line sonarjs/cognitive-complexity
   async run() {
     const { args, flags } = await this.parse(Template); // NOSONAR
     const interactive = !flags['no-interactive'];

src/apps/cli/commands/generate/models.ts

@@ -26,6 +26,7 @@ export default class Models extends Command {
     ...proxyFlags(),
   };
 
+  // eslint-disable-next-line sonarjs/cognitive-complexity
   async run() {
     const { args, flags } = await this.parse(Models);
     let { language, file } = args;

src/apps/cli/commands/start/preview.ts

@@ -23,7 +23,7 @@ export default class PreviewStudio extends Command {
 
     let filePath: string | undefined = args['spec-file'] ?? flags.file;
 
-    const previewPort = flags.port;
+    const previewPort = parseInt(flags.port ?? '0',10);
 
     if (!filePath) {
       filePath = (await load()).getFilePath();
@@ -37,7 +37,7 @@ export default class PreviewStudio extends Command {
       }
     }
     this.metricsMetadata.port = previewPort;
-    startPreview(
+    await startPreview(
       filePath as string,
       flags.base,
       flags.baseDir,

src/apps/cli/commands/start/studio.ts

@@ -22,7 +22,7 @@ export default class StartStudio extends Command {
 
     let filePath = args['spec-file'] ?? flags.file;
 
-    let port = flags.port;
+    let port = parseInt(flags.port ?? '0',10);
 
     if (flags.file) {
       this.warn(
@@ -55,7 +55,7 @@ export default class StartStudio extends Command {
       }
     }
     this.metricsMetadata.port = port;
-    startStudio(filePath as string, port);
+    await startStudio(filePath as string, port);
   }
 
   private async parseArgs(args: Record<string, any>, port?: string) {

src/apps/cli/internal/flags/start/preview.flags.ts

@@ -3,7 +3,7 @@ import { Flags } from '@oclif/core';
 export const previewFlags = () => {
   return {
     help: Flags.help({ char: 'h' }),
-    port: Flags.integer({
+    port: Flags.string({
       char: 'p',
       description: 'port in which to start Studio in the preview mode',
     }),

src/apps/cli/internal/flags/start/studio.flags.ts

@@ -8,7 +8,7 @@ export const studioFlags = () => {
       description: 'path to the AsyncAPI file to link with Studio',
       deprecated: true,
     }),
-    port: Flags.integer({
+    port: Flags.string({
       char: 'p',
       description: 'port in which to start Studio',
     }),