Skip to content

feat: synchronize 이벤트 시 변경된 파일만 패턴 재분석#20

Open
jylee2033 wants to merge 1 commit intomainfrom
issue-7-synchronize-optimization
Open

feat: synchronize 이벤트 시 변경된 파일만 패턴 재분석#20
jylee2033 wants to merge 1 commit intomainfrom
issue-7-synchronize-optimization

Conversation

@jylee2033
Copy link
Copy Markdown
Contributor

@jylee2033 jylee2033 commented Apr 9, 2026

Summary

  • synchronize 이벤트 시 Compare API(/compare/{before}...{after})로 이번 push에서 실제 변경된 파일만 식별
  • 변경된 파일의 봇 코멘트만 삭제/재작성하여 불필요한 OpenAI API 호출과 GitHub 알림 스팸 방지
  • Compare API 실패 시 기존 동작(전체 파일 분석)으로 안전하게 fallback

Closes #7

Test plan

  • opened 이벤트: 기존처럼 모든 솔루션 파일이 분석되는지 확인
  • synchronize 이벤트: 변경된 파일만 분석되고, 미변경 파일의 봇 코멘트가 유지되는지 확인
  • Compare API 실패 시 전체 파일 분석으로 fallback되는지 확인
  • 실제 PR에서 재push 후 동작 확인

🤖 Generated with Claude Code

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Apr 9, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
github 73869b0 Commit Preview URL

Branch Preview URL
Apr 09 2026, 02:09 AM

@sounmind sounmind self-requested a review April 9, 2026 01:56
Use Compare API (before...after SHA) to identify files changed in the
latest push, so only those files get pattern-analyzed and their bot
comments deleted/recreated. Falls back to full analysis if Compare API
fails.

Co-Authored-By: sounmind <37020415+sounmind@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jylee2033 jylee2033 force-pushed the issue-7-synchronize-optimization branch from 0ce6805 to 73869b0 Compare April 9, 2026 02:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

최적화: synchronize 이벤트 시 변경된 파일만 패턴 재분석

2 participants