add MAX_PATCH_LENGTH

Author: anc95

.env.example

@@ -19,4 +19,5 @@ temperature=
 top_p=
 max_tokens=
 TARGET_LABEL=
+MAX_PATCH_LENGTH=
 PROMPT=Below there is a code diff please help me do a code review

README.md

@@ -66,6 +66,7 @@ jobs:
           top_p: 1
           temperature: 1
           max_tokens: 10000
+          MAX_PATCH_LENGTH: 10000 # if the patch/diff length is large than MAX_PATCH_LENGTH, will be ignored and won't review. By default, with no MAX_PATCH_LENGTH set, there is also no limit for the patch/diff length.
 ```
 
 ## Self-hosting

src/bot.ts

@@ -3,7 +3,9 @@ import { Context, Probot } from 'probot';
 import { Chat } from './chat.js';
 
 const OPENAI_API_KEY = 'OPENAI_API_KEY';
-const MAX_PATCH_COUNT = 4000;
+const MAX_PATCH_COUNT = process.env.MAX_PATCH_LENGTH
+  ? +process.env.MAX_PATCH_LENGTH
+  : Infinity;
 
 export const robot = (app: Probot) => {
   const loadChat = async (context: Context) => {
@@ -90,11 +92,15 @@ export const robot = (app: Probot) => {
           head: commits[commits.length - 1].sha,
         });
 
-        const ignoreList = (process.env.IGNORE || process.env.ignore || '').split('\n').filter(v => v !== '');
+        const ignoreList = (process.env.IGNORE || process.env.ignore || '')
+          .split('\n')
+          .filter((v) => v !== '');
 
         const filesNames = files?.map((file) => file.filename) || [];
-        changedFiles = changedFiles?.filter((file) =>
-          filesNames.includes(file.filename) && !ignoreList.includes(file.filename)
+        changedFiles = changedFiles?.filter(
+          (file) =>
+            filesNames.includes(file.filename) &&
+            !ignoreList.includes(file.filename)
         );
       }
 
@@ -119,18 +125,22 @@ export const robot = (app: Probot) => {
           );
           continue;
         }
-        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,
-            path: file.filename,
-            body: res,
-            position: patch.split('\n').length - 1,
-          });
+        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,
+              path: file.filename,
+              body: res,
+              position: patch.split('\n').length - 1,
+            });
+          }
+        } catch (e) {
+          console.error(`review ${file.filename} failed`, e);
         }
       }