[wip]

Author: avivkeller

.github/workflows/compare-builds.yml

@@ -103,15 +103,16 @@ jobs:
       - name: Combine results
         id: combine
         run: |
-          cat results/*/result.txt > combined.txt
+          cat results/**/*.txt > combined.txt
           echo "combined<<EOF" >> $GITHUB_OUTPUT
           cat combined.txt >> $GITHUB_OUTPUT
           echo "EOF" >> $GITHUB_OUTPUT
 
       - run: echo combined.txt
-      # - name: Add Comment to PR
-      #   uses: thollander/actions-comment-pull-request@24bffb9b452ba05a4f3f77933840a6a841d1b32b # v3.0.1
-      #   with:
-      #     comment-tag: 'compared'
-      #     message: ${{ steps.combine.outputs.combined }}
-      #     pr-number: ${{ github.event.workflow_run.pull_requests[0].number }}
+      - name: Add Comment to PR
+        uses: thollander/actions-comment-pull-request@24bffb9b452ba05a4f3f77933840a6a841d1b32b # v3.0.1
+        with:
+          comment-tag: 'compared'
+          message: ${{ steps.combine.outputs.combined }}
+          pr-number: ${{ github.event.pull_request.number }}
+          # pr-number: ${{ github.event.workflow_run.pull_requests[0].number }}

scripts/compare-builds/web.mjs

@@ -1 +1,107 @@
-console.log('Test!');
+import { stat, readdir } from 'node:fs/promises';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+const [BASE, HEAD] = ['BASE', 'HEAD'].map(env =>
+  fileURLToPath(import.meta.resolve(`../../out/${process.env[env] ?? './'}`))
+);
+
+/**
+ * Formats bytes into human-readable format
+ * @param {number} bytes - Number of bytes
+ * @returns {string} Formatted string (e.g., "1.50 KB")
+ */
+const formatBytes = bytes => {
+  if (!bytes) {
+    return '0 B';
+  }
+
+  const units = ['B', 'KB', 'MB', 'GB'];
+  const i = Math.floor(Math.log(Math.abs(bytes)) / Math.log(1024));
+
+  return `${(bytes / 1024 ** i).toFixed(2)} ${units[i]}`;
+};
+
+/**
+ * Formats the difference between base and head sizes
+ * @param {number} base - Base file size in bytes
+ * @param {number} head - Head file size in bytes
+ * @returns {string} Formatted diff string (e.g., "+1.50 KB (+10.00%)")
+ */
+const formatDiff = (base, head) => {
+  const diff = head - base;
+  const pct = base ? `${((diff / base) * 100).toFixed(2)}%` : 'N/A';
+  const sign = diff > 0 ? '+' : diff < 0 ? '' : '';
+
+  return `${sign}${formatBytes(diff)} (${sign}${pct})`;
+};
+
+/**
+ * Gets all files in a directory with their stats
+ * @param {string} dir - Directory path to search
+ * @returns {Promise<Map<string, number>>} Map of filename to size
+ */
+const getDirectoryStats = async dir => {
+  const entries = await readdir(dir);
+  const stats = await Promise.all(
+    entries.map(async file => [file, (await stat(path.join(dir, file))).size])
+  );
+
+  return new Map(stats);
+};
+
+/**
+ * Gets the base name of a file (without extension)
+ * @param {string} file - Filename
+ * @returns {string} Base name
+ */
+const getBaseName = file => file.replace(/\.[^. ]+$/, '');
+
+const [baseStats, headStats] = await Promise.all(
+  [BASE, HEAD].map(getDirectoryStats)
+);
+
+const allFiles = new Set([...baseStats.keys(), ...headStats.keys()]);
+
+// Count occurrences of each base name to identify paired files
+const baseNameCounts = new Map();
+for (const file of allFiles) {
+  const name = getBaseName(file);
+  baseNameCounts.set(name, (baseNameCounts.get(name) || 0) + 1);
+}
+
+// Find and sort changed files: non-paired first (alphabetically), then paired (alphabetically)
+const changedFiles = [...allFiles]
+  // .filter(file => baseStats.get(file) !== headStats.get(file))
+  .sort((a, b) => {
+    const aPaired = baseNameCounts.get(getBaseName(a)) > 1;
+    const bPaired = baseNameCounts.get(getBaseName(b)) > 1;
+    return aPaired === bPaired ? a.localeCompare(b) : aPaired - bPaired;
+  });
+
+if (changedFiles.length) {
+  const rows = changedFiles.map(file => {
+    const baseSize = baseStats.get(file);
+    const headSize = headStats.get(file);
+
+    if (baseSize === undefined) {
+      return `| \`${file}\` | - | ${formatBytes(headSize)} | Added |`;
+    }
+
+    if (headSize === undefined) {
+      return `| \`${file}\` | ${formatBytes(baseSize)} | - | Removed |`;
+    }
+
+    return `| \`${file}\` | ${formatBytes(baseSize)} | ${formatBytes(headSize)} | ${formatDiff(baseSize, headSize)} |`;
+  });
+
+  console.log(
+    `
+## Web Generator
+
+| File | Base | Head | Diff |
+|------|------|------|------|
+${rows.join('\n')}
+`.trim()
+  );
+}