Merge pull request #1 from fastfive-dev/feature/max-file-count

feat: MAX_FILE_COUNT 최대 파일 수를 20에서 30으로 증가

Author: jonghyeokFF

src/bot.ts

@@ -8,15 +8,15 @@ const OPENAI_API_KEY = "OPENAI_API_KEY";
 const MAX_PATCH_COUNT = process.env.MAX_PATCH_LENGTH
   ? +process.env.MAX_PATCH_LENGTH
   : Infinity;
-const MAX_FILE_COUNT = 20;
+const MAX_FILE_COUNT = 30;
 
 // 파일 본문을 가져올 때 허용하는 최대 크기 (바이트 단위)
 // 너무 큰 파일은 그대로 두되, 이후 prompt 구성 단계에서 잘라냅니다.
 const MAX_FILE_BYTES = process.env.MAX_FILE_BYTES
   ? +process.env.MAX_FILE_BYTES
   : 64 * 1024; // 기본 64KB
 
-  // 프롬프트에 포함할 수 있는 최대 문자 수 (가드레일)
+// 프롬프트에 포함할 수 있는 최대 문자 수 (가드레일)
 // 이 값을 넘어가면 중간 부분을 잘라내고 앞/뒤 일부만 남깁니다.
 const MAX_CONTEXT_CHARS = process.env.MAX_CONTEXT_CHARS
   ? +process.env.MAX_CONTEXT_CHARS
