Skip to content

fix(root): AI 리뷰 작성자 계정 식별 - #11

Merged
wantkdd merged 2 commits into
developfrom
fix/ai-review-actor
Aug 10, 2026
Merged

fix(root): AI 리뷰 작성자 계정 식별#11
wantkdd merged 2 commits into
developfrom
fix/ai-review-actor

Conversation

@wantkdd

@wantkdd wantkdd commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

작업내용

  • AI 리뷰 작성 계정을 whylog-dev로 고정합니다.
  • 종합·인라인 댓글을 GitHub 사용자 타입이 아닌 고정 login으로 식별합니다.
  • 기존 종합 댓글은 같은 ID에 PATCH하고, 기존 인라인은 재등록하지 않도록 유지합니다.

포함된 영역

  • FE
  • BE
  • AI
  • docs
  • infra
  • 직접 작성

예측 (작업 초기 PR 생성 시 작성)

  • 예상 소요: 30분
  • 막힐 것 같은 곳: PAT 계정이 API상 Bot이 아니라 User로 반환되는 기존 가정
  • 전문 밖 영역 중 걱정되는 것: 없음

결과 (머지 전 작성)

  • 실제 소요: 약 20분
  • 실제로 막힌 곳: 기존 E2E 테스트 환경에 고정 계정 값이 없어 실패한 부분을 테스트 설정에 반영
  • 배운 것: 공용 PAT 계정 댓글은 user.type이 아니라 user.login으로 식별해야 안정적으로 갱신할 수 있음

리뷰 요청

  • 종합 리뷰가 새 댓글을 만들지 않고 기존 whylog-dev 댓글을 갱신하는지 확인
  • 인라인 리뷰가 기존 댓글을 유지하고 사라진 finding만 resolve하는지 확인

메모

  • python -m unittest discover -s .github/scripts -p "test_*.py": 56개 통과
  • python -m py_compile: 통과
  • git diff --check: 통과

@vercel

vercel Bot commented Aug 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
whylog Ready Ready Preview Aug 10, 2026 1:57am

@whylog-dev

Copy link
Copy Markdown
Contributor

WhyLog AI 리뷰

결과: ✅ 차단 항목 없음 · 모델: Google gemini-3.6-flash

AI 리뷰어 작성자 계정을 whylog-dev로 고정하고, 댓글 식별 방식을 user.type에서 user.login 기반으로 보완한 CI 스크립트 수정입니다. 워크플로우 설정 및 관련 테스트 코드가 함께 업데이트되어 문제없이 정상 동작합니다.

PR 리뷰 문서: docs/pr-reviews/PR-11.md 서명·부모 SHA·입력 digest 확인 · 이전 판단 유지

차단

없음

제안

없음

이 코멘트는 새 실행 때 갱신됩니다. 차단 항목은 사람이 타당성을 확인한 뒤 수정하세요.

@wantkdd
wantkdd merged commit 3fe76f2 into develop Aug 10, 2026
9 checks passed
@WhyLog-App WhyLog-App deleted a comment from whylog-dev Aug 10, 2026
@WhyLog-App WhyLog-App deleted a comment from whylog-dev Aug 10, 2026
@wantkdd
wantkdd deleted the fix/ai-review-actor branch August 10, 2026 02:02
@wantkdd wantkdd self-assigned this Aug 11, 2026
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.

2 participants