풀이 추가/수정 커밋시 기존 분석은 유지하고 추가/수정 파일에 대해서만 분석하도록 변경 - #50
Draft
parkhojeong wants to merge 1 commit into
Draft
Conversation
기존에는 풀이를 추가하거나 수정해서 커밋하면 기존 분석 댓글을 모두 삭제하고 모든 풀이에 대한 분석 댓글을 다시 달았음. 분석 댓글에서 대화를 주고 받는 경우가 있는데 커밋이 되면 분석 내용이 삭제되고 주고 받은 댓글 일부가 Conversations 탭에서 사라지는 문제가 있었음. 변경된 방식은 풀이를 추가하거나 수정해도 기존 분석 댓글을 유지함. 추가되거나 수정된 풀이에 대해서만 분석하여 댓글을 남김. 분석했던 코드 원본을 함께 분석 댓글에 포함하여 맥락을 파악하는데 용이하도록 함. 기존과 동일한 부분 - PR 오픈과 재오픈한 경우엔 모든 풀이에 대해서 분석 댓글을 남김. - 삭제된 풀이는 분석하지 않음. Test: bun run test Issue: DaleStudy#48
DaleSeo
reviewed
Jul 29, 2026
Comment on lines
+238
to
+258
| async function resolveChangedFilenames(payload, repoOwner, repoName, appToken) { | ||
| if (payload.action !== "synchronize" || !payload.before || !payload.after) { | ||
| return null; | ||
| } | ||
|
|
||
| let changedFilenames = null; | ||
|
|
||
| try { | ||
| changedFilenames = await getChangedFilenames( | ||
| repoOwner, | ||
| repoName, | ||
| payload.before, | ||
| payload.after, | ||
| appToken | ||
| ); | ||
| } catch (error) { | ||
| console.error(`[resolveChangedFilenames] failed: ${error.message}`); | ||
| } | ||
|
|
||
| return changedFilenames; | ||
| } |
Member
There was a problem hiding this comment.
일반 push는 before가 after의 조상이라 정확하지만, 충돌 해결 등으로 rebase 후 강제 푸시하면 merge base가 브랜치 분기점까지 내려가서 PR의 모든 풀이 파일이 changedFilenames에 포함도지 않을까요? 삭제 로직이 있을 때는
지우고 다시 다니 상관없었는데 이제는 전 파일에 댓글이 한 벌씩 더 쌓이게 될텐데 새로운 엣지 케이스가 우려가 되네요.
Comment on lines
+216
to
+223
| return `<details> | ||
| <summary>${filename}</summary> | ||
|
|
||
| \`\`\`${language} | ||
| ${content}${truncated} | ||
| \`\`\` | ||
|
|
||
| </details>`; |
Member
There was a problem hiding this comment.
풀이 파일의 주석이나 docstring에 ```이 포함되면 혹시 details 블록이 깨지지는 않겠죠?
|
|
||
| function renderAnalyzedSource(filename, content) { | ||
| const language = filename.includes(".") ? filename.split(".").pop() : ""; | ||
| const truncated = content.length >= MAX_FILE_SIZE ? "\n... (이하 생략)" : ""; |
Member
There was a problem hiding this comment.
downloadFileEntries는 > MAX_FILE_SIZE일 때만 자르는데 여기서는 >= MAX_FILE_SIZE로 판정하네요. 의도하신 건가요?
Comment on lines
+243
to
+257
| let changedFilenames = null; | ||
|
|
||
| try { | ||
| changedFilenames = await getChangedFilenames( | ||
| repoOwner, | ||
| repoName, | ||
| payload.before, | ||
| payload.after, | ||
| appToken | ||
| ); | ||
| } catch (error) { | ||
| console.error(`[resolveChangedFilenames] failed: ${error.message}`); | ||
| } | ||
|
|
||
| return changedFilenames; |
Member
There was a problem hiding this comment.
이렇게 살짝 다르게 작성하는 패턴에 대해서는 어떻게 생각하세요? 실패하면 null이 반환되는 부분이 더 쉽게 읽힐 수 있거든요.
Suggested change
| let changedFilenames = null; | |
| try { | |
| changedFilenames = await getChangedFilenames( | |
| repoOwner, | |
| repoName, | |
| payload.before, | |
| payload.after, | |
| appToken | |
| ); | |
| } catch (error) { | |
| console.error(`[resolveChangedFilenames] failed: ${error.message}`); | |
| } | |
| return changedFilenames; | |
| try { | |
| return await getChangedFilenames( | |
| repoOwner, | |
| repoName, | |
| payload.before, | |
| payload.after, | |
| appToken | |
| ); | |
| } catch (error) { | |
| console.error(`[resolveChangedFilenames] failed: ${error.message}`); | |
| return null; | |
| } |
sounmind
reviewed
Jul 29, 2026
| let body = `${COMMENT_MARKER} | ||
| ### 🏷️ 알고리즘 패턴 분석 | ||
|
|
||
| ${renderAnalyzedSource(file.filename, fileContent)} |
Member
There was a problem hiding this comment.
원본 소스코드를 보여주는 아이디어 좋네요!
코드 변경 이후의 히스토리 저장 측면에서도 유용하다고 생각합니다.
다만 여기서 더 나아가 diff를 보여준다던지 하는 건 없을까요? 어디서 어떻게 변경됐는지도 알 수 있으면 좋겠다 생각했기 떄문이에요.
그러면 깃허브에서 자동으로 제공하는 diff 링크 같은 게 있을까도 궁금합니다.
아래 코멘트에서 언급된 rebase로 인한 force push 업데이트에서도 깃허브에서 변경사항을 보여주는 링크가 제공되기에 해결할 수 있는 방법 중 하나가 되지 않을까 생각해봤습니다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
문제
커밋으로 풀이가 수정/추가되면 기존 분석 댓글을 모두 삭제하고 모든 파일에 대한 분석 댓글을 다시 달고 있음.
분석 댓글에서 대화를 주고 받는 경우가 있는데 커밋이 되면 분석 내용이 삭제되고 주고 받은 댓글 일부가
Conversations 탭에서 사라짐.
변경
풀이가 수정/추가되어도 기존 분석 댓글을 삭제하지 않고 변경된 파일에 대해서만 분석하여 댓글을 남김.
분석했던 코드 원본을 함께 댓글에 포함하여 맥락을 파악하는데 용이하도록 함.
이 pr에서 스크린샷의 동작 결과는 살펴보실 수 있습니다.
유지
고민사항
수정이 많아지면 계속 댓글이 추가되어 길어질 수 있음. 현재 기수의 경우 수정하는 경우가 수 회 정도라 수용가능한 정도로 보임.
테스트
Test: bun run test
cloudflare 개인 계정을 만들고 organization Dalestudy-test을 만들어서 테스트함.
Close: #48