@@ -36,7 +36,7 @@ const GLOBAL_CONTEXT_MODE = process.env.GLOBAL_CONTEXT_MODE || "meta"; // meta |
 const fetchFileContent = async (
   context: Context,
   path: string,
-  ref: string,
+  ref: string
 ): Promise<string | null> => {
   try {
     const repo = context.repo();
@@ -77,18 +77,28 @@ const buildReviewInput = (
   filename: string,
   content: string | null,
   patch: string,
-  globalContext: string | null,
+  globalContext: string | null
 ): string => {
   const parts: string[] = [];
 
   // 파일별 리뷰 지침을 최상단에 명시해 전역 컨텍스트 편향을 줄임 (STRICT)
   parts.push("## 리뷰 지침 (STRICT)");
   parts.push("다음을 반드시 준수하세요 — 어기면 산출물은 실패로 간주됩니다.");
-  parts.push("- **스코프: 이 파일의 Diff만** 다룹니다. 다른 파일/일반론 언급 금지.");
-  parts.push("- **근거 필수:** 모든 지적은 Diff의 `+`(추가) 라인에 근거해, 해당 코드 스니펫을 인라인 코드로 포함하세요.");
-  parts.push("- **전역 컨텍스트는 참고용**입니다. 스코프를 벗어난 내용이 필요하면 그 항목은 **작성하지 마세요**.");
-  parts.push("- **우선순위:** 버그/보안/성능/테스트 중심. 사소한 스타일/개인 취향 금지.");
-  parts.push("- **스코프 위반 방지:** 다른 파일이나 일반 베스트 프랙티스만을 언급하려 할 경우, 해당 코멘트는 생략합니다.");
+  parts.push(
+    "- **스코프: 이 파일의 Diff만** 다룹니다. 다른 파일/일반론 언급 금지."
+  );
+  parts.push(
+    "- **근거 필수:** 모든 지적은 Diff의 `+`(추가) 라인에 근거해, 해당 코드 스니펫을 인라인 코드로 포함하세요."
+  );
+  parts.push(
+    "- **전역 컨텍스트는 참고용**입니다. 스코프를 벗어난 내용이 필요하면 그 항목은 **작성하지 마세요**."
+  );
+  parts.push(
+    "- **우선순위:** 버그/보안/성능/테스트 중심. 사소한 스타일/개인 취향 금지."
+  );
+  parts.push(
+    "- **스코프 위반 방지:** 다른 파일이나 일반 베스트 프랙티스만을 언급하려 할 경우, 해당 코멘트는 생략합니다."
+  );
 
   parts.push(`\n# File: ${filename}`);
 
@@ -102,15 +112,19 @@ const buildReviewInput = (
   if (content) {
     parts.push("\n## 현재 파일 내용 (truncated)");
     parts.push("```");
-    parts.push(truncateForPrompt(content, allocateBudget(MAX_CONTEXT_CHARS, 0.5)));
+    parts.push(
+      truncateForPrompt(content, allocateBudget(MAX_CONTEXT_CHARS, 0.5))
+    );
     parts.push("```");
   }
 
   // (선택) PR 전역 컨텍스트는 맨 아래에 위치시켜 영향 최소화
   if (globalContext) {
     parts.push("\n---\n");
     parts.push("# PR 전역 컨텍스트 (참고용)");
-    parts.push(truncateForPrompt(globalContext, allocateBudget(MAX_CONTEXT_CHARS, 0.4)));
+    parts.push(
+      truncateForPrompt(globalContext, allocateBudget(MAX_CONTEXT_CHARS, 0.4))
+    );
   }
 
   return parts.join("\n");
@@ -163,7 +177,7 @@ export const robot = (app: Probot) => {
           owner: repo.owner,
           repo: repo.repo,
           name: OPENAI_API_KEY,
-        },
+        }
       )) as any;
 
       if (!data?.value) {
@@ -230,7 +244,7 @@ export const robot = (app: Probot) => {
         "compareCommits, base:",
         context.payload.pull_request.base.sha,
         "head:",
-        context.payload.pull_request.head.sha,
+        context.payload.pull_request.head.sha
       );
       log.debug("compareCommits.commits:", commits);
       log.debug("compareCommits.files", changedFiles);
@@ -254,8 +268,8 @@ export const robot = (app: Probot) => {
           pull_number: pull_request.number,
         });
 
-        hasSummaryReview = reviews.data.some(
-          (review) => review.body?.includes("<!-- chatgpt-summary:v1 -->"),
+        hasSummaryReview = reviews.data.some((review) =>
+          review.body?.includes("<!-- chatgpt-summary:v1 -->")
         );
 
         if (hasSummaryReview) {
@@ -310,7 +324,7 @@ export const robot = (app: Probot) => {
       let inlineFallback: string[] = [];
 
       if (isFileCountWithinLimit) {
-          const aggregatedPatch = changedFiles
+        const aggregatedPatch = changedFiles
           .filter(
             (file) =>
               (file.status === "modified" || file.status === "added") &&
@@ -335,26 +349,43 @@ export const robot = (app: Probot) => {
         let globalContext: string | null = null;
         {
           const prTitle = pull_request.title || "(no title)";
-          const changedFileList = changedFiles.map(f => `- ${f.filename} (${f.status})`).join("\n");
-          const summary = overallReview?.summary ? `\n\n## 변경 요약\n${overallReview.summary}` : "";
+          const changedFileList = changedFiles
+            .map((f) => `- ${f.filename} (${f.status})`)
+            .join("\n");
+          const summary = overallReview?.summary
+            ? `\n\n## 변경 요약\n${overallReview.summary}`
+            : "";
 
           // meta 모드: 메타데이터만 제공 (제목/커밋 메시지/파일 목록)
           if (GLOBAL_CONTEXT_MODE === "meta") {
             globalContext = [
               `## PR 제목\n${prTitle}`,
-              `\n## 커밋 메시지(집계)\n${truncateForPrompt(aggregatedCommitMessages, allocateBudget(MAX_GLOBAL_CONTEXT_CHARS, 0.4))}`,
+              `\n## 커밋 메시지(집계)\n${truncateForPrompt(
+                aggregatedCommitMessages,
+                allocateBudget(MAX_GLOBAL_CONTEXT_CHARS, 0.4)
+              )}`,
               `\n## 변경 파일 목록\n${changedFileList}`,
               summary,
             ].join("\n");
           } else {
             // full 모드: 전체 diff 발췌를 소량만 포함 (하위 호환)
-            const diffDigest = aggregatedPatch ? truncateForPrompt(aggregatedPatch, allocateBudget(MAX_GLOBAL_CONTEXT_CHARS, 0.3)) : "";
+            const diffDigest = aggregatedPatch
+              ? truncateForPrompt(
+                  aggregatedPatch,
+                  allocateBudget(MAX_GLOBAL_CONTEXT_CHARS, 0.3)
+                )
+              : "";
             globalContext = [
               `## PR 제목\n${prTitle}`,
-              `\n## 커밋 메시지(집계)\n${truncateForPrompt(aggregatedCommitMessages, allocateBudget(MAX_GLOBAL_CONTEXT_CHARS, 0.3))}`,
+              `\n## 커밋 메시지(집계)\n${truncateForPrompt(
+                aggregatedCommitMessages,
+                allocateBudget(MAX_GLOBAL_CONTEXT_CHARS, 0.3)
+              )}`,
               `\n## 변경 파일 목록\n${changedFileList}`,
               summary,
-              diffDigest ? `\n\n## 전체 diff (일부 발췌)\n\n\`\`\`diff\n${diffDigest}\n\`\`\`` : "",
+              diffDigest
+                ? `\n\n## 전체 diff (일부 발췌)\n\n\`\`\`diff\n${diffDigest}\n\`\`\``
+                : "",
             ].join("\n");
           }
         }
@@ -388,7 +419,9 @@ export const robot = (app: Probot) => {
           }
 
           if (!patch || patch.length > MAX_PATCH_COUNT) {
-            log.info(`${file.filename} skipped caused by its diff is too large`);
+            log.info(
+              `${file.filename} skipped caused by its diff is too large`
+            );
             continue;
           }
 
@@ -398,16 +431,24 @@ export const robot = (app: Probot) => {
             fullContent = await fetchFileContent(
               context,
               file.filename,
-              context.payload.pull_request.head.sha,
+              context.payload.pull_request.head.sha
             );
-            if (fullContent && Buffer.byteLength(fullContent, "utf-8") > MAX_FILE_BYTES) {
+            if (
+              fullContent &&
+              Buffer.byteLength(fullContent, "utf-8") > MAX_FILE_BYTES
+            ) {
               // Keep but it will be truncated by buildReviewInput/truncateForPrompt
             }
           } catch (e) {
             log.debug(`content fetch failed for ${file.filename}`, e);
           }
 
-          const reviewInput = buildReviewInput(file.filename, fullContent, patch, globalContext);
+          const reviewInput = buildReviewInput(
+            file.filename,
+            fullContent,
+            patch,
+            globalContext
+          );
 
           try {
             const res = await chat?.codeReview(reviewInput);
@@ -420,20 +461,28 @@ export const robot = (app: Probot) => {
 
             if (!res.lgtm || (res.lgtm && REVIEW_ON_LGTM)) {
               if (!commentablePaths.has(file.filename)) {
-                log.info(`PR 파일에 patch가 없어 ${file.filename}에 대한 코멘트 생략`);
-                inlineFallback.push(`\n### ${file.filename}\n${res.review_comment}`);
+                log.info(
+                  `PR 파일에 patch가 없어 ${file.filename}에 대한 코멘트 생략`
+                );
+                inlineFallback.push(
+                  `\n### ${file.filename}\n${res.review_comment}`
+                );
                 continue;
               }
               // Compute position against the PR-wide patch to avoid misalignment
               const prWidePatch = prFilePatchByPath.get(file.filename) || patch;
-              if (!prWidePatch || !prWidePatch.includes('@@')) {
-                inlineFallback.push(`\n### ${file.filename}\n${res.review_comment}`);
+              if (!prWidePatch || !prWidePatch.includes("@@")) {
+                inlineFallback.push(
+                  `\n### ${file.filename}\n${res.review_comment}`
+                );
                 continue;
               }
 
               const position = firstChangedPosition(prWidePatch);
               if (!Number.isSafeInteger(position) || position <= 0) {
-                inlineFallback.push(`\n### ${file.filename}\n${res.review_comment}`);
+                inlineFallback.push(
+                  `\n### ${file.filename}\n${res.review_comment}`
+                );
                 continue;
               }
               ress.push({
@@ -448,7 +497,9 @@ export const robot = (app: Probot) => {
             log.error(`리뷰 실패: ${file.filename} (status=${status})`);
             if (data) {
               try {
-                log.error(`GitHub API error payload: ${JSON.stringify(data, null, 2)}`);
+                log.error(
+                  `GitHub API error payload: ${JSON.stringify(data, null, 2)}`
+                );
               } catch {}
             }
             log.error("원시 오류:", e);
@@ -482,7 +533,11 @@ export const robot = (app: Probot) => {
         const data = e?.response?.data;
         log.error(`리뷰 생성 실패 (status=${status})`);
         if (data) {
-          try { log.error(`GitHub API error payload: ${JSON.stringify(data, null, 2)}`); } catch {}
+          try {
+            log.error(
+              `GitHub API error payload: ${JSON.stringify(data, null, 2)}`
+            );
+          } catch {}
         }
 
         if (status === 422) {
@@ -492,8 +547,13 @@ export const robot = (app: Probot) => {
               body = `<!-- chatgpt-summary:v1 -->\n${overallReview.summary}`;
             }
 
-            if (typeof inlineFallback !== "undefined" && inlineFallback.length) {
-              body += `\n\n---\n⚠️ 인라인 주석을 diff에 고정하지 못해 본문에 첨부합니다 (422).` + inlineFallback.join("\n");
+            if (
+              typeof inlineFallback !== "undefined" &&
+              inlineFallback.length
+            ) {
+              body +=
+                `\n\n---\n⚠️ 인라인 주석을 diff에 고정하지 못해 본문에 첨부합니다 (422).` +
+                inlineFallback.join("\n");
             } else if (ress.length) {
               body += `\n\n(인라인 코멘트가 위치 불일치로 생략되었습니다.)`;
             }
@@ -523,7 +583,7 @@ export const robot = (app: Probot) => {
       log.info("successfully reviewed", context.payload.pull_request.html_url);
 
       return "success";
-    },
+    }
   );
 };
 
@@ -537,7 +597,7 @@ const matchPatterns = (patterns: string[], path: string) => {
           : pattern.startsWith("**")
           ? pattern
           : "**/" + pattern,
-        { dot: true, nocase: true },
+        { dot: true, nocase: true }
       );
     } catch {
       // if the pattern is not a valid glob pattern, try to match it as a regular expression