Skip to content

feat: Gemini PR code review#4

Merged
mjk25 merged 2 commits intomainfrom
chore/3-gemini-pr
Jan 29, 2026
Merged

feat: Gemini PR code review#4
mjk25 merged 2 commits intomainfrom
chore/3-gemini-pr

Conversation

@mjk25
Copy link
Contributor

@mjk25 mjk25 commented Jan 24, 2026

#️⃣반영 브랜치

chore/#3

📝작업 내용

github action 설정
gemini 연동 설정
제미나이 모델 : gemini-flash-latest

  • 최초 1회만 리뷰
  • 추가 리뷰가 필요한 경우, GitHub Actions 탭에서 Run workflow 버튼을 통해 수동으로 실행 가능

@mjk25 mjk25 self-assigned this Jan 24, 2026
@mjk25 mjk25 linked an issue Jan 24, 2026 that may be closed by this pull request
2 tasks
@github-actions
Copy link

🤖 Gemini Code Review

✅ 변경 사항 요약

  • Gemini를 활용한 자동화된 GitHub 코드 리뷰 워크플로우를 신규 구축했습니다.
  • PR 발생 시 Node.js 스크립트를 통해 Git Diff를 추출하고 Gemini API를 호출합니다.
  • API 요청 제한(Rate Limit)에 대비하여 재시도 로직을 포함했습니다.

✏️ 파일별 상세 리뷰

.github/scripts/gemini-review.js

  • 📍 Issue: diff.txt 파일을 읽는 과정(L15)에서 오류가 발생했을 경우, 스크립트가 오류 메시지 출력 없이 조용히 종료됩니다 (return;). 디프 생성 단계가 실패했는데도 스크립트가 성공적으로 종료되어 리뷰가 생성되지 않는 경우를 인지하기 어렵습니다.
  • Improvement: diff.txt 파일 읽기 실패 시 console.error로 로그를 남기거나, 프로세스를 종료(exit 1)하여 워크플로우의 실패를 명확히 표시해야 합니다. 현재의 구현은 디버깅을 어렵게 만듭니다.

.github/workflows/code-review.yml

  • 📍 Issue: 디프 추출(Get git diff for PR) 단계에서, fetch-depth: 0 설정만으로는 충분하지 않을 수 있으며, 특히 대상 브랜치(base.ref)와 헤드 브랜치(head.ref)의 정확한 상태를 보장하기 위해 명시적인 git fetch가 필요합니다. 현재 코드는 두 브랜치 모두를 명시적으로 가져오고 있으므로 기능적으로는 올바르지만, 최초 Checkout 시 fetch-depth: 0을 설정했음에도 불구하고 두 개의 추가적인 git fetch 호출을 수행하는 것은 워크플로우 실행 시간을 소폭 증가시킵니다. 현재 로직이 가장 안전하므로 유지합니다.
  • 📍 Issue: github-script를 사용한 주석 생성 단계에서 review_result.txt 파일 읽기가 실패할 경우 catch (e) { return; }로 인해 주석 생성 시도가 조용히 무시됩니다. 리뷰 스크립트 실행이 성공했으나 결과 파일이 생성되지 않았거나 읽을 수 없는 경우에 대한 명확한 실패 처리 로직이 없습니다.

@mjk25 mjk25 merged commit 97634e7 into main Jan 29, 2026
@mjk25 mjk25 deleted the chore/3-gemini-pr branch January 29, 2026 10:02
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.

[chore] Gemini PR 코드 리뷰 설정

1 participant