From b58bbc330f598541fb04e4dcd830e87b94d34c3e Mon Sep 17 00:00:00 2001 From: vltbaudbot <264001112+vltbaudbot@users.noreply.github.com> Date: Tue, 29 Sep 2026 00:45:17 +0000 Subject: [PATCH] fix: add upm to package-count and process-count collectors MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The upm package manager was added to benchmarks but its count file entries were missing from collect-package-count.js and collect-process-count.js. This caused: - package-count.sh correctly writes upm-count.txt - collect-package-count.js didn't read it → no package count data - generate-chart.js couldn't compute per-package times for upm - Leaderboard cards and per-fixture charts (which use per-package data) excluded upm, while the total-time histogram (which uses raw data) showed it correctly Add the missing { filename: 'upm-count.txt', pmName: 'upm' } entry to both collector scripts. --- scripts/collect-package-count.js | 1 + scripts/collect-process-count.js | 1 + 2 files changed, 2 insertions(+) diff --git a/scripts/collect-package-count.js b/scripts/collect-package-count.js index a522685233..83b026546e 100644 --- a/scripts/collect-package-count.js +++ b/scripts/collect-package-count.js @@ -31,6 +31,7 @@ const countFiles = [ { filename: 'bun-count.txt', pmName: 'bun' }, { filename: 'deno-count.txt', pmName: 'deno' }, { filename: 'aube-count.txt', pmName: 'aube' }, + { filename: 'upm-count.txt', pmName: 'upm' }, // Registry names (for registry benchmark variations) { filename: 'aws-count.txt', pmName: 'aws' }, { filename: 'cloudsmith-count.txt', pmName: 'cloudsmith' }, diff --git a/scripts/collect-process-count.js b/scripts/collect-process-count.js index 1e26176be4..0f33d0b6b9 100644 --- a/scripts/collect-process-count.js +++ b/scripts/collect-process-count.js @@ -31,6 +31,7 @@ const countFiles = [ { filename: 'bun-process-count.txt', pmName: 'bun' }, { filename: 'deno-process-count.txt', pmName: 'deno' }, { filename: 'aube-process-count.txt', pmName: 'aube' }, + { filename: 'upm-process-count.txt', pmName: 'upm' }, { filename: 'nx-process-count.txt', pmName: 'nx' }, { filename: 'turbo-process-count.txt', pmName: 'turbo' }, { filename: 'node-process-count.txt', pmName: 'node' },