ci: refactor container size diff action to use github-script (#908)

* ci: refactor container size diff action to Node.js

Hi 👋 

I came across your composite action and found it useful so I decided to re-write it using github action script instead. Free free to discard this PR if you are happy with bash but I thought I should share.

Thanks!

Signed-off-by: Devin Buhl <[email protected]>

* Delete .github/actions/container-size-diff/container-size-diff.sh

Signed-off-by: Devin Buhl <[email protected]>

* Update .github/actions/container-size-diff/action.yml

Signed-off-by: Devin Buhl <[email protected]>

* Use environment variables for container inputs

Signed-off-by: Devin Buhl <[email protected]>

* Refactor container size diff action with new dependencies

Signed-off-by: Devin Buhl <[email protected]>

* Fix percentage calculation formatting in action.yml

Signed-off-by: Devin Buhl <[email protected]>

---------

Signed-off-by: Devin Buhl <[email protected]>
Co-authored-by: Ron <[email protected]>

Author: onedr0p

.github/actions/container-size-diff/action.yml

@@ -19,12 +19,123 @@ runs:
   using: "composite"
   steps:
     - run: |
-        EOF=$(dd if=/dev/urandom bs=15 count=1 status=none | base64)
-        echo "markdown<<${EOF}" >> "${GITHUB_OUTPUT}"
-        echo "$(${GITHUB_ACTION_PATH}/container-size-diff.sh ${INPUT_FROM_CONTAINER} ${INPUT_TO_CONTAINER})" >> "${GITHUB_OUTPUT}"
-        echo "${EOF}" >> "${GITHUB_OUTPUT}"
-      id: size-diff
+        cat > package.json << 'EOF'
+        {
+          "dependencies": {
+            "execa": "^9.6.0",
+            "filesize": "^11.0.2",
+            "lodash": "^4.0.0",
+            "markdown-table": "^3.0.4"
+          }
+        }
+        EOF
+      shell: bash
+
+    - uses: actions/setup-node@a0853c24544627f65ddf259abe73b1d18a591444 # v5.0.0
+      with:
+        cache: npm
+        cache-dependency-path: package.json
+        node-version: 22.x
+
+    - run: npm install
       shell: bash
+
+    - uses: actions/github-script@ed597411d8f924073f98dfc5c65a23a2325f34cd # v8.0.0
+      id: size-diff
       env:
-        INPUT_FROM_CONTAINER: ${{ inputs.from-container }}
-        INPUT_TO_CONTAINER: ${{ inputs.to-container }}
+        FROM_CONTAINER: ${{ inputs.from-container }}
+        TO_CONTAINER: ${{ inputs.to-container }}
+      with:
+        script: |
+          const { execaSync } = require("execa");
+          const { filesize } = require("filesize");
+          const { markdownTable } = require("markdown-table");
+          const _ = require("lodash");
+
+          const fromContainer = process.env.FROM_CONTAINER;
+          const toContainer = process.env.TO_CONTAINER;
+
+          async function getSizes(container) {
+              try {
+                  const { stdout } = execaSync("docker", ["manifest", "inspect", "-v", container]);
+                  const manifests = _.castArray(JSON.parse(stdout.trim()));
+
+                  return _(manifests)
+                      .map(m => {
+                          const platform = _.get(m, 'Descriptor.platform', {});
+
+                          const key = _.compact([
+                              platform.os,
+                              platform.architecture,
+                              platform.variant,
+                              platform["os.version"]
+                          ])
+                          .join("/");
+
+                          const layers = _.get(m, 'OCIManifest.layers') || _.get(m, 'layers', []);
+                          const total = _.sumBy(layers, 'size');
+
+                          return [key, total];
+                      })
+                      .fromPairs()
+                      .value();
+              } catch (e) {
+                  core.error(`Failed to inspect ${container}: ${e.message}`);
+                  return {};
+              }
+          }
+
+          const fromSizes = await getSizes(fromContainer);
+          const toSizes   = await getSizes(toContainer);
+
+          const platforms = _(fromSizes)
+              .keys()
+              .union(_.keys(toSizes))
+              .filter(platform => platform && !platform.includes("unknown"))
+              .sort()
+              .value();
+
+          const platformRow = (platform) => {
+              const previous = _.get(fromSizes, platform, 0);
+              const current = _.get(toSizes, platform, 0);
+              const change = current - previous;
+
+              const percentage = previous
+                  ? `${change >= 0 ? "+" : ""}${_.round(change / previous * 100, 2)}%`
+                  : current ? "+∞" : "+0.00%";
+
+              const icon = _.cond([
+                  [change => change < 0, _.constant("🔽")],
+                  [change => change > 0, _.constant("🔼")],
+                  [_.stubTrue, _.constant("🔄")]
+              ])(change);
+
+              return [
+                  platform,
+                  filesize(previous),
+                  filesize(current),
+                  `${change >= 0 ? "+" : ""}${filesize(Math.abs(change))} (${percentage})`,
+                  icon
+              ];
+          };
+
+          const rows = [
+              ["OS/Platform", "Previous", "Current", "Change", "Trend"],
+              ..._.map(platforms, platformRow)
+          ];
+
+          const output = `## 📦 Container Size Analysis
+
+          > [!NOTE]
+          > Comparing \`${fromContainer}\` ➔ \`${toContainer}\`
+
+          ### 📈 Size Comparison Table
+
+          ${markdownTable(rows, { align: ["l", "c", "c", "c", "c"] })}`;
+
+          core.setOutput("markdown", output);
+
+              output += `| ${platform} | ${formatBytes(fromSize)} | ${formatBytes(toSize)} | ${sign}${formatBytes(Math.abs(delta))} (${percentChange}%) | ${icon} |\n`;
+          }
+
+          core.setOutput('markdown', output);