test

.github/workflows/ecosystem-benchmark.yml

@@ -10,6 +10,7 @@ on:
   push:
     branches:
       - main
+      - jerry/test
     paths-ignore:
       - "**/*.md"
       - "website/**"
@@ -29,7 +30,7 @@ jobs:
       target: x86_64-unknown-linux-gnu
       native: ${{ needs.get-runner-labels.outputs.LINUX_RUNNER_LABELS == '"ubuntu-22.04"' }}
       runner: ${{ needs.get-runner-labels.outputs.LINUX_RUNNER_LABELS }}
-      ref: ${{ github.event_name == 'workflow_dispatch' && format('pull/{0}/head', inputs.pr) || "refs/heads/main" }}
+      ref: ${{ github.event_name == 'workflow_dispatch' && format('pull/{0}/head', inputs.pr) || github.sha }}
       test: false
       bench: false
 
@@ -41,39 +42,41 @@ jobs:
       - id: create-comment
         uses: actions/github-script@v7
         with:
-          github-token: ${{ GITHUB_TOKEN }}
+          github-token: ${{ secrets.GITHUB_TOKEN }}
           result-encoding: string
           script: |
             const url = `${context.serverUrl}//${context.repo.owner}/${context.repo.repo}/actions/runs/${context.runId}`
             const urlLink = `[Open](${url})`
+            const body = `⏳ Triggered benchmark: ${urlLink}`
 
             if (context.eventName === 'workflow_dispatch') {
               const { data: comment } = await github.rest.issues.createComment({
                 issue_number: context.payload.inputs.pr,
                 owner: context.repo.owner,
-                repo: 'rspack',
-                body: `⏳ Triggered benchmark: ${urlLink}`
+                repo: context.repo.repo,
+                body
               })
               return comment.id
             }
 
             const { data: comment } = await github.rest.repos.createCommitComment({
               commit_sha: context.sha,
               owner: context.repo.owner,
-              repo: 'rspack',
+              repo: context.repo.repo,
               body
             })
             return comment.id
 
   bench:
+    needs: [build]
     runs-on: [self-hosted]
     outputs:
       diff-result: ${{ steps.run-benchmark.outputs.diff-result }}
     steps:
       - name: Checkout
         uses: actions/checkout@v4
         with:
-          ref: ${{ inputs.ref }}
+          ref: ${{ github.event_name == 'workflow_dispatch' && format('pull/{0}/head', inputs.pr) || github.sha }}
 
       - name: Clean
         uses: ./.github/actions/clean
@@ -104,7 +107,7 @@ jobs:
           git clone --single-branch --depth 1 https://github.com/web-infra-dev/rspack-ecosystem-benchmark.git
           cd rspack-ecosystem-benchmark
           pnpm i
-          RSPACK_DIR=RSPACK_DIR node bin/cli.js bench
+          RSPACK_DIR="$RSPACK_DIR" node bin/cli.js bench
           result=$(node bin/cli.js compare --base latest --current current)
           echo "$result"
           echo "diff-result=${result//$'\n'/'@@'}" >> $GITHUB_OUTPUT
@@ -116,7 +119,7 @@ jobs:
     steps:
       - uses: actions/github-script@v7
         with:
-          github-token: ${{ secrets.RSPACK_BOT_ACCESS_TOKEN }}
+          github-token: ${{ secrets.GITHUB_TOKEN }}
           script: |
             const diffResult = `${{ needs.bench.outputs.diff-result }}`
 
@@ -133,12 +136,21 @@ jobs:
             ${result}
             `
 
-            await github.rest.issues.updateComment({
-              owner: context.repo.owner,
-              repo: 'rspack',
-              comment_id: `${{ needs.create-comment.outputs.comment-id }}`,
-              body
-            })
+            if (context.eventName === 'workflow_dispatch') {
+              await github.rest.issues.updateComment({
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                comment_id: `${{ needs.create-comment.outputs.comment-id }}`,
+                body
+              })
+            } else {
+              await github.rest.repos.updateCommitComment({
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                comment_id: `${{ needs.create-comment.outputs.comment-id }}`,
+                body,
+              });
+            }
 
             if (result.includes("Threshold exceeded")) {
               console.log("Some benchmark cases exceed the threshold, please visit the previous step for more information");