feat: Refactoring setRawMode and disableRawMode into the reporter

Author: kevinwang5658

src/common/base-command.ts

@@ -77,15 +77,8 @@ export abstract class BaseCommand extends Command {
         }
 
         if (data.options.stdin) {
-          await this.reporter.hide();
           console.log(chalk.blue(`Plugin "${pluginName}" is requesting stdin`));
-
-          if (this.reporter instanceof DefaultReporter) {
-            this.reporter.rawOutput = true;
-          }
-
-          // Raw mode is needed by stdin applications to function properly
-          process.stdin.setRawMode(true);
+          await this.reporter.setRawMode();
         }
 
         const result = await spawnSafe(data.command, data.options, pluginName, password)
@@ -97,13 +90,7 @@ export abstract class BaseCommand extends Command {
       } finally {
         // Always disable raw mode after
         if (data.options.stdin) {
-          process.stdin.setRawMode(false);
-
-          if (this.reporter instanceof DefaultReporter) {
-            this.reporter.rawOutput = false;
-          }
-
-          await this.reporter.displayProgress();
+          await this.reporter.disableRawMode();
         }
       }
     });

src/ui/reporters/default-reporter.tsx

@@ -111,7 +111,19 @@ export class DefaultReporter implements Reporter {
   }
 
   async hide(): Promise<void> {
-    await this.updateRenderState(RenderStatus.NOTHING);
+    store.set(store.renderState, { status: RenderStatus.NOTHING, data: null });
+  }
+
+  async setRawMode(): Promise<void> {
+    this.rawOutput = true;
+    process.stdin.setRawMode(true);
+    await this.hide();
+  }
+
+  async disableRawMode(): Promise<void> {
+    this.rawOutput = false;
+    process.stdin.setRawMode(false);
+    await this.displayProgress();
   }
 
   async displayImportWarning(requiresParameters: string[], noParametersRequired: string[]): Promise<void> {

src/ui/reporters/json-reporter.ts

@@ -63,4 +63,12 @@ export class JsonReporter implements Reporter {
 
   async displayImportWarning(): Promise<void> {
   }
+
+  async setRawMode(): Promise<void> {
+    throw new Error('Json reporter error: setRawMode is not supported. Raw stdin mode requires interactive terminal access.');
+  }
+
+  async disableRawMode(): Promise<void> {
+    throw new Error('Json reporter error: disableRawMode is not supported. Raw stdin mode requires interactive terminal access.');
+  }
 }

src/ui/reporters/plain-reporter.ts

@@ -39,6 +39,14 @@ export class PlainReporter implements Reporter {
 
   async hide(): Promise<void> {}
 
+  async setRawMode(): Promise<void> {
+    process.stdin.setRawMode(true);
+  }
+
+  async disableRawMode(): Promise<void> {
+    process.stdin.setRawMode(false);
+  }
+
   async displayImportWarning(): Promise<void> {
     ctx.log(chalk.bold('Additional information is required to continue import'))
     ctx.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.')

src/ui/reporters/reporter.ts

@@ -77,6 +77,10 @@ export interface Reporter {
   displayMessage(message: string): void
 
   displayImportWarning(requiresParameters: string[], noParametersRequired: string[]): Promise<void>
+
+  setRawMode(): Promise<void>
+
+  disableRawMode(): Promise<void>
 }
 
 export enum ReporterType {

src/ui/reporters/stub-reporter.ts

@@ -22,5 +22,7 @@ export class StubReporter implements Reporter {
     async displayImportResult(importResult: ImportResult): Promise<void> {}
     async displayFileModifications(diff: Array<{ file: string, modification: FileModificationResult }>): Promise<void> {}
     async displayMessage(message: string): Promise<void> {}
-    async displayImportWarning(requiresParameters: string[], noParametersRequired: string[]): Promise<void> {}  
+    async displayImportWarning(requiresParameters: string[], noParametersRequired: string[]): Promise<void> {}
+    async setRawMode(): Promise<void> {}
+    async disableRawMode(): Promise<void> {}
 }