ci(root): AI 리뷰 자동 커밋 메시지 한 줄로 통일 - #7
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| GEMINI_API_KEY: ${{ secrets.GEMINI_API_KEY }} | ||
| OPENROUTER_API_KEY: ${{ secrets.OPENROUTER_API_KEY }} | ||
| AI_REVIEW_PUSH_TOKEN: ${{ secrets.AI_REVIEW_PUSH_TOKEN }} | ||
| AI_REVIEW_PUSH_TOKEN: "" |
There was a problem hiding this comment.
현재 실행에서 재검출되지 않음(자동 추정). 사람이 실제 반영 여부를 확인하세요.
WhyLog AI 리뷰결과: ✅ 차단 항목 없음 · 모델: Google AI 리뷰 자동 커밋 메시지 규격을 한 줄로 통일하고 줄 단위 리뷰 댓글 생성 방식을 개별 댓글 요청으로 변경한 작업입니다. 전반적인 테스트와 로직이 잘 보완되어 있으나, GitHub API 커밋 메시지의 개행 문자 처리 관련 개선 사항이 있습니다. 인라인: 신규 1 · 갱신 0 · 재검출 안 됨 1 · 요약 대체 0 차단없음 제안
이 코멘트는 새 실행 때 갱신됩니다. 차단 항목은 사람이 타당성을 확인한 뒤 수정하세요. |
| or not isinstance(parents[0], Mapping) | ||
| ): | ||
| return None | ||
| document_pr_number = int(Path(paths[0]).stem.removeprefix("PR-")) |
There was a problem hiding this comment.
새 커밋의 같은 위치에 최신 자동 리뷰를 다시 등록했습니다.
| ): | ||
| return None | ||
| document_pr_number = _pr_number_from_review_doc_path(paths[0]) | ||
| if document_pr_number is None or message != _doc_commit_message( |
There was a problem hiding this comment.
WhyLog AI 리뷰
- 제안: GitHub API 커밋 메시지의 개행 문자(\n) 트림 처리 권장
- 이유: GitHub REST API로 커밋 정보를 조회할 때
commit.message문자열 끝에 개행 문자(\n)가 포함되어 들어올 수 있습니다.message와_doc_commit_message(...)를 직접 등가 비교(!=)할 경우 개행 문자로 인해 감지 실패(None반환)가 발생할 가능성이 있습니다. - 근거:
CI 자동화 스크립트 예외 처리 규칙 - 제안 수정:
message.strip() != _doc_commit_message(document_pr_number)와 같이.strip()을 사용하여 개행 여부와 관계없이 안전하게 일치 여부를 검증하도록 수정하는 것을 권장합니다.
- 이유: GitHub REST API로 커밋 정보를 조회할 때
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
1 participant
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.
작업내용
포함된 영역
예측 (작업 초기 PR 생성 시 작성)
결과 (머지 전 작성)
리뷰 요청
메모