feat: fail action if there are destructive changes

corymhall · Oct 30, 2023 · fd6b6fd · fd6b6fd
1 parent 3762896
commit fd6b6fd
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 2 deletions.
diff --git a/.projenrc.ts b/.projenrc.ts
@@ -73,5 +73,5 @@ jestConfig?.patch(JsonPatch.add('/transform', {
   '^.+\\.(t|j)sx?$': new Transform('@swc/jest'),
 }));
 const actionYml = project.tryFindObjectFile('action.yml');
-actionYml?.addOverride('runs.using', 'node18');
+actionYml?.addOverride('runs.using', 'node20');
 project.synth();
diff --git a/action.yml b/action.yml
diff --git a/src/action.ts b/src/action.ts
@@ -26,6 +26,9 @@ export async function run() {
   const processor = new StageProcessor(stages, inputs.allowedDestroyTypes);
   await processor.processStages();
   await processor.commentStages(comments);
+  if (processor.hasDestructiveChanges && inputs.failOnDestructiveChanges) {
+    throw new Error('There are destructive changes! See PR comment for details.');
+  }
   return;
 }
 
diff --git a/src/diff.ts b/src/diff.ts
@@ -137,6 +137,15 @@ export class StageProcessor {
     }
   }
 
+  public get hasDestructiveChanges(): boolean {
+    for (const comments of Object.values(this.stageComments)) {
+      if (comments.destructiveChanges) {
+        return true;
+      }
+    }
+    return false;
+  }
+
   private async diffStack(stack: StackInfo): Promise<{comment: string[]; changes: number}> {
     const stackDiff = new StackDiff(stack, this.allowedDestroyTypes);
     const { diff, destructiveChanges } = await stackDiff.diffStack();