Reduce remove reuse

Author: Cliff Odijk

dist/index.js

@@ -214,7 +214,7 @@ export class AssemblyProcessor {
     } catch (e: any) {
       this.handleError(
         e,
-        `Comment for stack ${stackName} is too long, please report this as a bug https://github.com/corymhall/cdk-diff-action/issues/new`,
+        `Comment for directory ${this.options.cdkOutDir} is too long, please report this as a bug https://github.com/corymhall/cdk-diff-action/issues/new`,
       );
     }
   }
@@ -328,19 +328,19 @@ export class AssemblyProcessor {
   }
 
   private formatStackComment(
-    stackName: string,
+    _stackName: string,
     diff: TemplateDiff,
     changes: ChangeDetails,
   ): string[] {
     const output: string[] = [];
     const emoji = this.getEmoji(changes);
     if (diff.isEmpty) {
-      output.push(`No Changes for stack: ${stackName} ${emoji}`);
+      output.push(`No Changes for directory: ${this.options.cdkOutDir} ${emoji}`);
       return output;
     }
     output.push(
       ...[
-        `#### Diff for stack: ${stackName} - ` +
+        `#### CDK Diff for ${this.options.cdkOutDir} - ` +
           `***${changes.createdResources} to add, ${changes.updatedResources} to update, ${changes.removedResources} to destroy***  ` +
           emoji,
         '<details><summary>Details</summary>',
@@ -352,7 +352,7 @@ export class AssemblyProcessor {
       output.push('> [!WARNING]\n> ***Destructive Changes*** :bangbang:'),
         changes.destructiveChanges.forEach((change) => {
           output.push(
-            `> **Stack: ${change.stackName} - Resource: ${change.logicalId} - Impact:** ***${change.impact}***`,
+            `> **Resource: ${change.logicalId} - Impact:** ***${change.impact}***`,
           );
           output.push('');
         });
@@ -370,12 +370,12 @@ export class AssemblyProcessor {
   }
 
   /**
-   * Only used when the stage comment is too long and we are creating
-   * a separate comment for each stack
+   * Only used when the comment is too long and we are creating
+   * a separate comment for the directory
    */
   private getCommentForStack(
-    stageName: string,
-    stackName: string,
+    _stageName: string,
+    _stackName: string,
     comment: string[],
   ): string[] {
     const output: string[] = [];
@@ -386,7 +386,7 @@ export class AssemblyProcessor {
       output.push(`## ${this.options.title}`);
       output.push('');
     }
-    output.push(`### Diff for stack: ${stageName} / ${stackName} (${this.options.cdkOutDir})`);
+    output.push(`### CDK Diff for ${this.options.cdkOutDir}`);
 
     return output.concat(comment);
   }
@@ -404,7 +404,7 @@ export class AssemblyProcessor {
       output.push(`## ${this.options.title}`);
       output.push('');
     }
-    output.push(`### Diff for stage: ${stageName} (${this.options.cdkOutDir})`);
+    output.push(`### CDK Diff for ${this.options.cdkOutDir}`);
 
     if (stageComments.destructiveChanges) {
       output.push(

dist/index.js.map

@@ -126,7 +126,7 @@ describe('StageProcessor', () => {
       SomeStage: expect.any(Object),
     });
     expect(p.SomeStage.stackComments['SomeStage/test-stack']).toEqual([
-      'No Changes for stack: SomeStage/test-stack :white_check_mark:',
+      'No Changes for directory: cdk.out :white_check_mark:',
     ]);
   });
 
@@ -330,7 +330,7 @@ describe('default stage', () => {
       DefaultStage: expect.any(Object),
     });
     expect(p.DefaultStage.stackComments['test-stack']).toEqual([
-      'No Changes for stack: test-stack :white_check_mark:',
+      'No Changes for directory: cdk.out :white_check_mark:',
     ]);
   });
 });
@@ -432,7 +432,7 @@ describe('stack comments', () => {
     await processor.processStages();
     await expect(
       processor.commentStages(new Comments({} as any, {} as any)),
-    ).rejects.toThrow(/Comment for stack SomeStage\/my-stack1 is too long/);
+    ).rejects.toThrow(/Comment for directory cdk\.out is too long/);
     expect(findPreviousMock).toHaveBeenCalledTimes(11);
     expect(createCommentMock).toHaveBeenCalledTimes(0);
     expect(updateCommentMock).toHaveBeenCalledTimes(11);