feat: reply with workflow page

opensumi · Oct 9, 2024 · 482eac4 · 482eac4
1 parent ba0af0b
commit 482eac4
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 31 deletions.
diff --git a/src/github/commands/pullRequests/index.ts b/src/github/commands/pullRequests/index.ts
@@ -203,9 +203,12 @@ Please see: <${getActionsUrl(ActionsRepo.BACKPORT_PR_WORKFLOW)}>`,
     constraintRepo(fullname, updateLockfileAllowedRepo);
     await checkIsPullRequestAndUserHasPermission(ctx, 'write');
 
-    await app.octoService.updateLockfileForPr({
-      pull_number: issue.number,
-    });
+    await app.octoService.dispatchWorkflow(
+      ActionsRepo.UPDATE_LOCKFILE_WORKFLOW,
+      {
+        pull_number: String(issue.number),
+      },
+    );
     await app.createReactionForIssueComment(ctx, 'rocket');
   });
   it.on(
@@ -219,11 +222,18 @@ Please see: <${getActionsUrl(ActionsRepo.BACKPORT_PR_WORKFLOW)}>`,
       constraintRepo(fullname, mergeCommitAllowedRepo);
       await checkIsPullRequestAndUserHasPermission(ctx, 'admin');
 
-      await app.octoService.createMergeCommitForPr({
-        pull_number: issue.number,
-        owner,
-        repo,
-      });
+      await app.octoService.dispatchWorkflow(
+        ActionsRepo.CREATE_MERGE_COMMIT_WORKFLOW,
+        {
+          pull_number: String(issue.number),
+          owner,
+          repo,
+        },
+      );
+      await app.replyComment(
+        ctx,
+        `[Start merging the pull request](${getActionsUrl(ActionsRepo.CREATE_MERGE_COMMIT_WORKFLOW)})`,
+      );
       await app.createReactionForIssueComment(ctx, 'rocket');
     },
     ['merge-this'],

diff --git a/src/github/service/opensumi.ts b/src/github/service/opensumi.ts
@@ -168,29 +168,19 @@ export class OpenSumiOctoService extends GitHubService {
     return workflow;
   }
 
-  async updateLockfileForPr({ pull_number }: { pull_number: number }) {
-    const workflow = await this.octo.actions.createWorkflowDispatch({
-      ...ActionsRepo.UPDATE_LOCKFILE_WORKFLOW,
-      inputs: {
-        pull_number: pull_number.toString(),
-      },
-    });
-    return workflow;
-  }
-
-  async createMergeCommitForPr({
-    pull_number,
-    owner,
-    repo,
-  }: { pull_number: number; owner: string; repo: string }) {
-    const workflow = await this.octo.actions.createWorkflowDispatch({
-      ...ActionsRepo.CREATE_MERGE_COMMIT_WORKFLOW,
-      inputs: {
-        pull_number: pull_number.toString(),
-        owner,
-        repo,
-      },
+  async dispatchWorkflow(
+    workflow: {
+      workflow_id: string;
+      ref: string;
+      owner: string;
+      repo: string;
+    },
+    inputs: Record<string, any>,
+  ) {
+    const result = await this.octo.actions.createWorkflowDispatch({
+      ...workflow,
+      inputs,
     });
-    return workflow;
+    return result;
   }
 }