feat: PR 리뷰 프로세스 개선을 위한 패치 없는 파일에 대한 코멘트 생략 및 오류 로깅 강화

Author: kim-the9

action/index.cjs

@@ -150670,6 +150670,7 @@ const robot = (app) => {
             }
             // Fetch PR-wide file patches to compute accurate review comment positions
             let prFilePatchByPath = new Map();
+            let commentablePaths = new Set();
             try {
                 const prFiles = await context.octokit.pulls.listFiles({
                     owner: repo.owner,
@@ -150680,6 +150681,7 @@ const robot = (app) => {
                 for (const f of prFiles.data) {
                     if (f && typeof f.patch === "string") {
                         prFilePatchByPath.set(f.filename, f.patch);
+                        commentablePaths.add(f.filename);
                     }
                 }
             }
@@ -150716,6 +150718,10 @@ const robot = (app) => {
                         continue;
                     }
                     if (!res.lgtm || (res.lgtm && REVIEW_ON_LGTM)) {
+                        if (!commentablePaths.has(file.filename)) {
+                            loglevel_1.default.info(`PR 파일에 patch가 없어 ${file.filename}에 대한 코멘트 생략`);
+                            continue;
+                        }
                         // Compute position against the PR-wide patch to avoid misalignment
                         const prWidePatch = prFilePatchByPath.get(file.filename) || patch;
                         const position = firstChangedPosition(prWidePatch);
@@ -150727,7 +150733,16 @@ const robot = (app) => {
                     }
                 }
                 catch (e) {
-                    loglevel_1.default.info(`review ${file.filename} failed`, e);
+                    const status = e?.status || e?.response?.status;
+                    const data = e?.response?.data;
+                    loglevel_1.default.error(`리뷰 실패: ${file.filename} (status=${status})`);
+                    if (data) {
+                        try {
+                            loglevel_1.default.error(`GitHub API error payload: ${JSON.stringify(data, null, 2)}`);
+                        }
+                        catch { }
+                    }
+                    loglevel_1.default.error("원시 오류:", e);
                 }
             }
         }
@@ -150753,6 +150768,15 @@ const robot = (app) => {
             });
         }
         catch (e) {
+            const status = e?.status || e?.response?.status;
+            const data = e?.response?.data;
+            loglevel_1.default.error(`리뷰 생성 실패 (status=${status})`);
+            if (data) {
+                try {
+                    loglevel_1.default.error(`GitHub API error payload: ${JSON.stringify(data, null, 2)}`);
+                }
+                catch { }
+            }
             loglevel_1.default.info(`Failed to create review`, e);
         }
         console.timeEnd("gpt cost");

src/bot.ts

@@ -358,6 +358,7 @@ export const robot = (app: Probot) => {
 
         // Fetch PR-wide file patches to compute accurate review comment positions
         let prFilePatchByPath = new Map<string, string>();
+        let commentablePaths = new Set<string>();
         try {
           const prFiles = await context.octokit.pulls.listFiles({
             owner: repo.owner,
@@ -368,6 +369,7 @@ export const robot = (app: Probot) => {
           for (const f of prFiles.data as any[]) {
             if (f && typeof f.patch === "string") {
               prFilePatchByPath.set(f.filename, f.patch as string);
+              commentablePaths.add(f.filename);
             }
           }
         } catch (e) {
@@ -414,6 +416,10 @@ export const robot = (app: Probot) => {
             }
 
             if (!res.lgtm || (res.lgtm && REVIEW_ON_LGTM)) {
+              if (!commentablePaths.has(file.filename)) {
+                log.info(`PR 파일에 patch가 없어 ${file.filename}에 대한 코멘트 생략`);
+                continue;
+              }
               // Compute position against the PR-wide patch to avoid misalignment
               const prWidePatch = prFilePatchByPath.get(file.filename) || patch;
               const position = firstChangedPosition(prWidePatch);
@@ -423,8 +429,16 @@ export const robot = (app: Probot) => {
                 position,
               });
             }
-          } catch (e) {
-            log.info(`review ${file.filename} failed`, e);
+          } catch (e: any) {
+            const status = e?.status || e?.response?.status;
+            const data = e?.response?.data;
+            log.error(`리뷰 실패: ${file.filename} (status=${status})`);
+            if (data) {
+              try {
+                log.error(`GitHub API error payload: ${JSON.stringify(data, null, 2)}`);
+              } catch {}
+            }
+            log.error("원시 오류:", e);
           }
         }
       }
@@ -450,7 +464,15 @@ export const robot = (app: Probot) => {
           commit_id: commits[commits.length - 1].sha,
           comments: ress,
         });
-      } catch (e) {
+      } catch (e: any) {
+        const status = e?.status || e?.response?.status;
+        const data = e?.response?.data;
+        log.error(`리뷰 생성 실패 (status=${status})`);
+        if (data) {
+          try {
+            log.error(`GitHub API error payload: ${JSON.stringify(data, null, 2)}`);
+          } catch {}
+        }
         log.info(`Failed to create review`, e);
       }