chore: use pnpm compare (#53)

* chore: add github token

* chore: use pnpm compare

Author: Timeless0911

.github/workflows/pr-bench.yaml

@@ -67,10 +67,13 @@ jobs:
       - name: Install Dependencies
         run: pnpm run install:scripts
 
+      - name: 🚀 Run specified benchmark
+        run: cd scripts && PR_NUMBER=${{ inputs.prNumber }} GITHUB_TOKEN=${{ secrets.GITHUB_TOKEN }} pnpm start ${{ inputs.product }}
+
       - id: print-results
-        name: 🚀 Run specified benchmark and Print results
+        name: Print results
         run: |
-          result=$(cd scripts && PR_NUMBER=${{ inputs.prNumber }} pnpm start ${{ inputs.product }})
+          result=$(cd scripts && node ./dist/compare.js ${{ inputs.product }})
           echo "$result"
           echo "diff-result=${result//$'\n'/'@@'}" >> $GITHUB_OUTPUT
   update-comment:

scripts/src/shared/compare.ts renamed to scripts/src/compare.ts

@@ -1,5 +1,8 @@
 import { readJson } from 'fs-extra';
-import { Metrics } from './types';
+import { Metrics } from './shared/types';
+import { DefaultBenchCase, getMetricsPath } from './shared';
+
+const productName = process.argv[2] || 'MODERNJS_FRAMEWORK';
 
 const formatFileSize = (size: number, target = 'KB') => {
   if (target === 'KB') {
@@ -60,7 +63,11 @@ function generateTable(
   return table.join('\n');
 }
 
-export async function compare(jsonPath: string) {
+export async function compare(productName: string) {
+  const caseName =
+    process.argv[3] ||
+    DefaultBenchCase[productName as keyof typeof DefaultBenchCase];
+  const { jsonPath } = await getMetricsPath(productName, caseName);
   const allMetrics: Metrics[] = await readJson(jsonPath);
   const keys = Object.keys(allMetrics);
   const currentKey = keys[keys.length - 1];
@@ -71,3 +78,5 @@ export async function compare(jsonPath: string) {
   const formatTable = generateTable(base, current);
   console.log(formatTable);
 }
+
+compare(productName);

scripts/src/main.ts

@@ -9,7 +9,6 @@ import {
 } from './shared';
 import { remove } from 'fs-extra';
 import { yarnInstall } from './runners/yarn-install';
-import { compare } from './shared/compare';
 
 const productName = process.argv[2] || 'MODERNJS_FRAMEWORK';
 const caseName =
@@ -43,11 +42,7 @@ async function main() {
       console.log('failed to collect install size metrics:', err);
     }
 
-    const jsonPath = await mergeMetrics(productName, caseName);
-
-    if (process.env.PR_NUMBER) {
-      await compare(jsonPath);
-    }
+    await mergeMetrics(productName, caseName);
   } else {
     logger.error(`Case not found: ${productName} ${caseName}`);
   }

scripts/src/shared/fs.ts

@@ -40,7 +40,7 @@ export async function updateFile(
   }
 }
 
-async function getMetricsPath(productName: string, caseName: string) {
+export async function getMetricsPath(productName: string, caseName: string) {
   const jsonName = `${caseName}.json`;
   return {
     jsonName,
@@ -102,10 +102,7 @@ const cleanData = (nums: number[]) => {
   return nums;
 };
 
-export async function mergeMetrics(
-  productName: string,
-  caseName: string,
-): Promise<string> {
+export async function mergeMetrics(productName: string, caseName: string) {
   const { jsonPath, jsonName, remoteURL } = await getMetricsPath(
     productName,
     caseName,
@@ -146,6 +143,4 @@ export async function mergeMetrics(
     await outputJson(jsonPath, allData);
     logger.success(`Successfully merged metrics to ${jsonName}.`);
   }
-
-  return jsonPath;
 }