fix: additional reporters

Author: kevinwang5658

src/ui/reporters/json-reporter.ts

@@ -1,18 +1,22 @@
-import { SudoRequestData, SudoRequestResponseData } from 'codify-schemas';
+import { SudoRequestData } from 'codify-schemas';
 
 import { Plan } from '../../entities/plan.js';
 import { ResourceConfig } from '../../entities/resource-config.js';
-import { ResourceInfo } from '../../entities/resource-info.js';
 import { ImportResult } from '../../orchestrators/import.js';
-import { FileModificationResult } from '../../utils/file-modification-calculator.js';
-import { PromptType, Reporter } from './reporter.js';
+import { Reporter } from './reporter.js';
 
 export class JsonReporter implements Reporter {
-    displayPlan(plan: Plan): void {
-      console.log(JSON.stringify(plan.resources.map((r) => r.raw), null, 2));
-    }
+  displayPlan(plan: Plan): void {
+    console.log(JSON.stringify(plan.resources.map((r) => r.raw), null, 2));
+  }
+
+  promptPressKeyToContinue(message?: string): Promise<void> {
+    throw new Error(`Press key to continue is not supported by the JsonReporter. 
+
+ ${message}`)
+  }
 
-    async displayInitBanner(): Promise<void> {}
+  async displayInitBanner(): Promise<void> {}
 
     async displayProgress(): Promise<void> {}
 
@@ -22,33 +26,33 @@ export class JsonReporter implements Reporter {
       return availableTypes;
     }
 
-    async promptInput(prompt: string, error?: string | undefined, validation?: (() => Promise<boolean>) | undefined, autoComplete?: ((input: string) => string[]) | undefined): Promise<string> {
+    async promptInput(prompt: string): Promise<string> {
       throw new Error(`Json reporter error: user input is required for prompt: ${prompt}. The Json reporter doesn't support user input. Make sure to have parameters preconfigured`);
     }
 
-    async promptConfirmation(message: string): Promise<boolean> {
+    async promptConfirmation(): Promise<boolean> {
       return true;
     }
 
-    async promptOptions(message: string, options: string[]): Promise<number> {
+    async promptOptions(): Promise<number> {
       throw new Error('Json reporter error: this reporter does not support prompting options. Use another reporter.')
     }
 
-    async promptSudo(pluginName: string, data: SudoRequestData, secureMode: boolean): Promise<SudoRequestResponseData> {
+    async promptSudo(pluginName: string, data: SudoRequestData): Promise<string | undefined> {
       throw new Error(`Json reporter error: sudo required for command: ${data.command}. Make sure to preconfigure the sudo password for the Json reporter using --sudoPassword`);
     }
 
-    async promptUserForValues(resources: ResourceInfo[], promptType: PromptType): Promise<ResourceConfig[]> {
+    async promptUserForValues(): Promise<ResourceConfig[]> {
       throw new Error('Json reporter error: cannot prompt user for values while using Json reporter. Use a different reporter.');
     }
 
-    async displayImportResult(importResult: ImportResult, showConfigs: boolean): void {
+    async displayImportResult(importResult: ImportResult): Promise<void> {
       console.log(JSON.stringify(importResult.result.map((r) => r.raw)));
     }
 
-    async displayFileModifications(diff: { file: string; modification: FileModificationResult; }[]): void {}
+    async displayFileModifications(): Promise<void> {}
 
-    displayMessage(message: string): void {}
+    displayMessage(): void {}
 
-    async displayImportWarning(requiresParameters: string[], noParametersRequired: string[]): Promise<void> {}
+    async displayImportWarning(): Promise<void> {}
 }

src/ui/reporters/plain-reporter.ts

@@ -1,5 +1,5 @@
 import chalk from 'chalk';
-import { SudoRequestData, SudoRequestResponseData } from 'codify-schemas';
+import { SudoRequestData } from 'codify-schemas';
 import readline from 'node:readline';
 
 import { Plan } from '../../entities/plan.js';
@@ -8,7 +8,6 @@ import { ResourceInfo } from '../../entities/resource-info.js';
 import { Event, ctx } from '../../events/context.js';
 import { ImportResult } from '../../orchestrators/import.js';
 import { FileModificationResult } from '../../utils/file-modification-calculator.js';
-import { SudoUtils } from '../../utils/sudo.js';
 import { prettyFormatPlan } from '../plan-pretty-printer.js';
 import { PromptType, Reporter } from './reporter.js';
 
@@ -25,9 +24,21 @@ export class PlainReporter implements Reporter {
     }
   }
 
+  promptPressKeyToContinue(message?: string | undefined): Promise<void> {
+    console.log(message);
+    console.log(chalk.dim.gray('<press any key to continue>'))
+    process.stdin.setRawMode(true)
+    return new Promise((resolve) => {
+      process.stdin.once('data', () => {
+        process.stdin.setRawMode(false)
+        resolve()
+      })
+    })
+  }
+
   async hide(): Promise<void> {}
 
-  async displayImportWarning(requiresParameters: string[], noParametersRequired: string[]): Promise<void> {
+  async displayImportWarning(): Promise<void> {
     console.log(chalk.bold('Additional information is required to continue import'))
     console.log('Some of the resources specified in the import support multiple instances. Additional information is required to identify the specific instance to import. If importing multiple instances is desired (for ex: multiple git clones) additional imports can be added in the prompt.')
   }