chore: integrate pm version to UI (#25)

Make sure to pass the `versions` data to the generated json result
so that the UI can properly render version numbers.

Author: ruyadorno

.github/workflows/benchmark.yaml

@@ -201,10 +201,6 @@ jobs:
             }
 
             console.log('Benchmark cleaning completed');
-      - name: Copy Versions File
-        run: |
-          # Find any versions.json file from the downloaded versions artifacts and copy it to results
-          find versions-temp -name "versions.json" -type f | head -1 | xargs -I {} cp {} ./results/versions.json || echo "No versions.json found"
       - name: Process Results
         run: |
           bash ./scripts/process-results.sh

app/src/components/leaderboard.tsx

@@ -9,10 +9,6 @@ interface LeaderboardProps {
   chartData: BenchmarkChartData;
 }
 
-
-
-
-
 export const Leaderboard = ({ chartData }: LeaderboardProps) => {
   const { enabledPackageManagers } = usePackageManagerFilter();
   const { theme } = useTheme();
@@ -39,10 +35,6 @@ export const Leaderboard = ({ chartData }: LeaderboardProps) => {
             </p>
           </div>
         </div>
-        <div className="flex items-center gap-1 text-xs text-muted-foreground">
-          <div className="w-2 h-2 rounded-full bg-yellow-500" />
-          <span>wins</span>
-        </div>
       </div>
 
       {/* Flexible grid layout for tools */}
@@ -81,13 +73,6 @@ export const Leaderboard = ({ chartData }: LeaderboardProps) => {
                   />
                 </div>
               )}
-
-              {/* Superscript-style win indicator */}
-              {item.wins > 0 && (
-                <div className="absolute -top-1 -right-1 bg-yellow-500 text-white rounded-full px-1.5 py-0.5 text-[10px] font-medium">
-                  {item.wins}
-                </div>
-              )}
             </div>
           );
         })}

app/src/hooks/use-chart-data.ts

@@ -37,15 +37,10 @@ export const useChartData = (): UseChartDataReturn => {
 
       // Fetch versions data (optional)
       let versions: PackageManagerVersions | undefined;
-      try {
-        const versionsResponse = await fetch(`/latest/versions.json`);
-        if (versionsResponse.ok) {
-          versions = await versionsResponse.json() as PackageManagerVersions;
-        } else {
-          console.warn("Versions data not available");
-        }
-      } catch (versionsError) {
-        console.warn("Failed to fetch versions data:", versionsError);
+      if (chartData.versions) {
+        versions = chartData.versions;
+      } else {
+        console.warn("Versions data not available");
       }
 
       // Combine chart data with versions

scripts/generate-chart.js

@@ -151,9 +151,31 @@ function generateChartData(option = {}) {
   };
 }
 
+// Load package manager version data
+function loadPackageManagersVersionData() {
+  const versionsFile = path.resolve(RESULTS_DIR, "versions.json");
+  
+  try {
+    if (!fs.existsSync(versionsFile)) {
+      console.warn(`Warning: Versions file ${versionsFile} does not exist`);
+      return {};
+    }
+    
+    const data = JSON.parse(fs.readFileSync(versionsFile, "utf8"));
+    return data || {};
+  } catch (error) {
+    console.warn(
+      `Warning: Could not read versions from ${versionsFile}:`,
+      error.message,
+    );
+    return {};
+  }
+}
+
 const dumpChartData = () => {
   const chartData = generateChartData();
   const perPackageCountChartData = generateChartData({ perPackageCount: true });
+  const versions = loadPackageManagersVersionData()
 
   const results = {
     chartData: {
@@ -168,6 +190,7 @@ const dumpChartData = () => {
       packageManagers: perPackageCountChartData.packageManagers,
       colors: perPackageCountChartData.colors,
     },
+    versions,
   };
 
   fs.writeFileSync(

scripts/process-results.sh

@@ -8,6 +8,9 @@ DATE=$(date +%Y-%m-%d)
 mkdir -p "results/$DATE"
 mkdir -p "results/latest"
 
+# Find any versions.json file from the downloaded versions artifacts and copy it to results
+find versions-temp -name "versions.json" -type f | head -1 | xargs -I {} cp {} ./results/$DATE/versions.json || echo "No versions.json found"
+
 # Function to print summary of results
 print_summary() {
     local file=$1