Added an error catcher to plan and apply commands

Author: kevinwang5658

src/commands/apply/index.ts

@@ -18,50 +18,41 @@ export default class Apply extends Command {
   ]
 
   static flags = {
-    // flag with no value (-f, --force)
-    force: Flags.boolean({ char: 'f' }),
-    // flag with a value (-n, --name=VALUE)
-    name: Flags.string({ char: 'n', description: 'name to print' }),
     // flag with a value (-p, --path=VALUE)
     path: Flags.string({ char: 'p', description: 'path to project' }),
   }
 
   public async run(): Promise<void> {
-    const { args, flags } = await this.parse(Apply)
+    const { flags } = await this.parse(Apply)
     const reporter = new DefaultReporter()
 
-    const name = flags.name ?? 'world'
-    this.log(`hello ${name} from /Users/kevinwang/Projects/codify/codify-core/src/commands/apply.ts`)
-    if (args.file && flags.force) {
-      this.log(`you input --force and --file: ${args.file}`)
-    }
-
-    if (flags.path) {
-      this.log(`Applying Codify from: ${flags.path}`);
-    }
+    try {
+      if (flags.path) {
+        this.log(`Applying Codify from: ${flags.path}`);
+      }
 
-    const resolvedPath = path.resolve(flags.path ?? '.');
+      const resolvedPath = path.resolve(flags.path ?? '.');
 
-    const planResult = await PlanOrchestrator.run(resolvedPath, false);
+      const planResult = await PlanOrchestrator.run(resolvedPath, false);
+      reporter.displayPlan(planResult.plan);
 
-    // Short circuit and exit if every change is NOOP
-    if (planResult.plan.every((p) => p.operation === ResourceOperation.NOOP)) {
-      console.log('No changes necessary. Exiting');
-      await planResult.pluginCollection.destroy();
-      return;
-    }
+      // Short circuit and exit if every change is NOOP
+      if (planResult.plan.every((p) => p.operation === ResourceOperation.NOOP)) {
+        console.log('No changes necessary. Exiting');
+        await planResult.pluginCollection.destroy();
+        return this.exit(0);
+      }
 
-    reporter.displayPlan(planResult.plan);
+      const confirm = await reporter.promptApplyConfirmation()
+      if (!confirm) {
+        return this.exit(0);
+      }
 
-    const confirm = await reporter.promptApplyConfirmation()
-
-    if (!confirm) {
-      return this.exit(0);
+      await ApplyOrchestrator.run(planResult);
+    } catch (error: unknown) {
+      console.error(error);
     }
 
-    await ApplyOrchestrator.run(planResult);
-
-    // this.exit(0);
-
+    this.exit(0);
   }
 }

src/commands/plan/index.ts

@@ -25,23 +25,22 @@ export default class Plan extends Command {
   }
 
   public async run(): Promise<void> {
-    const { args, flags } = await this.parse(Plan)
+    const { flags } = await this.parse(Plan)
     const reporter = new DefaultReporter()
 
-    const name = flags.name ?? 'world'
-    this.log(`hello ${name} from /Users/kevinwang/Projects/codify/codify-core/src/commands/plan.ts`)
-    if (args.file && flags.force) {
-      this.log(`you input --force and --file: ${args.file}`)
-    }
+    try {
+      if (flags.path) {
+        this.log(`Applying Codify from: ${flags.path}`);
+      }
 
-    if (flags.path) {
-      this.log(`Applying Codify from: ${flags.path}`);
-    }
+      const resolvedPath = path.resolve(flags.path ?? '.');
 
-    const resolvedPath = path.resolve(flags.path ?? '.');
+      const { plan } = await PlanOrchestrator.run(resolvedPath);
+      reporter.displayPlan(plan);
 
-    const { plan } = await PlanOrchestrator.run(resolvedPath);
-    reporter.displayPlan(plan);
+    } catch (error) {
+      console.error(error);
+    }
 
     this.exit(0);
   }

src/ui/reporters/reporter.ts

@@ -1,10 +1,10 @@
 import { PlanResponseData } from 'codify-schemas';
 
 export enum RenderEvent {
-  STATE_TRANSITION = 'stateTransition',
   LOG = 'log',
   PROGRESS_UPDATE = 'progressUpdate',
-  PROMPT_RESULT = 'promptResult'
+  PROMPT_RESULT = 'promptResult',
+  STATE_TRANSITION = 'stateTransition'
 }
 
 /**
@@ -26,7 +26,7 @@ export interface DisplayPlanStateTransition extends StateTransition {
 }
 
 export interface Reporter {
-  promptApplyConfirmation(): Promise<boolean>
-
   displayPlan(plan: PlanResponseData[]): void
+
+  promptApplyConfirmation(): Promise<boolean>
 }