feat: Add collapse_changes option

README.md

@@ -163,6 +163,7 @@ Please check the above setup example to use this action with `pull_request_targe
 | `percent_extra_attention` | `20`                                  | If an out file size has increased more than this percent, display a "‼️" to draw extra attention to the change.  |
 | `show_details`            | `true`                                | If `true`, a collapsed "details" section is rendered. It explains the details of the numbers provided and icons. |
 | `show_no_change`          | `true`                               | If `true`, all bundles are shown in the analysis regardless of size change. If `false`, only bundles with size changes are shown. |
+| `collapse_changes`        | `false`                               | If `true`, hide the analysis table inside an accordion. If `false`, display the analysis table directly. |
 | `top_n_largest_paths`     | `20`                                  | The number of largest paths (e.g.) `node_modules/foo`) to be collected. If 0 or lower, skipped.                  |
 
 ## Action outputs

__tests__/no-base.test.ts

@@ -22,6 +22,7 @@ describe("examples w/o base analysis", () => {
 				name: "test",
 				showDetails: false,
 				showNoChange: true,
+				collapseChanges: false,
 				topNLargestPaths: 10,
 			};
 

__tests__/with-base.test.ts

@@ -34,6 +34,7 @@ describe("examples w/ base analysis", () => {
 				name: "test",
 				showDetails: false,
 				showNoChange: isEven,
+				collapseChanges: !isEven,
 				topNLargestPaths: 10,
 			};
 
@@ -60,8 +61,10 @@ describe("examples w/ base analysis", () => {
 				expect(comment).toMatch(/✅ {2}No change/i);
 				if (isEven) {
 					expect(comment).not.toMatch(/\d bundles with no change are hidden./i);
+					expect(comment).not.toMatch("View analysis table");
 				} else {
 					expect(comment).toMatch(/\d bundles with no change are hidden./i);
+					expect(comment).toMatch("View analysis table");
 				}
 				expect(comment).toMatch(/🆕 Added/i);
 				expect(comment).toMatch(/🗑️ Deleted/i);

action.yaml

@@ -59,6 +59,11 @@ inputs:
     default: "true"
     description: |
       If `true`, all bundles are shown in the analysis regardless of size change. If `false`, only bundles with size changes are shown.
+  collapse_changes:
+    required: false
+    default: "false"
+    description: |
+      If `true`, hide the analysis table inside an accordion. If `false`, display the analysis table directly.
   top_n_largest_paths:
     required: false
     default: "20"
@@ -89,6 +94,7 @@ runs:
         INPUT_PERCENT_EXTRA_ATTENTION: ${{ inputs.percent_extra_attention }}
         INPUT_SHOW_DETAILS: ${{ inputs.show_details }}
         INPUT_SHOW_NO_CHANGE: ${{ inputs.show_no_change }}
+        INPUT_COLLAPSE_CHANGES: ${{ inputs.collapse_changes }}
         INPUT_TOP_N_LARGEST_PATHS: ${{ inputs.top_n_largest_paths }}
       run: |
         node ${{ github.action_path }}/dist/index.mjs

src/compare.ts

@@ -73,6 +73,7 @@ This analysis was generated by [esbuild-bundle-analyzer](https://github.com/exoe
 			comparison,
 			input.percentExtraAttention,
 			input.showNoChange,
+			input.collapseChanges,
 		);
 		output += noChangesTable(comparison, input.showNoChange);
 		output += fileSizeTable(comparison, input.topNLargestPaths);
@@ -229,6 +230,7 @@ function markdownTable(
 	data: Array<CompareResult>,
 	redThreshold: number,
 	showNoChange: boolean,
+	collapseChanges: boolean,
 ): string {
 	const rows = data
 		.filter((d) => shouldShowBundle(d, showNoChange))
@@ -240,10 +242,22 @@ function markdownTable(
 		})
 		.join("");
 
-	return `
+	const table = `
 Meta File | Out File  | Size (raw) | Note
 ----------|----------|-----------:|------
 ${rows}`;
+
+	if (collapseChanges) {
+		return `
+		<details>
+		<summary>View analysis table</summary>
+
+		${table}
+
+		</details>
+		`;
+	}
+	return table;
 }
 
 function noChangesTable(

src/index.ts

@@ -24,6 +24,9 @@ function getInput(): Input {
 		showNoChange: ["true", "True", "TRUE"].includes(
 			getSingleInput("show_no_change") || "true",
 		),
+		collapseChanges: ["true", "True", "TRUE"].includes(
+			getSingleInput("collapse_changes"),
+		),
 		topNLargestPaths: Number.parseInt(
 			getSingleInput("top_n_largest_paths") || "20",
 			10,

src/types.ts

@@ -23,6 +23,7 @@ export interface Input {
 	percentExtraAttention: number;
 	showDetails: boolean;
 	showNoChange: boolean;
+	collapseChanges: boolean;
 }
 
 export interface TreeMapNode {