Make one review for files (#148)

* Make one review comment for files * Add build diff
anc95 · Nov 5, 2024 · 0adfa2a · 0adfa2a
1 parent c086468
commit 0adfa2a
Show file tree

Hide file tree

Showing 2 changed files with 33 additions and 12 deletions.
diff --git a/action/index.cjs b/action/index.cjs
@@ -148211,6 +148211,7 @@ const robot = (app) => {
             return 'no change';
         }
         console.time('gpt cost');
+        const ress = [];
         for (let i = 0; i < changedFiles.length; i++) {
             const file = changedFiles[i];
             const patch = file.patch || '';
@@ -148224,11 +148225,7 @@ const robot = (app) => {
             try {
                 const res = await chat?.codeReview(patch);
                 if (!!res) {
-                    await context.octokit.pulls.createReviewComment({
-                        repo: repo.repo,
-                        owner: repo.owner,
-                        pull_number: context.pullRequest().pull_number,
-                        commit_id: commits[commits.length - 1].sha,
+                    ress.push({
                         path: file.filename,
                         body: res,
                         position: patch.split('\n').length - 1,
@@ -148239,6 +148236,20 @@ const robot = (app) => {
                 console.error(`review ${file.filename} failed`, e);
             }
         }
+        try {
+            await context.octokit.pulls.createReview({
+                repo: repo.repo,
+                owner: repo.owner,
+                pull_number: context.pullRequest().pull_number,
+                body: "Code review by ChatGPT",
+                event: 'COMMENT',
+                commit_id: commits[commits.length - 1].sha,
+                comments: ress,
+            });
+        }
+        catch (e) {
+            console.error(`Failed to create review`, e);
+        }
         console.timeEnd('gpt cost');
         console.info('successfully reviewed', context.payload.pull_request.html_url);
         return 'success';

diff --git a/src/bot.ts b/src/bot.ts
@@ -113,6 +113,8 @@ export const robot = (app: Probot) => {
 
       console.time('gpt cost');
 
+      const ress = [];
+
       for (let i = 0; i < changedFiles.length; i++) {
         const file = changedFiles[i];
         const patch = file.patch || '';
@@ -129,22 +131,30 @@ export const robot = (app: Probot) => {
         }
         try {
           const res = await chat?.codeReview(patch);
-
           if (!!res) {
-            await context.octokit.pulls.createReviewComment({
-              repo: repo.repo,
-              owner: repo.owner,
-              pull_number: context.pullRequest().pull_number,
-              commit_id: commits[commits.length - 1].sha,
+            ress.push({
               path: file.filename,
               body: res,
               position: patch.split('\n').length - 1,
-            });
+            })
           }
         } catch (e) {
           console.error(`review ${file.filename} failed`, e);
         }
       }
+      try {
+        await context.octokit.pulls.createReview({
+          repo: repo.repo,
+          owner: repo.owner,
+          pull_number: context.pullRequest().pull_number,
+          body: "Code review by ChatGPT",
+          event: 'COMMENT',
+          commit_id: commits[commits.length - 1].sha,
+          comments: ress,
+        });
+      } catch (e) {
+        console.error(`Failed to create review`, e);
+      }
 
       console.timeEnd('gpt cost');
       console.info(