Skip to content

[#39] fine-grained PAT 권한과 check metadata 실패 처리를 정리한다#40

Merged
opficdev merged 1 commit into
developfrom
fix/#39-check-metadata
Jun 24, 2026
Merged

[#39] fine-grained PAT 권한과 check metadata 실패 처리를 정리한다#40
opficdev merged 1 commit into
developfrom
fix/#39-check-metadata

Conversation

@opficdev

Copy link
Copy Markdown
Owner

🔗 연관된 이슈

🎯 의도

  • fine-grained PAT UI에서 설정할 수 없는 Checks: Read-only 안내를 제거
  • check run metadata 조회 실패가 전체 Watcher 실행을 중단하지 않도록 보조 metadata 수집을 격리

📝 작업 내용

📌 요약

  • README의 fine-grained PAT 권한 안내에서 Checks: Read-only 제거
  • Troubleshooting의 Checks: Read-only 확인 문구 제거
  • GitHub check run metadata 조회 실패 시 빈 check 목록으로 계속 진행
  • check run API 실패 격리 테스트 추가

🔍 상세

  • WATCHER_GITHUB_TOKEN 권한 안내를 실제 fine-grained PAT 설정 화면 기준으로 조정
  • PAT 권한은 Contents: Read-only, Pull requests: Read-only, 자동 포함되는 Metadata: Read-only 기준으로 정리
  • githubMetadataClientFor().checksFor()에서 check run API 실패를 warning으로만 남기고 [] 반환
  • PR metadata 조회는 기존처럼 실패를 전파하여 Pull requests: Read-only 설정 오류를 확인할 수 있도록 유지
  • check run API가 403을 반환해도 checksFor()가 빈 배열을 반환하는 테스트 추가

📸 영상 / 이미지 (Optional)

@opficdev opficdev self-assigned this Jun 24, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

이번 풀 리퀘스트에서는 GitHub check-runs API 호출 실패 시 전체 프로세스가 실패하지 않도록 예외 처리를 추가하여 안정성을 개선했습니다. checksFor 함수에 try-catch 블록을 적용하여 오류 발생 시 경고 로그를 남기고 빈 배열을 반환하도록 변경했으며, 이에 따른 테스트 코드와 README.md 문서의 권한 안내를 업데이트했습니다. 리뷰어의 피드백은 없습니다.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@opficdev opficdev merged commit c9a7d92 into develop Jun 24, 2026
3 checks passed
@opficdev opficdev deleted the fix/#39-check-metadata branch June 24, 2026 14:12
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.

fine-grained PAT 권한과 check metadata 실패 처리를 정리한다

1 participant