fix: fixed issue with the management sdk params

Author: Vikram Kalta

src/commands/app/uninstall.ts

@@ -44,7 +44,7 @@ export default class Uninstall extends BaseCommand<typeof Uninstall> {
 
         const factory = new UninstallAppFactory()
         const strategy = factory.getStrategyInstance(this.flags['uninstall-all'])
-        await strategy.run(this.flags, this.sharedConfig.org, {managementSdk: this.managementAppSdk, log: this.log}, appType)
+        await strategy.run(this.flags, this.sharedConfig.org, this.managementSdk, {managementSdk: this.managementAppSdk, log: this.log}, appType)
 
         this.log(this.$t(uninstallAppMsg.APP_UNINSTALLED, { app: app?.name || this.flags["app-uid"] }), "info")
 

src/interfaces/uninstall-app.ts

@@ -1,7 +1,7 @@
-import { FlagInput } from "@contentstack/cli-utilities";
+import { ContentstackClient, FlagInput } from "@contentstack/cli-utilities";
 import { CommonOptions } from "../util";
 import { AppTarget } from "@contentstack/management/types/app/index"
 
 export interface UninstallApp {
-  run(flags: FlagInput, org: string, options: CommonOptions, appType: AppTarget): Promise<void>;
+  run(flags: FlagInput, org: string, managementSdk: ContentstackClient, options: CommonOptions, appType: AppTarget): Promise<void>;
 }

src/strategies/uninstall-all.ts

@@ -1,19 +1,19 @@
-import { FlagInput } from "@contentstack/cli-utilities";
+import { ContentstackClient, FlagInput } from "@contentstack/cli-utilities";
 import { UninstallApp } from "../interfaces/uninstall-app";
 import { CommonOptions, getInstallation, uninstallApp } from "../util";
 import { AppTarget } from "@contentstack/management/types/app/index"
 
 export class UninstallAll implements UninstallApp {
-  public async run(flags: FlagInput, org: string, options: CommonOptions, appType: AppTarget): Promise<void> {
+  public async run(flags: FlagInput, org: string, managementSdk: ContentstackClient, options: CommonOptions, appType: AppTarget): Promise<void> {
     // get all installation uids to uninstall
-    const installationUids = await this.getInstallations(flags, org, options, appType)
+    const installationUids = await this.getInstallations(flags, org, managementSdk, options, appType)
     for (const installationUid of installationUids) {
       await uninstallApp(flags, org, options, installationUid)
     }
   }
 
-  public async getInstallations(flags: FlagInput, org: string, options: CommonOptions, appType: AppTarget): Promise<string[]> {
-    let installationUids: string = await getInstallation(flags, org, options.managementSdk, appType, options, true)
+  public async getInstallations(flags: FlagInput, org: string, managementSdk: ContentstackClient, options: CommonOptions, appType: AppTarget): Promise<string[]> {
+    let installationUids: string = await getInstallation(flags, org, managementSdk, appType, options, true)
     return installationUids.split(',')
   }
 }

src/strategies/uninstall-selected.ts

@@ -1,21 +1,21 @@
-import { FlagInput } from "@contentstack/cli-utilities";
+import { ContentstackClient, FlagInput } from "@contentstack/cli-utilities";
 import { UninstallApp } from "../interfaces/uninstall-app";
 import { CommonOptions, getInstallation, uninstallApp } from "../util";
 import { AppTarget } from "@contentstack/management/types/app/index"
 
 export class UninstallSelected implements UninstallApp {
-  public async run(flags: FlagInput, org: string, options: CommonOptions, appType: AppTarget): Promise<void> {
+  public async run(flags: FlagInput, org: string, managementSdk: ContentstackClient, options: CommonOptions, appType: AppTarget): Promise<void> {
     // select installation uid to uninstall
-    const installationUids = await this.getInstallations(flags, org, options, appType)
+    const installationUids = await this.getInstallations(flags, org, managementSdk, options, appType)
     for (const installationUid of installationUids) {
       await uninstallApp(flags, org, options, installationUid)
     }
   }
 
-  public async getInstallations(flags: FlagInput, org: string, options: CommonOptions, appType: AppTarget): Promise<string[]> {
+  public async getInstallations(flags: FlagInput, org: string, managementSdk: ContentstackClient, options: CommonOptions, appType: AppTarget): Promise<string[]> {
     let installationUids: any = flags['installation-uid'];
     if (!installationUids) {
-      installationUids = await getInstallation(flags, org, options.managementSdk, appType, options)
+      installationUids = await getInstallation(flags, org, managementSdk, appType, options)
     }
     return installationUids.split(',')
   }

src/util/inquirer.ts

@@ -204,7 +204,7 @@ async function getInstallation(
     installations = populateMissingDataInInstallations(installations, stacks)
     // To support uninstall all flag
     if (uninstallAll) {
-      return installations.join(',')
+      return installations.map(installation => installation.uid).join(',')
     }
     let _selectedInstallation = await cliux
     .inquire({